[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20201130211600.GA1779005@ravnborg.org>
Date: Mon, 30 Nov 2020 22:16:00 +0100
From: Sam Ravnborg <sam@...nborg.org>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: Andrey Zhizhikin <andrey.zhizhikin@...ca-geosystems.com>,
linux@...linux.org.uk, nicolas.ferre@...rochip.com,
alexandre.belloni@...tlin.com, ludovic.desroches@...rochip.com,
tony@...mide.com, mripard@...nel.org, wens@...e.org,
jernej.skrabec@...l.net, thierry.reding@...il.com,
jonathanh@...dia.com, catalin.marinas@....com, will@...nel.org,
tsbogend@...ha.franken.de, James.Bottomley@...senpartnership.com,
deller@....de, mpe@...erman.id.au, benh@...nel.crashing.org,
paulus@...ba.org, lee.jones@...aro.org, emil.l.velikov@...il.com,
daniel.thompson@...aro.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org,
linux-tegra@...r.kernel.org, linux-mips@...r.kernel.org,
linux-parisc@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org,
Arnd Bergmann <arnd@...db.de>, Olof Johansson <olof@...om.net>
Subject: Re: [PATCH 0/5] drop unused BACKLIGHT_GENERIC option
Hi Krzysztof,
On Mon, Nov 30, 2020 at 10:25:01PM +0200, Krzysztof Kozlowski wrote:
> On Mon, Nov 30, 2020 at 08:11:33PM +0100, Sam Ravnborg wrote:
> > On Mon, Nov 30, 2020 at 03:21:32PM +0000, Andrey Zhizhikin wrote:
> > > Since the removal of generic_bl driver from the source tree in commit
> > > 7ecdea4a0226 ("backlight: generic_bl: Remove this driver as it is
> > > unused") BACKLIGHT_GENERIC config option became obsolete as well and
> > > therefore subject to clean-up from all configuration files.
> > >
> > > This series introduces patches to address this removal, separated by
> > > architectures in the kernel tree.
> > >
> > > Andrey Zhizhikin (5):
> > > ARM: configs: drop unused BACKLIGHT_GENERIC option
> > > arm64: defconfig: drop unused BACKLIGHT_GENERIC option
> > > MIPS: configs: drop unused BACKLIGHT_GENERIC option
> > > parisc: configs: drop unused BACKLIGHT_GENERIC option
> > > powerpc/configs: drop unused BACKLIGHT_GENERIC option
> >
> > For defconfigs I expect arch maintainers to do a make xxxdefconfig / make
> > savedefconfig / cp defconfig ... run now and then - this will remove
> > all such symbols.
>
> savedefconfig can be tricky because of risk of loosing options:
> 1. it will remove options which became the default or became selected,
> 2. later when the default is changed or selecting option is removed, the
> first option from #1 will not be brought back.
>
> This was already for example with DEBUG_FS and the conclusion that time
> was - do not run savedefconfig automatically.
>
> Therefore if some symbol(s) can be safely removed, patch is welcomed.
Thanks for letting me know, I have missed that discussion and
was obviously not aware.
I already acked'ed the patches and hope the soc people will pick them
up.
Sam
Powered by blists - more mailing lists