| Message ID | 20240623083413.2051412-1-andyshrk@163.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | arm64: dts: rockchip: Fix the i2c address of es8316 on Cool Pi 4B | expand |
On Sunday, 23 June 2024 10:34:13 CEST Andy Yan wrote: > According to the hardware design, the i2c address of audio codec es8316 > on Cool Pi 4B is 0x10. > > ... > --- > > arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts > b/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts index > 20a0a60acd16..091dee37d068 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts > @@ -302,7 +302,7 @@ &i2c7 { > > es8316: audio-codec@11 { Shouldn't this be `@10` then too? > compatible = "everest,es8316"; > - reg = <0x11>; > + reg = <0x10>; > assigned-clocks = <&cru I2S0_8CH_MCLKOUT>; > assigned-clock-rates = <12288000>; > clocks = <&cru I2S0_8CH_MCLKOUT>;
On Sun, 23 Jun 2024 16:34:13 +0800, Andy Yan wrote: > According to the hardware design, the i2c address of audio codec es8316 > on Cool Pi 4B is 0x10. > > This fix the read/write error like bellow: > es8316 7-0011: ASoC: error at soc_component_write_no_lock on es8316.7-0011 for register: [0x0000000c] -6 > es8316 7-0011: ASoC: error at soc_component_write_no_lock on es8316.7-0011 for register: [0x00000003] -6 > es8316 7-0011: ASoC: error at soc_component_read_no_lock on es8316.7-0011 for register: [0x00000016] -6 > es8316 7-0011: ASoC: error at soc_component_read_no_lock on es8316.7-0011 for register: [0x00000016] -6 > > Fixes: 3f5d336d64d6 ("arm64: dts: rockchip: Add support for rk3588s based board Cool Pi 4B") > Signed-off-by: Andy Yan <andyshrk@163.com> > --- > > arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y rockchip/rk3588s-coolpi-4b.dtb' for 20240623083413.2051412-1-andyshrk@163.com: arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts:286.25-300.4: Warning (i2c_bus_reg): /i2c@fec90000/audio-codec@11: I2C bus unit address format error, expected "10"
diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts b/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts index 20a0a60acd16..091dee37d068 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts @@ -302,7 +302,7 @@ &i2c7 { es8316: audio-codec@11 { compatible = "everest,es8316"; - reg = <0x11>; + reg = <0x10>; assigned-clocks = <&cru I2S0_8CH_MCLKOUT>; assigned-clock-rates = <12288000>; clocks = <&cru I2S0_8CH_MCLKOUT>;
According to the hardware design, the i2c address of audio codec es8316 on Cool Pi 4B is 0x10. This fix the read/write error like bellow: es8316 7-0011: ASoC: error at soc_component_write_no_lock on es8316.7-0011 for register: [0x0000000c] -6 es8316 7-0011: ASoC: error at soc_component_write_no_lock on es8316.7-0011 for register: [0x00000003] -6 es8316 7-0011: ASoC: error at soc_component_read_no_lock on es8316.7-0011 for register: [0x00000016] -6 es8316 7-0011: ASoC: error at soc_component_read_no_lock on es8316.7-0011 for register: [0x00000016] -6 Fixes: 3f5d336d64d6 ("arm64: dts: rockchip: Add support for rk3588s based board Cool Pi 4B") Signed-off-by: Andy Yan <andyshrk@163.com> --- arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)