On Tue, 18 Jul 2023 13:40:13 +0200, Stephan Gerhold wrote: > SPMI devices typically have a single address cell and no size cell, > i.e. reg = <0xf000> for the audio codec instead of reg = <0xf000 0x200>. > > The example is a bit misleading because it uses the latter. Copying > this into the device tree would be incorrect and was fixed before for > pm8916.dtsi in commit c2f0cbb57dba ("arm64: dts: qcom: pm8916: Remove > invalid reg size from wcd_codec"). > > Make the example more clear by adding the outer "pmic" node which > specifies the same #address/size-cells that would be used in the > real deivce tree. > > Signed-off-by: Stephan Gerhold <stephan@xxxxxxxxxxx> > --- > .../sound/qcom,pm8916-wcd-analog-codec.yaml | 92 ++++++++++++---------- > 1 file changed, 50 insertions(+), 42 deletions(-) > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>