Skip to content

Commit ee346dc

Browse files
JihedChaibigregkh
authored andcommitted
ARM: dts: am335x-sl50: Fix audio bitclock and frame master endpoint
[ Upstream commit 2bc564f ] The cpu_endpoint in mcasp0 specifies the TLV320AIC3106 codec as the bitclock and frame master, but the phandles point to the codec's port node (codec_port) rather than its endpoint node (codec_endpoint). audio-graph-card calls simple_util_parse_daifmt() with ep_codec set to the endpoint node (codec_endpoint). The function resolves the bitclock-master phandle and checks whether it equals ep_codec. Since codec_port is the parent of codec_endpoint, not the endpoint itself, the comparison always evaluates to false. This causes the mcasp0 CPU side to be silently configured as bitclock and frame master instead of the codec, which is the opposite of the intended configuration. Fix by pointing bitclock-master and frame-master to codec_endpoint. Fixes: e5f89db ("ARM: dts: am335x-sl50: use audio-graph-card for sound") Signed-off-by: Jihed Chaibi <jihed.chaibi.dev@gmail.com> Link: https://patch.msgid.link/20260325223411.123666-1-jihed.chaibi.dev@gmail.com Signed-off-by: Kevin Hilman (TI) <khilman@baylibre.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
1 parent 46ca145 commit ee346dc

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

arch/arm/boot/dts/ti/omap/am335x-sl50.dts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -557,8 +557,8 @@
557557
remote-endpoint = <&codec_endpoint>;
558558

559559
dai-format = "dsp_b";
560-
bitclock-master = <&codec_port>;
561-
frame-master = <&codec_port>;
560+
bitclock-master = <&codec_endpoint>;
561+
frame-master = <&codec_endpoint>;
562562
bitclock-inversion;
563563
clocks = <&audio_mclk>;
564564
};

0 commit comments

Comments
 (0)