From: Alexander Stein <alexander.stein@mailbox.org> To: Rob Herring <robh+dt@kernel.org>, Kevin Hilman <khilman@baylibre.com>, Neil Armstrong <narmstrong@baylibre.com>, Jerome Brunet <jbrunet@baylibre.com>, Martin Blumenstingl <martin.blumenstingl@googlemail.com> Cc: Alexander Stein <alexander.stein@mailbox.org>, devicetree@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 1/1] arm64: dts: amlogic: Assign a fixed index to mmc devices Date: Tue, 2 Feb 2021 18:03:28 +0100 [thread overview] Message-ID: <20210202170328.79929-1-alexander.stein@mailbox.org> (raw) In-Reply-To: <20210127230852.66686-1-alexander.stein@mailbox.org> Recently introduced async probe on mmc devices can shuffle block IDs. Pin them to fixed values to ease booting in environments where UUIDs are not practical. Use newly introduced aliases for mmcblk devices from [1]. [1] https://patchwork.kernel.org/patch/11747669/ Commit message taken from commit 0011c6d18277 ("arm64: dts: rockchip: Assign a fixed index to mmc devices on rk3399 boards.") The unconventional order (B, C, A) is due to the fact that sd_emmc_a is (according to the comments) only used for SDIO. AFAICS all boards either have both sd_emmc_b and sd_emmc_c or only one of them enabled. So the alias order should match the previous non-async order for all of them. Signed-off-by: Alexander Stein <alexander.stein@mailbox.org> --- Just for the records, I only tested this on my ODROID-N2 (G12 based) board. arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 6 ++++++ arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi index 9c90d562ada1..15525f3aa4a6 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi @@ -17,6 +17,12 @@ / { #address-cells = <2>; #size-cells = <2>; + aliases { + mmc0 = &sd_emmc_b; + mmc1 = &sd_emmc_c; + mmc2 = &sd_emmc_a; + }; + chosen { #address-cells = <2>; #size-cells = <2>; diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi index 726b91d3a905..769d7e8fda13 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi @@ -21,6 +21,12 @@ / { #address-cells = <2>; #size-cells = <2>; + aliases { + mmc0 = &sd_emmc_b; + mmc1 = &sd_emmc_c; + mmc2 = &sd_emmc_a; + }; + reserved-memory { #address-cells = <2>; #size-cells = <2>; -- 2.30.0
WARNING: multiple messages have this Message-ID
From: Alexander Stein <alexander.stein@mailbox.org> To: Rob Herring <robh+dt@kernel.org>, Kevin Hilman <khilman@baylibre.com>, Neil Armstrong <narmstrong@baylibre.com>, Jerome Brunet <jbrunet@baylibre.com>, Martin Blumenstingl <martin.blumenstingl@googlemail.com> Cc: Alexander Stein <alexander.stein@mailbox.org>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org Subject: [PATCH v2 1/1] arm64: dts: amlogic: Assign a fixed index to mmc devices Date: Tue, 2 Feb 2021 18:03:28 +0100 [thread overview] Message-ID: <20210202170328.79929-1-alexander.stein@mailbox.org> (raw) In-Reply-To: <20210127230852.66686-1-alexander.stein@mailbox.org> Recently introduced async probe on mmc devices can shuffle block IDs. Pin them to fixed values to ease booting in environments where UUIDs are not practical. Use newly introduced aliases for mmcblk devices from [1]. [1] https://patchwork.kernel.org/patch/11747669/ Commit message taken from commit 0011c6d18277 ("arm64: dts: rockchip: Assign a fixed index to mmc devices on rk3399 boards.") The unconventional order (B, C, A) is due to the fact that sd_emmc_a is (according to the comments) only used for SDIO. AFAICS all boards either have both sd_emmc_b and sd_emmc_c or only one of them enabled. So the alias order should match the previous non-async order for all of them. Signed-off-by: Alexander Stein <alexander.stein@mailbox.org> --- Just for the records, I only tested this on my ODROID-N2 (G12 based) board. arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 6 ++++++ arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi index 9c90d562ada1..15525f3aa4a6 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi @@ -17,6 +17,12 @@ / { #address-cells = <2>; #size-cells = <2>; + aliases { + mmc0 = &sd_emmc_b; + mmc1 = &sd_emmc_c; + mmc2 = &sd_emmc_a; + }; + chosen { #address-cells = <2>; #size-cells = <2>; diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi index 726b91d3a905..769d7e8fda13 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi @@ -21,6 +21,12 @@ / { #address-cells = <2>; #size-cells = <2>; + aliases { + mmc0 = &sd_emmc_b; + mmc1 = &sd_emmc_c; + mmc2 = &sd_emmc_a; + }; + reserved-memory { #address-cells = <2>; #size-cells = <2>; -- 2.30.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID
From: Alexander Stein <alexander.stein@mailbox.org> To: Rob Herring <robh+dt@kernel.org>, Kevin Hilman <khilman@baylibre.com>, Neil Armstrong <narmstrong@baylibre.com>, Jerome Brunet <jbrunet@baylibre.com>, Martin Blumenstingl <martin.blumenstingl@googlemail.com> Cc: Alexander Stein <alexander.stein@mailbox.org>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org Subject: [PATCH v2 1/1] arm64: dts: amlogic: Assign a fixed index to mmc devices Date: Tue, 2 Feb 2021 18:03:28 +0100 [thread overview] Message-ID: <20210202170328.79929-1-alexander.stein@mailbox.org> (raw) In-Reply-To: <20210127230852.66686-1-alexander.stein@mailbox.org> Recently introduced async probe on mmc devices can shuffle block IDs. Pin them to fixed values to ease booting in environments where UUIDs are not practical. Use newly introduced aliases for mmcblk devices from [1]. [1] https://patchwork.kernel.org/patch/11747669/ Commit message taken from commit 0011c6d18277 ("arm64: dts: rockchip: Assign a fixed index to mmc devices on rk3399 boards.") The unconventional order (B, C, A) is due to the fact that sd_emmc_a is (according to the comments) only used for SDIO. AFAICS all boards either have both sd_emmc_b and sd_emmc_c or only one of them enabled. So the alias order should match the previous non-async order for all of them. Signed-off-by: Alexander Stein <alexander.stein@mailbox.org> --- Just for the records, I only tested this on my ODROID-N2 (G12 based) board. arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 6 ++++++ arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi index 9c90d562ada1..15525f3aa4a6 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi @@ -17,6 +17,12 @@ / { #address-cells = <2>; #size-cells = <2>; + aliases { + mmc0 = &sd_emmc_b; + mmc1 = &sd_emmc_c; + mmc2 = &sd_emmc_a; + }; + chosen { #address-cells = <2>; #size-cells = <2>; diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi index 726b91d3a905..769d7e8fda13 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi @@ -21,6 +21,12 @@ / { #address-cells = <2>; #size-cells = <2>; + aliases { + mmc0 = &sd_emmc_b; + mmc1 = &sd_emmc_c; + mmc2 = &sd_emmc_a; + }; + reserved-memory { #address-cells = <2>; #size-cells = <2>; -- 2.30.0 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic
next prev parent reply other threads:[~2021-02-02 17:07 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-01-27 23:08 [PATCH 1/1] arm64: dts: meson: g12b: odroid-n2: " Alexander Stein 2021-01-27 23:08 ` Alexander Stein 2021-01-27 23:08 ` Alexander Stein 2021-01-29 8:07 ` Neil Armstrong 2021-01-29 8:07 ` Neil Armstrong 2021-01-29 20:51 ` Alexander Stein 2021-01-29 20:51 ` Alexander Stein 2021-01-29 20:51 ` Alexander Stein 2021-02-02 0:50 ` Kevin Hilman 2021-02-02 0:50 ` Kevin Hilman 2021-02-02 0:50 ` Kevin Hilman 2021-02-02 17:03 ` Alexander Stein [this message] 2021-02-02 17:03 ` [PATCH v2 1/1] arm64: dts: amlogic: " Alexander Stein 2021-02-02 17:03 ` Alexander Stein 2021-02-03 19:20 ` Kevin Hilman 2021-02-03 19:20 ` Kevin Hilman 2021-02-03 19:20 ` Kevin Hilman 2021-02-03 19:28 ` [PATCH v3 " Alexander Stein 2021-02-03 19:28 ` Alexander Stein 2021-02-03 19:28 ` Alexander Stein 2021-02-18 18:09 ` Alexander Stein 2021-02-18 18:09 ` Alexander Stein 2021-02-18 18:09 ` Alexander Stein 2021-03-08 17:43 ` Kevin Hilman 2021-03-08 17:43 ` Kevin Hilman 2021-03-08 17:43 ` Kevin Hilman 2021-10-02 6:56 ` Alexander Stein 2021-10-02 6:56 ` Alexander Stein 2021-10-02 6:56 ` Alexander Stein 2021-10-02 8:49 ` Neil Armstrong 2021-10-02 8:49 ` Neil Armstrong 2021-10-02 8:49 ` Neil Armstrong 2021-10-02 8:54 ` Alexander Stein 2021-10-02 8:54 ` Alexander Stein 2021-10-02 8:54 ` Alexander Stein 2021-03-08 17:39 ` Kevin Hilman 2021-03-08 17:39 ` Kevin Hilman 2021-03-08 17:39 ` Kevin Hilman
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=20210202170328.79929-1-alexander.stein@mailbox.org \ --to=alexander.stein@mailbox.org \ --cc=devicetree@vger.kernel.org \ --cc=jbrunet@baylibre.com \ --cc=khilman@baylibre.com \ --cc=linux-amlogic@lists.infradead.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=martin.blumenstingl@googlemail.com \ --cc=narmstrong@baylibre.com \ --cc=robh+dt@kernel.org \ --subject='Re: [PATCH v2 1/1] arm64: dts: amlogic: Assign a fixed index to mmc devices' \ /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
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.