All of lore.kernel.org
 help / color / mirror / Atom feed
From: sashiko-bot@kernel.org
To: "Stefano Radaelli" <stefano.radaelli21@gmail.com>
Cc: devicetree@vger.kernel.org, imx@lists.linux.dev, robh@kernel.org,
	Frank.Li@kernel.org, conor+dt@kernel.org
Subject: Re: [PATCH v3 11/12] ARM: dts: imx6ulz: Add Variscite i.MX6ULZ board variants
Date: Mon, 28 Sep 2026 12:57:39 +0000	[thread overview]
Message-ID: <20260928125739.BCB7E1F00898@smtp.kernel.org> (raw)
In-Reply-To: <4e63d123736daddc18dcddeabe96b6e2bfd65611.1790512621.git.stefano.r@variscite.com>

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dts: combinatorial explosion of static DT permutations
--

commit fe23e5c3bff1e80a38291b82cb1ef5a3d2caeacc
Author: Stefano Radaelli <stefano.r@variscite.com>

ARM: dts: imx6ulz: Add Variscite i.MX6ULZ board variants

This commit adds 30 new static DTS files for various combinations of
carrier, storage, wireless, and audio options on Variscite i.MX6ULZ boards.

> diff --git a/arch/arm/boot/dts/nxp/imx/imx6ulz-var-dart-emmc-sd-card-6ulcustomboard.dts b/arch/arm/boot/dts/nxp/imx/imx6ulz-var-dart-emmc-sd-card-6ulcustomboard.dts
> new file mode 100644
> index 0000000000000..612246fe5638e
> --- /dev/null
> +++ b/arch/arm/boot/dts/nxp/imx/imx6ulz-var-dart-emmc-sd-card-6ulcustomboard.dts
> @@ -0,0 +1,21 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright (C) 2026 Variscite Ltd. - https://www.variscite.com
> + */
> +
> +/dts-v1/;
> +
> +#include "imx6ulz.dtsi"
> +#include "imx6ulz-var-dart.dtsi"
> +#include "imx6ul-imx6ull-var-emmc.dtsi"
> +#include "imx6ul-imx6ull-var-dart-wm8904.dtsi"
> +#include "imx6ulz-var-dart-6ulcustomboard.dtsi"
> +#include "imx6ul-imx6ull-var-dart-6ulcustomboard-sd-card.dtsi"
> +
> +/ {
> +	model = "Variscite DART-6UL with i.MX6ULZ, eMMC, SD card & WM8904 support on VAR-6ULCustomBoard";
> +	compatible = "variscite,dart-6ul-imx6ulz-emmc-sd-card-6ulcustomboard",
> +		     "variscite,dart-6ul-imx6ulz",
> +		     "fsl,imx6ull",
> +		     "fsl,imx6ulz";
> +};

[Severity: Low]
Does this approach create a combinatorial explosion of static device tree
permutations?

Creating a static .dts file for every combination of storage, Wi-Fi, and audio
options adds 30 new files for the i.MX6ULZ variant alone.

Could device tree overlays (.dtbo) be used here instead of generating static
files for each hardware combination?

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/cover.1790512621.git.stefano.r@variscite.com?part=11

  reply	other threads:[~2026-09-28 12:57 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-27 12:46 [PATCH v3 00/12] ARM: dts: imx6ul: Add Variscite VAR-SOM-6UL and DART-6UL Stefano Radaelli
2026-09-27 12:46 ` [PATCH v3 01/12] dt-bindings: arm: fsl: Document Variscite i.MX6UL boards Stefano Radaelli
2026-09-29  8:49   ` Krzysztof Kozlowski
2026-09-29  9:06     ` Stefano Radaelli
2026-09-29  9:32       ` Krzysztof Kozlowski
2026-09-29  9:41         ` Stefano Radaelli
2026-09-27 12:46 ` [PATCH v3 02/12] net: phy: micrel: Check RMII clock select property presence Stefano Radaelli
2026-09-27 12:46 ` [PATCH v3 03/12] dt-bindings: net: micrel: Fix RMII clock select property type Stefano Radaelli
2026-09-27 12:46 ` [PATCH v3 04/12] ARM: dts: imx6ul: Add Variscite SoM and DART base files Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot
2026-09-27 12:46 ` [PATCH v3 05/12] ARM: dts: imx6ul: Add Variscite audio codec files Stefano Radaelli
2026-09-27 12:46 ` [PATCH v3 06/12] ARM: dts: imx6ul: Add Variscite storage and wireless files Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot
2026-09-27 12:46 ` [PATCH v3 07/12] ARM: dts: imx6ul: Add Variscite carrier board files Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot
2026-09-27 12:46 ` [PATCH v3 08/12] ARM: dts: imx6ul: Reuse Variscite DTSI files for Concerto Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot
2026-09-27 12:46 ` [PATCH v3 09/12] ARM: dts: imx6ul: Add Variscite i.MX6UL board variants Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot
2026-09-27 12:46 ` [PATCH v3 10/12] ARM: dts: imx6ull: Add Variscite i.MX6ULL " Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot
2026-09-27 12:46 ` [PATCH v3 11/12] ARM: dts: imx6ulz: Add Variscite i.MX6ULZ " Stefano Radaelli
2026-09-28 12:57   ` sashiko-bot [this message]
2026-09-28 19:55   ` Frank Li
2026-09-27 12:46 ` [PATCH v3 12/12] ARM: dts: imx6ul: Enable Variscite i.MX6UL board DTBs Stefano Radaelli

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=20260928125739.BCB7E1F00898@smtp.kernel.org \
    --to=sashiko-bot@kernel.org \
    --cc=Frank.Li@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=imx@lists.linux.dev \
    --cc=robh@kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    --cc=stefano.radaelli21@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 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.