Re: [PATCH 1/5] ARM: configs: drop unused BACKLIGHT_GENERIC option

From: Catalin Marinas <catalin.marinas@arm•com>
To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems•com>
Cc: "alexandre.belloni@bootlin•com" <alexandre.belloni@bootlin•com>,
	"tony@atomide•com" <tony@atomide•com>,
	"benh@kernel•crashing.org" <benh@kernel•crashing.org>,
	"linux-kernel@vger•kernel.org" <linux-kernel@vger•kernel.org>,
	"James.Bottomley@HansenPartnership•com"
	<James.Bottomley@HansenPartnership•com>,
	"thierry.reding@gmail•com" <thierry.reding@gmail•com>,
	"paulus@samba•org" <paulus@samba•org>,
	"sam@ravnborg•org" <sam@ravnborg•org>,
	"daniel.thompson@linaro•org" <daniel.thompson@linaro•org>,
	"linux-omap@vger•kernel.org" <linux-omap@vger•kernel.org>,
	Arnd Bergmann <arnd@arndb•de>,
	"mpe@ellerman•id.au" <mpe@ellerman•id.au>,
	"deller@gmx•de" <deller@gmx•de>,
	"linux@armlinux•org.uk" <linux@armlinux•org.uk>,
	Krzysztof Kozlowski <krzk@kernel•org>,
	"jonathanh@nvidia•com" <jonathanh@nvidia•com>,
	"ludovic.desroches@microchip•com"
	<ludovic.desroches@microchip•com>,
	arm@kernel•org,
	"linux-mips@vger•kernel.org" <linux-mips@vger•kernel.org>,
	"will@kernel•org" <will@kernel•org>,
	"mripard@kernel•org" <mripard@kernel•org>,
	"linux-tegra@vger•kernel.org" <linux-tegra@vger•kernel.org>,
	"lee.jones@linaro•org" <lee.jones@linaro•org>,
	"wens@csie•org" <wens@csie•org>,
	"linux-arm-kernel@lists•infradead.org"
	<linux-arm-kernel@lists•infradead.org>,
	"jernej.skrabec@siol•net" <jernej.skrabec@siol•net>,
	"tsbogend@alpha•franken.de" <tsbogend@alpha•franken.de>,
	"linux-parisc@vger•kernel.org" <linux-parisc@vger•kernel.org>,
	"emil.l.velikov@gmail•com" <emil.l.velikov@gmail•com>,
	Olof Johansson <olof@lixom•net>,
	"linuxppc-dev@lists•ozlabs.org" <linuxppc-dev@lists•ozlabs.org>
Subject: Re: [PATCH 1/5] ARM: configs: drop unused BACKLIGHT_GENERIC option
Date: Tue, 1 Dec 2020 14:40:53 +0000	[thread overview]
Message-ID: <20201201144052.GE31404@gaia> (raw)
In-Reply-To: <AM6PR06MB4691EC52BA41B86AB16EE14FA6F50@AM6PR06MB4691.eurprd06.prod.outlook.com>

On Mon, Nov 30, 2020 at 07:50:25PM +0000, ZHIZHIKIN Andrey wrote:
> From Krzysztof Kozlowski <krzk@kernel•org>:
> > On Mon, Nov 30, 2020 at 03:21:33PM +0000, Andrey Zhizhikin wrote:
> > > Commit 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is
> > > unused") removed geenric_bl driver from the tree, together with
> > > corresponding config option.
> > >
> > > Remove BACKLIGHT_GENERIC config item from all ARM configurations.
> > >
> > > Fixes: 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it
> > > is unused")
> > > Cc: Sam Ravnborg <sam@ravnborg•org>
> > > Signed-off-by: Andrey Zhizhikin
> > > <andrey.zhizhikin@leica-geosystems•com>
> > > ---
> > >  arch/arm/configs/at91_dt_defconfig        | 1 -
> > >  arch/arm/configs/cm_x300_defconfig        | 1 -
> > >  arch/arm/configs/colibri_pxa300_defconfig | 1 -
> > >  arch/arm/configs/jornada720_defconfig     | 1 -
> > >  arch/arm/configs/magician_defconfig       | 1 -
> > >  arch/arm/configs/mini2440_defconfig       | 1 -
> > >  arch/arm/configs/omap2plus_defconfig      | 1 -
> > >  arch/arm/configs/pxa3xx_defconfig         | 1 -
> > >  arch/arm/configs/qcom_defconfig           | 1 -
> > >  arch/arm/configs/sama5_defconfig          | 1 -
> > >  arch/arm/configs/sunxi_defconfig          | 1 -
> > >  arch/arm/configs/tegra_defconfig          | 1 -
> > >  arch/arm/configs/u8500_defconfig          | 1 -
> > >  13 files changed, 13 deletions(-)
> > 
> > You need to send it to arm-soc maintainers, otherwise no one might feel
> > responsible enough to pick it up.
> 
> Good point, thanks a lot!
> 
> I was not aware of the fact that there is a separate ML that should
> receive patches targeted ARM SOCs. Can you (or anyone else) please
> share it, so I can re-send it there as well?

It's not a mailing list as such (with archives etc.), just an alias to
the arm-soc maintainers: arm@kernel•org.

> > Reviewed-by: Krzysztof Kozlowski <krzk@kernel•org>
> > 
> > +CC Arnd and Olof,
> > 
> > Dear Arnd and Olof,
> > 
> > Maybe it is worth to add arm-soc entry to the MAINTAINERS file?
> > Otherwise how one could get your email address? Not mentioning the
> > secret-soc address. :)

I tried to convince them before, it didn't work. I guess they don't like
to be spammed ;). Or rather, SoC-specific patches, even to defconfig,
should go through the specific SoC maintainers. However, there are
occasional defconfig patches which are more generic or affecting
multiple SoCs. I just ignore them as the arm64 defconfig is usually
handled by the arm-soc folk (when I need a defconfig change, I go for
arch/arm64/Kconfig directly ;)).

Anyway, I still think that we should add a MAINTAINERS entry for
arch/arm64/configs/defconfig and arch/arm64/Kconfig.platforms.

-- 
Catalin

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists•infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel