Set BM poll size once during priv probe and do not
overwrite it during port probe procedure. Pool is common for
all CP ports.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NNadav Haklai <nadavh@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

TX drain in transmit procedure could cause issues due
to race between drain procedure and transmition of descriptor
between AGGR TXQ and physical TXQ.
TXQ will be cleared before moving to Linux by stop procedure.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NNadav Haklai <nadavh@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

MVPP22 driver support 64 Bit arch and require BM pool
high address configuration.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NNadav Haklai <nadavh@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

Remove IRQ configuration from U-Boot PP driver.
U-Boot don't use interrupts and configuration of IRQ in U-Boot
caused crashes in Linux shared interrupt mode.
Also interrupt use is redundant in RX routine since a single RX
queue is used.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NNadav Haklai <nadavh@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

MBUS driver were replaced by AXI in PPv22 and relevant
only for PPv21.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NNadav Haklai <nadavh@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

U-boot use single physical tx queue with size 16 descriptors.
So aggregated tx queue size should be equal to physical tx queue
and cpu descriptor chunk(number of descriptors delivered from
physical tx queue to aggregated tx queue by one chunk) shouldn't be
larger than physical tx queue.

Fix:
Set AGGR_TXQ and CPU_DESC_CHUNK to be 16 descriptors, same as
physical TXQ.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NNadav Haklai <nadavh@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

Issue:
BM counters were overrun by probe that called per Network interface and
caused release of wrong number of buffers during remove procedure.

Fix:
Use probe_done and num_ports to call init and remove procedure
once per communication controller.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

This patch enables padding of packets shorter than 64B in TX(set by default).
Disabling of padding causes crashes on MACCIATO board.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

A8K marvell SoC has two South Bridge communication controllers(CP0 and CP1).
Each communication controller has packet processor ports and MDIO.
On MACHIATOBin board ports from CP1 are connected to mdio on CP0.

Issue:
Wrong base address is assigned to MDIO interface during probe.

Fix:
Get MDIO address from PHY handler parent base address.

This should be refined in the future when MDIO driver is implemented.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

This patch add GPIO configuration support in mvpp2x driver.
Driver will handle 10G SFP gpio reset and SFP TX disable. GPIO pins should
be set in device tree.
Signed-off-by: NStefan Chulski <stefanc@marvell.com>
Tested-by: NiSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: NKostya Porotchkin <kostap@marvell.com>
Reviewed-by: NIgal Liberman <igall@marvell.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NStefan Roese <sr@denx.de>

Reviewed-by: NJoe Hershberger <joe.hershberger@ni.com>
Signed-off-by: NTom Rini <trini@konsulko.com>

This defconfig uses the PCIe x4 binary blobs from the congatec BIOS.
Signed-off-by: NStefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: NBin Meng <bmeng.cn@gmail.com>

- Disable debug UART
- Enable more partition support
Signed-off-by: NStefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: NBin Meng <bmeng.cn@gmail.com>

- Enable ACPI resume support
- Disable debug UART
- Enable Spansion and Winbond SPI flash support
- Move VGA BIOS binary address to enable bigger U-Boot images
Signed-off-by: NStefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: NBin Meng <bmeng.cn@gmail.com>

This patch adds the common header include file theadorable-x86-common.h
for the theadorable-x86 targets to define all common options and the
default environment.
Signed-off-by: NStefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: NBin Meng <bmeng.cn@gmail.com>

This patch adds the infrastructure to define different config headers
with different configurations and default environment for the baseboards
that can now be selected via Kconfig. The new configuration for the
theadorable-x86-conga-qa3-e3845 is also added. Also the new defconfig
file for this new target is added.
Signed-off-by: NStefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: NBin Meng <bmeng.cn@gmail.com>

This patch adds the infrastructure to define different config headers
with different configurations and default environment for the baseboards
that can now be selected via Kconfig. The new configuration for the
theadorable-x86-dfi-bt700 is also added.
Signed-off-by: NStefan Roese <sr@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: NBin Meng <bmeng.cn@gmail.com>

When CONFIG_EFI_PARTITION is not set, the following build error is
seen in arch/x86/lib/acpi_s3.c:

  error: expected declaration specifiers or '...' before '*' token
  static void asmlinkage (*acpi_do_wakeup)(void *vector) = (void*)WAKEUP_BASE;

This is actually caused by missing asmlinkage declaration, but with
CONFIG_EFI_PARTITION on, the declaration comes from part.h which
is included from common.h.
Signed-off-by: NBin Meng <bmeng.cn@gmail.com>
Reviewed-by: NStefan Roese <sr@denx.de>

This is defined twice in the same file, with the same value, likely
because of some patch merge issue. Pick the uglier one and nuke it.
Signed-off-by: NMarek Vasut <marek.vasut+renesas@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Signed-off-by: NStefan Roese <sr@denx.de>

The I2C reading in the PEX vs SATA detection code often fails on the
first try. Try three times, as the code for EEPROM reading does.
Signed-off-by: NMarek Behun <marek.behun@nic.cz>
Signed-off-by: NStefan Roese <sr@denx.de>

Neither new design uses ISA bus, nor does any U-Boot codes use these
codes. Remove them.
Signed-off-by: NBin Meng <bmeng.cn@gmail.com>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

The environment variable "disable_giga" can now be used to disable
1000baseTx on the Micrel's KSZ9031.
Signed-off-by: NSebastien Bourdelin <sebastien.bourdelin@savoirfairelinux.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

PXE boot is broken with GCC 7.1 due option '-fstore-merging' enabled
by default for '-O2':

BOOTP broadcast 1
data abort
pc : [<8ff8bb30>]          lr : [<00004f1f>]
reloc pc : [<17832b30>]    lr : [<878abf1f>]
sp : 8f558bc0  ip : 00000000     fp : 8ffef5a4
r10: 8ffed248  r9 : 8f558ee0     r8 : 8ffef594
r7 : 0000000e  r6 : 8ffed700     r5 : 00000000  r4 : 8ffed74e
r3 : 00060101  r2 : 8ffed230     r1 : 8ffed706  r0 : 00000ddd
Flags: nzcv  IRQs off  FIQs off  Mode SVC_32
Resetting CPU ...

Core reason is usage of structures for network headers without packed
attribute.
Reviewed-by: NYauheni Kaliuta <yauheni.kaliuta@redhat.com>
Signed-off-by: NDenis Pynkin <denis.pynkin@collabora.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

Add missing "defined" statement to fix the compile failures.
Signed-off-by: NHolger Dengler <dengler@linutronix.de>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

Added the AFE (Analog Front End) settings for stability to the
Broadcom Cygnus phy. This improves the time take to perform
auto negotiation.
Signed-off-by: NArun Parameswaran <arun.parameswaran@broadcom.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

move to Kconfig:
	CONFIG_BCM_SF2_ETH
	CONFIG_BCM_SF2_ETH_DEFAULT_PORT
	CONFIG_BCM_SF2_ETH_GMAC

Also modified defconfigs of all platforms that use these configs.
Signed-off-by: NSuji Velupillai <suji.velupillai@broadcom.com>
Tested-by: NSuji Velupillai <suji.velupillai@broadcom.com>
Reviewed-by: NJD Zheng <jiandong.zheng@broadcom.com>
Reviewed-by: NScott Branden <scott.branden@broadcom.com>
Signed-off-by: NSteve Rae <steve.rae@raedomain.com>
Reviewed-by: NTom Rini <trini@konsulko.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

The correct option is PHY_MICREL_KSZ90X1, but some configs still
select the 9021 and 9031 options, which are deprecated.
Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Reviewed-by: NPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

There should be no longer be any ksz9000 users that pick up the PHY
driver from ksz8xxx, so remove ksz9000 remnants from there.
Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Reviewed-by: NPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

These boards will now use the ksz90x1 driver instead of the mess in
ksz8xxx. This change is needed before the two drivers can be fully
separated.
Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

The KS8721BL and KSZ9021 PHYs are software-incompatible, yet they
share the same ID. Drivers for bothe PHYs cannot safely coexist, so
the solution was to use #ifdefs to select between the two drivers.

As a result KSZ9031, which has a unique ID, is now caught in the
crossfire. Unless CONFIG_PHY_MICREL_KSZ9031 is defined, the KSZ9031
will not function properly, as some essential configuration code is
ifdef'd-out.

To prevent such situations, move the KSZ9000 drivers to a separate
file, and place them under a separate Kconfig option. While it is
possible to enable both KSZ8000 and KSZ9000 drivers at the same time,
the assumption is that it is highly unlikely for a system to contain
both a KSZ8000 and a KSZ9000 PHY, and that only one of the drivers
will be enabled at any given time.
Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Reviewed-by: NPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

Signed-off-by: NAlexandru Gagniuc <alex.g@adaptrum.com>
Reviewed-by: NPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

For strncpy() select a minimal string length of destination and source
strings, here DEFAULT_NAME_LEN is preferable to MAX_LEN.

Due to the NUL-terminated contents of default_string the change is
a noop, however it removes a compilation warning if SH2/3/4 platform
specific strncpy() function is used:

  In file included from include/linux/string.h:21:0,
                   from include/common.h:28,
                   from net/tftp.c:9:

  net/tftp.c: In function 'tftp_start':
  arch/sh/include/asm/string.h:52:42: warning: array subscript is above array bounds [-Warray-bounds]
     : "0" (__dest), "1" (__src), "r" (__src+__n)
Signed-off-by: NVladimir Zapolskiy <vz@mleia.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

Don't wait forever.
Pass errors back to the caller.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>
Acked-by: NMarek Vasut <marex@denx.de>

The register constants don't use the exact names that are used in the
TRM, so add comments that use the exact names so that it is clear what
register is being referred to.

https://www.atheros-drivers.com/qualcomm-atheros-datasheets-for-AR9331.htmlSigned-off-by: NJoe Hershberger <joe.hershberger@ni.com>
Acked-by: NMarek Vasut <marex@denx.de>