diff mbox series

ARM: dts: logicpd-som-lv-baseboard: Fix broken audio

Message ID 20200814125338.2428209-1-aford173@gmail.com (mailing list archive)
State New, archived
Headers show
Series ARM: dts: logicpd-som-lv-baseboard: Fix broken audio | expand

Commit Message

Adam Ford Aug. 14, 2020, 12:53 p.m. UTC
Older versions of U-Boot would pinmux the whole board, but as
the bootloader got updated, it started to only pinmux the pins
it needed, and expected Linux to configure what it needed.

Unfortunately this caused an issue with the audio, because the
mcbsp2 pins were configured in the device tree but never
referenced by the driver. When U-Boot stopped muxing the audio
pins, the audio died.

This patch adds the references to the associate the pin controller
with the mcbsp2 driver which makes audio operate again.

Fixes: 5cb8b0fa55a9 ("ARM: dts: Move most of logicpd-som-lv-37xx-devkit.dts to logicpd-som-lv-baseboard.dtsi")

Signed-off-by: Adam Ford <aford173@gmail.com>

Comments

Tony Lindgren Aug. 17, 2020, 11:28 a.m. UTC | #1
* Adam Ford <aford173@gmail.com> [200814 15:53]:
> Older versions of U-Boot would pinmux the whole board, but as
> the bootloader got updated, it started to only pinmux the pins
> it needed, and expected Linux to configure what it needed.
> 
> Unfortunately this caused an issue with the audio, because the
> mcbsp2 pins were configured in the device tree but never
> referenced by the driver. When U-Boot stopped muxing the audio
> pins, the audio died.
> 
> This patch adds the references to the associate the pin controller
> with the mcbsp2 driver which makes audio operate again.

Thanks applying into fixes.

Tony
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/logicpd-som-lv-baseboard.dtsi b/arch/arm/boot/dts/logicpd-som-lv-baseboard.dtsi
index 79e9e24328b9..395e05f10d36 100644
--- a/arch/arm/boot/dts/logicpd-som-lv-baseboard.dtsi
+++ b/arch/arm/boot/dts/logicpd-som-lv-baseboard.dtsi
@@ -51,6 +51,8 @@ 
 
 &mcbsp2 {
 	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&mcbsp2_pins>;
 };
 
 &charger {