public inbox for linux-wireless@vger.kernel.org
 help / color / mirror / Atom feed
From: Xianwei Zhao <xianwei.zhao@amlogic.com>
To: Ronald Claveau <linux-kernel-dev@aliel.fr>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Jerome Brunet <jbrunet@baylibre.com>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	Johannes Berg <johannes@sipsolutions.net>,
	van Spriel <arend@broadcom.com>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-amlogic@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org,
	linux-wireless@vger.kernel.org
Subject: Re: [PATCH v3 1/9] arm64: dts: amlogic: t7: Add eMMC, SD card and SDIO pinctrl nodes
Date: Wed, 25 Mar 2026 10:48:10 +0800	[thread overview]
Message-ID: <316efeff-477d-4d8d-9c60-ac7c4e5b9b33@amlogic.com> (raw)
In-Reply-To: <20260323-add-emmc-t7-vim4-v3-1-5159d90a984c@aliel.fr>

Hi Ronald,

On 2026/3/23 17:55, Ronald Claveau wrote:
> These pinctrl nodes are required by the eMMC, SD card and SDIO drivers
> to configure pin muxing at runtime.
> 
> - eMMC: control, 4-bit/8-bit data, data strobe and clock gate pins
> - SD card: data, clock, command and clock gate pins
> - SDIO: data, clock, command and clock gate pins
> 
> Signed-off-by: Ronald Claveau<linux-kernel-dev@aliel.fr>
> ---
>   arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi | 98 +++++++++++++++++++++++++++++
>   1 file changed, 98 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> index 6510068bcff92..ac8de8e9b8010 100644
> --- a/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> @@ -250,6 +250,104 @@ gpio: bank@4000 {
>                                          #gpio-cells = <2>;
>                                          gpio-ranges = <&periphs_pinctrl 0 0 157>;
>                                  };
> +
> +                               emmc_ctrl_pins: emmc-ctrl {
> +                                       mux-0 {
> +                                               groups = "emmc_cmd";
> +                                               function = "emmc";
> +                                               bias-pull-up;
> +                                       };
> +
> +                                       mux-1 {
> +                                               groups = "emmc_clk";
> +                                               function = "emmc";
> +                                               bias-disable;
> +                                       };
> +                               };
> +
> +                               emmc_data_4b_pins: emmc-data-4b {
> +                                       mux-0 {
> +                                               groups = "emmc_nand_d0",
> +                                                        "emmc_nand_d1",
> +                                                        "emmc_nand_d2",
> +                                                        "emmc_nand_d3";
> +                                               function = "emmc";
> +                                               bias-pull-up;
> +                                       };
> +                               };
> +
> +                               emmc_data_8b_pins: emmc-data-8b {
> +                                       mux-0 {
> +                                               groups = "emmc_nand_d0",
> +                                                        "emmc_nand_d1",
> +                                                        "emmc_nand_d2",
> +                                                        "emmc_nand_d3",
> +                                                        "emmc_nand_d4",
> +                                                        "emmc_nand_d5",
> +                                                        "emmc_nand_d6",
> +                                                        "emmc_nand_d7";
> +                                               function = "emmc";
> +                                               bias-pull-up;
> +                                       };
> +                               };
> +
> +                               emmc_ds_pins: emmc-ds {
> +                                       mux {
> +                                               groups = "emmc_nand_ds";
> +                                               function = "emmc";
> +                                               bias-pull-down;
> +                                       };
> +                               };
> +
> +                               emmc_clk_gate_pins: emmc_clk_gate {

Node names should use hyphens ('-') instead of underscores ('_'), 
consistent with the following nodes.

> +                                       mux {
> +                                               groups = "GPIOB_8";
> +                                               function = "gpio_periphs";
> +                                               bias-pull-down;
> +                                       };
> +                               };
> +
> +                               sdcard_pins: sdcard {
> +                                       mux {
> +                                               groups = "sdcard_d0",
> +                                                        "sdcard_d1",
> +                                                        "sdcard_d2",
> +                                                        "sdcard_d3",
> +                                                        "sdcard_clk",
> +                                                        "sdcard_cmd";
> +                                               function = "sdcard";
> +                                               bias-pull-up;
> +                                       };
> +                               };
> +
> +                               sdcard_clk_gate_pins: sdcard_clk_gate {
> +                                       mux {
> +                                               groups = "GPIOC_4";
> +                                               function = "gpio_periphs";
> +                                               bias-pull-down;
> +                                       };
> +                               };
> +
> +                               sdio_pins: sdio {
> +                                       mux-0 {
> +                                               groups = "sdio_d0",
> +                                                        "sdio_d1",
> +                                                        "sdio_d2",
> +                                                        "sdio_d3",
> +                                                        "sdio_clk",
> +                                                        "sdio_cmd";
> +                                               function = "sdio";
> +                                               bias-pull-up;
> +                                       };
> +                               };
> +
> +                               sdio_clk_gate_pins: sdio_clk_gate {
> +                                       mux {
> +                                               groups = "GPIOX_4";
> +                                               function = "gpio_periphs";
> +                                               bias-pull-up;
> +                                       };
> +                               };
>                          };
> 
>                          gpio_intc: interrupt-controller@4080 {

  reply	other threads:[~2026-03-25  2:48 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-23  9:55 [PATCH v3 0/9] arm64: dts: amlogic: Add MMC/SD/SDIO support for Khadas VIM4 (Amlogic T7) Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 1/9] arm64: dts: amlogic: t7: Add eMMC, SD card and SDIO pinctrl nodes Ronald Claveau
2026-03-25  2:48   ` Xianwei Zhao [this message]
2026-03-25  8:40     ` Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 2/9] dt-bindings: mmc: amlogic: Add compatible for T7 mmc Ronald Claveau
2026-03-23 19:38   ` Conor Dooley
2026-03-24 15:03   ` Ulf Hansson
2026-03-25  2:38   ` Xianwei Zhao
2026-03-25  8:45     ` Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 3/9] arm64: dts: amlogic: t7: Add MMC controller nodes Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 4/9] arm64: dts: amlogic: t7: Add PWM pinctrl nodes Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 5/9] arm64: dts: amlogic: t7: Add PWM controller nodes Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 6/9] arm64: dts: amlogic: t7: khadas-vim4: Add power regulators Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 7/9] arm64: dts: amlogic: t7: khadas-vim4: Add SDIO power sequence and WiFi clock Ronald Claveau
2026-03-23  9:55 ` [PATCH v3 8/9] dt-bindings: net: wireless: brcm: Add compatible for bcm43752 Ronald Claveau
2026-03-23 19:37   ` Conor Dooley
2026-03-23  9:55 ` [PATCH v3 9/9] arm64: dts: amlogic: t7: khadas-vim4: Add MMC nodes Ronald Claveau

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=316efeff-477d-4d8d-9c60-ac7c4e5b9b33@amlogic.com \
    --to=xianwei.zhao@amlogic.com \
    --cc=arend@broadcom.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jbrunet@baylibre.com \
    --cc=johannes@sipsolutions.net \
    --cc=khilman@baylibre.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel-dev@aliel.fr \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=martin.blumenstingl@googlemail.com \
    --cc=neil.armstrong@linaro.org \
    --cc=robh@kernel.org \
    --cc=ulf.hansson@linaro.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