On 30.12.2022 17:08, Krzysztof Kozlowski wrote: > On 30/12/2022 14:50, Konrad Dybcio wrote: >> - Expand long clock-names into vertical lists >> - Shuffle properties around: >> - Make sure compatible goes first and status goes last >> - Make property order consistent between similar nodes >> - Fix up indentation >> - Remove stray newlines >> - Remove a redundant comment about swr2 being associated with TX macro >> (it's obvious by looking at the label property 10 lines below) >> - Change unnecessary interrupts-extended to interrupts >> - Disable SWR0 and WSA macro by default and enable them on SM8250 MTP and >> RB5, which were the only users >> - Remove stray #address/size-cells from txmacro, as it's not even >> supposed to have children > > You duplicate here some work: > https://lore.kernel.org/all/20221225115844.55126-4-krzysztof.kozlowski@xxxxxxxxxx/ > > and maybe: > https://lore.kernel.org/all/167233461775.1099840.3444272939352778399.b4-ty@xxxxxxxxxx/ > > If you wish to avoid trivial cleanup conflicts, feel free to poke to my > pending branch: > https://github.com/krzk/linux/commits/pending/dt-bindings-qcom-new-and-fixes-for-warnings-linux-next > > Also, this is a mixture of non-functional changes (re-ordering, > whitespace) with something close to functional (interrupts-extended -> > interrupts, disabling nodes in DTSI). These should be split. Thanks for pointing this out, I'll resubmit these soon! Konrad > > > > Best regards, > Krzysztof >
