| Message ID | 20240828074755.1320692-1-net147@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] arm64: dts: rockchip: Enable RK809 audio codec for Radxa ROCK 4C+ | expand |
On Wed, 28 Aug 2024 17:47:53 +1000, Jonathan Liu wrote: > This adds the necessary device tree changes to enable analog audio > output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with > its RK809 audio codec. > > Signed-off-by: Jonathan Liu <net147@gmail.com> > --- > .../boot/dts/rockchip/rk3399-rock-4c-plus.dts | 46 ++++++++++++++++++- > 1 file changed, 45 insertions(+), 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/rk3399-rock-4c-plus.dtb' for 20240828074755.1320692-1-net147@gmail.com: arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dtb: pmic: vsel1-gpio: {'rockchip,pins': [[1, 17, 0, 194]], 'phandle': 138} is not of type 'array' from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml# arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dtb: pmic: vsel2-gpio: {'rockchip,pins': [[1, 14, 0, 194]], 'phandle': 139} is not of type 'array' from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml#
On Wed, 28 Aug 2024 17:47:53 +1000, Jonathan Liu wrote: > This adds the necessary device tree changes to enable analog audio > output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with > its RK809 audio codec. > > Applied, thanks! [1/1] arm64: dts: rockchip: Enable RK809 audio codec for Radxa ROCK 4C+ commit: 174c306321355d8a189b8f01fcc3f897ba13f227 Best regards,
Hi Jonathan, Am Mittwoch, 28. August 2024, 09:47:53 CEST schrieb Jonathan Liu: > This adds the necessary device tree changes to enable analog audio > output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with > its RK809 audio codec. > > Signed-off-by: Jonathan Liu <net147@gmail.com> as it looks like you have one of those boards, could I possibly as you to take a look at the vdd_cpu_b and vdd_gpu nodes? I see (and devicetree check reports it too) a pinctrl-0 for the vsel-gpio without accompanying pinctrl-names. Could you check if things fail if you either add pinctrl-names or remove the pinctrl-0 entry? Thanks a lot Heiko
Am Mittwoch, 28. August 2024, 14:51:12 CEST schrieb Rob Herring (Arm): > > On Wed, 28 Aug 2024 17:47:53 +1000, Jonathan Liu wrote: > > This adds the necessary device tree changes to enable analog audio > > output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with > > its RK809 audio codec. > > > > Signed-off-by: Jonathan Liu <net147@gmail.com> > > --- > > .../boot/dts/rockchip/rk3399-rock-4c-plus.dts | 46 ++++++++++++++++++- > > 1 file changed, 45 insertions(+), 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/rk3399-rock-4c-plus.dtb' for 20240828074755.1320692-1-net147@gmail.com: > > arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dtb: pmic: vsel1-gpio: {'rockchip,pins': [[1, 17, 0, 194]], 'phandle': 138} is not of type 'array' > from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml# > arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dtb: pmic: vsel2-gpio: {'rockchip,pins': [[1, 14, 0, 194]], 'phandle': 139} is not of type 'array' > from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml# unrelated to the change in question, as that affects silergy regulators. But I just asked Jonathan to check on their board if we can just remove those seemingly unused pinctrl properties.
Hi Heiko, On Wed, 28 Aug 2024 at 22:59, Heiko Stübner <heiko@sntech.de> wrote: > Am Mittwoch, 28. August 2024, 09:47:53 CEST schrieb Jonathan Liu: > > This adds the necessary device tree changes to enable analog audio > > output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with > > its RK809 audio codec. > > > > Signed-off-by: Jonathan Liu <net147@gmail.com> > > as it looks like you have one of those boards, could I possibly as you > to take a look at the vdd_cpu_b and vdd_gpu nodes? > > I see (and devicetree check reports it too) a pinctrl-0 for the vsel-gpio > without accompanying pinctrl-names. > > Could you check if things fail if you either add pinctrl-names or remove > the pinctrl-0 entry? I tried both but neither resulted in any change to the warnings in dtbs_check. Reverting my change also results in the same warnings so I don't think it is related to my change. Regards, Jonathan
Hi Jonathan, Am Mittwoch, 28. August 2024, 15:24:56 CEST schrieb Jonathan Liu: > On Wed, 28 Aug 2024 at 22:59, Heiko Stübner <heiko@sntech.de> wrote: > > Am Mittwoch, 28. August 2024, 09:47:53 CEST schrieb Jonathan Liu: > > > This adds the necessary device tree changes to enable analog audio > > > output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with > > > its RK809 audio codec. > > > > > > Signed-off-by: Jonathan Liu <net147@gmail.com> > > > > as it looks like you have one of those boards, could I possibly as you > > to take a look at the vdd_cpu_b and vdd_gpu nodes? > > > > I see (and devicetree check reports it too) a pinctrl-0 for the vsel-gpio > > without accompanying pinctrl-names. > > > > Could you check if things fail if you either add pinctrl-names or remove > > the pinctrl-0 entry? > > I tried both but neither resulted in any change to the warnings in dtbs_check. > Reverting my change also results in the same warnings so I don't think > it is related to my change. I was more aiming at a functional test on the board :-) . You are correct in that this is not related to your change, I was just seeing someone with the hardware. I.e. while running dtbs check here I saw those regulator nodes and didn't want to change stuff without being able to test them. Heiko
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dts b/arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dts index d4b4dced3e39..14453b286446 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dts +++ b/arch/arm64/boot/dts/rockchip/rk3399-rock-4c-plus.dts @@ -52,6 +52,21 @@ led-1 { }; }; + rk809-sound { + compatible = "simple-audio-card"; + simple-audio-card,format = "i2s"; + simple-audio-card,name = "Analog RK809"; + simple-audio-card,mclk-fs = <256>; + + simple-audio-card,cpu { + sound-dai = <&i2s0>; + }; + + simple-audio-card,codec { + sound-dai = <&rk809>; + }; + }; + sdio_pwrseq: sdio-pwrseq { compatible = "mmc-pwrseq-simple"; clocks = <&rk809 1>; @@ -200,10 +215,13 @@ rk809: pmic@20 { interrupt-parent = <&gpio1>; interrupts = <RK_PC5 IRQ_TYPE_LEVEL_LOW>; #clock-cells = <1>; + clock-names = "mclk"; + clocks = <&cru SCLK_I2S_8CH_OUT>; clock-output-names = "rk808-clkout1", "rk808-clkout2"; pinctrl-names = "default"; - pinctrl-0 = <&pmic_int_l>; + pinctrl-0 = <&pmic_int_l>, <&i2s_8ch_mclk>; rockchip,system-power-controller; + #sound-dai-cells = <0>; wakeup-source; vcc1-supply = <&vcc5v0_sys>; @@ -445,6 +463,10 @@ &i2c3 { status = "okay"; }; +&i2s0 { + status = "okay"; +}; + &i2s2 { status = "okay"; }; @@ -472,6 +494,28 @@ bt_wake_l: bt-wake-l { }; }; + i2s0 { + i2s0_8ch_bus: i2s0-8ch-bus { + rockchip,pins = + <3 RK_PD0 1 &pcfg_pull_none>, + <3 RK_PD2 1 &pcfg_pull_none>, + <3 RK_PD3 1 &pcfg_pull_none>, + <3 RK_PD7 1 &pcfg_pull_none>; + }; + + i2s0_8ch_bus_bclk_off: i2s0-8ch-bus-bclk-off { + rockchip,pins = + <3 RK_PD0 RK_FUNC_GPIO &pcfg_pull_none>, + <3 RK_PD2 1 &pcfg_pull_none>, + <3 RK_PD3 1 &pcfg_pull_none>, + <3 RK_PD7 1 &pcfg_pull_none>; + }; + + i2s_8ch_mclk: i2s-8ch-mclk { + rockchip,pins = <4 RK_PA0 1 &pcfg_pull_none>; + }; + }; + leds { user_led1: user-led1 { rockchip,pins = <3 RK_PD4 RK_FUNC_GPIO &pcfg_pull_none>;
This adds the necessary device tree changes to enable analog audio output for the 3.5 mm TRS headphone jack on the Radxa ROCK 4C+ with its RK809 audio codec. Signed-off-by: Jonathan Liu <net147@gmail.com> --- .../boot/dts/rockchip/rk3399-rock-4c-plus.dts | 46 ++++++++++++++++++- 1 file changed, 45 insertions(+), 1 deletion(-)