From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Thu, 25 Jan 2024 17:06:23 +0100 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1rT2FC-0020h8-0E for lore@lore.pengutronix.de; Thu, 25 Jan 2024 17:06:23 +0100 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1rT2FC-0008VD-Rk; Thu, 25 Jan 2024 17:06:22 +0100 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rT2F7-0008Uj-GL; Thu, 25 Jan 2024 17:06:17 +0100 Received: from [2a0a:edc0:2:b01:1d::c5] (helo=pty.whiteo.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rT2F7-002Js7-3x; Thu, 25 Jan 2024 17:06:17 +0100 Received: from mol by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1rT2F7-00EaUn-04; Thu, 25 Jan 2024 17:06:17 +0100 Date: Thu, 25 Jan 2024 17:06:17 +0100 From: Michael Olbrich To: Michael Tretter , Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , distrokit@pengutronix.de, Marc Kleine-Budde , Lucas Stach Message-ID: Mail-Followup-To: Michael Tretter , Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , distrokit@pengutronix.de, Marc Kleine-Budde , Lucas Stach References: <20240125084253.3084797-1-m.olbrich@pengutronix.de> <20240125152547.GA422181@pengutronix.de> <20240125-93ea0e34689998dec1d2321b@ifak-system.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240125-93ea0e34689998dec1d2321b@ifak-system.com> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain Subject: Re: [DistroKit] [PATCH v2] platform-v8a: kernelconfig: cleanup X-BeenThere: distrokit@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: DistroKit Mailinglist List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "DistroKit" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: distrokit-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false Hi, On Thu, Jan 25, 2024 at 04:30:37PM +0100, Alexander Dahl wrote: > Am Thu, Jan 25, 2024 at 04:25:47PM +0100 schrieb Michael Tretter: > > On Thu, 25 Jan 2024 09:42:53 +0100, Michael Olbrich wrote: > > > This is the result of a review of the whole kernel config. > > > The options have been selected by the following rules: > > > > > > In general, the kernel config is optimized to keep the kernel small. So > > > any features that are not used at all are disabled. This includes things > > > like extra scheduling governors that require explicit configuration to > > > be used. > > > > > > All drivers that are not used by the boards configured in DistroKit are > > > now disabled. > > > > > > If there was no other strong preference then the value from the > > > defconfig was used. > > > > > > Signed-off-by: Michael Olbrich > > > Cc: Lucas Stach > > > Cc: Marc Kleine-Budde > > > --- > > > > > > Changes in v2: > > > - reenabled CONFIG_BLK_DEV_INITRD as requested by Ahmad > > > - enabled CONFIG_CRYPTO_DEV_SAFEXCEL with its dependencies. It's the > > > only driver from Uwes list that was not already enable. > > > > > > configs/platform-v8a/kernelconfig | 2249 +++++------------------------ > > > 1 file changed, 357 insertions(+), 1892 deletions(-) > > > > > > diff --git a/configs/platform-v8a/kernelconfig b/configs/platform-v8a/kernelconfig > > > index a37fd1e62a26..8dd0a0ceb7ee 100644 > > > --- a/configs/platform-v8a/kernelconfig > > > +++ b/configs/platform-v8a/kernelconfig > > [...] > > > @@ -3220,29 +2490,26 @@ CONFIG_GPIOLIB=y > > > CONFIG_GPIOLIB_FASTPATH_LIMIT=512 > > > CONFIG_OF_GPIO=y > > > CONFIG_GPIOLIB_IRQCHIP=y > > > -CONFIG_OF_GPIO_MM_GPIOCHIP=y > > > # CONFIG_DEBUG_GPIO is not set > > > # CONFIG_GPIO_SYSFS is not set > > > CONFIG_GPIO_CDEV=y > > > -CONFIG_GPIO_CDEV_V1=y > > > +# CONFIG_GPIO_CDEV_V1 is not set > > > > Disabling CONFIG_GPIO_CDEV_V1 breaks compatibility with libgpiod < 2.0. As > > PTXdist currently has libgpiod 1.6.3, trying to use the gpio tools with this > > configuration won't work. The kernel also enables CDEV_V1 by default. Given > > that libgpiod is disabled in DistroKit, I am not sure how relevant this is. > > We use DistroKit as a base layer and in our higher layers we do use the > libgpiod command line tools as well as the library itself. It is no > big deal because we of course have an adapted kernelconfig anyways, > but maybe it is nice to know for you? It's good to know and we should definitely keep this enabled. I'm somewhat confused, why it is disabled here in the first place. It is enabled in the defconfig so we didn't drop it during that sync and I cannot remember disabling it manually... Strange. But that's why we have review here :-). I'll send a new version. Regards, Michael -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |