Re: [U-Boot] BUG num banks 8 exceeds hardcoded limit 4

2018-08-12 Thread Ramon Fried
Adding Simon. On Sat, Aug 11, 2018 at 2:54 PM Lukasz Majewski wrote: > > Hi Anand, > > > Hi Lukasz / Minkyu, > > > > Today I wanted to test the latest u-boot on my Odroid-XU4 > > and I landed onto this bug and it failed to boot up. > > > > include/configs/odroid.h:#define CONFIG_NR_DRAM_BANKS 8

Re: [U-Boot] BUG num banks 8 exceeds hardcoded limit 4

2018-08-12 Thread Anand Moon
Hi Lukasz and Ramon. On Sun, 12 Aug 2018 at 13:21, Ramon Fried wrote: > > Adding Simon. > > On Sat, Aug 11, 2018 at 2:54 PM Lukasz Majewski wrote: > > > > Hi Anand, > > > > > Hi Lukasz / Minkyu, > > > > > > Today I wanted to test the latest u-boot on my Odroid-XU4 > > > and I landed onto this bu

[U-Boot] [PATCH] cmd: led.c: fix coding style

2018-08-12 Thread Akee Huang
Fix coding style according to checkpatch.pl Signed-off-by: Akee Huang --- cmd/led.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/cmd/led.c b/cmd/led.c index fc07ca95a3..b339c98dfb 100644 --- a/cmd/led.c +++ b/cmd/led.c @@ -133,10 +133,10 @@ int do_led(cmd_tbl_

[U-Boot] [U-boot] 2018.09-rc1 Browsing source code of SPL and U-boot proper

2018-08-12 Thread Sreekesh Sreelal
I am trying to study the execution flow of SPL and U-boot proper specifically for BeagleBone Black(am335x_boneblack_defconfig) by browsing through the source code. 1. How can i generate the list of source files which gets compiled into U-boot proper(u-boot.img) and SPL(MLO) binaries? If i get a li

[U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-12 Thread Ramon Fried
From: Ramon Fried Instead of relaying on user to configure MEMORY_BANKS_MAX correctly, use VLA (variable length array) to accommodate the required banks. Fixes: 2a1f4f1758b5 ("Revert "fdt_support: Use CONFIG_NR_DRAM_BANKS if defined"") Signed-off-by: Ramon Fried --- common/fdt_support.c | 9 +

[U-Boot] [PATCH v2] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-12 Thread Ramon Fried
Instead of relaying on user to configure MEMORY_BANKS_MAX correctly, use VLA (variable length array) to accommodate the required banks. Fixes: 2a1f4f1758b5 ("Revert "fdt_support: Use CONFIG_NR_DRAM_BANKS if defined"") Signed-off-by: Ramon Fried --- Resending same patch but with the right email a

Re: [U-Boot] [PATCH v2] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-12 Thread Lukasz Majewski
Hi Ramon, > Instead of relaying on user to configure MEMORY_BANKS_MAX > correctly, use VLA (variable length array) to accommodate the > required banks. > > Fixes: 2a1f4f1758b5 ("Revert "fdt_support: Use CONFIG_NR_DRAM_BANKS if > defined"") > > Signed-off-by: Ramon Fried > --- > Resending same p

Re: [U-Boot] [PATCH] arm: socfpga: make socfpga_socrates_defconfig boot from QSPI

2018-08-12 Thread Simon Goldschmidt
On Sun, Aug 12, 2018 at 12:05 AM Marek Vasut wrote: > > On 08/11/2018 09:26 PM, Simon Goldschmidt wrote: > > On Fri, Aug 10, 2018 at 10:32 PM Marek Vasut wrote: > >> > >> On 08/10/2018 10:11 PM, Simon Goldschmidt wrote: > >>> On 10.08.2018 15:15, Marek Vasut wrote: > On 08/10/2018 02:56 PM,

Re: [U-Boot] Please pull u-boot-video/master

2018-08-12 Thread Tom Rini
On Sat, Aug 11, 2018 at 07:15:11PM +0200, Anatolij Gustschin wrote: > Hi Tom, > > The following changes since commit f05ebbf47a9dc863ff4bb084649ecb34d728b5f2: > > bcm968380gerg: Add MAINTAINERS file (2018-08-09 11:48:39 -0400) > > are available in the git repository at: > > git://git.denx.

Re: [U-Boot] Pull request: u-boot-staging

2018-08-12 Thread Tom Rini
On Sat, Aug 11, 2018 at 07:15:26PM +0200, Anatolij Gustschin wrote: > Hi Tom, > > The following changes since commit a032e0a6aed208977f48e78d2cc497b91543beaf: > > travis: give every job a name (2018-08-10 13:50:30 -0400) > > are available in the git repository at: > > git://git.denx.de/u-b

Re: [U-Boot] [PATCH] pci: Support parsing PCI controller DT subnodes

2018-08-12 Thread Bin Meng
Hi Marek, On Fri, Aug 10, 2018 at 6:32 PM, Marek Vasut wrote: > On 08/10/2018 05:42 AM, Bin Meng wrote: >> Hi Marek, >> >> On Thu, Aug 9, 2018 at 6:25 PM, Marek Vasut wrote: >>> On 08/09/2018 11:41 AM, Bin Meng wrote: >>> >>> [...] >>> Sorry this is a hack to current U-Boot implementati

Re: [U-Boot] [PATCH] pci: Support parsing PCI controller DT subnodes

2018-08-12 Thread Bin Meng
Hi Marek, On Fri, Aug 10, 2018 at 8:38 PM, Marek Vasut wrote: > On 08/10/2018 02:01 PM, Tom Rini wrote: >> On Wed, Aug 08, 2018 at 09:37:25PM +0200, Marek Vasut wrote: >>> On 08/08/2018 05:32 PM, Bin Meng wrote: Hi Marek, On Wed, Aug 8, 2018 at 10:33 PM, Marek Vasut wrote: > O

Re: [U-Boot] [PATCH v2] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-12 Thread Ramon Fried
On Sun, Aug 12, 2018 at 11:02 PM Lukasz Majewski wrote: > > Hi Ramon, > > > Instead of relaying on user to configure MEMORY_BANKS_MAX > > correctly, use VLA (variable length array) to accommodate the > > required banks. > > > > Fixes: 2a1f4f1758b5 ("Revert "fdt_support: Use CONFIG_NR_DRAM_BANKS if

[U-Boot] [PATCH v6 01/14] test: regmap: Increase size of syscon0 memory

2018-08-12 Thread Mario Six
The upcoming changes to the regmap interface will contain a proper check for plausibility when reading/writing from/to a register map. To still have the current tests pass, increase the size of the memory region for the syscon0 device, since one of the tests reads and writes beyond this range. Rev

[U-Boot] [PATCH v6 06/14] regmap: Add error output

2018-08-12 Thread Mario Six
Add some debug output in cases where the initialization of a regmap fails. Reviewed-by: Anatolij Gustschin Reviewed-by: Simon Glass Signed-off-by: Mario Six --- v5 -> v6: No changes v4 -> v5: No changes v3 -> v4: No changes v2 -> v3: New in v3 --- drivers/core/regmap.c | 10 -- 1

[U-Boot] [PATCH v6 07/14] mips: Implement {in, out}_{le, be}_{16, 32, 64} and {in, out}_8

2018-08-12 Thread Mario Six
MIPS is the only architecture currently supported by U-Boot that does not implement any of the in/out register access functions. To have a interface that is useable across architectures, add the functions to the MIPS architecture (implemented using the __raw_write and __raw_read functions). Signe

[U-Boot] [PATCH v6 03/14] regmap: Add documentation

2018-08-12 Thread Mario Six
Document the regmap_alloc() function. Reviewed-by: Anatolij Gustschin Reviewed-by: Simon Glass Signed-off-by: Mario Six --- v5 -> v6: No changes v4 -> v5: No changes v3 -> v4: No changes v2 -> v3: New in v3 --- drivers/core/regmap.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[U-Boot] [PATCH v6 08/14] regmap: Add raw read/write functions

2018-08-12 Thread Mario Six
The regmap functions currently assume that all register map accesses have a data width of 32 bits, but there are maps that have different widths. To rectify this, implement the regmap_raw_read and regmap_raw_write functions from the Linux kernel API that specify the width of a desired read or writ

[U-Boot] [PATCH v6 02/14] regmap: Fix documentation

2018-08-12 Thread Mario Six
The documentation in regmap.h is not in kernel-doc format. Correct this. Reviewed-by: Anatolij Gustschin Reviewed-by: Simon Glass Signed-off-by: Mario Six --- v5 -> v6: No changes v4 -> v5: No changes v3 -> v4: No changes v2 -> v3: New in v3 --- include/regmap.h | 48

[U-Boot] [PATCH v6 05/14] regmap: Introduce init_range

2018-08-12 Thread Mario Six
Both fdtdec_get_addr_size_fixed and of_address_to_resource can fail with an error, which is not currently checked during regmap initialization. Since the indentation depth is already quite deep, extract a new 'init_range' method to do the initialization. Reviewed-by: Anatolij Gustschin Reviewed-

[U-Boot] [PATCH v6 04/14] regmap: Improve error handling

2018-08-12 Thread Mario Six
ofnode_read_simple_addr_cells may fail and return a negative error code. Check for this when initializing regmaps. Also check if both_len is zero, since this is perfectly possible, and would lead to a division-by-zero further down the line. Reviewed-by: Anatolij Gustschin Reviewed-by: Simon Glas

[U-Boot] [PATCH v6 09/14] regmap: Support reading from specific range

2018-08-12 Thread Mario Six
It is useful to be able to treat the different ranges of a regmap separately to be able to use distinct offset for them, but this is currently not implemented in the regmap API. To preserve backwards compatibility, add regmap_read_range and regmap_write_range functions that take an additional para

[U-Boot] [PATCH v6 12/14] misc: Sort Makefile entries

2018-08-12 Thread Mario Six
Makefile entries should be sorted. Reviewed-by: Anatolij Gustschin Reviewed-by: Simon Glass Signed-off-by: Mario Six Signed-off-by: Anatolij Gustschin --- v5 -> v6: No changes v4 -> v5: Drop re-ordered entries that are not in mainline (e.g. CONFIG_GDSYS_IOEP, CONFIG_MPC83XX_SERDES, misc_san

[U-Boot] [PATCH v6 13/14] misc: Add gdsys_soc driver

2018-08-12 Thread Mario Six
This patch adds a driver for the bus associated with a IHS FPGA. Reviewed-by: Simon Glass Signed-off-by: Mario Six --- v5 -> v6: No changes v4 -> v5: No changes v3 -> v4: No changes v2 -> v3: * Fixed style violations * Added bindings file * Added more debug output in case of errors * Switch

[U-Boot] [PATCH v6 11/14] test: regmap: Add test for regmap_{set, get}

2018-08-12 Thread Mario Six
Add test for regmap_{set,get} functions. Reviewed-by: Anatolij Gustschin Reviewed-by: Simon Glass Signed-off-by: Mario Six --- v5 -> v6: No changes v4 -> v5: No changes v3 -> v4: No changes v2 -> v3: New in v3 --- test/dm/regmap.c | 28 1 file changed, 28 ins

[U-Boot] [PATCH v6 10/14] regmap: Define regmap_{get,set}

2018-08-12 Thread Mario Six
It would be convenient if one could use the regmap API in conjunction with register maps defined as structs (i.e. structs that directly mirror the memory layout of the registers in question). A similar approach was planned with the regmap_write32/regmap_read32 macros, but was never used. Hence, im

[U-Boot] [PATCH v6 14/14] misc: Add IHS FPGA driver

2018-08-12 Thread Mario Six
Add a driver for gdsys IHS (Integrated Hardware Systems) FPGAs, which supports initialization of the FPGA, as well as information gathering. Reviewed-by: Simon Glass Signed-off-by: Mario Six --- v5 -> v6: No changes v4 -> v5: No changes v3 -> v4: * Switched from 'res' as the name for return