Re: [U-Boot] [PATCH 3/3] x86: theadorable-x86-xxx_defconfig: Enable setexpr command

2017-10-06 Thread Bin Meng
On Sat, Oct 7, 2017 at 11:59 AM, Bin Meng wrote: > On Fri, Oct 6, 2017 at 3:12 PM, Stefan Roese wrote: >> The setexpr command is useful for scripting, lets enable it for all >> x86 based theadorable platforms. >> >> Signed-off-by: Stefan Roese >> Cc: Bin Meng >> --- >> configs/theadorable-x86-

Re: [U-Boot] [PATCH 2/3] x86: theadorable-x86-common: Move "-generic" into kernel-ver variable

2017-10-06 Thread Bin Meng
On Sat, Oct 7, 2017 at 11:59 AM, Bin Meng wrote: > On Fri, Oct 6, 2017 at 3:12 PM, Stefan Roese wrote: >> This enables the easier usage of "-custom" kernel versions as well. >> >> Signed-off-by: Stefan Roese >> Cc: Bin Meng >> --- >> include/configs/theadorable-x86-common.h | 8 >> 1

Re: [U-Boot] [PATCH 1/3] x86: theadorable-x86-common: Add further pci hotplug cmdline parameters

2017-10-06 Thread Bin Meng
On Sat, Oct 7, 2017 at 11:59 AM, Bin Meng wrote: > On Fri, Oct 6, 2017 at 3:12 PM, Stefan Roese wrote: >> This is needed for the PCIe hotplug to work correctly on some boards. >> >> Signed-off-by: Stefan Roese >> Cc: Bin Meng >> --- >> include/configs/theadorable-x86-common.h | 3 ++- >> 1 fil

Re: [U-Boot] [PATCH 2/3] x86: theadorable-x86-common: Move "-generic" into kernel-ver variable

2017-10-06 Thread Bin Meng
On Fri, Oct 6, 2017 at 3:12 PM, Stefan Roese wrote: > This enables the easier usage of "-custom" kernel versions as well. > > Signed-off-by: Stefan Roese > Cc: Bin Meng > --- > include/configs/theadorable-x86-common.h | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > Reviewed-b

Re: [U-Boot] [PATCH 3/3] x86: theadorable-x86-xxx_defconfig: Enable setexpr command

2017-10-06 Thread Bin Meng
On Fri, Oct 6, 2017 at 3:12 PM, Stefan Roese wrote: > The setexpr command is useful for scripting, lets enable it for all > x86 based theadorable platforms. > > Signed-off-by: Stefan Roese > Cc: Bin Meng > --- > configs/theadorable-x86-conga-qa3-e3845-pcie-x4_defconfig | 1 - > configs/theadora

Re: [U-Boot] [PATCH 1/3] x86: theadorable-x86-common: Add further pci hotplug cmdline parameters

2017-10-06 Thread Bin Meng
On Fri, Oct 6, 2017 at 3:12 PM, Stefan Roese wrote: > This is needed for the PCIe hotplug to work correctly on some boards. > > Signed-off-by: Stefan Roese > Cc: Bin Meng > --- > include/configs/theadorable-x86-common.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > Reviewed-by: B

Re: [U-Boot] [PATCH v2 1/1] x86: provide CONFIG_BUILD_ROM

2017-10-06 Thread Bin Meng
Hi Heinrich, On Mon, Sep 25, 2017 at 10:13 AM, Simon Glass wrote: > Hi Heinrich, > > On 19 September 2017 at 11:44, Heinrich Schuchardt wrote: >> Up to now we depended on an exported variable to build u-boot.rom. >> What we build should be specified by the configuration file. >> >> With this pat

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-06 Thread Jonathan Gray
On Fri, Oct 06, 2017 at 08:21:26AM -0400, Rob Clark wrote: > On Fri, Oct 6, 2017 at 12:35 AM, Jonathan Gray wrote: > > On Thu, Oct 05, 2017 at 05:05:49AM -0400, Rob Clark wrote: > >> On Thu, Oct 5, 2017 at 12:36 AM, Jonathan Gray wrote: > >> > On Wed, Oct 04, 2017 at 01:12:48PM -0400, Rob Clark w

Re: [U-Boot] ImportError: No module named _libfdt

2017-10-06 Thread Conrad
On 17.06.2017 19:45, jm.beaune at gmail.com (Jean-Marc Beaune) wrote: > Hello, > > I am new to U-Boot and trying to compile source code in order to get U-Boot > working on a BananaPi. > > I get the following error while compiling the latest version of master > branch: > > ImportError: No modul

[U-Boot] [PATCH v4 00/18] rockchip: back-to-bootrom: replace assembly-implementation with C-code

2017-10-06 Thread Philipp Tomsich
Recent discussions confirmed (what the code always assumed): the Rockchip BROM always enters U-Boot with the stack-pointer valid (i.e. the U-Boot startup code is running off the BROM stack). We can thus replace the back-to-bootrom code (i.e. both the save_boot_params and back_to_bootrom implement

Re: [U-Boot] [PATCH v4 07/18] socfpga: boot0 hook: adjust to unified boot0 semantics

2017-10-06 Thread Marek Vasut
On 10/06/2017 09:28 PM, Philipp Tomsich wrote: > With the updated boot0 semantics (i.e. giving the boot0-hook control > over when and where the vector table is emitted), the boot0-hook for > the socfpga needs to be adjusted. > > Signed-off-by: Philipp Tomsich Does this still produce the same bin

[U-Boot] [PATCH] Travis-CI: Split out more vendors from the catch-all ARM job

2017-10-06 Thread Tom Rini
- Move SoCFPGA and K2 boards to their own job - Expand the microblaze job to cover ARM boards from Xilinx as well. Signed-off-by: Tom Rini --- .travis.yml | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index f028e580f2fe..a1564ed90718 10064

[U-Boot] [PATCH] Travis-CI: Move Boundary and Engicam devices to their own jobs

2017-10-06 Thread Tom Rini
The i.MX6 job can still be close to the time limit, move a few more devices out. Signed-off-by: Tom Rini --- .travis.yml | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index b1a7c497a737..f028e580f2fe 100644 --- a/.travis.yml +++ b/.travis.ym

[U-Boot] [PATCH] Travis-CI: Move Toradex into its own job

2017-10-06 Thread Tom Rini
A few of the big jobs are getting close to the time limit again, split a few more things out. Signed-off-by: Tom Rini --- .travis.yml | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/.travis.yml b/.travis.yml index 7012fd1d28e2..b1a7c497a737 100644 --- a/.tr

[U-Boot] [PATCH v4 12/18] arm: mark save_boot_params_ret as a function

2017-10-06 Thread Philipp Tomsich
As no '.type' was set for save_boot_params_ret in start.S, binutils did not track whether it was emitted as A32 or T32. By properly marking save_boot_params_ret as a potential function entry, we can make sure that the compiler will insert the appropriate instructions for branching to save_boot_par

[U-Boot] [PATCH v4 14/18] rockchip: back-to-bootrom: replace assembly-implementation with C-code

2017-10-06 Thread Philipp Tomsich
The back-to-bootrom implementation for Rockchip has always relied on the stack-pointer being valid on entry, so there was little reason to have this as an assembly implementation. This provides a new C-only implementation of save_boot_params and back_to_bootrom (relying on setjmp/longjmp) and remo

[U-Boot] [PATCH v4 10/18] rockchip: boot0 hook: support early return for RK3188/RK3066-style BROM

2017-10-06 Thread Philipp Tomsich
Some Rockchip BROM versions (e.g. the RK3188 and RK3066) first read 1KB data from NAND into SRAM and executes it. Then, following a return to bootrom, the BROM loads additional code to SRAM (not overwriting the first block read) and reenters at the same address as the first time. To support bootin

[U-Boot] [PATCH v4 06/18] rockchip: rk3036: use aligned address for SPL_TEXT_BASE

2017-10-06 Thread Philipp Tomsich
With the boot0-hook inserting the additional padding to receive our SPL magic, the SPL_TEXT_BASE can be aligned again. Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v2: None include/configs/rk3036_common.h | 2 +- 1 file changed, 1 insertion(+), 1 deleti

[U-Boot] [PATCH v4 15/18] rockchip: rk3188: use boot0 hook to load up SPL in 2 steps

2017-10-06 Thread Philipp Tomsich
For the RK3188, the BROM will attempt to load up the first stage image (SPL for the RK3188) in two steps: first 1KB to offset 0x800 in the SRAM and then the remainder to offset 0xc00 in the SRAM. It always enters at 0x804, though. With this changeset, the RK3188 boot removes the TPL (stub) stage a

[U-Boot] [PATCH v4 17/18] rockchip: rk3188: move CONFIG_SPL_* entries from rk3188_common.h to Kconfig

2017-10-06 Thread Philipp Tomsich
There still are a few CONFIG_SPL_* options selected using defines from rk3188_common.h instead of via Kconfig. This migrates those over to Kconfig. Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v2: None arch/arm/mach-rockchip/Kconfig | 7 ++- inclu

[U-Boot] [PATCH v4 18/18] rockchip: mkimage: remove unused code-paths (spl_boot0 is now implied)

2017-10-06 Thread Philipp Tomsich
With all targets converted to generate prepadded images, this removes the spl_boot0 field from our config structure and removes the unused code-path (for images that are not prepadded): i.e. spl_boot0 is now implied as 'true' and the code is specialised by removing the other case. Signed-off-by: P

[U-Boot] [PATCH v4 11/18] arm: make save_boot_params_ret prototype visible for AArch64

2017-10-06 Thread Philipp Tomsich
The save_boot_params_ret() prototype (for those of us, that have a valid SP on entry and can implement save_boot_params() in C), was previously only defined for !defined(CONFIG_ARM64). This moves the declaration to a common block to ensure the prototype is available to everyone that might need it.

[U-Boot] [PATCH v4 08/18] bcm235xx: boot0 hook: adjust to unified boot0 semantics

2017-10-06 Thread Philipp Tomsich
This updates the BCM235xx boot0-hook to the updated boot0 semantics by emitting _start and the vector table before the boot0 hook (as was the case before). Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v2: None arch/arm/include/asm/arch-bcm235xx/boot0.h

[U-Boot] [PATCH v4 16/18] rockchip: back-to-bootrom: allow passing a cmd to the bootrom

2017-10-06 Thread Philipp Tomsich
The BROM supports forcing it to enter download-mode, if an appropriate result/cmd-word is returned to it. There already is a series to support this in review, so this prepares the (newly C-version) of the back-to-bootrom code to accept a cmd to passed on to the BROM. All the existing call-sites a

[U-Boot] [PATCH v4 09/18] bcm281xx: boot0 hook: adjust to unified boot0 semantics

2017-10-06 Thread Philipp Tomsich
This updates the BCM281xx boot0-hook to the updated boot0 semantics by emitting _start and the vector table before the boot0 hook (as was the case before). Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v2: None arch/arm/include/asm/arch-bcm281xx/boot0.h

[U-Boot] [PATCH v4 07/18] socfpga: boot0 hook: adjust to unified boot0 semantics

2017-10-06 Thread Philipp Tomsich
With the updated boot0 semantics (i.e. giving the boot0-hook control over when and where the vector table is emitted), the boot0-hook for the socfpga needs to be adjusted. Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v2: None arch/arm/mach-socfpga/inclu

[U-Boot] [PATCH v4 13/18] arm: provide a PCS-compliant setjmp implementation

2017-10-06 Thread Philipp Tomsich
The previous setjmp-implementation (as a static inline function that contained an 'asm volatile' sequence) was extremely fragile: (some versions of) GCC optimised the set of registers. One critical example was the removal of 'r9' from the clobber list, if -ffixed-reg9 was supplied. To increase ro

[U-Boot] [PATCH v4 04/18] rockchip: mkimage: use spl_boot0 for all Rockchip SoCs

2017-10-06 Thread Philipp Tomsich
From: Kever Yang Enable the spl_boot0 in SPL and use the pre-padding TAG memory, the mkimage do not need to pad it but only need to replace the value with correct TAG value. Signed-off-by: Kever Yang [Updated:] Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Change

[U-Boot] [PATCH v4 05/18] rockchip: rk3288: use aligned address for SPL_TEXT_BASE

2017-10-06 Thread Philipp Tomsich
From: Kever Yang After we use boot0 hook, we can use offset '000' instead of '004' as SPL_TEXT_BASE. Signed-off-by: Kever Yang [Updated tag in commit summary:] Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v2: None include/configs/rk3288_common.h | 2

[U-Boot] [PATCH v4 03/18] rockchip: enable boot0-hook for all Rockchip SoCs

2017-10-06 Thread Philipp Tomsich
Rockchip SoCs bootrom design is like this: - First 2KB or 4KB internal memory is for bootrom stack and heap; - Then the first 4-byte suppose to be a TAG like 'RK33'; - The the following memory address end with '0004' is the first instruction load and running by bootrom; Let's use the boot0 hook

[U-Boot] [PATCH v4 02/18] rockchip: boot0: align to 0x20 for armv7 '_start'

2017-10-06 Thread Philipp Tomsich
From: Kever Yang The '_start' is using as vector table base address, and will write to VBAR register, so it needs to be aligned to 0x20 for armv7. Signed-off-by: Kever Yang [Updated to current code base:] Signed-off-by: Philipp Tomsich --- Changes in v4: None Changes in v3: None Changes in v

[U-Boot] [PATCH v4 01/18] arm: boot0 hook: move boot0 hook before '_start'

2017-10-06 Thread Philipp Tomsich
The boot0 hook on ARM does not insert its payload before the vector table. This is both a mismatch with thec comment above it and contradict usage of the boot0 hook on ARM64. To fix this (and unify the semantics for ARM and ARM64), we change the boot0-hook semantics on ARM to match those on ARM64:

Re: [U-Boot] [U-Boot,v3,04/19] dm: core: add dev_read_addr_ptr()

2017-10-06 Thread Heinrich Schuchardt
On 10/06/2017 02:00 PM, Dr. Philipp Tomsich wrote: > >> On 6 Oct 2017, at 13:47, Heinrich Schuchardt wrote: >> >> On 09/11/2017 10:04 PM, Philipp Tomsich wrote: >>> The dev_read_addr_ptr() mimics the behaviour of the devfdt_get_addr_ptr(), >>> retrieving the first address of the node's reg-proper

Re: [U-Boot] [PATCH v2 00/17] efi_loader: Simple Network Protocol

2017-10-06 Thread Heinrich Schuchardt
On 10/06/2017 02:08 PM, Alexander Graf wrote: > > > On 05.10.17 16:35, Heinrich Schuchardt wrote: >> This patch series focuses on >> * correction of parameter types for boot services >> * fixes for the SetMem and CopyMem boot services >> * fixes for the simple network protocol implementation (SNP

[U-Boot] [PATCH 1/1] efi_selftest: efi_st_memcmp return difference of bytes

2017-10-06 Thread Heinrich Schuchardt
If the memory regions are different efi_st_memcmp currently returns the difference of the addresses. Insted the difference of the first differing byte pair should be returned. Signed-off-by: Heinrich Schuchardt --- lib/efi_selftest/efi_selftest_util.c | 2 +- 1 file changed, 1 insertion(+), 1 de

Re: [U-Boot] [U-Boot, v4, 2/2] rockchip: configs: use rockchip-common.h for rk3368

2017-10-06 Thread Philipp Tomsich
> rockchip-common.h already defines values that are missing from > rk3368_common.h > > For example BOOT_TARGET_DEVICES was defined empty and therefore > distroboot had no boot targets. > > Signed-off-by: Klaus Goger > Acked-by: Philipp Tomsich > Reviewed-by: Philipp Tomsich > --- > > Changes

Re: [U-Boot] [U-Boot, v4, 1/2] rockchip: configs: only add available BOOT_TARGET_DEVICES

2017-10-06 Thread Philipp Tomsich
> BOOT_TARGET_DEVICES should only be added if the corresponding u-boot > command is enabled. > > Signed-off-by: Klaus Goger > Acked-by: Philipp Tomsich > Reviewed-by: Philipp Tomsich > --- > > Changes in v4: > - rebase on mаster > > Changes in v3: > - Use CONFIG_IS_ENABLED macro instead of #

[U-Boot] [PATCH v4 2/2] rockchip: configs: use rockchip-common.h for rk3368

2017-10-06 Thread Klaus Goger
rockchip-common.h already defines values that are missing from rk3368_common.h For example BOOT_TARGET_DEVICES was defined empty and therefore distroboot had no boot targets. Signed-off-by: Klaus Goger Acked-by: Philipp Tomsich Reviewed-by: Philipp Tomsich --- Changes in v4: None Changes in

[U-Boot] [PATCH v4 1/2] rockchip: configs: only add available BOOT_TARGET_DEVICES

2017-10-06 Thread Klaus Goger
BOOT_TARGET_DEVICES should only be added if the corresponding u-boot command is enabled. Signed-off-by: Klaus Goger Acked-by: Philipp Tomsich Reviewed-by: Philipp Tomsich --- Changes in v4: - rebase on mаster Changes in v3: - Use CONFIG_IS_ENABLED macro instead of #ifdefs - Add missing else

Re: [U-Boot] [PATCH] dm: video: make ANSI escape sequence support optional

2017-10-06 Thread Simon Glass
Hi Rob, On 6 October 2017 at 10:56, Rob Clark wrote: > > On Fri, Oct 6, 2017 at 12:15 PM, Anatolij Gustschin wrote: > > On Sat, 30 Sep 2017 10:19:17 +0200 > > Anatolij Gustschin ag...@denx.de wrote: > > > >> As mentioned in review comments for ANSI escape sequence > >> support patches, this shou

Re: [U-Boot] [PATCH] cmd/fdt.c align data buffer to avoid unaligned word access

2017-10-06 Thread Hannes Schmelzer
On 09/28/2017 11:29 AM, Bernhard Messerklinger wrote: Since the compiler is free to place a char array to any address in memory (in this case the stack), also to a non word aligned address the function "fdt_prop_parse" runs into troubles upon it wants to write some (fdt32_t *) to such a variable

Re: [U-Boot] [PATCH] dm: video: make ANSI escape sequence support optional

2017-10-06 Thread Rob Clark
On Fri, Oct 6, 2017 at 12:15 PM, Anatolij Gustschin wrote: > On Sat, 30 Sep 2017 10:19:17 +0200 > Anatolij Gustschin ag...@denx.de wrote: > >> As mentioned in review comments for ANSI escape sequence >> support patches, this should be optional to reduce code >> size. Disable escape sequence suppor

[U-Boot] [PATCH v3 2/2] rockchip: configs: use rockchip-common.h for rk3368

2017-10-06 Thread Klaus Goger
rockchip-common.h already defines values that are missing from rk3368_common.h For example BOOT_TARGET_DEVICES was defined empty and therefore distroboot had no boot targets. Signed-off-by: Klaus Goger Acked-by: Philipp Tomsich Reviewed-by: Philipp Tomsich --- Changes in v3: None Changes in

[U-Boot] [PATCH v3 1/2] rockchip: configs: only add available BOOT_TARGET_DEVICES

2017-10-06 Thread Klaus Goger
BOOT_TARGET_DEVICES should only be added if the corresponding u-boot command is enabled. Signed-off-by: Klaus Goger Acked-by: Philipp Tomsich Reviewed-by: Philipp Tomsich --- Changes in v3: - Use CONFIG_IS_ENABLED macro instead of #ifdefs - Add missing else path Changes in v2: - Add patch to

Re: [U-Boot] rockchip: Set up default PWM flags

2017-10-06 Thread Philipp Tomsich
> At present if the PWM set_invert() method is not called then the flags are > not set up and the driver does not work properly. Fix this by setting up > the default flags when probing. > > Signed-off-by: Simon Glass > Fixes: 874ee59 (rockchip: pwm: implement pwm_set_invert()) > --- > > drivers

Re: [U-Boot] rockchip: Set up default PWM flags

2017-10-06 Thread Philipp Tomsich
> At present if the PWM set_invert() method is not called then the flags are > not set up and the driver does not work properly. Fix this by setting up > the default flags when probing. > > Signed-off-by: Simon Glass > Fixes: 874ee59 (rockchip: pwm: implement pwm_set_invert()) > --- > > drivers

Re: [U-Boot] [PATCH] dm: video: make ANSI escape sequence support optional

2017-10-06 Thread Anatolij Gustschin
On Sat, 30 Sep 2017 10:19:17 +0200 Anatolij Gustschin ag...@denx.de wrote: > As mentioned in review comments for ANSI escape sequence > support patches, this should be optional to reduce code > size. Disable escape sequence support when CONFIG_VIDEO_ANSI > is not enabled. > > Signed-off-by: Anato

Re: [U-Boot] [PATCH v2 1/2] rockchip: configs: only add available BOOT_TARGET_DEVICES

2017-10-06 Thread Dr. Philipp Tomsich
> On 6 Oct 2017, at 17:08, Klaus Goger > wrote: > > BOOT_TARGET_DEVICES should only be added if the corresponding u-boot > command is enabled. > > Signed-off-by: Klaus Goger Reviewed-by: Philipp Tomsich Required changes below. > --- > > Changes in v2: > - Add patch to series to prevent b

Re: [U-Boot] [U-Boot, v2, 1/2] rockchip: configs: only add available BOOT_TARGET_DEVICES

2017-10-06 Thread Philipp Tomsich
> BOOT_TARGET_DEVICES should only be added if the corresponding u-boot > command is enabled. > > Signed-off-by: Klaus Goger > --- > > Changes in v2: > - Add patch to series to prevent build errors if boards don't enable the > required u-boot command for distroboot > > include/configs/rockchi

Re: [U-Boot] [U-Boot, 7/8] rockchip: rk3128: add defconfig for evb-rk3128

2017-10-06 Thread Philipp Tomsich
> Enable board config for evb-rk3128. > Serial output and eMMC works in this version. > > Signed-off-by: Kever Yang > --- > > configs/evb-rk3128_defconfig | 42 ++ > 1 file changed, 42 insertions(+) > create mode 100644 configs/evb-rk3128_defconfig > A

Re: [U-Boot] [U-Boot,5/8] rockchip: rk3128: add sysreset driver

2017-10-06 Thread Philipp Tomsich
> Add rk3128 sysreset driver. > > Signed-off-by: Kever Yang > --- > > drivers/sysreset/Makefile | 1 + > drivers/sysreset/sysreset_rk3128.c | 45 > ++ > 2 files changed, 46 insertions(+) > create mode 100644 drivers/sysreset/sysreset_rk3128.c >

Re: [U-Boot] [U-Boot,3/8] rockchip: rk3128: add clock driver

2017-10-06 Thread Philipp Tomsich
> Add rk3128 clock driver and cru structure definition. > > Signed-off-by: Kever Yang > --- > > arch/arm/include/asm/arch-rockchip/cru_rk3128.h | 173 > arch/arm/mach-rockchip/rk3128/Makefile | 1 + > arch/arm/mach-rockchip/rk3128/clk_rk3128.c | 32 +++ > drivers/c

Re: [U-Boot] [U-Boot,4/8] rockchip: rk3128: add pinctrl driver

2017-10-06 Thread Philipp Tomsich
> Add rk3128 pinctrl driver and grf/iomux structure definition. > > Signed-off-by: Kever Yang > --- > > arch/arm/include/asm/arch-rockchip/grf_rk3128.h | 551 > > drivers/pinctrl/Kconfig | 10 + > drivers/pinctrl/rockchip/Makefile

Re: [U-Boot] [U-Boot,8/8] rockchip: rk3128: add sdram driver

2017-10-06 Thread Philipp Tomsich
> RK3128 support up to 2GB DDR3 sdram, one channel, 32bit data width. > > This patch is only used for U-Boot, but not for SPL which will > comes later, maybe after we merge all the common code into a common > file. > > Signed-off-by: Kever Yang > --- > > drivers/ram/rockchip/Makefile |

Re: [U-Boot] [U-Boot,6/8] rockchip: rk3128: add evb-rk3128 support

2017-10-06 Thread Philipp Tomsich
> evb-rk3128 is an evb from Rockchip based on rk3128 SoC: > - 2 USB2.0 Host port; > - 1 HDMI port; > - 2 10/100M eth port; > - 2GB ddr; > - 16GB eMMC; > - UART to USB debug port; > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/rk3128/Kconfig | 23 +++ > board

Re: [U-Boot] [U-Boot,1/8] rockchip: rk3128: add device tree file

2017-10-06 Thread Philipp Tomsich
> Add dts binding header for rk3128, files origin from kernel. > > Signed-off-by: Kever Yang > Reviewed-by: Philipp Tomsich > --- > > arch/arm/dts/Makefile | 1 + > arch/arm/dts/rk3128-evb.dts| 77 > arch/arm/dts/rk3128.dtsi | 756 > +

Re: [U-Boot] [U-Boot,2/8] rockchip: rk3128: add soc basic support

2017-10-06 Thread Philipp Tomsich
> RK3128 is a SoC from Rockchip with quad-core Cortex-A7 CPU > and mali400 GPU. Support Nand flash, eMMC, SD card, USB 2.0 host > and device, HDMI/LVDS/MIPI display. > > Signed-off-by: Kever Yang > --- > > arch/arm/mach-rockchip/Kconfig| 10 ++ > arch/arm/mach-rockchip/Makefile

Re: [U-Boot] [PATCH 1/1] armv8: ls1012a: Memory Map modification of kernel, env

2017-10-06 Thread York Sun
On 10/06/2017 03:50 AM, Bhaskar Upadhaya wrote: > > > -Original Message- > From: York Sun > Sent: Wednesday, September 06, 2017 9:51 PM > To: Bhaskar Upadhaya ; u-boot@lists.denx.de > Subject: Re: [PATCH 1/1] armv8: ls1012a: Memory Map modification of kernel,env > > On 09/01/2017 02:54

Re: [U-Boot] [PATCH] driver: fsl-mc: memset pointers after malloc

2017-10-06 Thread York Sun
On 10/06/2017 04:06 AM, Prabhakar Kushwaha wrote: > Memory allocated via malloc is not guaranteed to be zeroized. > > So explicitly memset the memory allocated via malloc. > > Signed-off-by: Prabhakar Kushwaha > --- Have you tried calloc()? York ___

[U-Boot] [PATCH v2 2/2] rockchip: configs: use rockchip-common.h for rk3368

2017-10-06 Thread Klaus Goger
rockchip-common.h already defines values that are missing from rk3368_common.h For example BOOT_TARGET_DEVICES was defined empty and therefore distroboot had no boot targets. Signed-off-by: Klaus Goger Acked-by: Philipp Tomsich Reviewed-by: Philipp Tomsich --- Changes in v2: None include/c

[U-Boot] [PATCH v2 1/2] rockchip: configs: only add available BOOT_TARGET_DEVICES

2017-10-06 Thread Klaus Goger
BOOT_TARGET_DEVICES should only be added if the corresponding u-boot command is enabled. Signed-off-by: Klaus Goger --- Changes in v2: - Add patch to series to prevent build errors if boards don't enable the required u-boot command for distroboot include/configs/rockchip-common.h | 39 ++

[U-Boot] [PATCH] fs: Set .opendir to fs_opendir_unsupported for btrfs

2017-10-06 Thread Marek Behún
Otherwise fs_opendir will fault. Signed-off-by: Marek Behun --- fs/fs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/fs/fs.c b/fs/fs.c index 84349f3039..9c4d67faf8 100644 --- a/fs/fs.c +++ b/fs/fs.c @@ -233,6 +233,7 @@ static struct fstype_info fstypes[] = { .read = btrfs_

Re: [U-Boot] [PATCH 2/4] fs: btrfs: Fix usage of uninitialized variables

2017-10-06 Thread Marek Behún
On Fri, 6 Oct 2017 06:54:34 -0700 "J. William Campbell" wrote: > Hi Marek, >     If the searched key is not found, isn't it an error to use > the res value AT ALL? Setting it to 0 may cover up the warning, but > the function shouldn't actually use it for anything. If it does, it > is modifyi

Re: [U-Boot] [PATCH 3/4] fs: btrfs: Fix printf length modifier

2017-10-06 Thread Tom Rini
On Fri, Oct 06, 2017 at 03:14:53PM +0200, Marek Behún wrote: > Hmm, this is weird. Tom's commit 6b7fd350 > > http://git.denx.de/?p=u-boot.git;a=commit;h=6b7fd350e85a97892a2cb066364797f3f3dcb9c9 > > forces %lu here, because the %u issued a warning. > But my compiler warns when %lu is used. > Maybe

Re: [U-Boot] [PATCH 3/4] fs: btrfs: Fix printf length modifier

2017-10-06 Thread Marek Behún
Hmm, this is weird. Tom's commit 6b7fd350 http://git.denx.de/?p=u-boot.git;a=commit;h=6b7fd350e85a97892a2cb066364797f3f3dcb9c9 forces %lu here, because the %u issued a warning. But my compiler warns when %lu is used. Maybe we should use PRIu32 instead... ... On Fri, 6 Oct 2017 15:04:58 +0200 M

[U-Boot] [PATCH 1/4] linux/time.h: Remove dead code

2017-10-06 Thread Marek Behún
Since rem = ((long) *tim_p) % SECSPERDAY; the second while cycle while (rem >= SECSPERDAY) is dead. Reported-by: Coverity (CID: 167334) Signed-off-by: Marek Behun --- include/linux/time.h | 5 - 1 file changed, 5 deletions(-) diff --git a/include/linux/time.h b/include/linux/time.h inde

[U-Boot] [PATCH 3/4] fs: btrfs: Fix printf length modifier

2017-10-06 Thread Marek Behún
Change %lu to %u. The compiler issues a warning otherwise. Signed-off-by: Marek Behun --- fs/btrfs/super.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 2529c2b3b6..706286ee2d 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@

[U-Boot] [PATCH 2/4] fs: btrfs: Fix usage of uninitialized variables

2017-10-06 Thread Marek Behún
The variable res should be initialized to 0 in these functions, because if the searched key is not found, the variable is used uninitialized. Reported-by: Coverity (CID: 167335) Reported-by: Coverity (CID: 167336) Reported-by: Coverity (CID: 167337) Signed-off-by: Marek Behun --- fs/btrfs/chunk-

[U-Boot] [PATCH 4/4] fs: btrfs: Remove a foreign language note

2017-10-06 Thread Marek Behún
I accidentaly left a foreign language note in the code from development. Signed-off-by: Marek Behun --- fs/btrfs/inode.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 0af04278a3..f785b600fa 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -21

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-06 Thread Rob Clark
On Fri, Oct 6, 2017 at 8:21 AM, Rob Clark wrote: > On Fri, Oct 6, 2017 at 12:35 AM, Jonathan Gray wrote: >> On Thu, Oct 05, 2017 at 05:05:49AM -0400, Rob Clark wrote: >>> On Thu, Oct 5, 2017 at 12:36 AM, Jonathan Gray wrote: >>> > On Wed, Oct 04, 2017 at 01:12:48PM -0400, Rob Clark wrote: >>> >>

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-06 Thread Rob Clark
On Fri, Oct 6, 2017 at 12:35 AM, Jonathan Gray wrote: > On Thu, Oct 05, 2017 at 05:05:49AM -0400, Rob Clark wrote: >> On Thu, Oct 5, 2017 at 12:36 AM, Jonathan Gray wrote: >> > On Wed, Oct 04, 2017 at 01:12:48PM -0400, Rob Clark wrote: >> >> On Wed, Oct 4, 2017 at 12:29 PM, Fabio Estevam >> >>

Re: [U-Boot] [PATCH v2 00/17] efi_loader: Simple Network Protocol

2017-10-06 Thread Alexander Graf
On 05.10.17 16:35, Heinrich Schuchardt wrote: > This patch series focuses on > * correction of parameter types for boot services > * fixes for the SetMem and CopyMem boot services > * fixes for the simple network protocol implementation (SNP) > * a unit test for SetMem, CopyMem and the simple net

[U-Boot] [PATCH 4/6] mmc: uniphier-sd: Add Renesas RCar quirks

2017-10-06 Thread Marek Vasut
Add a quirk to identify that the controller is Renesas RCar variant of the Matsushita SD IP and another quirk indicating it can support Renesas RCar HS200/HS400/SDR104 modes. Signed-off-by: Marek Vasut Cc: Jaehoon Chung Cc: Masahiro Yamada --- drivers/mmc/uniphier-sd.c | 17 ++---

[U-Boot] [PATCH] ARM: rmobile: dts: Add SDR104/HS200 to Salvator-X

2017-10-06 Thread Marek Vasut
Add DT nodes to enable SDR104/HS200 modes on Salvator-X boards. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- arch/arm/dts/salvator-common.dtsi | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/arm/dts/salvator-common.dtsi b/arch/arm/dts/salvator-common.dtsi index 0a0963

[U-Boot] [PATCH 5/6] mmc: uniphier-sd: Handle Renesas div-by-1

2017-10-06 Thread Marek Vasut
On the Renesas version of the IP, the /1 divider is realized by setting the clock register [7:0] to 0xff instead of setting bit 10 of the register. Check the quirk and handle accordingly. Signed-off-by: Marek Vasut Cc: Jaehoon Chung Cc: Masahiro Yamada --- drivers/mmc/uniphier-sd.c | 4 +++- 1

[U-Boot] [PATCH 6/6] mmc: uniphier-sd: Add Renesas SDR104/HS200 tuning support

2017-10-06 Thread Marek Vasut
Add code for PHY tuning required for SDR104/HS200 support on Renesas RCar. Signed-off-by: Marek Vasut Cc: Jaehoon Chung Cc: Masahiro Yamada --- drivers/mmc/uniphier-sd.c | 280 ++ 1 file changed, 280 insertions(+) diff --git a/drivers/mmc/uniphier-s

[U-Boot] [PATCH 2/6] mmc: uniphier-sd: Use mmc_of_parse()

2017-10-06 Thread Marek Vasut
Drop the ad-hoc DT caps parsing in favor of common framework function. Signed-off-by: Marek Vasut Cc: Jaehoon Chung Cc: Masahiro Yamada --- drivers/mmc/uniphier-sd.c | 23 +++ 1 file changed, 7 insertions(+), 16 deletions(-) diff --git a/drivers/mmc/uniphier-sd.c b/drivers

[U-Boot] [PATCH 3/6] mmc: uniphier-sd: Properly handle pin voltage configuration

2017-10-06 Thread Marek Vasut
Factor out the regulator handling into set_ios and add support for selecting pin configuration based on the voltage to support UHS modes. Signed-off-by: Marek Vasut Cc: Jaehoon Chung Cc: Masahiro Yamada --- drivers/mmc/uniphier-sd.c | 33 - 1 file changed, 24 in

[U-Boot] [PATCH 1/6] mmc: uniphier-sd: Always check controller version

2017-10-06 Thread Marek Vasut
Handle the controller version even if quirks are set. The controller in Renesas Gen3 SoCs does provide the version register, which indicates a controller v10 and the controller does support internal DMA and /1024 divider. Signed-off-by: Marek Vasut Cc: Jaehoon Chung Cc: Masahiro Yamada --- dri

Re: [U-Boot] [PATCH 1/2] thermal: ti-bandgap: Add support for temperature sensor

2017-10-06 Thread Tom Rini
On Fri, Oct 06, 2017 at 05:31:23PM +0530, Faiz Abbas wrote: > Hi, > > On Friday 06 October 2017 05:20 PM, Tom Rini wrote: > > On Fri, Oct 06, 2017 at 05:05:01PM +0530, Faiz Abbas wrote: > > > >> The dra7xx series of SOCs contain a temperature sensor and an > >> associated analog-to-digital conver

Re: [U-Boot] [PATCH 1/2] thermal: ti-bandgap: Add support for temperature sensor

2017-10-06 Thread Faiz Abbas
Hi, On Friday 06 October 2017 05:20 PM, Tom Rini wrote: > On Fri, Oct 06, 2017 at 05:05:01PM +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 tem

Re: [U-Boot] [U-Boot,v3,04/19] dm: core: add dev_read_addr_ptr()

2017-10-06 Thread Dr. Philipp Tomsich
> On 6 Oct 2017, at 13:47, Heinrich Schuchardt wrote: > > On 09/11/2017 10:04 PM, Philipp Tomsich wrote: >> The dev_read_addr_ptr() mimics the behaviour of the devfdt_get_addr_ptr(), >> retrieving the first address of the node's reg-property and returning >> it as a pointer (or NULL on failure).

Re: [U-Boot] [PATCH v2 11/17] efi_loader: implement WaitForPacket event

2017-10-06 Thread Alexander Graf
On 05.10.17 16:36, Heinrich Schuchardt wrote: > The WaitForPacket event informs that a network package has been > received by the SimpleNetworkProtocol. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > signaled has been renamed to is_signaled and is bool > --- > lib/efi_loader/efi_net

Re: [U-Boot] [PATCH 1/2] thermal: ti-bandgap: Add support for temperature sensor

2017-10-06 Thread Tom Rini
On Fri, Oct 06, 2017 at 05:05:01PM +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,v3,04/19] dm: core: add dev_read_addr_ptr()

2017-10-06 Thread Heinrich Schuchardt
On 09/11/2017 10:04 PM, Philipp Tomsich wrote: > The dev_read_addr_ptr() mimics the behaviour of the devfdt_get_addr_ptr(), > retrieving the first address of the node's reg-property and returning > it as a pointer (or NULL on failure). > > Signed-off-by: Philipp Tomsich > Acked-by: Philipp Tomsic

[U-Boot] [PATCH 0/2] Support for TI bandgap temperature sensor

2017-10-06 Thread Faiz Abbas
Adding support for TI bandgap temperature sensor. Also add dt support for bandgap sensor in spl. Faiz Abbas (2): thermal: ti-bandgap: Add support for temperature sensor ARM: dts: OMAP5+: Add support for bandgap sensor in SPL arch/arm/dts/omap5-u-boot.dtsi | 4 + common/

[U-Boot] [PATCH 2/2] ARM: dts: OMAP5+: Add support for bandgap sensor in SPL

2017-10-06 Thread Faiz Abbas
Mark bandgap node as uboot,dm-spl so that it can be accessed in spl Signed-off-by: Faiz Abbas --- arch/arm/dts/omap5-u-boot.dtsi | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/dts/omap5-u-boot.dtsi b/arch/arm/dts/omap5-u-boot.dtsi index fdaa692..bf2684c 100644 --- a/arch/arm/dt

[U-Boot] [PATCH 1/2] thermal: ti-bandgap: Add support for temperature sensor

2017-10-06 Thread Faiz Abbas
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-off-by: Faiz Abbas --- common/spl/Kconfig

[U-Boot] [PATCH v2] iso: Reduce verbosity on test and info calls

2017-10-06 Thread Alexander Graf
The test and info callbacks into the partition callback struct are used by the "part list" command on the command line. That command is used by the distro script. With verb=1 set, "part list" thus throws a lot of warnings about partitions it can't find when an upper layer searches for partitions.

Re: [U-Boot] [PATCH] dwc: ep0: Allocate and flush dwc->ep0_trb in a cache aligned manner

2017-10-06 Thread Faiz Abbas
Hi, On Thursday 05 October 2017 04:57 PM, Marek Vasut wrote: > On 10/04/2017 03:11 PM, Faiz Abbas wrote: >> Hi, >> >> On Wednesday 04 October 2017 06:01 PM, Marek Vasut wrote: >>> On 10/04/2017 12:51 PM, Faiz Abbas wrote: Hi, On Tuesday 03 October 2017 06:48 PM, Marek Vasut wrote: >

Re: [U-Boot] [PATCH] driver: fsl-mc: memset pointers after malloc

2017-10-06 Thread Stefan Roese
On 06.10.2017 13:05, Prabhakar Kushwaha wrote: Memory allocated via malloc is not guaranteed to be zeroized. So explicitly memset the memory allocated via malloc. Signed-off-by: Prabhakar Kushwaha --- drivers/net/fsl-mc/mc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/n

[U-Boot] [PATCH] driver: fsl-mc: memset pointers after malloc

2017-10-06 Thread Prabhakar Kushwaha
Memory allocated via malloc is not guaranteed to be zeroized. So explicitly memset the memory allocated via malloc. Signed-off-by: Prabhakar Kushwaha --- drivers/net/fsl-mc/mc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/fsl-mc/mc.c b/drivers/net/fsl-mc/mc.c index 12db

Re: [U-Boot] [PATCH 1/1] armv8: ls1012a: Memory Map modification of kernel, env

2017-10-06 Thread Bhaskar Upadhaya
-Original Message- From: York Sun Sent: Wednesday, September 06, 2017 9:51 PM To: Bhaskar Upadhaya ; u-boot@lists.denx.de Subject: Re: [PATCH 1/1] armv8: ls1012a: Memory Map modification of kernel,env On 09/01/2017 02:54 AM, Bhaskar Upadhaya wrote: > This patch adjusts memory map for im

Re: [U-Boot] [U-Boot,8/8] rockchip: rk3128: add sdram driver

2017-10-06 Thread Philipp Tomsich
On Wed, 27 Sep 2017, Kever Yang wrote: RK3128 support up to 2GB DDR3 sdram, one channel, 32bit data width. This patch is only used for U-Boot, but not for SPL which will comes later, maybe after we merge all the common code into a common file. Signed-off-by: Kever Yang --- drivers/ram/rock

Re: [U-Boot] [U-Boot,2/8] rockchip: rk3128: add soc basic support

2017-10-06 Thread Philipp Tomsich
On Wed, 27 Sep 2017, Kever Yang wrote: RK3128 is a SoC from Rockchip with quad-core Cortex-A7 CPU and mali400 GPU. Support Nand flash, eMMC, SD card, USB 2.0 host and device, HDMI/LVDS/MIPI display. Signed-off-by: Kever Yang Requested changes below. --- arch/arm/mach-rockchip/Kconfig

Re: [U-Boot] [U-Boot, 5/6] net: gmac_rockchip: Use the proerty of "clock_in_out" to set mac clock

2017-10-06 Thread Philipp Tomsich
On Thu, 21 Sep 2017, David Wu wrote: If the mac clock if from the external IO, set clock rate with 0; If the mac clock if from the internal divider pll, set 50M for rmii mode and set 125M for rgmii. If we do this, we really shouldn't treat the external clock any different. So we should reque

Re: [U-Boot] [U-Boot, 6/6] rockchip: gmac_rockchip: Add gmac support for rv1108

2017-10-06 Thread Philipp Tomsich
On Thu, 21 Sep 2017, David Wu wrote: The rv1108 mac only support rmii interface, so need to add the set_rmii() ops. Use the phy current interface to set rmii or rgmii ops. Signed-off-by: David Wu Acked-by: Philipp Tomsich See below for required changes. --- drivers/net/gmac_rockchip.c

Re: [U-Boot] [U-Boot, 4/6] net: gmac_rockchip: Define the gmac grf register struct at gmac_rockchip.c

2017-10-06 Thread Philipp Tomsich
On Thu, 21 Sep 2017, David Wu wrote: If we include both the rk3288_grf.h and rv1108_grf.h, there is a number of compiling error for redefinition. So we define the reg structs of mac_grf at gmac_rockchip.c. Remove the rk**_grf.h files, give them own grf offset for their use. The reg offset sh

Re: [U-Boot] [U-Boot, 3/6] rockchip: configs: Enable CONFIG_NET_RANDOM_ETHADDR for rk3288-evb

2017-10-06 Thread Philipp Tomsich
> If the Ethernet address is not set, the network can't work, > enable the random address config for default use. > > Signed-off-by: David Wu > Acked-by: Philipp Tomsich > --- > > configs/evb-rk3288_defconfig | 1 + > 1 file changed, 1 insertion(+) > Reviewed-by: Philipp Tomsich ___

Re: [U-Boot] [U-Boot, 2/6] rockchip: dts: rk3399-evb: Change the tx/rx delay value for transmission quality

2017-10-06 Thread Philipp Tomsich
> Give the mac controller the correct tx-delay and rx-delay value > for the rgmii mode transmission. If they are not matched, there > would be Ethernet packets lost, the net feature may not work. > > Signed-off-by: David Wu > Acked-by: Philipp Tomsich > --- > > arch/arm/dts/rk3399-evb.dts | 4

  1   2   >