From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: "Russell King (Oracle)" <linux@armlinux.org.uk> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>, Shawn Guo <shawnguo@kernel.org>, Li Yang <leoyang.li@nxp.com>, Marek Vasut <marex@denx.de>, Marcel Ziswiler <marcel.ziswiler@toradex.com>, soc@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 8/9] ARM: multi_v7_defconfig: Add options to support TQMLS102xA series Date: Sat, 21 Jan 2023 21:27:52 +0100 [thread overview] Message-ID: <3114bdeb-c8eb-3a3a-0d68-ca3370ce64ec@linaro.org> (raw) In-Reply-To: <Y8qr8wl8WdJfQ9We@shell.armlinux.org.uk> On 20/01/2023 15:57, Russell King (Oracle) wrote: > On Fri, Jan 20, 2023 at 12:35:41PM +0100, Krzysztof Kozlowski wrote: >> Defconfig=y is only for critical stuff needed to bring initramfs. Time >> is not needed for initramfs. > > Are I2C drivers "critical stuff"? What about AHCI drivers? What about > all the USB HCI drivers? USB Storage can certainly be loaded from an > initramfs, yet it's =y. LED triggers? Loads of RTC drivers that are > marked as =y? XOR drivers? VIRTIO drivers? Filesystems? I seem to > remember Red Hat sets even popular filesystems to be a module in their > kernel. Almost all your answers above - these should be =m. USB not always, because it is used for USB Ethernet adapters which is used for network boot (although even then initramfs comes with actual drivers...) > > Clearly, "defconfig=y is only for critical stuff" is rather false in > practice, and I suspect is little more than a desire rather than a > rule. I suspect it's more "defconfig=y is for stuff that makes most > platforms successfully boot without an initramfs". Yes, that's true, although there was already pushback against this. Having an initramfs is not a big deal and nowadays I think all new ARMv8 platforms use it. Anyway this talk here is about RTC which is not needed at all as built in. Best regards, Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: "Russell King (Oracle)" <linux@armlinux.org.uk> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>, Shawn Guo <shawnguo@kernel.org>, Li Yang <leoyang.li@nxp.com>, Marek Vasut <marex@denx.de>, Marcel Ziswiler <marcel.ziswiler@toradex.com>, soc@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 8/9] ARM: multi_v7_defconfig: Add options to support TQMLS102xA series Date: Sat, 21 Jan 2023 21:27:52 +0100 [thread overview] Message-ID: <3114bdeb-c8eb-3a3a-0d68-ca3370ce64ec@linaro.org> (raw) In-Reply-To: <Y8qr8wl8WdJfQ9We@shell.armlinux.org.uk> On 20/01/2023 15:57, Russell King (Oracle) wrote: > On Fri, Jan 20, 2023 at 12:35:41PM +0100, Krzysztof Kozlowski wrote: >> Defconfig=y is only for critical stuff needed to bring initramfs. Time >> is not needed for initramfs. > > Are I2C drivers "critical stuff"? What about AHCI drivers? What about > all the USB HCI drivers? USB Storage can certainly be loaded from an > initramfs, yet it's =y. LED triggers? Loads of RTC drivers that are > marked as =y? XOR drivers? VIRTIO drivers? Filesystems? I seem to > remember Red Hat sets even popular filesystems to be a module in their > kernel. Almost all your answers above - these should be =m. USB not always, because it is used for USB Ethernet adapters which is used for network boot (although even then initramfs comes with actual drivers...) > > Clearly, "defconfig=y is only for critical stuff" is rather false in > practice, and I suspect is little more than a desire rather than a > rule. I suspect it's more "defconfig=y is for stuff that makes most > platforms successfully boot without an initramfs". Yes, that's true, although there was already pushback against this. Having an initramfs is not a big deal and nowadays I think all new ARMv8 platforms use it. Anyway this talk here is about RTC which is not needed at all as built in. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-01-21 20:27 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-01-19 14:42 [PATCH 0/9] TQMLS1021A support Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 14:42 ` [PATCH 1/9] dt-bindings: arm: fsl: add TQ-Systems LS1021A board Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 16:59 ` Krzysztof Kozlowski 2023-01-19 16:59 ` Krzysztof Kozlowski 2023-01-20 8:58 ` Alexander Stein 2023-01-20 8:58 ` Alexander Stein 2023-01-20 9:07 ` Krzysztof Kozlowski 2023-01-20 9:07 ` Krzysztof Kozlowski 2023-01-19 14:42 ` [PATCH 2/9] ARM: dts: ls1021a: add TQ-Systems MBLS102xA device tree Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 17:03 ` Krzysztof Kozlowski 2023-01-19 17:03 ` Krzysztof Kozlowski 2023-01-20 10:08 ` Alexander Stein 2023-01-20 10:08 ` Alexander Stein 2023-01-20 10:21 ` Krzysztof Kozlowski 2023-01-20 10:21 ` Krzysztof Kozlowski 2023-01-19 14:42 ` [PATCH 3/9] ARM: dts: ls1021a: add TQMLS1021A flash partition layout Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 14:42 ` [PATCH 4/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS TM070JVHG33 overlay Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 14:42 ` [PATCH 5/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA HDMI overlay Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 14:42 ` [PATCH 6/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS CDTECH DC44 overlay Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 14:42 ` [PATCH 7/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS CDTECH FC21 overlay Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 14:42 ` [PATCH 8/9] ARM: multi_v7_defconfig: Add options to support TQMLS102xA series Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 17:04 ` Krzysztof Kozlowski 2023-01-19 17:04 ` Krzysztof Kozlowski 2023-01-20 11:12 ` Alexander Stein 2023-01-20 11:12 ` Alexander Stein 2023-01-20 11:35 ` Krzysztof Kozlowski 2023-01-20 11:35 ` Krzysztof Kozlowski 2023-01-20 14:57 ` Russell King (Oracle) 2023-01-20 14:57 ` Russell King (Oracle) 2023-01-21 20:27 ` Krzysztof Kozlowski [this message] 2023-01-21 20:27 ` Krzysztof Kozlowski 2023-01-21 22:33 ` Russell King (Oracle) 2023-01-21 22:33 ` Russell King (Oracle) 2023-01-19 14:42 ` [PATCH 9/9] [DNI] ARM: multi_v7_defconfig: Enable CONFIG_ARM_LPAE for multi_v7_config Alexander Stein 2023-01-19 14:42 ` Alexander Stein 2023-01-19 15:09 ` Arnd Bergmann 2023-01-19 15:09 ` Arnd Bergmann 2023-01-19 15:27 ` Alexander Stein 2023-01-19 15:27 ` Alexander Stein 2023-01-19 16:07 ` Arnd Bergmann 2023-01-19 16:07 ` Arnd Bergmann 2023-01-20 12:43 ` Alexander Stein 2023-01-20 12:43 ` Alexander Stein 2023-01-20 14:00 ` Arnd Bergmann 2023-01-20 14:00 ` Arnd Bergmann 2023-01-24 10:30 ` Alexander Stein 2023-01-24 10:30 ` Alexander Stein 2023-01-24 11:37 ` Arnd Bergmann 2023-01-24 11:37 ` Arnd Bergmann 2023-01-19 16:00 ` Russell King (Oracle) 2023-01-19 16:00 ` Russell King (Oracle)
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=3114bdeb-c8eb-3a3a-0d68-ca3370ce64ec@linaro.org \ --to=krzysztof.kozlowski@linaro.org \ --cc=alexander.stein@ew.tq-group.com \ --cc=arnd@arndb.de \ --cc=devicetree@vger.kernel.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=leoyang.li@nxp.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux@armlinux.org.uk \ --cc=marcel.ziswiler@toradex.com \ --cc=marex@denx.de \ --cc=olof@lixom.net \ --cc=robh+dt@kernel.org \ --cc=shawnguo@kernel.org \ --cc=soc@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.