Re: [PATCH 2/8] env: Rename SYS_REDUNDAND_ENVIRONMENT to ENV_REDUNDAND_ENVIRONMENT

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1387 bytes --]

On Tue, Mar 25, 2025 at 10:02:46AM +0100, Quentin Schulz wrote:
> Hi Marek,
> 
> On 3/24/25 11:52 PM, Marek Vasut wrote:
> > On 3/24/25 1:33 PM, Quentin Schulz wrote:
> > > Hi Marek,
> > > 
> > > On 3/22/25 10:55 PM, Marek Vasut wrote:
> > > > Rename the variable and add ENV_ prefix, so that all configuration
> > > > options which are related to environment would have an CONFIG_ENV_
> > > > prefix. No functional change.
> > > > 
> > > > Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
> > > > ---
> > > [...]
> > > > diff --git a/env/Kconfig b/env/Kconfig
> > > > index a177a354c8b..9ef45a7e3fe 100644
> > > > --- a/env/Kconfig
> > > > +++ b/env/Kconfig
> > > > @@ -463,7 +463,7 @@ config ENV_IS_IN_UBI
> > > >         the environment in.  This will enable redundant
> > > > environments in UBI.
> > > >         It is assumed that both volumes are in the same MTD partition.
> > > > -config SYS_REDUNDAND_ENVIRONMENT
> > > > +config ENV_REDUNDAND_ENVIRONMENT
> > > 
> > > Do either of the following:
> > > 
> > > 1) fix typo: s/REDUNDAND/REDUNDANT/
> > > 2) shorten name to be consistent with other symbols and avoid
> > > "env[...]environment" redundancy: config ENV_REDUND
> > CONFIG_ENV_REDUNDANT works for you ?
> 
> Fine with me!

Me as well, thanks! And for v2, please make a cover letter for when I
merge this in.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]