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

[Andrew Davis]

On 1/24/24 12:04 PM, Nishanth Menon wrote:
> 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.

We are not enabling LPAE in common multi_v7_config, that can't be
done as many plats do not support it. Keystone will use the new
multi_v7_lpae_defconfig which was just added:

e9faf9b0b07a ("ARM: add multi_v7_lpae_defconfig")

That is what prompted me to make this change, we now have a
commonish config that works for Keystone.

Andrew


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