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

Wolfgang Denk wd at denx.de
Thu Aug 24 10:16:30 UTC 2017


Dear Thomas,

In message <20170823205906.20933-1-thomas.petazzoni at free-electrons.com> you wrote:
> 
> While adding support for new platforms, I found out that many, many
> platforms define pretty much the same values for:
> 
>  CONFIG_SYS_CBSIZE
>  CONFIG_SYS_PBSIZE
>  CONFIG_SYS_MAXARGS
>  CONFIG_SYS_BARGSIZE
> 
> with just a few platforms deviating from the values that have clearly
> been copy/pasted over and over again all over the place.

Thanks for cleaning this up!

> In this patch series, I propose to have U-Boot define some sane
> defaults for those 4 configuration options, and cleanup all the board
> configuration files that were using the default values. This leads to
> the removal of 1300+ lines.

Would it not be even easier (and less code) to move these
definitions to Kconfig settings?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If two witches would watch two watches, which witch would watch which
watch?


More information about the U-Boot mailing list