From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 22 Jan 2024 16:57:06 +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 1rRwfZ-00BgCK-1P for lore@lore.pengutronix.de; Mon, 22 Jan 2024 16:57:06 +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 1rRwfZ-0004ZR-Sb; Mon, 22 Jan 2024 16:57:05 +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 1rRwfW-0004Yj-9i; Mon, 22 Jan 2024 16:57:02 +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 1rRwfV-001dB9-TL; Mon, 22 Jan 2024 16:57:01 +0100 Received: from mol by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1rRwfV-00ACe2-2e; Mon, 22 Jan 2024 16:57:01 +0100 Date: Mon, 22 Jan 2024 16:57:01 +0100 From: Michael Olbrich To: Ahmad Fatoum Message-ID: Mail-Followup-To: Ahmad Fatoum , distrokit@pengutronix.de, Marc Kleine-Budde , Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Lucas Stach References: <20240117135625.2863370-1-m.olbrich@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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] 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: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , distrokit@pengutronix.de, Marc Kleine-Budde , 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 Hi, Thanks for the Review. On Mon, Jan 22, 2024 at 04:01:36PM +0100, Ahmad Fatoum wrote: > On 17.01.24 14:56, Michael Olbrich wrote: > > -CONFIG_BLK_DEV_INITRD=y > > -CONFIG_INITRAMFS_SOURCE="" > > -# CONFIG_RD_GZIP is not set > > -# CONFIG_RD_BZIP2 is not set > > -# CONFIG_RD_LZMA is not set > > -# CONFIG_RD_XZ is not set > > -# CONFIG_RD_LZO is not set > > -# CONFIG_RD_LZ4 is not set > > -CONFIG_RD_ZSTD=y > > +# CONFIG_BLK_DEV_INITRD is not set > > We want to keep CONFIG_BLK_DEV_INITRD, so DistroKit systems can easily > boot common distros like Debian that ship with an initramfs. I was not aware that this is a use-case for DistroKit. I'm not sure how that is supposed to work but fine. > > -CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y > > +# CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE is not set > > # CONFIG_CPU_FREQ_DEFAULT_GOV_POWERSAVE is not set > > # CONFIG_CPU_FREQ_DEFAULT_GOV_USERSPACE is not set > > # CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND is not set > > # CONFIG_CPU_FREQ_DEFAULT_GOV_CONSERVATIVE is not set > > -# CONFIG_CPU_FREQ_DEFAULT_GOV_SCHEDUTIL is not set > > +CONFIG_CPU_FREQ_DEFAULT_GOV_SCHEDUTIL=y > > Why? The default can be changed later on, but using > performance as default allows for faster boot. It does? By how much? DistroKit does not change the default later so it runs with the maximum frequency the whole time. That is not a sane default for me. > > CONFIG_CMA=y > > # CONFIG_CMA_DEBUG is not set > > -# CONFIG_CMA_DEBUGFS is not set > > +CONFIG_CMA_DEBUGFS=y > > I never used this. What do we need this for given that we don't enable > graphic support? CMA is not only used by graphics drivers. It's nice to have just in case, just like the other debug options we've enabled. > > -CONFIG_NET_DSA=m > > -# CONFIG_NET_DSA_TAG_NONE is not set > > -# CONFIG_NET_DSA_TAG_AR9331 is not set > > -# CONFIG_NET_DSA_TAG_BRCM is not set > > -# CONFIG_NET_DSA_TAG_BRCM_LEGACY is not set > > -# CONFIG_NET_DSA_TAG_BRCM_PREPEND is not set > > -# CONFIG_NET_DSA_TAG_HELLCREEK is not set > > -# CONFIG_NET_DSA_TAG_GSWIP is not set > > -# CONFIG_NET_DSA_TAG_DSA is not set > > -# CONFIG_NET_DSA_TAG_EDSA is not set > > -# CONFIG_NET_DSA_TAG_MTK is not set > > -# CONFIG_NET_DSA_TAG_KSZ is not set > > -CONFIG_NET_DSA_TAG_OCELOT=m > > -CONFIG_NET_DSA_TAG_OCELOT_8021Q=m > > -# CONFIG_NET_DSA_TAG_QCA is not set > > -# CONFIG_NET_DSA_TAG_RTL4_A is not set > > -# CONFIG_NET_DSA_TAG_RTL8_4 is not set > > -# CONFIG_NET_DSA_TAG_RZN1_A5PSW is not set > > -# CONFIG_NET_DSA_TAG_LAN9303 is not set > > -# CONFIG_NET_DSA_TAG_SJA1105 is not set > > -# CONFIG_NET_DSA_TAG_TRAILER is not set > > -# CONFIG_NET_DSA_TAG_XRS700X is not set > > -CONFIG_VLAN_8021Q=m > > -CONFIG_VLAN_8021Q_GVRP=y > > -CONFIG_VLAN_8021Q_MVRP=y > > -CONFIG_LLC=m > > +# CONFIG_BRIDGE is not set > > +# CONFIG_NET_DSA is not set > > The Espressobin has an on-board Ethernet switch, but it seems it was > not enabled before? I'm not sure what makes sense for the Espressobin, that's why I asked Uwe to look at that. But I mostly disabled this stuff, because it's not actually configured anywhere in DistroKit. > > -CONFIG_PCIE_ROCKCHIP=y > > -CONFIG_PCIE_ROCKCHIP_HOST=y > > -CONFIG_PCIE_ROCKCHIP_EP=y > > -CONFIG_PCI_XGENE=y > > -CONFIG_PCI_XGENE_MSI=y > > +# CONFIG_PCIE_ROCKCHIP_HOST is not set > > +# CONFIG_PCI_XGENE is not set > > In case someone is wondering too: These are rk3399-specific. rk3568 PCI remains enabled. Right. There were several drivers enabled that were not used by any of the boards supported by DistroKit. > The rest looks ok. 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 |