Re: [PATCH v2 1/3] arm64: dts: rockchip: Switch RK3399-Gru DP to SPDIF output
From: Heiko Stübner <heiko@sntech.de>
Date: 2022-01-17 08:41:15
Also in:
alsa-devel, dri-devel, linux-devicetree, linux-rockchip, lkml
Am Montag, 17. Januar 2022, 08:44:37 CET schrieb Chen-Yu Tsai:
On Sat, Jan 15, 2022 at 7:03 AM Brian Norris [off-list ref] wrote:quoted
Commit b18c6c3c7768 ("ASoC: rockchip: cdn-dp sound output use spdif") switched the platform to SPDIF, but we didn't fix up the device tree. Drop the pinctrl settings, because the 'spdif_bus' pins are either: * unused (on kevin, bob), so the settings is ~harmless * used by a different function (on scarlet), which causes probe failures (!!)I suppose that means the default pinctrl should be dropped? Or maybe this use case is the outlier. Up to Heiko?
Interesting question. Right now it looks like Gru is the only one using spdif in that way, so I'd think dropping the pinctrl here is the "saner" option at this time ;-) I guess we can reevaluate if this becomes more widespread
quoted
Fixes: b18c6c3c7768 ("ASoC: rockchip: cdn-dp sound output use spdif") Signed-off-by: Brian Norris <briannorris@chromium.org>Reviewed-by: Chen-Yu Tsai <wenst@chromium.org>
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel