* [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk
@ 2022-07-28 11:16 Shinjo Park
2022-07-28 11:51 ` Krzysztof Kozlowski
0 siblings, 1 reply; 7+ messages in thread
From: Shinjo Park @ 2022-07-28 11:16 UTC (permalink / raw)
Cc: peremen, David Heidelberg, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
devicetree, linux-kernel
Change the reference of sleep_clk to the same as qcom-apq8064.dtsi.
Signed-off-by: Shinjo Park <peremen@gmail.com>
Reviewed-by: David Heidelberg <david@ixit.cz>
---
arch/arm/boot/dts/qcom-msm8960.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
index e8cd1c9c0..991eb1948 100644
--- a/arch/arm/boot/dts/qcom-msm8960.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
@@ -71,7 +71,7 @@ pxo_board: pxo_board {
clock-output-names = "pxo_board";
};
- sleep_clk {
+ sleep_clk: sleep_clk {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32768>;
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk
2022-07-28 11:16 [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk Shinjo Park
@ 2022-07-28 11:51 ` Krzysztof Kozlowski
2022-07-29 11:33 ` Dmitry Baryshkov
0 siblings, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-28 11:51 UTC (permalink / raw)
To: Shinjo Park
Cc: David Heidelberg, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm, devicetree,
linux-kernel
On 28/07/2022 13:16, Shinjo Park wrote:
> Change the reference of sleep_clk to the same as qcom-apq8064.dtsi.
You add label, not change something.
>
> Signed-off-by: Shinjo Park <peremen@gmail.com>
> Reviewed-by: David Heidelberg <david@ixit.cz>
> ---
> arch/arm/boot/dts/qcom-msm8960.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
> index e8cd1c9c0..991eb1948 100644
> --- a/arch/arm/boot/dts/qcom-msm8960.dtsi
> +++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
> @@ -71,7 +71,7 @@ pxo_board: pxo_board {
> clock-output-names = "pxo_board";
> };
>
> - sleep_clk {
> + sleep_clk: sleep_clk {
Since you touch the line, make the device node sleep-clk (device node
names should not have underscores) and mention this in commit msg.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk
2022-07-28 11:51 ` Krzysztof Kozlowski
@ 2022-07-29 11:33 ` Dmitry Baryshkov
2022-07-29 12:01 ` Konrad Dybcio
0 siblings, 1 reply; 7+ messages in thread
From: Dmitry Baryshkov @ 2022-07-29 11:33 UTC (permalink / raw)
To: Krzysztof Kozlowski, Shinjo Park
Cc: David Heidelberg, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm, devicetree,
linux-kernel
On 28/07/2022 14:51, Krzysztof Kozlowski wrote:
> On 28/07/2022 13:16, Shinjo Park wrote:
>> Change the reference of sleep_clk to the same as qcom-apq8064.dtsi.
>
> You add label, not change something.
>
>>
>> Signed-off-by: Shinjo Park <peremen@gmail.com>
>> Reviewed-by: David Heidelberg <david@ixit.cz>
>> ---
>> arch/arm/boot/dts/qcom-msm8960.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
>> index e8cd1c9c0..991eb1948 100644
>> --- a/arch/arm/boot/dts/qcom-msm8960.dtsi
>> +++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
>> @@ -71,7 +71,7 @@ pxo_board: pxo_board {
>> clock-output-names = "pxo_board";
>> };
>>
>> - sleep_clk {
>> + sleep_clk: sleep_clk {
>
> Since you touch the line, make the device node sleep-clk (device node
> names should not have underscores) and mention this in commit msg.
Then we are back to the compat issues, since the gcc expects the
'sleep_clk' clock.
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk
2022-07-29 11:33 ` Dmitry Baryshkov
@ 2022-07-29 12:01 ` Konrad Dybcio
2022-07-29 12:10 ` Dmitry Baryshkov
2022-07-30 8:12 ` [PATCH v2 1/3] ARM: dts: qcom: msm8960: change the device node of sleep_clk Shinjo Park
0 siblings, 2 replies; 7+ messages in thread
From: Konrad Dybcio @ 2022-07-29 12:01 UTC (permalink / raw)
To: Dmitry Baryshkov, Krzysztof Kozlowski, Shinjo Park
Cc: David Heidelberg, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel
On 29.07.2022 13:33, Dmitry Baryshkov wrote:
> On 28/07/2022 14:51, Krzysztof Kozlowski wrote:
>> On 28/07/2022 13:16, Shinjo Park wrote:
>>> Change the reference of sleep_clk to the same as qcom-apq8064.dtsi.
>>
>> You add label, not change something.
>>
>>>
>>> Signed-off-by: Shinjo Park <peremen@gmail.com>
>>> Reviewed-by: David Heidelberg <david@ixit.cz>
>>> ---
>>> arch/arm/boot/dts/qcom-msm8960.dtsi | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
>>> index e8cd1c9c0..991eb1948 100644
>>> --- a/arch/arm/boot/dts/qcom-msm8960.dtsi
>>> +++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
>>> @@ -71,7 +71,7 @@ pxo_board: pxo_board {
>>> clock-output-names = "pxo_board";
>>> };
>>> - sleep_clk {
>>> + sleep_clk: sleep_clk {
>>
>> Since you touch the line, make the device node sleep-clk (device node
>> names should not have underscores) and mention this in commit msg.
>
>
> Then we are back to the compat issues, since the gcc expects the 'sleep_clk' clock.
clock-output-names = "sleep_clk";
Konrad
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk
2022-07-29 12:01 ` Konrad Dybcio
@ 2022-07-29 12:10 ` Dmitry Baryshkov
2022-07-30 8:12 ` [PATCH v2 1/3] ARM: dts: qcom: msm8960: change the device node of sleep_clk Shinjo Park
1 sibling, 0 replies; 7+ messages in thread
From: Dmitry Baryshkov @ 2022-07-29 12:10 UTC (permalink / raw)
To: Konrad Dybcio, Krzysztof Kozlowski, Shinjo Park
Cc: David Heidelberg, Andy Gross, Bjorn Andersson, Rob Herring,
Krzysztof Kozlowski, linux-arm-msm, devicetree, linux-kernel
On 29/07/2022 15:01, Konrad Dybcio wrote:
>
>
> On 29.07.2022 13:33, Dmitry Baryshkov wrote:
>> On 28/07/2022 14:51, Krzysztof Kozlowski wrote:
>>> On 28/07/2022 13:16, Shinjo Park wrote:
>>>> Change the reference of sleep_clk to the same as qcom-apq8064.dtsi.
>>>
>>> You add label, not change something.
>>>
>>>>
>>>> Signed-off-by: Shinjo Park <peremen@gmail.com>
>>>> Reviewed-by: David Heidelberg <david@ixit.cz>
>>>> ---
>>>> arch/arm/boot/dts/qcom-msm8960.dtsi | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
>>>> index e8cd1c9c0..991eb1948 100644
>>>> --- a/arch/arm/boot/dts/qcom-msm8960.dtsi
>>>> +++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
>>>> @@ -71,7 +71,7 @@ pxo_board: pxo_board {
>>>> clock-output-names = "pxo_board";
>>>> };
>>>> - sleep_clk {
>>>> + sleep_clk: sleep_clk {
>>>
>>> Since you touch the line, make the device node sleep-clk (device node
>>> names should not have underscores) and mention this in commit msg.
>>
>>
>> Then we are back to the compat issues, since the gcc expects the 'sleep_clk' clock.
> clock-output-names = "sleep_clk";
Ack then. Didn't notice that msm8960 has one.
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH v2 1/3] ARM: dts: qcom: msm8960: change the device node of sleep_clk
2022-07-29 12:01 ` Konrad Dybcio
2022-07-29 12:10 ` Dmitry Baryshkov
@ 2022-07-30 8:12 ` Shinjo Park
2022-08-02 8:34 ` Krzysztof Kozlowski
1 sibling, 1 reply; 7+ messages in thread
From: Shinjo Park @ 2022-07-30 8:12 UTC (permalink / raw)
Cc: peremen, David Heidelberg, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, linux-arm-msm,
devicetree, linux-kernel
Change the device node of sleep_clk to sleep-clk, and add a label. This
follows the similar convention as qcom-apq8064.dtsi.
Signed-off-by: Shinjo Park <peremen@gmail.com>
Reviewed-by: David Heidelberg <david@ixit.cz>
---
v2:
- Rewrite commit message
- Rename device node to not contain underscore
arch/arm/boot/dts/qcom-msm8960.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
index e8cd1c9c0924..e1f010c9643e 100644
--- a/arch/arm/boot/dts/qcom-msm8960.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
@@ -71,7 +71,7 @@ pxo_board: pxo_board {
clock-output-names = "pxo_board";
};
- sleep_clk {
+ sleep_clk: sleep-clk {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32768>;
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH v2 1/3] ARM: dts: qcom: msm8960: change the device node of sleep_clk
2022-07-30 8:12 ` [PATCH v2 1/3] ARM: dts: qcom: msm8960: change the device node of sleep_clk Shinjo Park
@ 2022-08-02 8:34 ` Krzysztof Kozlowski
0 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-02 8:34 UTC (permalink / raw)
To: Shinjo Park
Cc: David Heidelberg, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Rob Herring, Krzysztof Kozlowski, linux-arm-msm, devicetree,
linux-kernel
On 30/07/2022 10:12, Shinjo Park wrote:
> Change the device node of sleep_clk to sleep-clk, and add a label. This
> follows the similar convention as qcom-apq8064.dtsi.
>
> Signed-off-by: Shinjo Park <peremen@gmail.com>
> Reviewed-by: David Heidelberg <david@ixit.cz>
> ---
>
> v2:
> - Rewrite commit message
> - Rename device node to not contain underscore
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-08-02 8:34 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-28 11:16 [PATCH 1/3] ARM: dts: qcom: msm8960: add reference to sleep_clk Shinjo Park
2022-07-28 11:51 ` Krzysztof Kozlowski
2022-07-29 11:33 ` Dmitry Baryshkov
2022-07-29 12:01 ` Konrad Dybcio
2022-07-29 12:10 ` Dmitry Baryshkov
2022-07-30 8:12 ` [PATCH v2 1/3] ARM: dts: qcom: msm8960: change the device node of sleep_clk Shinjo Park
2022-08-02 8:34 ` Krzysztof Kozlowski
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).