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 15:48:41 +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: <20221019154841.1cb2a15a@kmaincent-XPS-13-7390>
In-Reply-To: <5e6858b7-231a-187c-acaf-f82b82a956bf@linaro.org>
References: <20221019133208.319626-1-kory.maincent@bootlin.com>
        <20221019133208.319626-4-kory.maincent@bootlin.com>
        <5e6858b7-231a-187c-acaf-f82b82a956bf@linaro.org>
Organization: bootlin
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hello Krzysztof,

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

> On 19/10/2022 09:32, K=C3=B6ry Maincent wrote:
> > From: Kory Maincent <kory.maincent@bootlin.com>
> >=20
> > Enable the PL110 DRM driver, used by the spear600.
> >=20
> > CONFIG_I2C is dropped from the defconfig as it is selected by CONFIG_DR=
M. =20
>=20
> I2C as a user-selectable option should stay. Don't remove such
> (user-selectable) top-level options just because something else selected =
it.

As the CONFIG_DRM selects it, the "make savedefconfig" removes it by defaul=
t.
Why keeping something that is already automatically enabled by CONFIG_DRM?