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

> On 24 Aug 2017, at 21:13, Thomas Petazzoni > wrote: > > 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

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

On Thu, Aug 24, 2017 at 04:23:18PM +0200, Thomas Petazzoni wrote: > Hello, > > On Thu, 24 Aug 2017 12:16:30 +0200, Wolfgang Denk wrote: > > > > In this patch series, I propose to have U-Boot define some sane > > > defaults for those 4 configuration options, and cleanup all the board > > >

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

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

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

Dear Thomas, In message <20170824162318.7132066a@windsurf> you wrote: > > > Would it not be even easier (and less code) to move these > > definitions to Kconfig settings? > > I believe this patch series is a step in that direction. With this > patch series applied, we can now clearly identify

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

Hello, On Thu, 24 Aug 2017 12:16:30 +0200, Wolfgang Denk wrote: > > 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

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

Dear Thomas, In message <20170823205906.20933-1-thomas.petazz...@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 >

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

Hi Thomas, On Thu, Aug 24, 2017 at 8:58 AM, Thomas Petazzoni wrote: > Hello, > > 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 >

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

Hello, 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