[PATCH v1 12/24] ringneck-px30: remove check for ENV_IS_NOWHERE, it is an SPL config

Tom Rini trini at konsulko.com
Wed Feb 22 16:03:20 CET 2023


On Wed, Feb 22, 2023 at 03:45:17PM +0100, Quentin Schulz wrote:
> Hi Troy,
> 
> On 2/22/23 02:38, Troy Kisky wrote:
> > When IS_ENABLED(CONFIG_ENV_IS_NOWHERE) is converted to
> > CONFIG_IS_ENABLED(ENV_IS_NOWHERE) this will give unwanted errors
> > on spl builds.
> > 
> > Signed-off-by: Troy Kisky <troykiskyboundary at gmail.com>
> > ---
> > 
> >   board/theobroma-systems/ringneck_px30/ringneck-px30.c | 4 ----
> >   1 file changed, 4 deletions(-)
> > 
> > diff --git a/board/theobroma-systems/ringneck_px30/ringneck-px30.c b/board/theobroma-systems/ringneck_px30/ringneck-px30.c
> > index 47d1a40ef7c..bb1bb4acf5c 100644
> > --- a/board/theobroma-systems/ringneck_px30/ringneck-px30.c
> > +++ b/board/theobroma-systems/ringneck_px30/ringneck-px30.c
> > @@ -118,10 +118,6 @@ int mmc_get_env_dev(void)
> >   	return CONFIG_SYS_MMC_ENV_DEV;
> >   }
> > -#if !IS_ENABLED(CONFIG_ENV_IS_NOWHERE)
> > -#error Please enable CONFIG_ENV_IS_NOWHERE
> > -#endif
> > -
> 
> NACK. We absolutely require CONFIG_ENV_IS_NOWHERE for our SoMs in U-Boot

OK, so please select it in the config TARGET_... stanza, and make sure
doc/board/... explains what's going on.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230222/455428ec/attachment.sig>


More information about the U-Boot mailing list