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;
Date:   Wed, 19 Oct 2022 16:05:09 +0200
From:   =?UTF-8?B?S8O2cnk=?= Maincent <kory.maincent@bootlin.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        thomas.petazzoni@bootlin.com, Russell King <linux@armlinux.org.uk>,
        Arnd Bergmann <arnd@arndb.de>,
        Gregory CLEMENT <gregory.clement@bootlin.com>,
        Sudeep Holla <sudeep.holla@arm.com>,
        Alexandre Ghiti <alexandre.ghiti@canonical.com>
Subject: Re: [PATCH 3/6] arm: configs: spear6xx: Enable PL110 display
 controller
Message-ID: <20221019160509.6252f9d1@kmaincent-XPS-13-7390>
In-Reply-To: <644c4a8a-8e30-5f00-7b9f-55c2d59e0d69@linaro.org>
References: <20221019133208.319626-1-kory.maincent@bootlin.com>
        <20221019133208.319626-4-kory.maincent@bootlin.com>
        <5e6858b7-231a-187c-acaf-f82b82a956bf@linaro.org>
        <20221019154841.1cb2a15a@kmaincent-XPS-13-7390>
        <644c4a8a-8e30-5f00-7b9f-55c2d59e0d69@linaro.org>
Organization: bootlin
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Krzysztof

On Wed, 19 Oct 2022 09:57:26 -0400
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:

> On 19/10/2022 09:48, K=C3=B6ry Maincent wrote:
> > Hello Krzysztof,
> >=20
> > On Wed, 19 Oct 2022 09:38:45 -0400
> > Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
> >  =20
> >> On 19/10/2022 09:32, K=C3=B6ry Maincent wrote: =20
> >>> From: Kory Maincent <kory.maincent@bootlin.com>
> >>>
> >>> Enable the PL110 DRM driver, used by the spear600.
> >>>
> >>> CONFIG_I2C is dropped from the defconfig as it is selected by CONFIG_=
DRM.
> >>>   =20
> >>
> >> I2C as a user-selectable option should stay. Don't remove such
> >> (user-selectable) top-level options just because something else select=
ed
> >> it. =20
> >=20
> > As the CONFIG_DRM selects it, the "make savedefconfig" removes it by
> > default. Why keeping something that is already automatically enabled by
> > CONFIG_DRM? =20
>=20
> Same question every few months :)
>=20
> Because otherwise we can drop select in DRM and do not update the
> defconfigs.

Alright, thanks, I will update it in V2.
I will be more careful with savedefconfig for now !

K=C3=B6ry