Re: [PATCH v6 1/5] media: dt-bindings: Add qcom,sc7280-camss

Krzysztof Kozlowski
2024-11-28T10:09:53Z

On 28/11/2024 10:43, Vikram Sharma wrote:
> Hi Bryan/Vladimir/Krzysztof,
>
> Thanks for your comments and suggestions.
> To address below warning.I guess sorting reg entries on the basis of
> register address is better idea than sorting alphanumerically with
> reg-name. Please confirm if I can sort it based on register addresses
> for V7 of this series?
>
> arch/arm64/boot/dts/qcom/sc7280.dtsi:4429.24-4597.5: Warning
> (simple_bus_reg): /soc@0/camss@acaf000: simple-bus unit address format
> error, expected "acb3000"
>
These two topics are not related. There was a feedback on this from Rob:
binding does not know addresses, so how can you enforce in the binding
sorting by address? Anyway, we talked about this so much already I am
not going back there. Use the tools to produce correct DTS and binding,
whatever you decide.

Best regards,
Krzysztof