On 16/05/2022 17:01, Doug Anderson wrote: > > >> Actually this change should be rather part of that other commit... > > I'm not sure I understand. Which of these are you suggesting? > > 1. You want me to squash the two commits into one, so we add the > generic doc in the same patch as the one adding sc7180 Chromebooks. > > 2. You want the generic doc to come first in the series and it just > adds the little blurb into > "Documentation/devicetree/bindings/arm/qcom.yaml" even though > Chromebooks aren't actually added until a later commit in the series. > > 3. You want to add the sc7180 Chromebooks first in the series with no > documentation and then later add in the generic documentation and the > blurb into "Documentation/devicetree/bindings/arm/qcom.yaml". Sorry for being unclear, I propose the third option. I mean, the hunk referencing "chromebook-boot-flow.rst" should be rather added by the patch adding that document. > I'm also interested to know if you think the generic documentation > patch should land through the Qualcomm tree. I think it would > absolutely have to if I take your suggestion, right? Yes, in that case would be a 5th patch in the set here, so I think everything would go via Qcom SoC. Best regards, Krzysztof
