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
next prev parent reply other threads:[~2023-01-21 20:27 UTC|newest]
Thread overview: 30+ 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 ` [PATCH 1/9] dt-bindings: arm: fsl: add TQ-Systems LS1021A board Alexander Stein
2023-01-19 16:59 ` Krzysztof Kozlowski
2023-01-20 8:58 ` Alexander Stein
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 17:03 ` Krzysztof Kozlowski
2023-01-20 10:08 ` Alexander Stein
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 ` [PATCH 4/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS TM070JVHG33 overlay 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 ` [PATCH 6/9] ARM: dts: ls1021a: add TQMLS1021A/MBLS102xA LVDS CDTECH DC44 overlay 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 ` [PATCH 8/9] ARM: multi_v7_defconfig: Add options to support TQMLS102xA series Alexander Stein
2023-01-19 17:04 ` Krzysztof Kozlowski
2023-01-20 11:12 ` Alexander Stein
2023-01-20 11:35 ` Krzysztof Kozlowski
2023-01-20 14:57 ` Russell King (Oracle)
2023-01-21 20:27 ` Krzysztof Kozlowski [this message]
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 15:09 ` Arnd Bergmann
2023-01-19 15:27 ` Alexander Stein
2023-01-19 16:07 ` Arnd Bergmann
2023-01-20 12:43 ` Alexander Stein
2023-01-20 14:00 ` Arnd Bergmann
2023-01-24 10:30 ` Alexander Stein
2023-01-24 11:37 ` Arnd Bergmann
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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox