Re: [U-Boot] [U-Boot, v3, 1/3] rockchip: make boot_mode related codes reused across all platforms

2017-11-21 Thread Dr. Philipp Tomsich
Andy, > On 21 Nov 2017, at 01:48, Andy Yan wrote: > > Hi Philipp: > > > On 2017年11月20日 22:50, Philipp Tomsich wrote: >> >> >> On Wed, 11 Oct 2017, Andy Yan wrote: >> >>> setup_boot_mode function use the same logic but different >>> mode register address across all the rockchip platforms, >>

Re: [U-Boot] [PATCH] arm: zynq: Remove double const qualifier

2017-11-21 Thread Alex Sadovsky
Hi Michal, I'm sorry to disturb you and others but it looks like you've removed a bit too much. There are two different types of usage of the "const" modifier in the areas that are touched by your patch: 1) for function prototypes/implementation - "const" before the function name is senseless, fun

Re: [U-Boot] [PATCH v5 1/2] sun50i: h5: Add NanoPi Neo Plus2 DT initial support

2017-11-21 Thread Antony Antony
On Tue, Nov 21, 2017 at 11:59:42AM +0530, Jagan Teki wrote: > On Fri, Nov 17, 2017 at 3:09 PM, Antony Antony wrote: > > Add initial DT for NanoPi NEO Plus2 by FriendlyARM > > - Allwinner quad core H5 Cortex A53 with an ARM Mali-450MP GPU > > - 1 GB DDR3 RAM > > - 8GB eMMC flash (Samsung KLM8G1WEPD

[U-Boot] [PATCH v6 1/2] arm64: sun50i-h5.dtsi : update to kernel 4.14

2017-11-21 Thread Antony Antony
last Kernel commit of sun50i-h5.dtsi d86e63e1f0b7 Signed-off-by: Antony Antony --- Changes v1->v2 removed memory {}, compatible ="allwinner,sun5i-a13-mmc" remove CONFIG_SPL_SPI_FLASH_SUPPORT=y and CONFIG_SPL_SPI_SUNXI=y (no SPI) remove CONFIG_SD_BOOT=y Changes v2->v3 removed compatible ="allw

[U-Boot] [PATCH v6 2/2] sun50i: h5: Add NanoPi Neo Plus2 DT initial support

2017-11-21 Thread Antony Antony
Add initial DT for NanoPi NEO Plus2 by FriendlyARM - Allwinner quad core H5 Cortex A53 with an ARM Mali-450MP GPU - 1 GB DDR3 RAM - 8GB eMMC flash (Samsung KLM8G1WEPD-B031) - micro SD card slot - Gigabit Ethernet (external RTL8211E-VB-CG chip) - 802.11 b/g/n WiFi, Bluetooth 4.0 (Ampak AP6212A modul

Re: [U-Boot] [PATCH 1/1] ubifs: no NULL check needed before free

2017-11-21 Thread Alex Sadovsky
Hi Heinrich and Heiko, do you see anything strange in this code (it was more obvious before the patch but it can still be spotted)? I should say that it's too late to check for "file != NULL" after the "file->private_data" dereference. Maybe it should look like this: if (file)

Re: [U-Boot] [PATCH] imx: Unify CONFIG_BOOTDELAY

2017-11-21 Thread Lukasz Majewski
Hi Fabio, > In order to provide a consistent user experience for imx board users, > remove the custom CONFIG_BOOTDELAY values from defconfig files, so > that all boards can use the default two second delay. Reviewed-by: Lukasz Majewski > > Signed-off-by: Fabio Estevam > --- > configs/mx23evk

Re: [U-Boot] [PATCH v2 1/1] x86: don't compare pointers to 0

2017-11-21 Thread Bin Meng
On Tue, Nov 21, 2017 at 2:11 PM, Bin Meng wrote: > On Tue, Nov 21, 2017 at 2:45 AM, Heinrich Schuchardt > wrote: >> x86_vendor_name is defined as >> >> static const char *const x86_vendor_name[] >> >> So its elements should not be compared to 0. >> >> Remove superfluous paranthesis. >> >> Proble

[U-Boot] U-Boot proper(not SPL) relocate option

2017-11-21 Thread Kever Yang
Hi Guys, I try to understand why we need to do the relocate in U-Boot. From the document README/crt0.S, I think the relocation feature comes from some SoC have limited SRAM whose size is enough to load the whole U-Boot, but not enough to run all the drivers. I don't know how many SoCs/Ar

Re: [U-Boot] [U-Boot,V9,1/4] usb: rockchip: add the rockusb gadget

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 16:54:51 +0100 "Dr. Philipp Tomsich" wrote: > Kever & Eddie, > > > On 2 Oct 2017, at 12:51, Dr. Philipp Tomsich > > wrote: > > > > +Kever, +Eddie > > > > Kever & Eddie, > > > >> On 2 Oct 2017, at 12:41, Marek Vasut wrote: > >> > >> On 10/02/2017 12:40 PM, Dr. Philipp

[U-Boot] U-Boot proper(not SPL) relocate option

2017-11-21 Thread Kever Yang
Hi Guys, I try to understand why we need to do the relocate in U-Boot. From the document README/crt0.S, I think the relocation feature comes from some SoC have limited SRAM whose size is enough to load the whole U-Boot, but not enough to run all the drivers. I don't know how many SoCs/Ar

Re: [U-Boot] [PATCH v3 04/14] Move debug and logging support to a separate header

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:25 -0700 Simon Glass wrote: > Before adding new features, move these definitions to a separate > header to avoid further cluttering common.h. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng Reviewed-by: Lukasz Majewski Best regards, Lukasz Majewski -- DEN

Re: [U-Boot] [PATCH v3 07/14] log: Add an implemention of logging

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:28 -0700 Simon Glass wrote: > Add the logging header file and implementation with some configuration > options to control it. Despite one question - Reviewed-by: Lukasz Majewski > > Signed-off-by: Simon Glass > --- > > Changes in v3: None > Changes in v2: > - Add

Re: [U-Boot] [PATCH v3 08/14] log: Add a console driver

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:29 -0700 Simon Glass wrote: > It is useful to display log messages on the console. Add a simple > driver to handle this. > > Note that this driver outputs to the console, which may be serial or > video. It does not specifically select serial output. > Reviewed-by: Luk

Re: [U-Boot] [PATCH v3 09/14] log: Add a 'log level' command

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:30 -0700 Simon Glass wrote: > Add a command for adjusting the log level. > Reviewed-by: Lukasz Majewski > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v3: None > Changes in v2: None > > cmd/Kconfig | 7 +++ > cmd/Makefile | 1 +

Re: [U-Boot] [PATCH v3 10/14] log: Add a test command

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:31 -0700 Simon Glass wrote: > Add a command which exercises the logging system. > Reviewed-by: Lukasz Majewski > Signed-off-by: Simon Glass > --- > > Changes in v3: None > Changes in v2: > - Fix function called when test command is selected > - Fix help output for

Re: [U-Boot] [PATCH v3 11/14] log: Plumb logging into the init sequence

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:32 -0700 Simon Glass wrote: > Set up logging both before and after relocation. > Reviewed-by: Lukasz Majewski > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v3: None > Changes in v2: None > > common/board_f.c | 5 -

Re: [U-Boot] [PATCH v3 12/14] log: sandbox: Enable logging

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:33 -0700 Simon Glass wrote: > Enable all logging features on sandbox so that the tests can be run. > Reviewed-by: Lukasz Majewski > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v3: None > Changes in v2: > - Change sandbox log level to 6

Re: [U-Boot] [PATCH v3 13/14] log: test: Add a pytest for logging

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:34 -0700 Simon Glass wrote: > Add a test which tries out various filters and options to make sure > that logging works as expected. > Reviewed-by: Lukasz Majewski > Signed-off-by: Simon Glass > --- > > Changes in v3: None > Changes in v2: > - Change log levels to m

Re: [U-Boot] [PATCH v3 14/14] log: Add documentation

2017-11-21 Thread Lukasz Majewski
On Mon, 20 Nov 2017 15:33:35 -0700 Simon Glass wrote: > Add documentation for the log system. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v3: > - Rebase to master > > Changes in v2: > - Drop the special log() functions from the README > > doc/README.log | 21

[U-Boot] re-use dm data for pre-relocate and post-relocate

2017-11-21 Thread Kever Yang
Hi Simon, In order to make people easier to maintain a U-Boot for product, we want to re-use the dtb from kernel. Some peripheral like power/pmic and display panel, key, charger which need work in U-Boot and may different in different product, we want to make people only maintain kernel dt

Re: [U-Boot] U-Boot proper(not SPL) relocate option

2017-11-21 Thread Lukasz Majewski
Hi Kever, > Hi Guys, > > I try to understand why we need to do the relocate in U-Boot. > From the document README/crt0.S, I think the relocation feature comes > from some SoC have limited SRAM whose size is enough to load the whole > U-Boot, but not enough to run all the drivers. > >

Re: [U-Boot] [PATCH] mtd: cfi: Fix checking status register feature

2017-11-21 Thread Marek Vasut
On 11/21/2017 03:52 AM, York Sun wrote: > On 11/18/2017 11:09 AM, York Sun wrote: >> Commit 72443c7f7d21 ("mtd: cfi: Add support for status register >> polling") added a feature check to determine if status register >> is available for certain flash chips. The "lower software bits" >> register used

Re: [U-Boot] [RFC PATCH] usb: ehci: do not invalidate a NULL buffer

2017-11-21 Thread Dirk Behme
On 17.11.2017 16:04, Marek Vasut wrote: On 11/17/2017 03:28 PM, Dirk Behme wrote: Its a valid use case to call ehci_submit_async() with a NULL buffer with length 0. E.g. from usb_set_configuration(). As invalidate_dcache_range() isn't able to judge if the address NULL is valid or not (depending

Re: [U-Boot] [PATCH 1/1] ubifs: no NULL check needed before free

2017-11-21 Thread Heinrich Schuchardt
On 11/21/2017 10:20 AM, Alex Sadovsky wrote: Hi Heinrich and Heiko, do you see anything strange in this code (it was more obvious before the patch but it can still be spotted)? I should say that it's too late to check for "file != NULL" after the "file->private_data" dereference. Maybe it shou

Re: [U-Boot] [PATCH 04/11] tools: mkimage: Extend mkimage to also include pmufw

2017-11-21 Thread Michal Simek
On 20.11.2017 16:38, Simon Glass wrote: > Hi Michal, > > On 6 November 2017 at 05:02, Michal Simek wrote: >> This is generated command line when PMUFW_INIT_FILE is setup. >> >> ./tools/mkimage -T zynqmpimage -R ./"" -n >> ./"board/xilinx/zynqmp/pmufw.bin" -d spl/u-boot-spl.bin spl/boot.bin >> >>

Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-11-21 Thread Jonathan Gray
On Tue, Nov 21, 2017 at 04:59:33PM +1100, Jonathan Gray wrote: > On Sat, Nov 18, 2017 at 03:25:29PM +1100, Jonathan Gray wrote: > > > > While U-Boot 2017.11 release works with vexpress/qemu with the > > efi loader it is broken on at least rpi_3 and tinker-rk3288. > > MEDIA_DEVICE is not found again

[U-Boot] [PATCH v2 07/11] arm64: zynqmp: Use u32 type instead of uint32_t

2017-11-21 Thread Michal Simek
Warning is reported by checkpatch. Signed-off-by: Michal Simek --- Changes in v2: None board/xilinx/zynqmp/zynqmp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c index a036ed6a35d3..5e22cc54ba69 100644 --- a/

[U-Boot] [PATCH v2 05/11] arm64: zynqmp: Enable config DEFINE_TCM_OCM_MMAP if CONFIG_MP defined

2017-11-21 Thread Michal Simek
From: Siva Durga Prasad Paladugu This modifies default value of config DEFINE_TCM_OCM_MMAP to yes if CONFIG_MP is defined MP supports needs OCM and TCM part of memory map. Signed-off-by: Siva Durga Prasad Paladugu Signed-off-by: Michal Simek --- Changes in v2: None arch/arm/cpu/armv8/zynqmp

[U-Boot] [PATCH v2 04/11] tools: mkimage: Extend mkimage to also include pmufw

2017-11-21 Thread Michal Simek
The patch is adding external pmufw "Platform Management Unit firmware" to boot.bin image. Boot.bin is a Xilinx format which bootrom is capable to read and boot the system. pmufw is copied to the header data section follows by u-boot-spl.bin. pmufw is consumed by PMU unit (Microblaze) and SPL runs o

[U-Boot] [PATCH v2 01/11] arm64: zynqmp: Remove slcr with mio status pin detection

2017-11-21 Thread Michal Simek
This code is not used on this platform and it is not called. Signed-off-by: Michal Simek --- Changes in v2: None arch/arm/cpu/armv8/zynqmp/Makefile | 1 - arch/arm/cpu/armv8/zynqmp/slcr.c | 63 arch/arm/include/asm/arch-zynqmp/sys_proto.h |

[U-Boot] [PATCH v2 03/11] arm64: zynqmp: Provide a Kconfig option to use specified memory for MMU table

2017-11-21 Thread Michal Simek
From: Siva Durga Prasad Paladugu This patch provides a Kconfig option to use specified memory for MMU table using reserve_mmu platform specific routine. Here we used TCM space for MMU table. Signed-off-by: Siva Durga Prasad Paladugu Signed-off-by: Michal Simek --- Changes in v2: None arch/a

[U-Boot] [PATCH v2 09/11] arm64: zynqmp: Add new ID for RFSoC

2017-11-21 Thread Michal Simek
This ID is available on zc1254. Signed-off-by: Michal Simek --- Changes in v2: None board/xilinx/zynqmp/zynqmp.c | 24 1 file changed, 24 insertions(+) diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c index 2b1d8119f567..f769db7e81b9 100644 ---

[U-Boot] [PATCH v2 08/11] arm64: zynqmp: Add support for CG/EG/EV device detection

2017-11-21 Thread Michal Simek
Version string has unused fields 31:20 which can be used for exporting 9 bits from efuse IPDISABLE regs to recognize eg/cg/ev devices. These efuse bits are setup for certain devices. Signed-off-by: Michal Simek --- Changes in v2: None arch/arm/include/asm/arch-zynqmp/hardware.h | 2 + arch/

[U-Boot] [PATCH v2 06/11] arm64: zynqmp: Add SD1 level shifter mode to alternative selection

2017-11-21 Thread Michal Simek
Extend Kconfig to cover SD1 level shifter mode. Reported-by: Jason Wu Signed-off-by: Michal Simek --- Changes in v2: None arch/arm/cpu/armv8/zynqmp/Kconfig | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/cpu/armv8/zynqmp/Kconfig b/arch/arm/cpu/armv8/zynqmp/Kconfig index fb12

[U-Boot] [PATCH v2 02/11] arm64: zynqmp: mp: Correct the R5 release sequence

2017-11-21 Thread Michal Simek
From: Siva Durga Prasad Paladugu This patch corrects the R5 release sequence by adding the below steps. 1. Flush dcache to ensure that image loaded into memory. 2. Keep R5 reset just to ensure R5 in reset. 3. Disable caches before accessing TCM as with out this A53 can do speculative and may r

[U-Boot] [PATCH v2 11/11] arm64: zynqmp: Wire QSPI boot mode for SPL

2017-11-21 Thread Michal Simek
From: Michal Simek ZynqMP qspi driver is on the way to mainline Signed-off-by: Michal Simek --- Changes in v2: None arch/arm/cpu/armv8/zynqmp/spl.c | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/cpu/armv8/zynqmp/spl.c b/arch/arm/cpu/armv8/zynqmp/spl.c index 468dc1dc4d54..4

[U-Boot] [PATCH v2 10/11] arm64: zynqmp: Enable debug uart for zc1751 dc5

2017-11-21 Thread Michal Simek
Showing uart earlier. Signed-off-by: Michal Simek --- Changes in v2: None configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig | 5 + 1 file changed, 5 insertions(+) diff --git a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig index c854ba7

[U-Boot] [PATCH] efi_loader: initialise partition_signature memory

2017-11-21 Thread Jonathan Gray
Zero partition_signature in the efi_device_path_hard_drive_path structure when signature_type is zero. This is required so that efi_dp_match() will work as expected when doing memcmp() comparisons. Corrects a problem where the loaded image protocol would not return a device path with MEDIA_DEVICE

[U-Boot] [PATCH] spl: make CONFIG_OF_EMBED pass dts through fdtgrep

2017-11-21 Thread Goldschmidt Simon
Building spl with CONFIG_OF_EMBED enabled results in an error message on my board: "SPL image too big". This is because the fdtgrep build step is only executed for CONFIG_OF_SEPARATE. Fix this by moving the fdtgrep build step ('cmd_fdtgreo') from scripts/Makefile.spl to dts/Makefile so that the re

Re: [U-Boot] [U-Boot, v3, 3/3] rockchip: check download key before bootup

2017-11-21 Thread Dr. Philipp Tomsich
Andy, > On 21 Nov 2017, at 02:02, Andy Yan wrote: > > Hi Philipp: > > > On 2017年11月20日 22:51, Philipp Tomsich wrote: >> >> >> On Wed, 11 Oct 2017, Andy Yan wrote: >> >>> Enter download mode if the download key pressed. >>> >>> Signed-off-by: Andy Yan >>> Acked-by: Philipp Tomsich >> >>

Re: [U-Boot] [U-Boot, v5, 1/3] thermal: ti-bandgap: Add support for temperature sensor

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 04:12:31PM +0530, Faiz Abbas wrote: > The dra7xx series of SOCs contain a temperature sensor and an > associated analog-to-digital converter (ADC) which produces > an output which is proportional to the SOC temperature. > Add support for this temperature sensor. > > Signed-

Re: [U-Boot] [U-Boot, v5, 1/3] thermal: ti-bandgap: Add support for temperature sensor

2017-11-21 Thread Lokesh Vutla
On Tuesday 21 November 2017 06:14 PM, Tom Rini wrote: > On Tue, Nov 14, 2017 at 04:12:31PM +0530, Faiz Abbas wrote: >> The dra7xx series of SOCs contain a temperature sensor and an >> associated analog-to-digital converter (ADC) which produces >> an output which is proportional to the SOC tempera

Re: [U-Boot] [U-Boot, v5, 1/3] thermal: ti-bandgap: Add support for temperature sensor

2017-11-21 Thread Tom Rini
On Tue, Nov 21, 2017 at 06:10:59PM +0530, Lokesh Vutla wrote: > > > On Tuesday 21 November 2017 06:14 PM, Tom Rini wrote: > > On Tue, Nov 14, 2017 at 04:12:31PM +0530, Faiz Abbas wrote: > >> The dra7xx series of SOCs contain a temperature sensor and an > >> associated analog-to-digital converter

[U-Boot] [PATCH] TI: am57xx; Remove am57xx_evm_nodt_defconfig

2017-11-21 Thread Tom Rini
We don't want this build anymore. Cc: Lokesh Vutla Signed-off-by: Tom Rini --- board/ti/am57xx/MAINTAINERS | 1 - configs/am57xx_evm_nodt_defconfig | 66 --- 2 files changed, 67 deletions(-) delete mode 100644 configs/am57xx_evm_nodt_defconfig diff -

Re: [U-Boot] [PATCH] TI: am57xx; Remove am57xx_evm_nodt_defconfig

2017-11-21 Thread Lokesh Vutla
On Tuesday 21 November 2017 06:17 PM, Tom Rini wrote: > We don't want this build anymore. > > Cc: Lokesh Vutla > Signed-off-by: Tom Rini Acked-by: Lokesh Vutla Thanks and regards, Lokesh ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.d

Re: [U-Boot] [U-Boot, 1/1] kconfig/symbol.c: use correct pointer type argument for sizeof

2017-11-21 Thread Tom Rini
On Wed, Nov 08, 2017 at 10:13:54PM +0100, Heinrich Schuchardt wrote: > sym_arr is of type struct symbol **. > So in malloc we need sizeof(struct symbol *). > > The problem was indicated by coccinelle. > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature

Re: [U-Boot] [U-Boot, v2, 1/1] scripts/ld-version.sh: regular expression compile fails

2017-11-21 Thread Tom Rini
On Wed, Nov 08, 2017 at 11:44:55PM +0100, Heinrich Schuchardt wrote: > ld --version | scripts/ld-version.sh > fails with > awk: scripts/ld-version.sh: > line 4: regular expression compile failed (missing '(') > .*) > > So let's refresh the script from Linux kernel v4.14-rc8. > > Signed-off-by: H

Re: [U-Boot] dra7x: fastboot: Increase recovery partition size

2017-11-21 Thread Tom Rini
On Wed, Nov 08, 2017 at 05:12:56PM -0600, Praneeth Bajjuri wrote: > As per current android recommendation > https://source.android.com/devices/architecture/kernel/modular-kernels > > 1. Android recovery mode should contain both SOC and ODM > kernel modules in the recovery partition. > > 2. If a

Re: [U-Boot] [U-Boot, 1/1] scripts/coccinelle: add some more coccinelle tests

2017-11-21 Thread Tom Rini
On Fri, Nov 10, 2017 at 07:15:02PM +0100, Heinrich Schuchardt wrote: > Add some useful static code analysis scripts for coccinelle > copied from the Linux kernel v4.14-rc8: > > Warn on check against NULL before calling free. > scripts/coccinelle/free/ifnullfree.cocci > > Detect superfluous NULL

Re: [U-Boot] [U-Boot,v2,1/1] yaffs2: rework yaffs_new_obj_id

2017-11-21 Thread Tom Rini
On Thu, Nov 09, 2017 at 01:26:43AM +0100, Heinrich Schuchardt wrote: > The iterator variable of list_for_each is never NULL. > if (1 || A) is always true. > Use break if entry found. > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature.asc Description: P

Re: [U-Boot] [U-Boot,v2,1/1] malloc: don't compare pointers to 0

2017-11-21 Thread Tom Rini
On Fri, Nov 10, 2017 at 09:46:34PM +0100, Heinrich Schuchardt wrote: > 0 is not a pointer. So do not compare pointers to 0. > > Do not return 0 from functions with a pointer return > type. > > Problem identified with Coccinelle. > > Signed-off-by: Heinrich Schuchardt > Reviewed-by: Bin Meng

Re: [U-Boot] envtools: make sure version/timestamp header file are available

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 03:47:18PM +0100, Stefan Agner wrote: > From: Stefan Agner > > With commit 84d46e7e8948 ("tools: env: allow to print U-Boot version") > the fw_env utilities need the version.h header file. Building only > the envtools in a pristine build directory will fail due to missing

Re: [U-Boot] [U-Boot, v5, 2/3] ARM: dts: OMAP5+: Add support for bandgap sensor in SPL

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 04:12:32PM +0530, Faiz Abbas wrote: > Mark bandgap node as uboot,dm-spl so that it can be accessed in spl > > Signed-off-by: Faiz Abbas > Reviewed-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signat

Re: [U-Boot] [U-Boot, v5, 1/3] thermal: ti-bandgap: Add support for temperature sensor

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 04:12:31PM +0530, Faiz Abbas wrote: > The dra7xx series of SOCs contain a temperature sensor and an > associated analog-to-digital converter (ADC) which produces > an output which is proportional to the SOC temperature. > Add support for this temperature sensor. > > Signed

Re: [U-Boot] [U-Boot,1/1] omap3: spi: the symbol for Hertz is Hz

2017-11-21 Thread Tom Rini
On Sun, Nov 12, 2017 at 09:02:52PM +0100, Heinrich Schuchardt wrote: > fix typo > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de h

Re: [U-Boot] README: update mailing list url

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 10:56:42PM -0800, S. Lockwood-Childs wrote: > Old url currently returns 403; modify to url that actually works > with current state of the list server > > Signed-off-by: S. Lockwood-Childs Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signat

Re: [U-Boot] [U-Boot,v2,1/1] Makefile: add coccicheck target

2017-11-21 Thread Tom Rini
On Sun, Nov 19, 2017 at 02:33:14PM +0100, Heinrich Schuchardt wrote: > Coccinelle is a program for static code analysis. > For details on Coccinelle see > > http://coccinelle.lip6.fr/ > > Add scripts/coccicheck copied from Linux kernel v4.14. > > The coccicheck script executes the tests *

Re: [U-Boot] Makefile: Correct SPL/TPL/DTB build race condition

2017-11-21 Thread Tom Rini
On Sat, Nov 11, 2017 at 10:48:40PM -0500, M. Vefa Bicakci wrote: > When building on a multi-core machine for an SPL-enabled board that > also uses CONFIG_OF_EMBED, the following error can be encountered > due to a race condition: > > make[3]: *** No rule to make target 'spl/dts/dt.dtb.o', neede

Re: [U-Boot] [PATCH] TI: am57xx; Remove am57xx_evm_nodt_defconfig

2017-11-21 Thread Tom Rini
On Tue, Nov 21, 2017 at 06:26:27PM +0530, Lokesh Vutla wrote: > On Tuesday 21 November 2017 06:17 PM, Tom Rini wrote: > > We don't want this build anymore. > > > > Cc: Lokesh Vutla > > Signed-off-by: Tom Rini > > Acked-by: Lokesh Vutla Applied to u-boot/master, thanks! -- Tom signature.a

Re: [U-Boot] env: Remove CONFIG_ENV_AES support

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 08:39:35AM -0500, Tom Rini wrote: > This support has been deprecated since v2017.09 due to security issues. > We now remove this support. > > Signed-off-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___

Re: [U-Boot] [U-Boot, v5, 3/3] ARM: dra7: Kconfig: Add thermal configs for dra7xx and am57xx

2017-11-21 Thread Tom Rini
On Tue, Nov 14, 2017 at 04:12:33PM +0530, Faiz Abbas wrote: > Configure thermal configs to remain set by default for dra7xx and am57xx > devices. > > Signed-off-by: Faiz Abbas > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature _

Re: [U-Boot] Convert CONFIG_SYS_NAND_ECCSIZE et al to Kconfig

2017-11-21 Thread Adam Ford
On Mon, Nov 20, 2017 at 7:19 PM, Tom Rini wrote: > On Sun, Nov 05, 2017 at 09:16:22AM -0600, Adam Ford wrote: > >> This converts the following to Kconfig: >>CONFIG_SYS_NAND_ECCSIZE >>CONFIG_SYS_NAND_ECCBYTES >>CONFIG_SYS_NAND_ECCSTEPS >>CONFIG_SYS_NAND_MAX_ECCPOS >>CONFIG_SYS_N

Re: [U-Boot] [RFC PATCH] usb: ehci: do not invalidate a NULL buffer

2017-11-21 Thread Marek Vasut
On 11/21/2017 07:30 AM, Dirk Behme wrote: > On 17.11.2017 16:04, Marek Vasut wrote: >> On 11/17/2017 03:28 PM, Dirk Behme wrote: >>> Its a valid use case to call ehci_submit_async() with a NULL buffer >>> with length 0. E.g. from usb_set_configuration(). >>> >>> As invalidate_dcache_range() isn't a

Re: [U-Boot] [PATCH v2 1/1] dm: usb: ehci: avoid possible NULL dereference

2017-11-21 Thread Marek Vasut
On 11/20/2017 07:33 PM, Heinrich Schuchardt wrote: > Currently we check in ehci_shutdown() if ctrl is NULL after > dereferencing it. > > Before this we have already dereferenced ctrl, ctrl->hccr, > and ctrl->hcor in ehci_get_portsc_register(), ehci_submit_root(), > and hci_common_init(). > > A be

Re: [U-Boot] [PATCH v6 2/2] sun50i: h5: Add NanoPi Neo Plus2 DT initial support

2017-11-21 Thread Jagan Teki
On Tue, Nov 21, 2017 at 2:41 PM, Antony Antony wrote: > Add initial DT for NanoPi NEO Plus2 by FriendlyARM > - Allwinner quad core H5 Cortex A53 with an ARM Mali-450MP GPU > - 1 GB DDR3 RAM > - 8GB eMMC flash (Samsung KLM8G1WEPD-B031) > - micro SD card slot > - Gigabit Ethernet (external RTL8211E-

Re: [U-Boot] [PATCH v6 1/2] arm64: sun50i-h5.dtsi : update to kernel 4.14

2017-11-21 Thread Jagan Teki
On Tue, Nov 21, 2017 at 2:41 PM, Antony Antony wrote: > last Kernel commit of sun50i-h5.dtsi d86e63e1f0b7 > > Signed-off-by: Antony Antony Applied to u-boot-sunxi/master ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v1] mtd/spi: add support for is25lq040b

2017-11-21 Thread Jagan Teki
On Mon, Nov 20, 2017 at 4:23 PM, Sean Nyekjaer wrote: > Signed-off-by: Sean Nyekjaer > --- Applied to u-boot-spi/master ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/1] vsprintf.c: add EFI device path printing

2017-11-21 Thread Wolfgang Denk
Dear Heinrich Schuchardt, In message you wrote: > > +u16 *str = efi_dp_str((struct efi_device_path *)dp); > + > +buf = string16(buf, end, str, field_width, precision, flags); > +efi_free_pool(str); > >>> > >>> efi_dp_str() can return NULL. Should this n

Re: [U-Boot] [PATCH v3 1/6] x86: baytrail: Add fsp-header verification for secure boot FSP

2017-11-21 Thread Bin Meng
Hi Anatolij, On Fri, Nov 17, 2017 at 9:13 AM, Anatolij Gustschin wrote: > From: Markus Valentin > > Introduce a new Kconfig variable for secure boot on Bay Trail based > platforms. If this variable is set, the build process tries to use > fsp-sb.bin instead of fsp.bin (-sb is the secure boot ena

[U-Boot] [PATCH 0/5] mmc: fixes for HS200/UHS core support

2017-11-21 Thread Jean-Jacques Hiblot
This series applies on top of "[PATCH v2 00/26] mmc: Add support for HS200 and UHS modes" It fixes a bug with old SD and MMC cards that support only the legacy mode. It also addresses the comments made on the mailing list: * dump card and host capabilities in debug mode * use 1-bit if the DTS prop

[U-Boot] [PATCH 1/5] dm: mmc: updated mmc_of_parse() to not fail because of an invalid bus-width

2017-11-21 Thread Jean-Jacques Hiblot
Instead of failing, the driver uses the default: 1-bit bus width. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc-uclass.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c index e30cde7..48fafce 100644 --- a/dri

[U-Boot] [PATCH 3/5] mmc: dump card and host capabilities if debug is enabled

2017-11-21 Thread Jean-Jacques Hiblot
This is a useful information while debugging the initialization process or performance issues. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index a5a521e..a30b6a2 100644 --- a/drivers/m

[U-Boot] [PATCH 4/5] mmc: Fixed a problem with old sd or mmc that do not support High speed

2017-11-21 Thread Jean-Jacques Hiblot
As the legacy modes were not added to the list of supported modes, old cards that do not support other modes could not be used. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c i

[U-Boot] [PATCH 5/5] mmc: all hosts support 1-bit bus width and legacy timings

2017-11-21 Thread Jean-Jacques Hiblot
Make sure that those basic capabilities are advertised by the host. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index 67f21ff..ec1dc49 100644 --- a/drivers/mmc/

[U-Boot] [PATCH 2/5] mmc: dm: support "mmc-ddr-1_2v" and "mmc-hs200-1_2v" boolean properties

2017-11-21 Thread Jean-Jacques Hiblot
Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc-uclass.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c index 48fafce..9723129 100644 --- a/drivers/mmc/mmc-uclass.c +++ b/drivers/mmc/mmc-uclass.c @@ -161,8 +161,12 @@ int mmc_of_p

[U-Boot] [PATCH v2] board: freescale: ls1012a: LS1012A-2G5RDB board support

2017-11-21 Thread Bhaskar Upadhaya
LS1012A-2G5RDB belongs to LS1012A family with features 2.5G SGMII PFE MAC, SATA, USB 2.0/3.0, WiFi, DDR, eMMC, QuadSPI, UART Signed-off-by: Bhaskar Upadhaya --- Changes for V2: 1. Use existing code base of RDB 2. Correct Copyrights Depends on https://patchwork.ozlabs.org/cover/823205 arch/ar

Re: [U-Boot] [PATCH 00/10] rockchip: add tpl and OPTEE support for rk3229

2017-11-21 Thread Andrew F. Davis
On 11/20/2017 03:25 PM, Heiko Stübner wrote: > Hi Kever, > > Am Mittwoch, 6. September 2017, 10:14:27 CET schrieb Kever Yang: >> Add some generic options for TPL support for arm 32bit, and then >> and TPL support for rk3229(cortex-A7), and then add OPTEE support >> in SPL. > > I was now finally a

[U-Boot] [PATCH 0/3] Update MAC address in "ethernet" node of Linux device tree

2017-11-21 Thread Prabhakar Kushwaha
Current implementation of MAC address fix-up of device tree uses tailing number behind "ethernet" found in "/aliases". It is not necessary for trailing number of “ethernet” to be sequential. There can be hole in between or any node disabled. hole can be created based on SerDes protocol supported

[U-Boot] [PATCH 1/3] common: Fix-up MAC addr in dts by fetching env variable serially

2017-11-21 Thread Prabhakar Kushwaha
Current implementation of MAC address fix-up of device tree uses tailing number behind "ethernet" found in "/aliases". It is not necessary for trailing number of “ethernet” to be sequential. There can be hole in between or any node disabled. So provide support device tree fix-up of “ethernent” no

[U-Boot] [PATCH 2/3] arm: Add support of updating dts before fix-up

2017-11-21 Thread Prabhakar Kushwaha
"ethernet" node fix-up for device tree happens before Linux boot. There can be requirement of updating "ethernet" node even before fix-up. So, add support of updating "ethernet" node. Signed-off-by: Prabhakar Kushwaha --- arch/arm/lib/bootm-fdt.c | 12 include/fdt_support.h| 3

[U-Boot] [PATCH 3/3] boards: ls1046ardb: disable unavailable "ethernet" node in dts

2017-11-21 Thread Prabhakar Kushwaha
Linux device tree contains "ethernet" node for all possible interface supported by SoC i.e. LS1046A. It is not necessary for a SerDes protocol to support all possible interface. So disable unavailable "ethernet" node in device tree. Also, enable FDT_SEQ_MACADDR_FROM_ENV to fetch MAC address seque

Re: [U-Boot] [PATCH] FSL PCI: Configure PCIe reference ratio

2017-11-21 Thread Joakim Tjernlund
On Tue, 2017-09-12 at 19:56 +0200, Joakim Tjernlund wrote: > Most FSL PCIe controllers expects 333 MHz PCI reference clock. > This clock is derived from the CCB but in many cases the ref. > clock is not 333 MHz and a divisor needs to be configured. > > This adds PEX_CCB_DIV #define which can be de

Re: [U-Boot] [PATCH 1/3] common: Fix-up MAC addr in dts by fetching env variable serially

2017-11-21 Thread York Sun
On 11/21/2017 08:26 AM, Prabhakar Kushwaha wrote: > Current implementation of MAC address fix-up of device tree uses > tailing number behind "ethernet" found in "/aliases". It is not > necessary for trailing number of “ethernet” to be sequential. There > can be hole in between or any node disabled

Re: [U-Boot] [PATCH 0/3] Update MAC address in "ethernet" node of Linux device tree

2017-11-21 Thread York Sun
On 11/21/2017 08:26 AM, Prabhakar Kushwaha wrote: > Current implementation of MAC address fix-up of device tree uses > tailing number behind "ethernet" found in "/aliases". It is not > necessary for trailing number of “ethernet” to be sequential. There > can be hole in between or any node disabled

Re: [U-Boot] [PATCH] FSL PCI: Configure PCIe reference ratio

2017-11-21 Thread York Sun
On 11/21/2017 09:18 AM, Joakim Tjernlund wrote: > On Tue, 2017-09-12 at 19:56 +0200, Joakim Tjernlund wrote: >> Most FSL PCIe controllers expects 333 MHz PCI reference clock. >> This clock is derived from the CCB but in many cases the ref. >> clock is not 333 MHz and a divisor needs to be configure

Re: [U-Boot] [PATCH] FSL PCI: Configure PCIe reference ratio

2017-11-21 Thread Joakim Tjernlund
On Tue, 2017-11-21 at 17:23 +, York Sun wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > On 11/21/2017 09:18 AM, Joakim Tjernlund wrote: > > On Tue, 2017-0

Re: [U-Boot] [PATCH] FSL PCI: Configure PCIe reference ratio

2017-11-21 Thread York Sun
On 11/21/2017 09:29 AM, Joakim Tjernlund wrote: > On Tue, 2017-11-21 at 17:23 +, York Sun wrote: >> CAUTION: This email originated from outside of the organization. Do not >> click links or open attachments unless you recognize the sender and know the >> content is safe. >> >> >> On 11/21/201

[U-Boot] [PATCH 00/22] mtd: nand: imports NAND core updates from Linux 4.15-rc1 and sync Denali driver

2017-11-21 Thread Masahiro Yamada
My motivation is to complete syncing the Denali driver with Linux. However, the last sync of NAND core was Linux 4.6 so a lots of parts are missing in U-Boot. I imported the following: - NAND_ECC_MAXIMIZE flag - nand_data_interface - sync nand_timings - NAND_ECC_CUSTOM_PAGE_ACCESS - mtd_oo

[U-Boot] [PATCH 08/22] mtd: nand: Expose data interface for ONFI mode 0

2017-11-21 Thread Masahiro Yamada
From: Sascha Hauer The nand layer will need ONFI mode 0 to use it as timing mode before and right after reset. Signed-off-by: Sascha Hauer Signed-off-by: Boris Brezillon [Linux commit: 6e1f9708dbf3c50a8da93c1952a01a7a2acb5e66] Signed-off-by: Masahiro Yamada --- drivers/mtd/nand/nand_timing

[U-Boot] [PATCH 04/22] mtd: nand: remove unnecessary 'extern' from function declarations

2017-11-21 Thread Masahiro Yamada
From: Sascha Hauer 'extern' is not necessary for function declarations. To prevent people from adding the keyword to new declarations remove the existing ones. Signed-off-by: Sascha Hauer Signed-off-by: Boris Brezillon [Linux commit: 79022591839f110f465cac0223e117b91d47d5db] Signed-off-by: Mas

[U-Boot] [PATCH 17/22] mtd: nand: allow drivers to request minimum alignment for passed buffer

2017-11-21 Thread Masahiro Yamada
In some cases, nand_do_{read,write}_ops is passed with unaligned ops->datbuf. Drivers using DMA will be unhappy about unaligned buffer. The new struct member, buf_align, represents the minimum alignment the driver require for the buffer. If the buffer passed from the upper MTD layer does not hav

[U-Boot] [PATCH 11/22] mtd: nand: Add a few more timings to nand_sdr_timings

2017-11-21 Thread Masahiro Yamada
From: Boris Brezillon Add the tR_max, tBERS_max, tPROG_max and tCCS_min timings to the nand_sdr_timings struct. Assign default/safe values for the statically defined timings, and extract them from the ONFI parameter table if the NAND is ONFI compliant. Signed-off-by: Boris Brezillon Tested-by:

[U-Boot] [PATCH 01/22] bitops: collect BIT macros to include/linux/bitops.h

2017-11-21 Thread Masahiro Yamada
Same macros are defined in various places. Collect them into include/linux/bitops.h like Linux. Signed-off-by: Masahiro Yamada --- arch/arm/include/asm/arch-sunxi/dram_sunxi_dw.h | 3 ++- arch/arm/mach-kirkwood/include/mach/gpio.h | 4 drivers/gpio/kw_gpio.c

[U-Boot] [PATCH 12/22] mtd: nand: Support controllers with custom page

2017-11-21 Thread Masahiro Yamada
From: Marc Gonzalez If your controller already sends the required NAND commands when reading or writing a page, then the framework is not supposed to send READ0 and SEQIN/PAGEPROG respectively. Signed-off-by: Marc Gonzalez Signed-off-by: Boris Brezillon [Linux commit: 3371d663bb4579f1b2003a921

[U-Boot] [PATCH 20/22] mtd: nand: add a shorthand to generate nand_ecc_caps structure

2017-11-21 Thread Masahiro Yamada
struct nand_ecc_caps was designed as flexible as possible to support multiple stepsizes (like sunxi_nand.c). So, we need to write multiple arrays even for the simplest case. I guess many controllers support a single stepsize, so here is a shorthand macro for the case. It allows to describe like .

[U-Boot] [PATCH 10/22] mtd: nand: Fix data interface configuration logic

2017-11-21 Thread Masahiro Yamada
From: Boris Brezillon When changing from one data interface setting to another, one has to ensure a specific sequence which is described in the ONFI spec. One of these constraints is that the CE line has go high after a reset before a command can be sent with the new data interface setting, whic

[U-Boot] [PATCH 21/22] mtd: nand: introduce NAND_ROW_ADDR_3 flag

2017-11-21 Thread Masahiro Yamada
Several drivers check ->chipsize to see if the third row address cycle is needed. Instead of embedding magic sizes such as 32MB, 128MB in drivers, introduce a new flag NAND_ROW_ADDR_3 for clean-up. Since nand_scan_ident() knows well about the device, it can handle this properly. The flag is set

[U-Boot] [PATCH 18/22] mtd: nand: Pass the CS line to ->setup_data_interface()

2017-11-21 Thread Masahiro Yamada
From: Boris Brezillon Some NAND controllers can assign different NAND timings to different CS lines. Pass the CS line information to ->setup_data_interface() so that the NAND controller driver knows which CS line is concerned by the setup_data_interface() request. Signed-off-by: Boris Brezillon

[U-Boot] [PATCH 13/22] mtd: add mtd_ooblayout_xxx() helper functions

2017-11-21 Thread Masahiro Yamada
From: Boris Brezillon In order to make the ecclayout definition completely dynamic we need to rework the way the OOB layout are defined and iterated. Create a few mtd_ooblayout_xxx() helpers to ease OOB bytes manipulation and hide ecclayout internals to their users. Signed-off-by: Boris Brezill

  1   2   >