[U-Boot] [PATCH] arm: mvebu: theadorable_debug_defconfig: Enable setexpr command

2017-10-06 Thread Stefan Roese
The setexpr command is useful for scripting, lets enable it for this platform. Signed-off-by: Stefan Roese --- configs/theadorable_debug_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/configs/theadorable_debug_defconfig b/configs/theadorable_debug_defconfig index a39497984b..c10b1

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

2017-10-06 Thread Stefan Roese
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(-) diff --git a/include/configs/theadorable-x86-common.h b/include/configs/theadorab

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

2017-10-06 Thread Stefan Roese
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(-) diff --git a/include/configs/theadorable-x86-common.h b/include/configs/thead

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

2017-10-06 Thread Stefan Roese
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/theadorable-x86-conga-qa3-e3845_defconfig | 1 - configs/theadorable

[U-Boot] [PATCH] rockchip: configs: fix typo in BOOT_TARGET_DEVICES

2017-10-06 Thread Klaus Goger
dchp should be dhcp. fixed copy-paste versions too. Signed-off-by: Klaus Goger --- include/configs/rockchip-common.h | 4 ++-- include/configs/tinker_rk3288.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/configs/rockchip-common.h b/include/configs/rockchip-

[U-Boot] [PATCH] 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 --- include/configs/rk3368_common.h | 6 ++ 1 file changed, 2 insertions(+), 4 deletion

Re: [U-Boot] rockchip: configs: fix typo in BOOT_TARGET_DEVICES

2017-10-06 Thread Philipp Tomsich
> dchp should be dhcp. fixed copy-paste versions too. > > Signed-off-by: Klaus Goger > --- > > include/configs/rockchip-common.h | 4 ++-- > include/configs/tinker_rk3288.h | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > Acked-by: Philipp Tomsich __

Re: [U-Boot] 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 > --- > > include/configs/rk3368_common.h | 6 ++ > 1 file changed, 2 inse

Re: [U-Boot] rockchip: configs: fix typo in BOOT_TARGET_DEVICES

2017-10-06 Thread Philipp Tomsich
> dchp should be dhcp. fixed copy-paste versions too. > > Signed-off-by: Klaus Goger > --- > > include/configs/rockchip-common.h | 4 ++-- > include/configs/tinker_rk3288.h | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > Reviewed-by: Philipp Tomsich ___

Re: [U-Boot] 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 > --- > > include/configs/rk3368_common.h | 6 ++ > 1 file changed, 2 inse

Re: [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368

2017-10-06 Thread Dr. Philipp Tomsich
Klaus, > On 6 Oct 2017, at 10:12, Klaus Goger > wrote: > > 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 > > --- > > inclu

Re: [U-Boot] rockchip: configs: fix typo in BOOT_TARGET_DEVICES

2017-10-06 Thread Philipp Tomsich
> dchp should be dhcp. fixed copy-paste versions too. > > Signed-off-by: Klaus Goger > Acked-by: Philipp Tomsich > Reviewed-by: Philipp Tomsich > --- > > include/configs/rockchip-common.h | 4 ++-- > include/configs/tinker_rk3288.h | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-)

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

2017-10-06 Thread Antony Antony
Hi Thanks Chen-Yu and Maxime. On Tue, Oct 03, 2017 at 10:11:00AM +0800, Chen-Yu Tsai wrote: > On Mon, Oct 2, 2017 at 1:34 AM, Antony Antony wrote: > > Hi Maxime, > > > > sorry for the delayed response. > > > > On Mon, Sep 25, 2017 at 01:12:23PM +0200, Maxime Ripard wrote: > >> On Sat, Sep 23, 20

Re: [U-Boot] [U-Boot,RFC] rockchip: asus c201 support

2017-10-06 Thread Philipp Tomsich
On Tue, 3 Oct 2017, Marty E. Plummer wrote: From: "Marty E. Plummer" I realize this patch is not up to standards for the sake of mainlining right now, but I'm mostly interested in getting some feedback on how to make it work before getting into the nicities of mainline inclusion. As of righ

Re: [U-Boot] [U-Boot, 1/6] rockchip: clk: Add mac clock set for rk3399

2017-10-06 Thread Philipp Tomsich
> Assuming mac_clk is fed by an external clock, set clk_rmii_src > clock select control register from IO for rgmii interface. > > Signed-off-by: David Wu > Acked-by: Philipp Tomsich > --- > > drivers/clk/rockchip/clk_rk3399.c | 21 +++-- > 1 file changed, 19 insertions(+), 2 de

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
> 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. > > Signed-off-by: David Wu > Acked-by: Philipp Tomsich > --- > > drivers/net/gmac_rockchip.c | 23 +++ >

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

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, 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, 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, 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,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,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] [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

[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] 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

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: >

[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.

[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 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 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/

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

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] [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] [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 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] [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

[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

[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 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 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 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] 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 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 ++---

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

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] 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: >>> >>

[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

[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 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 @@

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

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 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

[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_

[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 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

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 ___

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] [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] [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,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,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,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,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,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, 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, 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] [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] [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] 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

[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

[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

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

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 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

[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

[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

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 #

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

[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] [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

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

[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:

[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 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 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 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 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 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 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 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 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 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 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 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 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 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 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

  1   2   >