* [PATCH] ARM: mstar: remove unused config MACH_MERCURY
@ 2023-03-21 3:38 Lukas Bulwahn
2023-03-21 8:33 ` Daniel Palmer
0 siblings, 1 reply; 6+ messages in thread
From: Lukas Bulwahn @ 2023-03-21 3:38 UTC (permalink / raw)
To: Daniel Palmer, Romain Perier, Arnd Bergmann, Russell King, soc,
linux-arm-kernel
Cc: kernel-janitors, linux-kernel, Lukas Bulwahn
Commit 312b62b6610c ("ARM: mstar: Add machine for MStar/Sigmastar Armv7
SoCs") adds the config MACH_INFINITY and MACH_MERCURY. The MACH_INFINITY
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.
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
arch/arm/mach-mstar/Kconfig | 7 -------
1 file changed, 7 deletions(-)
diff --git a/arch/arm/mach-mstar/Kconfig b/arch/arm/mach-mstar/Kconfig
index 5dbea7b485af..fa9709f30b46 100644
--- a/arch/arm/mach-mstar/Kconfig
+++ b/arch/arm/mach-mstar/Kconfig
@@ -20,11 +20,4 @@ config MACH_INFINITY
help
Support for MStar/Sigmastar infinity IP camera SoCs.
-config MACH_MERCURY
- bool "MStar/Sigmastar mercury SoC support"
- default ARCH_MSTARV7
- help
- Support for MStar/Sigmastar mercury dash camera SoCs.
- Note that older Mercury2 SoCs are ARM9 based and not supported.
-
endif
--
2.17.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: mstar: remove unused config MACH_MERCURY
2023-03-21 3:38 [PATCH] ARM: mstar: remove unused config MACH_MERCURY Lukas Bulwahn
@ 2023-03-21 8:33 ` Daniel Palmer
2023-03-21 9:29 ` Dan Carpenter
2023-03-21 9:30 ` Arnd Bergmann
0 siblings, 2 replies; 6+ messages in thread
From: Daniel Palmer @ 2023-03-21 8:33 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Romain Perier, Arnd Bergmann, Russell King, soc,
linux-arm-kernel, kernel-janitors, linux-kernel
Hi Lukas
On Tue, 21 Mar 2023 at 12:38, Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
> Commit 312b62b6610c ("ARM: mstar: Add machine for MStar/Sigmastar Armv7
> SoCs") adds the config MACH_INFINITY and MACH_MERCURY. The MACH_INFINITY
> 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.
Thanks,
Daniel
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: mstar: remove unused config MACH_MERCURY
2023-03-21 8:33 ` Daniel Palmer
@ 2023-03-21 9:29 ` Dan Carpenter
2023-03-21 9:30 ` Arnd Bergmann
1 sibling, 0 replies; 6+ messages in thread
From: Dan Carpenter @ 2023-03-21 9:29 UTC (permalink / raw)
To: Daniel Palmer
Cc: Lukas Bulwahn, Romain Perier, Arnd Bergmann, Russell King, soc,
linux-arm-kernel, kernel-janitors, linux-kernel
On Tue, Mar 21, 2023 at 05:33:19PM +0900, Daniel Palmer wrote:
> Hi Lukas
>
> On Tue, 21 Mar 2023 at 12:38, Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
> >
> > Commit 312b62b6610c ("ARM: mstar: Add machine for MStar/Sigmastar Armv7
> > SoCs") adds the config MACH_INFINITY and MACH_MERCURY. The MACH_INFINITY
> > 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.
Is there a link to the patchset?
>
> nak from me.
This stuff hasn't been used for years. We generally don't push dummy
code. It's super easy to add it back when the time comes.
regards,
dan carpenter
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: mstar: remove unused config MACH_MERCURY
2023-03-21 8:33 ` Daniel Palmer
2023-03-21 9:29 ` Dan Carpenter
@ 2023-03-21 9:30 ` Arnd Bergmann
2023-03-21 11:21 ` Daniel Palmer
2023-03-23 7:26 ` Romain Perier
1 sibling, 2 replies; 6+ messages in thread
From: Arnd Bergmann @ 2023-03-21 9:30 UTC (permalink / raw)
To: Daniel Palmer, Lukas Bulwahn
Cc: Romain Perier, Russell King, soc, linux-arm-kernel,
kernel-janitors, linux-kernel
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> wrote:
>>
>> Commit 312b62b6610c ("ARM: mstar: Add machine for MStar/Sigmastar Armv7
>> SoCs") adds the config MACH_INFINITY and MACH_MERCURY. The MACH_INFINITY
>> 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: mstar: remove unused config MACH_MERCURY
2023-03-21 9:30 ` Arnd Bergmann
@ 2023-03-21 11:21 ` Daniel Palmer
2023-03-23 7:26 ` Romain Perier
1 sibling, 0 replies; 6+ messages in thread
From: Daniel Palmer @ 2023-03-21 11:21 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Lukas Bulwahn, Romain Perier, Russell King, soc,
linux-arm-kernel, kernel-janitors, linux-kernel
Hi Arnd,
On Tue, 21 Mar 2023 at 18:30, Arnd Bergmann <arnd@arndb.de> wrote:
> I'd still prefer removing it now and only bringing it
> back when you actually add the code for it.
Fair enough. Code does exist for it in my tree but I never got around
to sending it.
I rescind my nak.
Cheers,
Daniel
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: mstar: remove unused config MACH_MERCURY
2023-03-21 9:30 ` Arnd Bergmann
2023-03-21 11:21 ` Daniel Palmer
@ 2023-03-23 7:26 ` Romain Perier
1 sibling, 0 replies; 6+ messages in thread
From: Romain Perier @ 2023-03-23 7:26 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Daniel Palmer, Lukas Bulwahn, Russell King, soc,
linux-arm-kernel, kernel-janitors, linux-kernel
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 à 10:30, Arnd Bergmann <arnd@arndb.de> a écrit :
>
> 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> wrote:
> >>
> >> Commit 312b62b6610c ("ARM: mstar: Add machine for MStar/Sigmastar Armv7
> >> SoCs") adds the config MACH_INFINITY and MACH_MERCURY. The MACH_INFINITY
> >> 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-03-23 7:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-21 3:38 [PATCH] ARM: mstar: remove unused config MACH_MERCURY Lukas Bulwahn
2023-03-21 8:33 ` Daniel Palmer
2023-03-21 9:29 ` Dan Carpenter
2023-03-21 9:30 ` Arnd Bergmann
2023-03-21 11:21 ` Daniel Palmer
2023-03-23 7:26 ` Romain Perier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).