Re: [PATCH 2/2] ARM: configs: keystone: Remove this defconfig

[Nishanth Menon]

On 11:31-20240124, Andrew Davis wrote:
> On 1/24/24 10:41 AM, Nishanth Menon wrote:
> > On 10:28-20240124, Andrew Davis wrote:
> > > TI Keystone devices can and should use the common multi-v7 defconfig.
> > > Currently it is not clear which defconfig should be used and so config
> > > options for Keystone boards are added to either both defconfigs, or only
> > > one or the other. As nice as it is to have a config just for this platform
> > > it is a maintenance burden. As it is not used by generic distros it is not
> > > very useful to continue to maintain. Remove this defconfig.
> > > 
> > > Signed-off-by: Andrew Davis <afd@xxxxxx>
> > > ---
> > >   arch/arm/configs/keystone_defconfig | 238 ----------------------------
> > >   1 file changed, 238 deletions(-)
> > >   delete mode 100644 arch/arm/configs/keystone_defconfig
> > > 
> > 
> > There are a bunch of downstream folks who will have recipe fails etc if
> > we do that. I am not sure we need to go down that route.
> > 
> 
> That is the point of this patch, we want to stop any remaining downstream
> folks from using this defconfig. It is not maintained nor updated like
> the multi_v7_defconfig, any new or needed options will only be added to
> multi-v7 defconfig.
> 

I am going to have to defer to ARM maintainers what they think..
enabling LPAE etc in common multi_v7_config.

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D