9 Jan
2019
9 Jan
'19
10:31 p.m.
On 1/9/19 10:30 PM, Simon Goldschmidt wrote:
Am Mi., 9. Jan. 2019, 22:27 hat Marek Vasut <marex@denx.de mailto:marex@denx.de> geschrieben:
On 1/9/19 8:56 PM, Simon Goldschmidt wrote: > socfpga_common.h defines CONFIG_SYS_BOOTMAPSZ to 64 MiB. > > Since having this define overrides the 'bootm_size' env variable for > the whole socfpga platform, let's remove this define from socfpga_common.h > and instead rely on the 'bootm_size' env variable (which is initialized > to 160 MiB in the same file's default env). This gives users the > chance to override it in their own environment. > > Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com <mailto:simon.k.r.goldschmidt@gmail.com>> > --- > > include/configs/socfpga_common.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/include/configs/socfpga_common.h b/include/configs/socfpga_common.h > index e9b368d93a..04e0f06230 100644 > --- a/include/configs/socfpga_common.h > +++ b/include/configs/socfpga_common.h > @@ -10,8 +10,6 @@ > */ > #define CONFIG_CLOCKS > > -#define CONFIG_SYS_BOOTMAPSZ (64 * 1024 * 1024) > - > #define CONFIG_TIMESTAMP /* Print image info with timestamp */ > > /* add target to build it automatically upon "make" */ > Can you at least "imply" it to 64 MiB, so we don't change the behavior ?You mean change the "boot_size" value in default environment to 64 MiB? Sure.
Sure, just so we won't change the behavior, some people like to run gigantic kernels.
--
Best regards,
Marek Vasut