On Thu, Aug 07, 2025 at 03:41:57PM +0200, Marek Vasut wrote:
> On 8/7/25 12:24 PM, Philip Oberfichtner wrote:
> > For coherency, use the same help text for all related size checks
> > options. Furthermore, simplify the depends-on logic.
> >
> > Signed-off-by: Philip Oberfichtner <p...@denx.de>
> > ---
> >
> > Notes:
> > Changes in v2: new
> >
> > Kconfig | 8 +++-----
> > 1 file changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/Kconfig b/Kconfig
> > index ec20603883b..490a706101a 100644
> > --- a/Kconfig
> > +++ b/Kconfig
> > @@ -543,10 +543,8 @@ config BOARD_SIZE_LIMIT
> > default 1048576 if RCAR_64
> > default 0
> > help
> > - Maximum size of the U-Boot image. When defined, the build system
> > - checks that the actual size does not exceed it. This does not
> > - include SPL nor TPL, on platforms that use that functionality, they
> > - have separate options to restrict size.
> > + Specifies the maximum length of the U-Boot image.
> > + If this value is zero, it is ignored.
>
> This makes the description less accurate, sorry, NAK.
What about using just the first half:
Maximum size of the U-Boot image. When defined, the build system
checks that the actual size does not exceed it.
[plus/minus treatment of the value zero]
And dropping the remainder "This does not include ..."?
I think the hint on the separate options would be unnecessary, if they
are placed placed just below.
Best regards,
Philip
--
=====================================================================
DENX Software Engineering GmbH,
Managing Director: Johanna Denk, Tabea Lutz
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
=====================================================================
