Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4b97dab7 authored by Heiner Kallweit's avatar Heiner Kallweit Committed by Greg Kroah-Hartman
Browse files

arm64: dts: meson-g12-common: Make mmc host controller interrupts level-sensitive 

commit ac8db4cceed218cca21c84f9d75ce88182d8b04f upstream.

The usage of edge-triggered interrupts lead to lost interrupts under load,
see [0]. This was confirmed to be fixed by using level-triggered
interrupts.
The report was about SDIO. However, as the host controller is the same
for SD and MMC, apply the change to all mmc controller instances.

[0] https://www.spinics.net/lists/linux-mmc/msg73991.html



Fixes: 4759fd87 ("arm64: dts: meson: g12a: add mmc nodes")
Tested-by: default avatarFUKAUMI Naoki <naoki@radxa.com>
Tested-by: default avatarMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Tested-by: default avatarJerome Brunet <jbrunet@baylibre.com>
Cc: stable@vger.kernel.org
Signed-off-by: default avatarHeiner Kallweit <hkallweit1@gmail.com>
Acked-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://lore.kernel.org/r/27d89baa-b8fa-baca-541b-ef17a97cde3c@gmail.com


Signed-off-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f0302e98

arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi

+3 −3
Original line number Diff line number Diff line
@@ -2317,7 +2317,7 @@ 
		sd_emmc_a: sd@ffe03000 {

			compatible = "amlogic,meson-axg-mmc";

			reg = <0x0 0xffe03000 0x0 0x800>;

			interrupts = <GIC_SPI 189 IRQ_TYPE_EDGE_RISING>;

			interrupts = <GIC_SPI 189 IRQ_TYPE_LEVEL_HIGH>;

			status = "disabled";

			clocks = <&clkc CLKID_SD_EMMC_A>,

				 <&clkc CLKID_SD_EMMC_A_CLK0>,
@@ -2329,7 +2329,7 @@ 
		sd_emmc_b: sd@ffe05000 {

			compatible = "amlogic,meson-axg-mmc";

			reg = <0x0 0xffe05000 0x0 0x800>;

			interrupts = <GIC_SPI 190 IRQ_TYPE_EDGE_RISING>;

			interrupts = <GIC_SPI 190 IRQ_TYPE_LEVEL_HIGH>;

			status = "disabled";

			clocks = <&clkc CLKID_SD_EMMC_B>,

				 <&clkc CLKID_SD_EMMC_B_CLK0>,
@@ -2341,7 +2341,7 @@ 
		sd_emmc_c: mmc@ffe07000 {

			compatible = "amlogic,meson-axg-mmc";

			reg = <0x0 0xffe07000 0x0 0x800>;

			interrupts = <GIC_SPI 191 IRQ_TYPE_EDGE_RISING>;

			interrupts = <GIC_SPI 191 IRQ_TYPE_LEVEL_HIGH>;

			status = "disabled";

			clocks = <&clkc CLKID_SD_EMMC_C>,

				 <&clkc CLKID_SD_EMMC_C_CLK0>,