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

2018-08-13 Thread Ramon Fried
On August 13, 2018 7:59:05 PM GMT+03:00, Tom Rini wrote: >On Mon, Aug 13, 2018 at 10:55:03PM +0300, Ramon Fried wrote: >> On Mon, Aug 13, 2018 at 7:22 PM Ramon Fried >wrote: >> >> > On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini > >> > wrote: >> > >On Mon, Aug 13, 2018 at 07:14:00PM +0300,

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

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 10:55:03PM +0300, Ramon Fried wrote: > On Mon, Aug 13, 2018 at 7:22 PM Ramon Fried wrote: > > > On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini > > wrote: > > >On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: > > >> On August 13, 2018 7:08:22 PM GMT+03:00,

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

2018-08-13 Thread Ramon Fried
On Mon, Aug 13, 2018 at 7:22 PM Ramon Fried wrote: > On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini > wrote: > >On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: > >> On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini > > wrote: > >> >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon

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

2018-08-13 Thread Ramon Fried
On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini wrote: >On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: >> On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini > wrote: >> >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: >> >> On Mon, Aug 13, 2018 at 5:52 PM Tom Rini

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

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: > On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini wrote: > >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: > >> On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: > >> > >> > On Mon, Aug 13, 2018 at 08:20:03AM +0100,

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

2018-08-13 Thread Ramon Fried
On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini wrote: >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: >> On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: >> >> > On Mon, Aug 13, 2018 at 08:20:03AM +0100, Peter Robinson wrote: >> > >> > > On Sun, Aug 12, 2018 at 9:37 PM, Ramon

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

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: > On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: > > > On Mon, Aug 13, 2018 at 08:20:03AM +0100, Peter Robinson wrote: > > > > > On Sun, Aug 12, 2018 at 9:37 PM, Ramon Fried > > wrote: > > > > From: Ramon Fried > > > > > > > >

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

2018-08-13 Thread Ramon Fried
On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: > On Mon, Aug 13, 2018 at 08:20:03AM +0100, Peter Robinson wrote: > > > On Sun, Aug 12, 2018 at 9:37 PM, Ramon Fried > wrote: > > > From: Ramon Fried > > > > > > Instead of relaying on user to configure MEMORY_BANKS_MAX > > > correctly, use VLA

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

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 08:20:03AM +0100, Peter Robinson wrote: > On Sun, Aug 12, 2018 at 9:37 PM, Ramon Fried wrote: > > From: Ramon Fried > > > > Instead of relaying on user to configure MEMORY_BANKS_MAX > > correctly, use VLA (variable length array) to accommodate the > > required banks. >

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

2018-08-13 Thread Peter Robinson
On Sun, Aug 12, 2018 at 9:37 PM, Ramon Fried wrote: > From: Ramon Fried > > Instead of relaying on user to configure MEMORY_BANKS_MAX > correctly, use VLA (variable length array) to accommodate the > required banks. With the kernel actively removing VLAs [1] does it make sense for us to use

[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