* [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings [not found] <20210519001802.1863-1-jonathan@marek.ca> @ 2021-05-19 0:18 ` Jonathan Marek 2021-06-02 21:27 ` Stephen Boyd 0 siblings, 1 reply; 5+ messages in thread From: Jonathan Marek @ 2021-05-19 0:18 UTC (permalink / raw) To: linux-arm-msm Cc: Rob Herring, Andy Gross, Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring, open list:COMMON CLK FRAMEWORK, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list Add sm8350 DISPCC bindings, which are simply a symlink to the sm8250 bindings. Update the documentation with the new compatible. Signed-off-by: Jonathan Marek <jonathan@marek.ca> Reviewed-by: Rob Herring <robh@kernel.org> --- .../devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml | 6 ++++-- include/dt-bindings/clock/qcom,dispcc-sm8350.h | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) create mode 120000 include/dt-bindings/clock/qcom,dispcc-sm8350.h diff --git a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml index 0cdf53f41f84..8f414642445e 100644 --- a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml +++ b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml @@ -4,24 +4,26 @@ $id: http://devicetree.org/schemas/clock/qcom,dispcc-sm8x50.yaml# $schema: http://devicetree.org/meta-schemas/core.yaml# -title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250 +title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250/SM8350 maintainers: - Jonathan Marek <jonathan@marek.ca> description: | Qualcomm display clock control module which supports the clocks, resets and - power domains on SM8150 and SM8250. + power domains on SM8150/SM8250/SM8350. See also: dt-bindings/clock/qcom,dispcc-sm8150.h dt-bindings/clock/qcom,dispcc-sm8250.h + dt-bindings/clock/qcom,dispcc-sm8350.h properties: compatible: enum: - qcom,sm8150-dispcc - qcom,sm8250-dispcc + - qcom,sm8350-dispcc clocks: items: diff --git a/include/dt-bindings/clock/qcom,dispcc-sm8350.h b/include/dt-bindings/clock/qcom,dispcc-sm8350.h new file mode 120000 index 000000000000..0312b4544acb --- /dev/null +++ b/include/dt-bindings/clock/qcom,dispcc-sm8350.h @@ -0,0 +1 @@ +qcom,dispcc-sm8250.h \ No newline at end of file -- 2.26.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings 2021-05-19 0:18 ` [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings Jonathan Marek @ 2021-06-02 21:27 ` Stephen Boyd 2021-06-04 17:25 ` Jonathan Marek 2021-06-06 4:11 ` Bjorn Andersson 0 siblings, 2 replies; 5+ messages in thread From: Stephen Boyd @ 2021-06-02 21:27 UTC (permalink / raw) To: Jonathan Marek, linux-arm-msm Cc: Rob Herring, Andy Gross, Bjorn Andersson, Michael Turquette, Rob Herring, linux-clk, devicetree, linux-kernel Quoting Jonathan Marek (2021-05-18 17:18:02) > Add sm8350 DISPCC bindings, which are simply a symlink to the sm8250 > bindings. Update the documentation with the new compatible. > > Signed-off-by: Jonathan Marek <jonathan@marek.ca> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > .../devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml | 6 ++++-- > include/dt-bindings/clock/qcom,dispcc-sm8350.h | 1 + > 2 files changed, 5 insertions(+), 2 deletions(-) > create mode 120000 include/dt-bindings/clock/qcom,dispcc-sm8350.h Why the symlink? Can we have the dt authors use the existing header file instead? > > diff --git a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > index 0cdf53f41f84..8f414642445e 100644 > --- a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > +++ b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > @@ -4,24 +4,26 @@ > $id: http://devicetree.org/schemas/clock/qcom,dispcc-sm8x50.yaml# > $schema: http://devicetree.org/meta-schemas/core.yaml# > > -title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250 > +title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250/SM8350 Maybe just "Binding for SM8x50 SoCs" > > maintainers: > - Jonathan Marek <jonathan@marek.ca> > > description: | > Qualcomm display clock control module which supports the clocks, resets and > - power domains on SM8150 and SM8250. > + power domains on SM8150/SM8250/SM8350. same 8x50 comment. > > See also: > dt-bindings/clock/qcom,dispcc-sm8150.h > dt-bindings/clock/qcom,dispcc-sm8250.h > + dt-bindings/clock/qcom,dispcc-sm8350.h > > properties: > compatible: > enum: > - qcom,sm8150-dispcc > - qcom,sm8250-dispcc > + - qcom,sm8350-dispcc > > clocks: > items: > diff --git a/include/dt-bindings/clock/qcom,dispcc-sm8350.h b/include/dt-bindings/clock/qcom,dispcc-sm8350.h > new file mode 120000 > index 000000000000..0312b4544acb > --- /dev/null > +++ b/include/dt-bindings/clock/qcom,dispcc-sm8350.h > @@ -0,0 +1 @@ > +qcom,dispcc-sm8250.h > \ No newline at end of file ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings 2021-06-02 21:27 ` Stephen Boyd @ 2021-06-04 17:25 ` Jonathan Marek 2021-06-28 2:39 ` Stephen Boyd 2021-06-06 4:11 ` Bjorn Andersson 1 sibling, 1 reply; 5+ messages in thread From: Jonathan Marek @ 2021-06-04 17:25 UTC (permalink / raw) To: Stephen Boyd, linux-arm-msm Cc: Rob Herring, Andy Gross, Bjorn Andersson, Michael Turquette, Rob Herring, linux-clk, devicetree, linux-kernel On 6/2/21 5:27 PM, Stephen Boyd wrote: > Quoting Jonathan Marek (2021-05-18 17:18:02) >> Add sm8350 DISPCC bindings, which are simply a symlink to the sm8250 >> bindings. Update the documentation with the new compatible. >> >> Signed-off-by: Jonathan Marek <jonathan@marek.ca> >> Reviewed-by: Rob Herring <robh@kernel.org> >> --- >> .../devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml | 6 ++++-- >> include/dt-bindings/clock/qcom,dispcc-sm8350.h | 1 + > >> 2 files changed, 5 insertions(+), 2 deletions(-) >> create mode 120000 include/dt-bindings/clock/qcom,dispcc-sm8350.h > > Why the symlink? Can we have the dt authors use the existing header file > instead? > It would be strange to include bindings with the name of a different SoC. I guess it is a matter a preference, is there any good reason to *not* do it like this? >> >> diff --git a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml >> index 0cdf53f41f84..8f414642445e 100644 >> --- a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml >> +++ b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml >> @@ -4,24 +4,26 @@ >> $id: http://devicetree.org/schemas/clock/qcom,dispcc-sm8x50.yaml# >> $schema: http://devicetree.org/meta-schemas/core.yaml# >> >> -title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250 >> +title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250/SM8350 > > Maybe just "Binding for SM8x50 SoCs" > Its likely these bindings won't be compatible with future "SM8x50" SoCs, listing supported SoCs explicitly will avoid confusion in the future. >> >> maintainers: >> - Jonathan Marek <jonathan@marek.ca> >> >> description: | >> Qualcomm display clock control module which supports the clocks, resets and >> - power domains on SM8150 and SM8250. >> + power domains on SM8150/SM8250/SM8350. > > same 8x50 comment. > >> >> See also: >> dt-bindings/clock/qcom,dispcc-sm8150.h >> dt-bindings/clock/qcom,dispcc-sm8250.h >> + dt-bindings/clock/qcom,dispcc-sm8350.h >> >> properties: >> compatible: >> enum: >> - qcom,sm8150-dispcc >> - qcom,sm8250-dispcc >> + - qcom,sm8350-dispcc >> >> clocks: >> items: >> diff --git a/include/dt-bindings/clock/qcom,dispcc-sm8350.h b/include/dt-bindings/clock/qcom,dispcc-sm8350.h >> new file mode 120000 >> index 000000000000..0312b4544acb >> --- /dev/null >> +++ b/include/dt-bindings/clock/qcom,dispcc-sm8350.h >> @@ -0,0 +1 @@ >> +qcom,dispcc-sm8250.h >> \ No newline at end of file ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings 2021-06-04 17:25 ` Jonathan Marek @ 2021-06-28 2:39 ` Stephen Boyd 0 siblings, 0 replies; 5+ messages in thread From: Stephen Boyd @ 2021-06-28 2:39 UTC (permalink / raw) To: Jonathan Marek, linux-arm-msm Cc: Rob Herring, Andy Gross, Bjorn Andersson, Michael Turquette, Rob Herring, linux-clk, devicetree, linux-kernel Quoting Jonathan Marek (2021-06-04 10:25:41) > On 6/2/21 5:27 PM, Stephen Boyd wrote: > > Quoting Jonathan Marek (2021-05-18 17:18:02) > >> Add sm8350 DISPCC bindings, which are simply a symlink to the sm8250 > >> bindings. Update the documentation with the new compatible. > >> > >> Signed-off-by: Jonathan Marek <jonathan@marek.ca> > >> Reviewed-by: Rob Herring <robh@kernel.org> > >> --- > >> .../devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml | 6 ++++-- > >> include/dt-bindings/clock/qcom,dispcc-sm8350.h | 1 + > > > >> 2 files changed, 5 insertions(+), 2 deletions(-) > >> create mode 120000 include/dt-bindings/clock/qcom,dispcc-sm8350.h > > > > Why the symlink? Can we have the dt authors use the existing header file > > instead? > > > > It would be strange to include bindings with the name of a different > SoC. I guess it is a matter a preference, is there any good reason to > *not* do it like this? $ find include/dt-bindings -type l include/dt-bindings/input/linux-event-codes.h include/dt-bindings/clock/qcom,dispcc-sm8150.h It seems to not be common at all. > > >> > >> diff --git a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > >> index 0cdf53f41f84..8f414642445e 100644 > >> --- a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > >> +++ b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > >> @@ -4,24 +4,26 @@ > >> $id: http://devicetree.org/schemas/clock/qcom,dispcc-sm8x50.yaml# > >> $schema: http://devicetree.org/meta-schemas/core.yaml# > >> > >> -title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250 > >> +title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250/SM8350 > > > > Maybe just "Binding for SM8x50 SoCs" > > > > Its likely these bindings won't be compatible with future "SM8x50" SoCs, > listing supported SoCs explicitly will avoid confusion in the future. The yaml file has sm8x50 in the name. What's the plan there? ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings 2021-06-02 21:27 ` Stephen Boyd 2021-06-04 17:25 ` Jonathan Marek @ 2021-06-06 4:11 ` Bjorn Andersson 1 sibling, 0 replies; 5+ messages in thread From: Bjorn Andersson @ 2021-06-06 4:11 UTC (permalink / raw) To: Stephen Boyd Cc: Jonathan Marek, linux-arm-msm, Rob Herring, Andy Gross, Michael Turquette, Rob Herring, linux-clk, devicetree, linux-kernel On Wed 02 Jun 16:27 CDT 2021, Stephen Boyd wrote: > Quoting Jonathan Marek (2021-05-18 17:18:02) > > Add sm8350 DISPCC bindings, which are simply a symlink to the sm8250 > > bindings. Update the documentation with the new compatible. > > > > Signed-off-by: Jonathan Marek <jonathan@marek.ca> > > Reviewed-by: Rob Herring <robh@kernel.org> > > --- > > .../devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml | 6 ++++-- > > include/dt-bindings/clock/qcom,dispcc-sm8350.h | 1 + > > > 2 files changed, 5 insertions(+), 2 deletions(-) > > create mode 120000 include/dt-bindings/clock/qcom,dispcc-sm8350.h > > Why the symlink? Can we have the dt authors use the existing header file > instead? > > > > > diff --git a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > > index 0cdf53f41f84..8f414642445e 100644 > > --- a/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > > +++ b/Documentation/devicetree/bindings/clock/qcom,dispcc-sm8x50.yaml > > @@ -4,24 +4,26 @@ > > $id: http://devicetree.org/schemas/clock/qcom,dispcc-sm8x50.yaml# > > $schema: http://devicetree.org/meta-schemas/core.yaml# > > > > -title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250 > > +title: Qualcomm Display Clock & Reset Controller Binding for SM8150/SM8250/SM8350 > > Maybe just "Binding for SM8x50 SoCs" > That seems like a certain way to ensure that SM8450 etc will be different enough to not match this binding :) Regards, Bjorn ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-06-28 2:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210519001802.1863-1-jonathan@marek.ca>
2021-05-19 0:18 ` [PATCH v2 2/2] dt-bindings: clock: add QCOM SM8350 display clock bindings Jonathan Marek
2021-06-02 21:27 ` Stephen Boyd
2021-06-04 17:25 ` Jonathan Marek
2021-06-28 2:39 ` Stephen Boyd
2021-06-06 4:11 ` Bjorn Andersson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).