Hi Owen, On Fri, Nov 25, 2022 at 05:44:25PM +0800, Owen Yang wrote: > Add DT for sc7280-herobrine-zombie > > Signed-off-by: Owen Yang <ecs.taipeikernel@xxxxxxxxx> > Changes in v5: > - Overwrite pm8350c_pwm_backlight setting in sc7280-herobrine-zombie.dtsi for chrome zombie. v5. > - Dropping the redundant 'DT binding for' as requested by Krzysztof. v4. > - Adding an empty line here before "/dts-v1/;" in "sc7280-herobrine-zombie-lte.dts", "sc7280-herobrine-zombie.dts" as requested by Matthias. v4. > - Deleteing "/dts-v1/;" in "sc7280-herobrine-zombie.dtsi" as requested by Matthias. v4. > - Droping changing file path in description. v3. as requested by Matthias. v3. > - Changing Patch order, binding patch first and dt file second, as requested by Douglas. v2. > - Adding "arm64: dts: qcom: sc7280:" in dt patch ${SUBJECT}, as requested by Douglas. v2. > - Adding "dt-bindings: arm: qcom:" in bind patch ${SUBJECT}, as requested by Douglas. v2. > - Adding '#include "sc7280-herobrine-wifi-sku.dtsi"' in sc7280-herobrine-zombie.dts, as requested by Douglas. v2. > - Adding "(newest rev)" for zombie entry description in qcom.yaml, as requested by Douglas. v2. > - Adding "post-power-on-delay-ms = <100>;" for trackpad in "sc7280-herobrine-zombie.dtsi". v2 > - Changing "vcc-supply" to "vdd-supply" for trackpad in "sc7280-herobrine-zombie.dtsi", as requested by Douglas. v2. For future patches please better organize the change log like this: Changes in v5: - Overwrite pm8350c_pwm_backlight setting in sc7280-herobrine-zombie.dtsi for chrome zombie Changes in v4: - Dropping the redundant 'DT binding for' as requested by Krzysztof - Adding an empty line here before "/dts-v1/;" in "sc7280-herobrine-zombie-lte.dts", "sc7280-herobrine-zombie.dts" as requested by Matthias - Deleteing "/dts-v1/;" in "sc7280-herobrine-zombie.dtsi" as requested by Matthias ... If you are using patman you can keep the existing 'Commit-changes: vN' entries from the previous versions and just add a new entry for the current version. No need to re-spin the patches just for change log. > diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtsi > new file mode 100644 > index 000000000000..d10362ecdafb > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-zombie.dtsi > > ... > > +&pm8350c_pwm_backlight{ > + pwms = <&pm8350c_pwm 3 200000>; > +}; Why is this change needed?