Messages in this thread | | | Date | Tue, 30 Aug 2022 19:50:25 +0300 | Subject | Re: [PATCH 1/1] arm64: dts: qcom: sc7280: Add device tree for herobrine evoker | From | Krzysztof Kozlowski <> |
| |
On 30/08/2022 19:10, Doug Anderson wrote: > Hi, > > On Tue, Aug 30, 2022 at 2:33 AM Krzysztof Kozlowski > <krzysztof.kozlowski@linaro.org> wrote: >> >>> +}; >>> + >>> +/* >>> + * ADDITIONS TO FIXED REGULATORS DEFINED IN PARENT DEVICE TREE FILES >> >> What does it mean and why it's SCREAMING? >>> + * >>> + * Sort order matches the order in the parent files (parents before children). >> >> Why? Sorting should be rather alphabetical. > > We've had this discussion on the lists in the past. See, for instance: > > https://lore.kernel.org/r/CAD=FV=U2C1W+JHWyGRfyRB=WiPKLYvtjO90UDoJ9p+Xwe09+ow@mail.gmail.com/
Good explanation, such sorting rule is quite nice. The part about regulators is still a bit confusing, I guess it is about some other pieces in some other place? But isn't this kind of obvious from including other DTSI?
Best regards, Krzysztof
| |