* [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-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
* 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
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).