mbox series

[0/3] arm64: dts: imx93-11x11-evk: Add several sound cards

Message ID 1721897948-6306-1-git-send-email-shengjiu.wang@nxp.com (mailing list archive)
Headers show
Series arm64: dts: imx93-11x11-evk: Add several sound cards | expand

Message

Shengjiu Wang July 25, 2024, 8:59 a.m. UTC
Add bt-sco, PDM microphone, XCVR sound card support.

Shengjiu Wang (3):
  arm64: dts: imx93-11x11-evk: add bt-sco sound card support
  arm64: dts: imx93-11x11-evk: Add PDM microphone sound card support
  arm64: dts: imx93-11x11-evk: Add audio XCVR sound card

 .../boot/dts/freescale/imx93-11x11-evk.dts    | 135 ++++++++++++++++++
 1 file changed, 135 insertions(+)

Comments

Rob Herring (Arm) July 25, 2024, 10:20 p.m. UTC | #1
On Thu, 25 Jul 2024 16:59:05 +0800, Shengjiu Wang wrote:
> Add bt-sco, PDM microphone, XCVR sound card support.
> 
> Shengjiu Wang (3):
>   arm64: dts: imx93-11x11-evk: add bt-sco sound card support
>   arm64: dts: imx93-11x11-evk: Add PDM microphone sound card support
>   arm64: dts: imx93-11x11-evk: Add audio XCVR sound card
> 
>  .../boot/dts/freescale/imx93-11x11-evk.dts    | 135 ++++++++++++++++++
>  1 file changed, 135 insertions(+)
> 
> --
> 2.34.1
> 
> 
> 


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 freescale/imx93-11x11-evk.dtb' for 1721897948-6306-1-git-send-email-shengjiu.wang@nxp.com:

arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: pinctrl@443c0000: 'pdmgrpsleep', 'sai1grpsleep', 'spdifgrpsleep' do not match any of the regexes: 'grp$', 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/pinctrl/fsl,imx9-pinctrl.yaml#
Krzysztof Kozlowski July 28, 2024, 9:08 a.m. UTC | #2
On 26/07/2024 00:20, Rob Herring (Arm) wrote:
> 
> On Thu, 25 Jul 2024 16:59:05 +0800, Shengjiu Wang wrote:
>> Add bt-sco, PDM microphone, XCVR sound card support.
>>
>> Shengjiu Wang (3):
>>   arm64: dts: imx93-11x11-evk: add bt-sco sound card support
>>   arm64: dts: imx93-11x11-evk: Add PDM microphone sound card support
>>   arm64: dts: imx93-11x11-evk: Add audio XCVR sound card
>>
>>  .../boot/dts/freescale/imx93-11x11-evk.dts    | 135 ++++++++++++++++++
>>  1 file changed, 135 insertions(+)
>>
>> --
>> 2.34.1
>>
>>
>>
> 
> 
> 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 freescale/imx93-11x11-evk.dtb' for 1721897948-6306-1-git-send-email-shengjiu.wang@nxp.com:
> 
> arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: pinctrl@443c0000: 'pdmgrpsleep', 'sai1grpsleep', 'spdifgrpsleep' do not match any of the regexes: 'grp$', 'pinctrl-[0-9]+'

And that's a proof that you did not test it.

Is anyone in NXP testing patches before sending?

Best regards,
Krzysztof