[U-Boot] [PATCH 0/8] Clean up CONFIG_SYS_{CBSIZE, PBSIZE, MAXARGS, BARGSIZE} definitions

[Thomas Petazzoni]

Hello,

On Thu, 24 Aug 2017 20:53:35 +0200, Wolfgang Denk wrote:

> > I believe this patch series is a step in that direction. With this
> > patch series applied, we can now clearly identify which boards have
> > custom values for CONFIG_SYS_{CBSIZE, PBSIZE, MAXARGS, BARGSIZE}, and
> > therefore more easily move these from the .h files to defconfig files.  
> 
> So you think moving this to Kconfig would be a second step?  Ok,
> fine with me.

Yes, this would be a second step. The current patch series eliminates
the situations where the .h files use the default values, or values
that are close enough to the default values for the default value to
remain reasonable.

Of course, the patches should be reviewed, and the decision taken by
them potentially discussed.

> > The choice for such values is totally independent from the HW, but
> > rather tied to the specific use of a given board. For the same board,  
> 
> No, it is not. Buffer sizes eat on RAM, and there are environments
> (like today in SPL) where this may be a precious resource.

Right, true.

> > most people will find the default value to be OK, some people will need
> > a larger number of command arguments or a bigger Linux kernel command
> > line.
> > 
> > Let me know what your preference is.  
> 
> I think it makes a lot of sense to keep such configuration parameters
> changable, but we hould not stop half way but rather move this to
> Kconfig so it's easy to use a default setting, and only boards with
> special needs have to adjust this in their defconfig files.

OK. So let's review this first patch series. Once we agree on it and it
gets merged, I can work on the second step of moving those
configuration options to Kconfig.

Thanks for the very useful feedback!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com