Re: [U-Boot] [PATCH] arm: omap3: Detect boot mode very early

2017-07-30 Thread Paul Kocialkowski
Hi, I'd like to point out that HTML messages are not welcome in mailing lists, so please make sure you send plain-text in the future! Le jeudi 27 juillet 2017 à 15:44 -0500, Adam Ford a écrit : > On Jul 27, 2017 3:40 PM, "Paul Kocialkowski" wrote: > > Le mardi 25 juillet 2017 à 13:19 +0300, Siar

[U-Boot] [PATCH v5 00/10] mips: bmips: add SPI support

2017-07-30 Thread Álvaro Fernández Rojas
BCM63xx SPI controller is a bit tricky since it doesn't allow keeping CS active between transfers, so I had to modify the spi_flash driver in order to allow limiting reads. v5: Introduce changes suggested by Jagan Teki: - Use long structs for registers v4: Introduce changes suggested by Jagan Teki

[U-Boot] [PATCH v5 01/10] drivers: spi: allow limiting reads

2017-07-30 Thread Álvaro Fernández Rojas
For some SPI controllers it's not possible to keep the CS active between transfers and they are limited to a known number of bytes. This splits spi_flash reads into different iterations in order to respect the SPI controller limits. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass

[U-Boot] [PATCH v5 04/10] mips: bmips: add bcm63xx-spi driver support for BCM6338

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: no changes v4: no changes v3: rename BCM6338 SPI driver to BCM6348 v2: add spi alias arch/mips/dts/brcm,bcm6338.dtsi | 17 + 1 file chan

[U-Boot] [PATCH v5 02/10] drivers: spi: consider command bytes when sending transfers

2017-07-30 Thread Álvaro Fernández Rojas
Command bytes are part of the written bytes and they should be taken into account when sending a spi transfer. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass Reviewed-by: Daniel Schwierzeck --- v5: no changes v4: no changes v3: Fix bug introduced in v2: sizeof(cmd) vs len v2

[U-Boot] [PATCH v5 03/10] dm: spi: add BCM63xx SPI driver

2017-07-30 Thread Álvaro Fernández Rojas
This driver is a simplified version of linux/drivers/spi/spi-bcm63xx.c Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass Reviewed-by: Daniel Schwierzeck --- v5: Introduce changes suggested by Jagan Teki: - Use long structure instead of a custom bmips_spi_hw structure. - Define

[U-Boot] [PATCH v5 09/10] mips: bmips: enable the SPI flash on the Sagem F@ST1704

2017-07-30 Thread Álvaro Fernández Rojas
It's a Winbond (w25x32) 4 MB SPI flash. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: sync with master v4: switch to CONFIG_BCM63XX_SPI v3: rename BCM6338 SPI driver to BCM6348 v2: remove spi alias arch/mips/dts/sagem,f...@st1704.dts | 12 c

[U-Boot] [PATCH v5 08/10] mips: bmips: add bcm63xx-spi driver support for BCM63268

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the low speed SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: no changes v4: no changes v3: no changes v2: add spi alias arch/mips/dts/brcm,bcm63268.dtsi | 17 + 1 file changed, 17 inserti

[U-Boot] [PATCH v5 07/10] mips: bmips: add bcm63xx-spi driver support for BCM3380

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: no changes v4: no changes v3: no changes v2: add spi alias arch/mips/dts/brcm,bcm3380.dtsi | 17 + 1 file changed, 17 insertions(+) dif

[U-Boot] [PATCH v5 06/10] mips: bmips: add bcm63xx-spi driver support for BCM6358

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: no changes v4: no changes v3: no changes v2: add spi alias arch/mips/dts/brcm,bcm6358.dtsi | 17 + 1 file changed, 17 insertions(+) dif

[U-Boot] [PATCH v5 10/10] mips: bmips: enable the SPI flash on the Netgear CG3100D

2017-07-30 Thread Álvaro Fernández Rojas
It's a Spansion (s25fl064a) 8 MB SPI flash. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: sync with master v4: switch to CONFIG_BCM63XX_SPI v3: no changes v2: remove spi alias arch/mips/dts/netgear,cg3100d.dts | 12 configs/netgear_cg3100d_r

[U-Boot] [PATCH v5 05/10] mips: bmips: add bcm63xx-spi driver support for BCM6348

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Daniel Schwierzeck --- v5: no changes v4: no changes v3: rename BCM6338 SPI driver to BCM6348 v2: add spi alias arch/mips/dts/brcm,bcm6348.dtsi | 17 + 1 file chan

[U-Boot] [PATCH v4 3/4] mips: bmips: add bcm63xx-hsspi driver support for BCM63268

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the high speed SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass Reviewed-by: Daniel Schwierzeck --- v4: no changes v3: no changes v2: no changes arch/mips/dts/brcm,bcm63268.dtsi | 21 + 1 file changed

[U-Boot] [PATCH v4 2/4] mips: bmips: add bcm63xx-hsspi driver support for BCM6328

2017-07-30 Thread Álvaro Fernández Rojas
This driver manages the SPI controller present on this SoC. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass Reviewed-by: Daniel Schwierzeck --- v4: no changes v3: no changes v2: no changes arch/mips/dts/brcm,bcm6328.dtsi | 24 1 file changed, 24 inse

[U-Boot] [PATCH v4 0/4] mips: bmips: add HSSPI support

2017-07-30 Thread Álvaro Fernández Rojas
BCM63xx HSSPI controller has the same issue as BCM63xx SPI controller: it doesn't allow keeping CS active between transfers. However, this controller allows changing CS polarities, which is used in the linux upstream driver to manage CS as desired. v4: Sync with master. v3: Switch to CONFIG_BCM63X

[U-Boot] [PATCH v4 4/4] mips: bmips: enable the SPI flash on the Comtrend AR-5387un

2017-07-30 Thread Álvaro Fernández Rojas
It's a Macronix (mx25l12805d) 16 MB SPI flash. Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass Reviewed-by: Daniel Schwierzeck --- v4: Sync with master. v3: switch to CONFIG_BCM63XX_HSSPI v2: no changes arch/mips/dts/comtrend,ar-5387un.dts| 12 configs/comtr

[U-Boot] [PATCH v4 1/4] dm: spi: add BCM63xx HSSPI driver

2017-07-30 Thread Álvaro Fernández Rojas
This driver is a simplified version of linux/drivers/spi/spi-bcm63xx-hsspi.c Signed-off-by: Álvaro Fernández Rojas Reviewed-by: Simon Glass Reviewed-by: Daniel Schwierzeck --- v4: no changes. v3: Switch to CONFIG_BCM63XX_HSSPI and rebase on top of SPI v4. v2: Introduce changes suggested by S

Re: [U-Boot] [PATCH v3 2/2] MIPS: Stop building position independent code

2017-07-30 Thread Álvaro Fernández Rojas
I've been a bit busy lately and I couldn't test this until now, but I have to say that this commit breaks u-boot bmips support :( BTW, I tried this on several bmips boards (one of them is a Netgear CG3100D, which uses u-boot.bin instead of u-boot.elf) Do you have any idea on what could be hapen

[U-Boot] [PATCH v2 00/22] x86: kconfig clean up

2017-07-30 Thread Bin Meng
This series cleans up x86 Kconfig files a lot. The rules are: - Select all x86 must have Kconfig options from X86 in arch/Kconfig - Imply all DM uclass drivers from X86 in arch/Kconfig - Select platform-dependent initialization hooks from arch/x86/cpu//Kconfig - Imply platform-specific driver

[U-Boot] [PATCH v2 02/22] x86: kconfig: Select USE_PRIVATE_LIBGCC

2017-07-30 Thread Bin Meng
x86 is using the built-in libgcc implementation and this cannot be turned off. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/Kconfig| 1 + configs/bayleybay_defconfig | 1 - configs/chr

[U-Boot] [PATCH v2 03/22] x86: kconfig: Select PCI and DM_PCI

2017-07-30 Thread Bin Meng
PCI is the de facto interconnect bus in an x86 system. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/Kconfig| 2 ++ configs/bayleybay_defconfig | 1 - configs/chromebook_link64_defconfi

[U-Boot] [PATCH v2 01/22] x86: kconfig: Select TIMER and X86_TSC_TIMER

2017-07-30 Thread Bin Meng
Without a timer, U-Boot just doesn't boot. This is not something we can turn off. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: - sync defconfigs arch/Kconfig| 2 ++ configs/bayleybay_defconfig

[U-Boot] [PATCH v2 09/22] x86: kconfig: Select OF_CONTROL

2017-07-30 Thread Bin Meng
This is a must have for all x86 boards. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/Kconfig| 1 + configs/bayleybay_defconfig | 1 - configs/chromebook_link64_defconfig

[U-Boot] [PATCH v2 11/22] x86: baytrail: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
BayTrail integrates lots of peripherals that have U-Boot drivers. Imply those in the platform Kconfig. Signed-off-by: Bin Meng --- Changes in v2: None arch/x86/cpu/baytrail/Kconfig | 11 ++- configs/bayleybay_defconfig | 9

[U-Boot] [PATCH v2 06/22] x86: kconfig: Select ARCH_MISC_INIT in the platform Kconfig

2017-07-30 Thread Bin Meng
arch_misc_init() is intended to do architecture-dependent stuff. This is required by each platform. Signed-off-by: Bin Meng --- Changes in v2: None arch/x86/cpu/baytrail/Kconfig | 1 + arch/x86/cpu/coreboot/coreboot.c| 5 - arch/x86/cpu/efi

[U-Boot] [PATCH v2 16/22] x86: tangier: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
Imply Tangier-specific drivers in the platform Kconfig. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/x86/cpu/tangier/Kconfig | 7 +++ configs/edison_defconfig | 7 --- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/arch/x86/cpu/

[U-Boot] [PATCH v2 04/22] x86: kconfig: Imply HAVE_INTEL_ME in the platform Kconfig

2017-07-30 Thread Bin Meng
Intel Management Engine is required by the platform, however it's not a must have when building a U-Boot image. For example, during development normally programming ME firmware is a one-time effort. Signed-off-by: Bin Meng --- Changes in v2: None arch/x86/cpu/baytrail/Kconfig

[U-Boot] [PATCH v2 17/22] x86: quark: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
Imply Quark-specific drivers in the platform Kconfig. Signed-off-by: Bin Meng --- Changes in v2: - imply ENV_IS_IN_SPI_FLASH for quark arch/x86/cpu/quark/Kconfig | 9 + configs/galileo_defconfig | 9 - 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/arch/x86/cp

[U-Boot] [PATCH v2 07/22] x86: kconfig: Select ARCH_EARLY_INIT_R in the platform Kconfig

2017-07-30 Thread Bin Meng
This is architecture-dependent early initialization hence should be put in the platform Kconfig. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/x86/cpu/broadwell/Kconfig | 1 + arch/x86/cpu/broadwell/refcode.c

[U-Boot] [PATCH v2 19/22] x86: qemu: Remove SPI flash from defconfigs

2017-07-30 Thread Bin Meng
QEMU does not support ICH SPI controller yet. It's meaningless to include SPI flash support. Signed-off-by: Bin Meng --- Changes in v2: None configs/qemu-x86_64_defconfig| 5 - configs/qemu-x86_defconfig | 4 configs/qemu-x86_efi_payload32_defconfig | 4

[U-Boot] [PATCH v2 21/22] x86: kconfig: Move USB to platform Kconfig

2017-07-30 Thread Bin Meng
Like other peripheral drivers, move USB related drivers to platform Kconfig as well. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/Kconfig | 2 -- arch/x86/cpu/baytrail/Kconfig | 3 +++ arch/x86/cpu/broadwell/Kconfig | 2 ++ arch/x86/cpu/

[U-Boot] [PATCH v2 20/22] x86: kconfig: Let board select SPI flash

2017-07-30 Thread Bin Meng
Only a specific type of SPI flash exists on a board, having board Kconfig to select the SPI flash seems to make more sense. Other flash types are not necessary except coreboot, which implies all available flash drivers there. Signed-off-by: Bin Meng --- Changes in v2: None board/advantech/som-

[U-Boot] [PATCH v2 10/22] x86: kconfig: Imply DM uclass drivers

2017-07-30 Thread Bin Meng
Now that all x86 boards have been converted to use DM, we can imply these uclass drivers (DM_ETH, DM_RTC, DM_USB, DM_VIDEO) from the top level. Previously DM_GPIO, DM_KEYBOARD, DM_SERIAL, DM_SPI, DM_SPI_FLASH are selected. Change to use 'imply' to allow them to be removed. Note with this change,

[U-Boot] [PATCH v2 13/22] x86: broadwell: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
Imply Broadwell-specific drivers in the platform Kconfig. Signed-off-by: Bin Meng --- Changes in v2: - imply ENV_IS_IN_SPI_FLASH for broadwell arch/x86/cpu/broadwell/Kconfig | 6 ++ configs/chromebook_samus_defconfig | 6 -- 2 files changed, 6 insertions(+), 6 deletions(-) diff -

[U-Boot] [PATCH v2 22/22] x86: Convert INTEL_ICH6_GPIO to Kconfig

2017-07-30 Thread Bin Meng
This converts Intel ICH6 GPIO driver to Kconfig, and add it to the imply list of platform drivers. Signed-off-by: Bin Meng --- Changes in v2: - rebase on top of u-boot/master arch/x86/cpu/baytrail/Kconfig | 1 + arch/x86/cpu/ivybridge/Kconfig | 1 + arch/x86/cpu/quark/Kconfig | 1 + arch

[U-Boot] [PATCH v2 12/22] x86: ivybridge: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
Imply drivers that are working with Ivybridge platform in the platform Kconfig. Signed-off-by: Bin Meng --- Changes in v2: - imply ENV_IS_IN_SPI_FLASH for ivybridge arch/x86/cpu/ivybridge/Kconfig | 5 + configs/chromebook_link64_defconfig | 5 - configs/chromebook_link_defconfig

[U-Boot] [PATCH v2 14/22] x86: coreboot: kconfig: Imply drivers that are useful

2017-07-30 Thread Bin Meng
U-Boot as coreboot payload can run on any x86 hardware ideally. Let's imply some common drivers that are useful. Signed-off-by: Bin Meng --- Changes in v2: - imply more common drivers for coreboot arch/x86/cpu/coreboot/Kconfig | 10 ++ configs/coreboot-x86_defconfig | 5 - 2 fil

[U-Boot] [PATCH v2 08/22] x86: kconfig: Imply ENABLE_MRC_CACHE in the platform Kconfig

2017-07-30 Thread Bin Meng
Platform knows whether MRC cache is implemented, but using it can be a choice of a specific board. Signed-off-by: Bin Meng --- Changes in v2: None arch/x86/cpu/baytrail/Kconfig | 1 + arch/x86/cpu/broadwell/Kconfig | 1 + arch/x86/cpu/ivybridg

[U-Boot] [PATCH v2 05/22] x86: kconfig: Let board select BOARD_EARLY_INIT_F

2017-07-30 Thread Bin Meng
CONFIG_BOARD_EARLY_INIT_F, by its mean is provided by board-specific codes and should be not 'default y' for all x86 boards. Signed-off-by: Bin Meng Reviewed-by: Andy Shevchenko --- Changes in v2: None arch/x86/cpu/coreboot/coreboot.c | 5 - arch/x86/cpu/efi/efi.c

[U-Boot] [PATCH v2 18/22] x86: qemu: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
Imply Qemu-specific drivers in the platform Kconfig. Signed-off-by: Bin Meng --- Changes in v2: - imply ENV_IS_NOWHERE for qemu arch/x86/cpu/qemu/Kconfig| 4 configs/qemu-x86_64_defconfig| 4 configs/qemu-x86_defconfig | 4 configs/qemu

[U-Boot] [PATCH v2 15/22] x86: queensbay: kconfig: Imply platform specific drivers

2017-07-30 Thread Bin Meng
Imply drivers that work with Intel Queensbay platform. Signed-off-by: Bin Meng --- Changes in v2: - imply ENV_IS_IN_SPI_FLASH for queensbay arch/x86/cpu/queensbay/Kconfig | 11 +++ configs/crownbay_defconfig | 11 --- 2 files changed, 11 insertions(+), 11 deletions(-) dif

Re: [U-Boot] [PATCH 1/1] Add support Raspberry-Pi Ziro W

2017-07-30 Thread Rask Ingemann Lambertsen
On Thu, Jul 27, 2017 at 11:42:36PM +0400, Dmitry Korunov wrote: > Signed-off-by: Dmitry Korunov How about a summary of the board? CPU Memory Storage USB Wi-fi Ethernet etc. > --- > arch/arm/dts/bcm2835-rpi-zero-w.dts| 44 > ++ > arch/arm/dts/bcm283x-rpi-usb

Re: [U-Boot] [PATCH v3 2/2] MIPS: Stop building position independent code

2017-07-30 Thread Daniel Schwierzeck
Am 30.07.2017 um 14:27 schrieb Álvaro Fernández Rojas: > I've been a bit busy lately and I couldn't test this until now, but I have to > say that this commit breaks u-boot bmips support :( > > BTW, I tried this on several bmips boards (one of them is a Netgear CG3100D, > which uses u-boot.bin

[U-Boot] [PATCH 1/1] efi_loader: write protocol GUID in OpenProtocol

2017-07-30 Thread Heinrich Schuchardt
To understand what happens in OpenProtol it is necessary to know the protocol interface GUID. Let's write a debug message. Signed-off-by: Heinrich Schuchardt --- include/efi_loader.h | 14 ++ lib/efi_loader/efi_boottime.c | 2 ++ 2 files changed, 16 insertions(+) diff --gi

[U-Boot] [PATCH 1/1] efi_loader: debug messages for protocol interfaces

2017-07-30 Thread Heinrich Schuchardt
Write debug messages with the adresses of the protocol interfaces. This information is needed to understand the debug messages for efi_open_protocol and other functions. Signed-off-by: Heinrich Schuchardt --- cmd/bootefi.c| 5 + lib/efi_loader/efi_console.c | 4 lib/efi_

Re: [U-Boot] [PATCH v3 2/2] MIPS: Stop building position independent code

2017-07-30 Thread Álvaro Fernández Rojas
El 30/07/2017 a las 16:05, Daniel Schwierzeck escribió: > > > Am 30.07.2017 um 14:27 schrieb Álvaro Fernández Rojas: >> I've been a bit busy lately and I couldn't test this until now, but I have >> to say that this commit breaks u-boot bmips support :( >> >> BTW, I tried this on several bmips

Re: [U-Boot] [PATCH 06/17] omap4: Drop redundant CONFIG_OMAP4430 symbol

2017-07-30 Thread Paul Kocialkowski
Le vendredi 12 mai 2017 à 22:33 -0400, Tom Rini a écrit : > While there are a few different OMAP4 SoCs, today we always set > CONFIG_OMAP4430 and CONFIG_OMAP44XX. Convert the few test of > CONFIG_OMAP4430 to CONFIG_OMAP44XX. I don't have any omap4 device in hands these days, but I definitely agre

[U-Boot] [PATCH 1/1] api: remove superfluous assignment

2017-07-30 Thread Heinrich Schuchardt
No need to assign a value to sig if the next statement using sig is itself an assignment of a value to sig. Signed-off-by: Heinrich Schuchardt --- api/api.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/api/api.c b/api/api.c index c368511704..4fae95d280 100644 --- a/api/api

Re: [U-Boot] Please pull u-boot-x86

2017-07-30 Thread Tom Rini
On Sun, Jul 30, 2017 at 10:45:06AM +0800, Bin Meng wrote: > Hi Tom, > > The following changes since commit 19d1f1a2f3ccfbf85125150f7876ce22714b38bd: > > Merge git://git.denx.de/u-boot-socfpga (2017-07-29 11:44:08 -0400) > > are available in the git repository at: > > git://git.denx.de/u-bo

[U-Boot] [PATCH 1/1] arm: spear: do not return random value from stack

2017-07-30 Thread Heinrich Schuchardt
If the NOR device is not available do not return a random value from the stack. Signed-off-by: Heinrich Schuchardt --- arch/arm/cpu/arm926ejs/spear/spl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/cpu/arm926ejs/spear/spl.c b/arch/arm/cpu/arm926ejs/spear/spl.c i

[U-Boot] [PATCH 1/1] arm: bcm235xx: avoid possible NULL dereference

2017-07-30 Thread Heinrich Schuchardt
It does not make sense to first dereference c and then check if it is NULL. Signed-off-by: Heinrich Schuchardt --- arch/arm/cpu/armv7/bcm235xx/clk-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/cpu/armv7/bcm235xx/clk-core.c b/arch/arm/cpu/armv7/bcm235xx/clk-

[U-Boot] [PATCH 1/1] arm: bcm235xx: clk_set_rate avoid possible NULL deref

2017-07-30 Thread Heinrich Schuchardt
It does not make sense first to dereference c and then to check if it is NULL. Signed-off-by: Heinrich Schuchardt --- arch/arm/cpu/armv7/bcm235xx/clk-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/cpu/armv7/bcm235xx/clk-core.c b/arch/arm/cpu/armv7/bcm235xx/c

[U-Boot] [PATCH 1/1] bcm281xx: clock: avoid possible NULL dereference

2017-07-30 Thread Heinrich Schuchardt
It does not make sense first to dereference c and then to check if it is NULL. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- arch/arm/cpu/armv7/bcm281xx/clk-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/cpu/armv7/bcm281xx

[U-Boot] [PATCH 1/1] armv7m: mpu_config add missing break

2017-07-30 Thread Heinrich Schuchardt
For DEVICE_NON_SHARED the newly assigned value of attr is overwritten due to a missing break. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- arch/arm/cpu/armv7m/mpu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/cpu/armv7m/mpu.c b/arch/arm/cpu/armv

[U-Boot] [PATCH 1/1] arm: mvebu: avoid possible NULL dereference

2017-07-30 Thread Heinrich Schuchardt
It does not make sense to check if info is NULL after dereferencing it. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- arch/arm/mach-mvebu/serdes/axp/high_speed_env_lib.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/arm/mach-

[U-Boot] [PATCH 1/1] stm32: remove redundant 'else if'

2017-07-30 Thread Heinrich Schuchardt
The if in the else branch is superfluous. We can use a simple if. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- arch/arm/mach-stm32/stm32f1/flash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-stm32/stm32f1/flash.c b/arch/arm

[U-Boot] [RFC 1/1] m68k: mfc5227x: do not rely on uninitialized stack values

2017-07-30 Thread Heinrich Schuchardt
The behavior get_clocks depends on the unitialized value of bootmode. By setting it to zero we get a defined behavior and can get rid of superfluos coding. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- I have no hardware to test this. Please, check thorougly befo

[U-Boot] [PATCH 1/1] x86: ivybridge: remove unused uma_memory_size

2017-07-30 Thread Heinrich Schuchardt
The value of uma_memory_size depends on an undefined value from the stack. The value of uma_memory_size is changed but never used. So simply remove this superfluous code. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- arch/x86/cpu/ivybridge/sdram.c | 5 - 1 f

[U-Boot] [PATCH 1/1] ARM: hisilicon: hikey: do not rely on random stack value

2017-07-30 Thread Heinrich Schuchardt
If CONFIG_MMC_DW is not defined the return value of init_dwmmc should not rely on a random stack value. Instead indicate that no error occured. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- board/hisilicon/hikey/hikey.c | 2 +- 1 file changed, 1 insertion(+), 1

[U-Boot] [PATCH 1/1] tpm: add missing va_end

2017-07-30 Thread Heinrich Schuchardt
va_start must always be matched by va_end. Signed-off-by: Heinrich Schuchardt --- lib/tpm.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/tpm.c b/lib/tpm.c index fb520e3d73..d1cf5a8a16 100644 --- a/lib/tpm.c +++ b/lib/tpm.c @@ -95,8 +95,10 @@ int pack_byte_string(ui

[U-Boot] [PATCH 1/1] arm: mvebu: remove self assignment

2017-07-30 Thread Heinrich Schuchardt
Assigning dev_num to itself is superfluous. Signed-off-by: Heinrich Schuchardt --- drivers/ddr/marvell/a38x/ddr3_training_centralization.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/ddr/marvell/a38x/ddr3_training_centralization.c b/drivers/ddr/marvell/a38x/ddr3_training_centra

[U-Boot] [PATCH 1/1] arm: mvebu: ddr3_debug: remove self assignments

2017-07-30 Thread Heinrich Schuchardt
Remove superfluous self assignements. Signed-off-by: Heinrich Schuchardt --- drivers/ddr/marvell/a38x/ddr3_debug.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/ddr/marvell/a38x/ddr3_debug.c b/drivers/ddr/marvell/a38x/ddr3_debug.c index 12b5b04109..a704a3e9d3 100644 --- a/

[U-Boot] [PATCH 1/1] lcd: avoid possible NULL dereference

2017-07-30 Thread Heinrich Schuchardt
Do not dereference bmp before the check if it is NULL. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- common/lcd.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/common/lcd.c b/common/lcd.c index 7e399cee2d..35914960a5 100644 --- a/common/lc

[U-Boot] [PATCH 1/1] ARM64: zynqmp: avoid out of buffer access

2017-07-30 Thread Heinrich Schuchardt
strncat(a, b, c) appends a maximum of c characters plus the 0 byte to a. In board_init we first write 4 characters plus 0 byte to version. So only ZYNQMP_VERSION_SIZE - 5 additional characters fit into version. The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- board

Re: [U-Boot] [PATCH 1/1] x86: ivybridge: remove unused uma_memory_size

2017-07-30 Thread Bin Meng
On Mon, Jul 31, 2017 at 3:17 AM, Heinrich Schuchardt wrote: > The value of uma_memory_size depends on an undefined value > from the stack. The value of uma_memory_size is changed but > never used. > > So simply remove this superfluous code. > > The problem was indicated by cppcheck. > > Signed-off

Re: [U-Boot] [PATCH 3/3] usb: configs: Clean up CONFIG_SYS_USB_EVENT_POLL_(xxx) in board configs

2017-07-30 Thread Bin Meng
On Mon, Jul 31, 2017 at 5:01 AM, Rask Ingemann Lambertsen wrote: > On Mon, Jun 26, 2017 at 04:05:41AM -0700, Bin Meng wrote: >> The following 3 Kconfig options >> - CONFIG_SYS_USB_EVENT_POLL >> - CONFIG_SYS_USB_EVENT_POLL_VIA_CONTROL_EP >> - CONFIG_SYS_USB_EVENT_POLL_VIA_INT_QUEUE >> >> are alread

Re: [U-Boot] [PATCH 3/3] usb: configs: Clean up CONFIG_SYS_USB_EVENT_POLL_(xxx) in board configs

2017-07-30 Thread Rask Ingemann Lambertsen
On Mon, Jun 26, 2017 at 04:05:41AM -0700, Bin Meng wrote: > The following 3 Kconfig options > - CONFIG_SYS_USB_EVENT_POLL > - CONFIG_SYS_USB_EVENT_POLL_VIA_CONTROL_EP > - CONFIG_SYS_USB_EVENT_POLL_VIA_INT_QUEUE > > are already converted to Kconfig. Let's clean up all board configs to > switch over

[U-Boot] [PATCH] arm: omap: Fix 'get_device_type()' for OMAP34XX

2017-07-30 Thread Derald D. Woods
Fixes 00bbe96ebabb (arm: omap: Unify get_device_type() function) where the control status register value is embedded in a structure somewhere in SRAM. This patch allows OMAP3 EVM (TMDSEVM3530) to boot again using a known global constant and offset for 'readl'. --- arch/arm/mach-omap2/sysinfo-commo

[U-Boot] [PATCH] rockchip: rk3288: fix EMMC_DIV_MASK definition in header

2017-07-30 Thread Kever Yang
It shoulb be '<<' instead of '<' for _MASK definition, fix it. Signed-off-by: Ziyuan Xu Signed-off-by: Kever Yang --- arch/arm/include/asm/arch-rockchip/cru_rk3288.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3288.h b/arch/arm/

[U-Boot] [PATCH] i2c: at91: Add missing probe function to device driver

2017-07-30 Thread Wenyou Yang
Add missing probe function to the device driver to active a device. Signed-off-by: Wenyou Yang --- drivers/i2c/at91_i2c.c | 26 -- 1 file changed, 24 insertions(+), 2 deletions(-) diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c index b7298cf774..d394044f80

[U-Boot] [PATCH v2 0/4] x86: Convert all boards to DM MMC and SCSI

2017-07-30 Thread Bin Meng
This series converts all x86 boards to use DM MMC and SCSI. It is based on previous x86 kconfig clean up series, and is available at u-boot-x86/blk-working for testing. Changes in v2: - Fix unmet direct dependencies on BLK Bin Meng (3): dm: scsi: Add a generic PCI-based AHCI driver block: id

[U-Boot] [PATCH v2 1/4] dm: scsi: Add a generic PCI-based AHCI driver

2017-07-30 Thread Bin Meng
This adds support for PCI-based AHCI controller based on DM SCSI. Signed-off-by: Bin Meng Reviewed-by: Simon Glass Tested-by: Simon Glass --- Changes in v2: - Fix unmet direct dependencies on BLK drivers/ata/Kconfig| 6 ++ drivers/ata/Makefile | 1 + drivers/ata/ahci-pci.c | 42

[U-Boot] [PATCH v2 2/4] block: ide: Fix build error when CONFIG_BLK is on

2017-07-30 Thread Bin Meng
Add missing #ifndef CONFIG_BLK to wrap dev_desc->block_read. Signed-off-by: Bin Meng --- Changes in v2: None drivers/block/ide.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/block/ide.c b/drivers/block/ide.c index 308ad73..edcf87b 100644 --- a/drivers/block/ide.c +++ b/drivers

[U-Boot] [PATCH v2 4/4] x86: Switch all boards to use DM SCSI

2017-07-30 Thread Bin Meng
After MMC is converted to DM, convert to use DM SCSI as well for all x86 boards and imply BLK for both MMC and SCSI drivers. CONFIG_SCSI_DEV_LIST is no longer used. Clean them up. Signed-off-by: Bin Meng --- Changes in v2: None arch/Kconfig | 2 ++ arch/x86/cpu/

[U-Boot] [PATCH v2 3/4] x86: Convert MMC to driver model

2017-07-30 Thread Bin Meng
From: Simon Glass Convert the pci_mmc driver over to driver model and migrate all x86 boards that use it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng [bmeng: remove DM_MMC from edison_defconfig] Signed-off-by: Bin Meng --- Changes in v2: None arch/Kconfig

Re: [U-Boot] [PATCH 3/3] usb: configs: Clean up CONFIG_SYS_USB_EVENT_POLL_(xxx) in board configs

2017-07-30 Thread Bin Meng
On Mon, Jul 31, 2017 at 7:03 AM, Marek Vasut wrote: > On 07/31/2017 12:30 AM, Bin Meng wrote: >> On Mon, Jul 31, 2017 at 5:01 AM, Rask Ingemann Lambertsen >> wrote: >>> On Mon, Jun 26, 2017 at 04:05:41AM -0700, Bin Meng wrote: The following 3 Kconfig options - CONFIG_SYS_USB_EVENT_POLL

Re: [U-Boot] [PATCH 3/3] usb: configs: Clean up CONFIG_SYS_USB_EVENT_POLL_(xxx) in board configs

2017-07-30 Thread Marek Vasut
On 07/31/2017 12:30 AM, Bin Meng wrote: > On Mon, Jul 31, 2017 at 5:01 AM, Rask Ingemann Lambertsen > wrote: >> On Mon, Jun 26, 2017 at 04:05:41AM -0700, Bin Meng wrote: >>> The following 3 Kconfig options >>> - CONFIG_SYS_USB_EVENT_POLL >>> - CONFIG_SYS_USB_EVENT_POLL_VIA_CONTROL_EP >>> - CONFIG_

Re: [U-Boot] [PATCH 00/39] dm: sata: Complete driver-model support for SATA

2017-07-30 Thread Jaehoon Chung
Hi Simon, On 07/28/2017 11:40 PM, Simon Glass wrote: > Hi Jaehoon, > > On 28 July 2017 at 06:25, Jaehoon Chung wrote: >> Dear Simon, >> >> On 07/27/2017 11:43 AM, Jaehoon Chung wrote: >>> On 07/23/2017 02:15 AM, Simon Glass wrote: (pruning the cc list a bit) Hi Jaehoon,

[U-Boot] [GIT PULL] Please pull u-boot-mmc master

2017-07-30 Thread Jaehoon Chung
Dear Tom, Could you pull these patches into u-boot/master? The following changes since commit 55f228b07e2d84938a88737445441e736de41340: x86: minnowmax: Remove CONFIG_SMSC_LPC47M (2017-07-30 10:30:25 +0800) are available in the git repository at: git://git.denx.de/u-boot-mmc.git master for

[U-Boot] [PATCH 0/2] misc: i2c_eeprom: Add compatibles

2017-07-30 Thread Wenyou Yang
Add the compatibles to support the I2C EEPROM 24AA02E48 and AT24MAC402 respectively. - microchip,24aa02e48 - atmel,24mac402 Wenyou Yang (2): misc: i2c_eeprom: Add compatible for 24AA02E48 misc: i2c_eeprom: Add compatible for AT24MAC402 drivers/misc/i2c_eeprom.c | 2 ++ 1 file changed, 2 i

[U-Boot] [PATCH 2/2] misc: i2c_eeprom: Add compatible for AT24MAC402

2017-07-30 Thread Wenyou Yang
Add the new compatible "atmel,24mac402" to accommodate AT24MAC402. The AT24MAC402 is a 2K Serial EEPROM and the 2-Kbit memory array is internally organized as 16 pages of 16 bytes of EEPROM each. The 48-bit EUI address in the AT24MAC402 are located in the extended memory block. Signed-off-by: Weny

[U-Boot] [PATCH 1/2] misc: i2c_eeprom: Add compatible for 24AA02E48

2017-07-30 Thread Wenyou Yang
Add the new compatible "microchip,24aa02e48" to accommodate 24AA02E48, the 24AA02E48 is a 2K I2C Serial EEPROM with pre-programmed globally unique, 48-bit node address, and 8-byte page size. Signed-off-by: Wenyou Yang --- drivers/misc/i2c_eeprom.c | 1 + 1 file changed, 1 insertion(+) diff --g

[U-Boot] [PATCH v2 0/3] board: atmel: Set the ethernet mac address from eeprom

2017-07-30 Thread Wenyou Yang
Create board/$(VENDOR)/common folder to accommodate the common code for the boards from atmel. Now put the code to set the ethernet mac address here, using the function to set the ethernet mac address on sama5d2 and sama5d4 Xplained boards. Changes in v2: - Add a new patch to create board/$(VENDO

[U-Boot] [PATCH v2 1/3] board: atmel: Create board/$(VENDOR)/common folder

2017-07-30 Thread Wenyou Yang
Create board/$(VENDOR)/common folder to accommodate the common code shared by other atmel boards, now put the code to set ethernet mac address from eeprom, which uses the i2c eeprom driver. Signed-off-by: Wenyou Yang --- Changes in v2: None arch/arm/mach-at91/include/mach/at91_common.h | 2 ++

[U-Boot] [PATCH v2 3/3] board: sama5d4_xplained: Set mac address from eeprom

2017-07-30 Thread Wenyou Yang
Add the code to set the ethernet mac address from eeprom by using the common code from the common folder. Signed-off-by: Wenyou Yang --- Changes in v2: - Add a new patch to create board/$(VENDOR)/common folder to accommodate the common code and put the code to set the ethernet mac address

[U-Boot] [PATCH v2 2/3] board: sama5d2_xplained: Replace code to set mac address

2017-07-30 Thread Wenyou Yang
Replace the code to set the ethernet mac address with the code from the common folder. Signed-off-by: Wenyou Yang --- Changes in v2: None arch/arm/dts/at91-sama5d2_xplained.dts | 5 +++ board/atmel/sama5d2_xplained/sama5d2_xplained.c | 47 +++-- configs/sama5d2_xp

Re: [U-Boot] [PATCH 2/6] ARM: at91: spl: Add mck function to lower rate while switching

2017-07-30 Thread Yang, Wenyou
Hi Simon, On 2017/7/28 12:20, Simon Glass wrote: Hi Wenyou, On 20 July 2017 at 02:12, Wenyou Yang wrote: Refer to the commit 70f8c8316ad(PMC: add new mck function to lower rate while switching) from AT91Bootstrap. While switching to a lower clock source, we must switch the clock source firs

Re: [U-Boot] [PATCH 6/6] board: atmel: Add SAMA5D27 SOM1 EK board

2017-07-30 Thread Yang, Wenyou
Hi Simon, On 2017/7/28 12:20, Simon Glass wrote: Hi Wenyou, On 20 July 2017 at 02:13, Wenyou Yang wrote: From: Wenyou Yang The SAMA5D27-SiP (System in Package) integrates the SAMA5D2 with 1Gbit DDR2-SDRAM in a single package. The SAMA5D27 SOM1 embeds a 64Mbit QSPI flash, KSZ8081 Phy and M

[U-Boot] [PATCH 0/7] board: atmel: Convert to support DM_VIDEO

2017-07-30 Thread Wenyou Yang
Add a common code to display the company's logo and board information by using the API from DM_VIDEO under the board/$(VENDOR)/common folder, and the logo files can be used by the API from DM_VIDEO, and then convert the board to enable the driver model support for LCD/video. It is based on the pat

[U-Boot] [PATCH 3/7] board: sama5d2_xplained: Convert to support DM_VIDEO

2017-07-30 Thread Wenyou Yang
Convert the board to enable the driver model support for LCD/video. Signed-off-by: Wenyou Yang --- arch/arm/dts/at91-sama5d2_xplained.dts | 60 arch/arm/dts/sama5d2.dtsi | 7 ++ arch/arm/mach-at91/Kconfig | 1 + board/atmel/

[U-Boot] [PATCH 5/7] board: sama5d4ek: Convert to support DM_VIDEO

2017-07-30 Thread Wenyou Yang
From: Wenyou Yang Convert the board to enable the driver model support for LCD/video. Signed-off-by: Wenyou Yang --- arch/arm/dts/at91-sama5d4ek.dts | 26 ++ arch/arm/mach-at91/Kconfig| 1 + board/atmel/sama5d4ek/sama5d4ek.c | 97 +++-

[U-Boot] [PATCH 2/7] atmel: common: Add function to display via DM_VIDEO's API

2017-07-30 Thread Wenyou Yang
Add a function to display the company's logo and board information by using the API from DM_VIDEO. This function can be shared by other atmel boards, so locate it in common folder. Signed-off-by: Wenyou Yang --- arch/arm/mach-at91/include/mach/at91_common.h | 1 + board/atmel/common/Makefile

[U-Boot] [PATCH 6/7] board: sama5d4_xplained: Convert to support DM_VIDEO

2017-07-30 Thread Wenyou Yang
From: Wenyou Yang Convert the board to enable the driver model support for LCD/video. Signed-off-by: Wenyou Yang --- arch/arm/dts/at91-sama5d4_xplained.dts | 25 ++ arch/arm/dts/sama5d4.dtsi | 21 + arch/arm/mach-at91/Kconfig | 1

[U-Boot] [PATCH 4/7] board: sama5d3xek: Convert to support DM_VIDEO

2017-07-30 Thread Wenyou Yang
From: Wenyou Yang Convert the board to enable the driver model support for LCD/video. Signed-off-by: Wenyou Yang --- arch/arm/dts/sama5d36ek_cmp.dts| 1 + arch/arm/dts/sama5d3_lcd.dtsi | 21 +-- arch/arm/dts/sama5d3xdm.dtsi | 26 + board/atm

[U-Boot] [PATCH 7/7] board: at91sam9x5ek: Convert to support DM_VIDEO

2017-07-30 Thread Wenyou Yang
Convert the board to enable the driver model support for LCD/video. Signed-off-by: Wenyou Yang --- arch/arm/mach-at91/Kconfig | 1 + board/atmel/at91sam9x5ek/at91sam9x5ek.c | 111 ++- configs/at91sam9x5ek_dataflash_defconfig | 3 +- configs/at91sam