On Wed, May 18, 2022 at 05:52:36PM +0200, Daniel Schwierzeck wrote: > > > On 18.05.22 17:22, Tom Rini wrote: > > On Wed, May 18, 2022 at 05:12:13PM +0200, Daniel Schwierzeck wrote: > > > > > > > > > On 18.05.22 14:18, Tom Rini wrote: > > > > On Tue, May 17, 2022 at 10:53:53PM +0200, Daniel Schwierzeck wrote: > > > > > Those options show up in menuconfig when selecting ARM or MIPS which > > > > > is dangerous if a user accidently sets them. This also clutters up the > > > > > menuconfig top-level screen. Because those options should only be set > > > > > by SoC specific or board specific configs, make them invisible. > > > > > > > > > > Signed-off-by: Daniel Schwierzeck <daniel.schwierz...@gmail.com> > > > > > > > > > > --- > > > > > > > > > > arch/Kconfig | 14 +++++++------- > > > > > 1 file changed, 7 insertions(+), 7 deletions(-) > > > > > > > > > > diff --git a/arch/Kconfig b/arch/Kconfig > > > > > index 12de8a1165..e3e28d0628 100644 > > > > > --- a/arch/Kconfig > > > > > +++ b/arch/Kconfig > > > > > @@ -365,7 +365,7 @@ config SYS_DISABLE_DCACHE_OPS > > > > > this functionality. > > > > > config SYS_IMMR > > > > > - hex "Address for the Internal Memory-Mapped Registers (IMMR) > > > > > window" > > > > > + hex > > > > > depends on PPC || FSL_LSCH2 || FSL_LSCH3 || ARCH_LS1021A > > > > > default 0xFF000000 if MPC8xx > > > > > default 0xF0000000 if ARCH_MPC8313 > > > > > > > > But now these aren't user selectable on the right platforms either. > > > > Maybe we need more menu organization, and so can have sub-menus depend > > > > on PPC, or whatever, and so things aren't cluttered on other > > > > architectures? > > > > > > > > > > my main concern was with SKIP_LOWLEVEL_INIT which definitely should not be > > > selectable by the user. Maybe I was to overambitious with changing the > > > SYS_IMMR option too, I could revert that part ;) > > > > I'd be agreeable to a patch that makes everyone that enables it today > > select it instead. > > I'll look into it. Would moving this to top-level Kconfig and the "General > setup" menu be an option?
Sure. Note that it's only used on ARM and some MIPS platforms today, so we could also tighten the dependncy line. -- Tom
signature.asc
Description: PGP signature