Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230321033810.22017-1-lukas.bulwahn@gmail.com>
 <CAFr9PXmKsequA3V7QWC2fKQYZgaNRj+q3q=UOBLmg6wxYKiVYg@mail.gmail.com> <59c05b36-3370-4d7b-b8cc-39d6dc36cdd0@app.fastmail.com>
In-Reply-To: <59c05b36-3370-4d7b-b8cc-39d6dc36cdd0@app.fastmail.com>
From:   Romain Perier <romain.perier@gmail.com>
Date:   Thu, 23 Mar 2023 08:26:26 +0100
Message-ID: <CABgxDoKMhdhzEpNEC_ciXoa7xeAEbMubapr9b8WvufJdKQqx8w@mail.gmail.com>
Subject: Re: [PATCH] ARM: mstar: remove unused config MACH_MERCURY
To:     Arnd Bergmann <arnd@arndb.de>
Cc:     Daniel Palmer <daniel@thingy.jp>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>,
        Russell King <linux@armlinux.org.uk>, soc@kernel.org,
        linux-arm-kernel@lists.infradead.org,
        kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi,

After discussion with Daniel, we agree that's an unused and an
orphaned Kconfig symbol (we cannot really argue against it).
We agree to remove it, it will be reverted back for a future patches set.

Acked-by: Romain Perier <romain.perier@gmail.com>

Le mar. 21 mars 2023 =C3=A0 10:30, Arnd Bergmann <arnd@arndb.de> a =C3=A9cr=
it :
>
> On Tue, Mar 21, 2023, at 09:33, Daniel Palmer wrote:
> > Hi Lukas
> >
> > On Tue, 21 Mar 2023 at 12:38, Lukas Bulwahn <lukas.bulwahn@gmail.com> w=
rote:
> >>
> >> Commit 312b62b6610c ("ARM: mstar: Add machine for MStar/Sigmastar Armv=
7
> >> SoCs") adds the config MACH_INFINITY and MACH_MERCURY. The MACH_INFINI=
TY
> >> config is used in the gpio-msc313 driver, but the MACH_MERCURY config =
is
> >> never used to configure anything in the kernel tree.
> >>
> >> Remove the unused config MACH_MERCURY.
> >
> > Because work is still on going to push that stuff.
> >
> > nak from me.
>
> The symbol has been unused three year though, and will
> show up every time someone runs a script to look for
> orphaned Kconfig entries.
>
> I'd still prefer removing it now and only bringing it
> back when you actually add the code for it.
>
>      Arnd