Re: [PATCH 5/6] common: kconfig: raise SYS_CBSIZE default to 1024 for FSL_LSCH2/LSCH3

Vladimir Oltean <[email protected]>
Newsgroups org.u-boot-project.lists.u-boot
Message-ID <20260825081938.b74bey6zpuewig7l@skbuf>
On Tue, Aug 25, 2026 at 04:01:27PM +0800, Peng Fan wrote:
> On Mon, Aug 24, 2026 at 03:28:24PM +0300, Vladimir Oltean wrote:
> >The 512-byte console input buffer limit for Layerscape SoCs originates
> >from copy-pasted boilerplate in the original LS2085A board support added
> >in commit f749db3a75ec ("ARMv8/ls2085a_emu: Add LS2085A emulator and
> >simulator board support"). No rationale was given for the 512-byte
> >choice; every subsequent FSL_LSCH2 and FSL_LSCH3 common header
> >(ls1043a, ls1046a, ls1088a, lx2160a, ...) inherited it by convention.
> >
> >When CONFIG_SYS_CBSIZE was moved to Kconfig in commit d31466b382dd
> >("Convert CONFIG_SYS_CBSIZE to Kconfig"), the existing 512-byte values
> >were transcribed mechanically into Kconfig defaults, cementing the
> >limit without re-examining its validity.
> >
> >The practical consequence is that environment variables holding
> >multi-step boot scripts are silently truncated at 510 usable characters
> >on all Layerscape boards, making it impossible to store commands such
> >as combined TFTP+flash sequences in a single variable.
> >
> >The same problem was already identified and fixed for i.MX6 and i.MX7
> >in commit f7b237b81e5a ("MLK-24380 imx6: Enlarge the console I/O buffer
> nit:
> 
> This is a downstream commit.
> 
> Regards
> Peng

Correct, I failed to update the commit message when rebasing.

Would you prefer replacing the commit sha1sum reference with just
"downstream", or just remove the justification altogether?
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.