From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Thu, 25 Jan 2024 16:30:51 +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 1rT1gn-001wKU-2s for lore@lore.pengutronix.de; Thu, 25 Jan 2024 16:30:51 +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 1rT1go-00068r-Ly; Thu, 25 Jan 2024 16:30:50 +0100 Received: from mail.thorsis.com ([92.198.35.195]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rT1gk-00068J-Kj; Thu, 25 Jan 2024 16:30:47 +0100 Date: Thu, 25 Jan 2024 16:30:37 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=thorsis.com; s=default; t=1706196644; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:content-type:content-type:in-reply-to:in-reply-to: references:references; bh=cnbhb04gb6vKxzQsiDRWEMGixt2egDqVhUoEZR3dE8k=; b=NAgnvS4iAkhDUcs6lG5AevvRImFZHF8AE1lZmlw8LY1aBV0dVM0Ag7JkLxv+1RDNIeG9T9 iM/OdYSztHTJKIUQUQUgN8+qDPd5amsEGQbCJqhZi3yKBAFSCyjX6VJEMHCzmGa8KoIvoD hDHwdgpz9wwFUnHUVeY5iliNUOdNahspHD4MNAap+w6DeTBg6QBQdhaygPFEEQo7uV2WzA fDBU7Y1jaxSZVbyOdaAKCXD4l9cZLncftFsu+oZd7naJvjiQO3GpBrUy26TD6MVG1hi2uo tPlZMXcXpRZeO7ZmSd/Yt6tjwsuSGLSoOT09eFAT8FelITQJElN7AxIcft2ziQ== From: Alexander Dahl To: Michael Tretter Message-ID: <20240125-93ea0e34689998dec1d2321b@ifak-system.com> Mail-Followup-To: Michael Tretter , Michael Olbrich , 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> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240125152547.GA422181@pengutronix.de> X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.whiteo.stw.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-2.8 required=4.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.2 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: , Cc: Marc Kleine-Budde , Michael Olbrich , distrokit@pengutronix.de, Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Lucas Stach 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 Hei hei, 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? Greets Alex