public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: "Arnd Bergmann" <arnd@arndb.de>
To: "Stefan Dösinger" <stefandoesinger@gmail.com>,
	"Jonathan Corbet" <corbet@lwn.net>,
	"Shuah Khan" <skhan@linuxfoundation.org>,
	"Russell King" <linux@armlinux.org.uk>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
	"Linus Walleij" <linusw@kernel.org>,
	"Drew Fustini" <fustini@kernel.org>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Jiri Slaby" <jirislaby@kernel.org>
Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	soc@lists.linux.dev, linux-serial@vger.kernel.org
Subject: Re: [PATCH v5 8/8] ARM: defconfig: Add a zx29 defconfig file
Date: Fri, 24 Apr 2026 09:48:46 +0200	[thread overview]
Message-ID: <61452117-0cdc-4ec2-83eb-dc03ccbd410b@app.fastmail.com> (raw)
In-Reply-To: <20260421-send-v5-8-ace038e63515@gmail.com>

On Tue, Apr 21, 2026, at 22:23, Stefan Dösinger wrote:
> This enables existing drivers that already are (UART) or will be (USB,
> GPIO) necessary to operate this board even if they aren't declared in
> the DTS yet.
>
> Signed-off-by: Stefan Dösinger <stefandoesinger@gmail.com>

I'll reply to Linus' comment as well, the defconfigs are generally
not in a great shape across many platforms, so we should come up
with some better policies there.

Either way, the patch description above should at least explain
why you think you need your own defconfig, as we don't normally
take those.

Some comments about the contents of this file:

> +++ b/arch/arm/configs/zx29_defconfig
> @@ -0,0 +1,89 @@
> +CONFIG_SYSVIPC=y
> +CONFIG_BLK_DEV_INITRD=y
> +# CONFIG_RD_BZIP2 is not set
> +# CONFIG_RD_LZMA is not set
> +# CONFIG_RD_XZ is not set
> +# CONFIG_RD_LZ4 is not set
> +CONFIG_EXPERT=y

What is the reason for CONFIG_EXPERT here? Can you avoid this?

> +CONFIG_CMDLINE="console=ttyAMA0 earlyprintk root=/dev/ram rw"

A definconfig should normall not rely on earlyprintk, just add
that when you actually need to debug the super-early boot
stages. With "earlycon" it should pick up the right console
from the stdout path and work almost as early.

> +CONFIG_BINFMT_FLAT=y

Are you actually using flat binaries? I wasn't aware that this
is still possible on MMU-enabled kernels.

> +CONFIG_BLK_DEV_RAM=y
> +CONFIG_BLK_DEV_RAM_COUNT=4

The old ramdisk boot is going away in the future, please use
initramfs instead. This should also save a good amount of RAM.

> +CONFIG_DEVTMPFS=y       # FIXME: This is specific to my initrd. Remove 
> before upstream

stale comment?

> +CONFIG_CONFIG_TMPFS=y

Typo?


       Arnd


      parent reply	other threads:[~2026-04-24  7:49 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-21 20:23 [PATCH v5 0/8] Add support for ZTE zx297520v3 Stefan Dösinger
2026-04-21 20:23 ` [PATCH v5 1/8] ARM: zte: Add zx297520v3 platform support Stefan Dösinger
2026-04-24  7:06   ` Linus Walleij
2026-04-24 11:23     ` Stefan Dösinger
2026-04-21 20:23 ` [PATCH v5 2/8] dt-bindings: arm: Add zx297520v3 board binding Stefan Dösinger
2026-04-22  7:06   ` Krzysztof Kozlowski
2026-04-21 20:23 ` [PATCH v5 3/8] ARM: dts: Add D-Link DWR-932M support Stefan Dösinger
2026-04-22  7:09   ` Krzysztof Kozlowski
2026-04-21 20:23 ` [PATCH v5 4/8] ARM: zte: Add support for zx29 low level debug Stefan Dösinger
2026-04-24  7:07   ` Linus Walleij
2026-04-21 20:23 ` [PATCH v5 5/8] ARM: dts: Add an armv7 timer for zx297520v3 Stefan Dösinger
2026-04-22  7:10   ` Krzysztof Kozlowski
2026-04-21 20:23 ` [PATCH v5 6/8] amba/serial: amba-pl011: Bring back zx29 UART support Stefan Dösinger
2026-04-24  7:05   ` Linus Walleij
2026-04-21 20:23 ` [PATCH v5 7/8] ARM: dts: Declare UARTs on zx297520v3 boards Stefan Dösinger
2026-04-22  7:10   ` Krzysztof Kozlowski
2026-04-22  7:11     ` Krzysztof Kozlowski
2026-04-21 20:23 ` [PATCH v5 8/8] ARM: defconfig: Add a zx29 defconfig file Stefan Dösinger
2026-04-24  7:13   ` Linus Walleij
2026-04-24  8:54     ` Arnd Bergmann
2026-04-24  7:48   ` Arnd Bergmann [this message]

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=61452117-0cdc-4ec2-83eb-dc03ccbd410b@app.fastmail.com \
    --to=arnd@arndb.de \
    --cc=alexandre.belloni@bootlin.com \
    --cc=conor+dt@kernel.org \
    --cc=corbet@lwn.net \
    --cc=devicetree@vger.kernel.org \
    --cc=fustini@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jirislaby@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linusw@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=robh@kernel.org \
    --cc=skhan@linuxfoundation.org \
    --cc=soc@lists.linux.dev \
    --cc=stefandoesinger@gmail.com \
    /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