| Message ID | 20230526180712.8481-2-y.oudjana@protonmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/3] dt-bindings: media: camss: qcom,msm8996-camss: Add CAMSS power domain | expand |
On Fri, May 26 2023 at 10:36:32 PM +02:00:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote: > > > On 26.05.2023 22:21, Bryan O'Donoghue wrote: >> On 26/05/2023 21:19, Conor Dooley wrote: >>> On Fri, May 26, 2023 at 09:05:47PM +0100, Bryan O'Donoghue wrote: >>>> On 26/05/2023 20:46, Conor Dooley wrote: >>>>>> + - power-domain-names >>>>> Why is this now required? >>>>> >>>>> Thanks, >>>>> Conor. >>>>> >>>> >>>> Its an accurate description of the power/clock tree to have the >>>> top power >>>> domain be switched on prior to the clocks that depend on it. >>> >>> But what does that have to do with the *names* now being required? >> >> oh the names >> >> no toss that > this should be > > if:properties:compatible:blahblahmsm8996:then:required:power-domain-names The only compatible in this binding is qcom,msm8996-camss, so what would this achieve? > > Konrad >> >>> >>>> I think Yassine, you could probably include the majority of your >>>> cover-letter text in this commit to explain this change a bit >>>> better. >>> >>> I think it would be good to have that regardless. >>> >>> Cheers, >>> Conor. >>> >>>> bod >>> ^^^ I've been trying not to think about rugby since the weekend >>> :( >> >> Pockets O'Gara should have his paddy papers rescinded
On 27.05.2023 08:05, Yassine Oudjana wrote: > > On Fri, May 26 2023 at 10:36:32 PM +02:00:00, Konrad Dybcio <konrad.dybcio@linaro.org> wrote: >> >> >> On 26.05.2023 22:21, Bryan O'Donoghue wrote: >>> On 26/05/2023 21:19, Conor Dooley wrote: >>>> On Fri, May 26, 2023 at 09:05:47PM +0100, Bryan O'Donoghue wrote: >>>>> On 26/05/2023 20:46, Conor Dooley wrote: >>>>>>> + - power-domain-names >>>>>> Why is this now required? >>>>>> >>>>>> Thanks, >>>>>> Conor. >>>>>> >>>>> >>>>> Its an accurate description of the power/clock tree to have the top power >>>>> domain be switched on prior to the clocks that depend on it. >>>> >>>> But what does that have to do with the *names* now being required? >>> >>> oh the names >>> >>> no toss that >> this should be >> >> if:properties:compatible:blahblahmsm8996:then:required:power-domain-names > > The only compatible in this binding is qcom,msm8996-camss, so what would this achieve? You're right - I didn't notice and assumed it was a camss-common one. Konrad > >> >> Konrad >>> >>>> >>>>> I think Yassine, you could probably include the majority of your >>>>> cover-letter text in this commit to explain this change a bit better. >>>> >>>> I think it would be good to have that regardless. >>>> >>>> Cheers, >>>> Conor. >>>> >>>>> bod >>>> ^^^ I've been trying not to think about rugby since the weekend :( >>> >>> Pockets O'Gara should have his paddy papers rescinded > >
diff --git a/Documentation/devicetree/bindings/media/qcom,msm8996-camss.yaml b/Documentation/devicetree/bindings/media/qcom,msm8996-camss.yaml index 8a10aa1cafc5..27c9a11f0df9 100644 --- a/Documentation/devicetree/bindings/media/qcom,msm8996-camss.yaml +++ b/Documentation/devicetree/bindings/media/qcom,msm8996-camss.yaml @@ -85,6 +85,13 @@ properties: items: - description: VFE0 GDSC - Video Front End, Global Distributed Switch Controller. - description: VFE1 GDSC - Video Front End, Global Distributed Switch Controller. + - description: CAMSS GDSC - Camera Subsystem, Global Distributed Switch Controller. + + power-domain-names: + items: + - const: vfe0 + - const: vfe1 + - const: camss ports: $ref: /schemas/graph.yaml#/properties/ports @@ -209,6 +216,7 @@ required: - interrupts - iommus - power-domains + - power-domain-names - reg - reg-names - vdda-supply @@ -326,7 +334,10 @@ examples: <&vfe_smmu 3>; power-domains = <&mmcc VFE0_GDSC>, - <&mmcc VFE1_GDSC>; + <&mmcc VFE1_GDSC>, + <&mmcc CAMSS_GDSC>; + + power-domain-names = "vfe0", "vfe1", "camss"; reg = <0x00a34000 0x1000>, <0x00a00030 0x4>,