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

Wolfgang Denk
2017-08-24T10:16:30Z

Dear Thomas,

In message <20170823205906.20933-1-thomas.petazzoni@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?