linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 1/2] thermal: armada: fix register offsets for AXP
       [not found] <1575485367-18262-1-git-send-email-zhays@lexmark.com>
@ 2019-12-05 14:19 ` Zak Hays
  2019-12-09 15:42   ` Daniel Lezcano
       [not found] ` <1575485367-18262-2-git-send-email-zhays@lexmark.com>
  1 sibling, 1 reply; 7+ messages in thread
From: Zak Hays @ 2019-12-05 14:19 UTC (permalink / raw)
  To: Zak Hays
  Cc: miquel.raynal, rui.zhang, daniel.lezcano, amit.kucheria,
	linux-pm, linux-kernel

As shown in its device tree, Armada XP has the control1 register at
0x184d0, not 0x182d0.

Signed-off-by: Zachary Hays <zhays@lexmark.com>
---
v2: update commit title and add "Signed-off-by"
---
 drivers/thermal/armada_thermal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/armada_thermal.c b/drivers/thermal/armada_thermal.c
index 709a22f455e9..88363812033c 100644
--- a/drivers/thermal/armada_thermal.c
+++ b/drivers/thermal/armada_thermal.c
@@ -578,7 +578,7 @@ static const struct armada_thermal_data armadaxp_data = {
        .coef_m = 10000000ULL,
        .coef_div = 13825,
        .syscon_status_off = 0xb0,
-       .syscon_control1_off = 0xd0,
+       .syscon_control1_off = 0x2d0,
 };

 static const struct armada_thermal_data armada370_data = {
--
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v2 2/2] thermal: armada: clear reset in armadaxp_init
       [not found] ` <1575485367-18262-2-git-send-email-zhays@lexmark.com>
@ 2019-12-05 14:19   ` Zak Hays
  0 siblings, 0 replies; 7+ messages in thread
From: Zak Hays @ 2019-12-05 14:19 UTC (permalink / raw)
  To: Zak Hays
  Cc: miquel.raynal, rui.zhang, daniel.lezcano, amit.kucheria,
	linux-pm, linux-kernel

The reset bit needs to be cleared in the init sequence otherwise it
holds the block in reset.

Signed-off-by: Zachary Hays <zhays@lexmark.com>
---
v2: update commit title and add "Signed-off-by"
---
 drivers/thermal/armada_thermal.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/thermal/armada_thermal.c b/drivers/thermal/armada_thermal.c
index 88363812033c..8c4d1244ee7a 100644
--- a/drivers/thermal/armada_thermal.c
+++ b/drivers/thermal/armada_thermal.c
@@ -155,6 +155,9 @@ static void armadaxp_init(struct platform_device *pdev,

        regmap_write(priv->syscon, data->syscon_control1_off, reg);

+       reg &= ~PMU_TDC0_SW_RST_MASK;
+       regmap_write(priv->syscon, data->syscon_control1_off, reg);
+
        /* Enable the sensor */
        regmap_read(priv->syscon, data->syscon_status_off, &reg);
        reg &= ~PMU_TM_DISABLE_MASK;
--
2.7.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v2 1/2] thermal: armada: fix register offsets for AXP
  2019-12-05 14:19 ` [PATCH v2 1/2] thermal: armada: fix register offsets for AXP Zak Hays
@ 2019-12-09 15:42   ` Daniel Lezcano
  2019-12-09 16:31     ` Zak Hays
  0 siblings, 1 reply; 7+ messages in thread
From: Daniel Lezcano @ 2019-12-09 15:42 UTC (permalink / raw)
  To: Zak Hays; +Cc: miquel.raynal, rui.zhang, amit.kucheria, linux-pm, linux-kernel

Hi Zak,

what patch to pick up? V2 or V2?

What email is the correct one?

	Zachary Hays <zhays@lexmark.com>
or
	Zak Hays <zak.hays@lexmark.com>

Also waiting for Miquel to ack the patch.

Thanks

  -- Daniel


On 05/12/2019 15:19, Zak Hays wrote:
> As shown in its device tree, Armada XP has the control1 register at
> 0x184d0, not 0x182d0.
> 
> Signed-off-by: Zachary Hays <zhays@lexmark.com>
> ---
> v2: update commit title and add "Signed-off-by"
> ---
>  drivers/thermal/armada_thermal.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/thermal/armada_thermal.c b/drivers/thermal/armada_thermal.c
> index 709a22f455e9..88363812033c 100644
> --- a/drivers/thermal/armada_thermal.c
> +++ b/drivers/thermal/armada_thermal.c
> @@ -578,7 +578,7 @@ static const struct armada_thermal_data armadaxp_data = {
>         .coef_m = 10000000ULL,
>         .coef_div = 13825,
>         .syscon_status_off = 0xb0,
> -       .syscon_control1_off = 0xd0,
> +       .syscon_control1_off = 0x2d0,
>  };
> 
>  static const struct armada_thermal_data armada370_data = {
> --
> 2.7.4
> 


-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2 1/2] thermal: armada: fix register offsets for AXP
  2019-12-09 15:42   ` Daniel Lezcano
@ 2019-12-09 16:31     ` Zak Hays
  2019-12-09 16:39       ` Daniel Lezcano
  0 siblings, 1 reply; 7+ messages in thread
From: Zak Hays @ 2019-12-09 16:31 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: miquel.raynal, rui.zhang, amit.kucheria, linux-pm, linux-kernel

Hi Daniel,

>Hi Zak,
>
>what patch to pick up? V2 or V2?
I'm not sure I entirely understand your question. There are two patches:

Zachary Hays  thermal: armada: fix register offsets for AXP
Zachary Hays  thermal: armada: clear reset in armadaxp_init

Version 2 of each patch updates the commit message to add a "Signed-off-by"
and cleans up the commit message per Miquèl's earlier comments. Otherwise the
patches are the same as they were in version 1. The patch below is version 2 of that first patch.

Does that answer your question?
>
>What email is the correct one?
>
>        Zachary Hays <zhays@lexmark.com>
>or
>        Zak Hays <zak.hays@lexmark.com>
>
Both addresses route to the same place but "Zachary Hays <zhays@lexmark.com>" is preferable. Sorry for the confusion.

>Also waiting for Miquel to ack the patch.
>
>Thanks
>
> -- Daniel
>
>
>On 05/12/2019 15:19, Zak Hays wrote:
>> As shown in its device tree, Armada XP has the control1 register at
>> 0x184d0, not 0x182d0.
>>
>> Signed-off-by: Zachary Hays <zhays@lexmark.com>
>> ---
>> v2: update commit title and add "Signed-off-by"
>> ---
>>  drivers/thermal/armada_thermal.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/thermal/armada_thermal.c b/drivers/thermal/armada_thermal.c
>> index 709a22f455e9..88363812033c 100644
>> --- a/drivers/thermal/armada_thermal.c
>> +++ b/drivers/thermal/armada_thermal.c
>> @@ -578,7 +578,7 @@ static const struct armada_thermal_data armadaxp_data = {
>>         .coef_m = 10000000ULL,
>>         .coef_div = 13825,
>>         .syscon_status_off = 0xb0,
>> -       .syscon_control1_off = 0xd0,
>> +       .syscon_control1_off = 0x2d0,
>>  };
>>
>>  static const struct armada_thermal_data armada370_data = {
>> --
>> 2.7.4
>>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2 1/2] thermal: armada: fix register offsets for AXP
  2019-12-09 16:31     ` Zak Hays
@ 2019-12-09 16:39       ` Daniel Lezcano
  2019-12-09 16:58         ` Zak Hays
  0 siblings, 1 reply; 7+ messages in thread
From: Daniel Lezcano @ 2019-12-09 16:39 UTC (permalink / raw)
  To: Zak Hays; +Cc: miquel.raynal, rui.zhang, amit.kucheria, linux-pm, linux-kernel

On 09/12/2019 17:31, Zak Hays wrote:
> Hi Daniel,
> 
>> Hi Zak,
>>
>> what patch to pick up? V2 or V2?
> I'm not sure I entirely understand your question. There are two patches:
> 
> Zachary Hays  thermal: armada: fix register offsets for AXP
> Zachary Hays  thermal: armada: clear reset in armadaxp_init

I'm seeing multiple posting of the V2 and reply to V2 with I guess a
changelog added.

It is very difficult to deal with patches when it is unclear, duplicate
series with different changelog.

In the future, post a resend version explaining what was missing, so it
is easier to understand what is happening. Or send a V3.

> Version 2 of each patch updates the commit message to add a "Signed-off-by"
> and cleans up the commit message per Miquèl's earlier comments. Otherwise the
> patches are the same as they were in version 1. The patch below is version 2 of that first patch.
> 
> Does that answer your question?
>>
>> What email is the correct one?
>>
>>        Zachary Hays <zhays@lexmark.com>
>> or
>>        Zak Hays <zak.hays@lexmark.com>
>>
> Both addresses route to the same place but "Zachary Hays <zhays@lexmark.com>" is preferable. Sorry for the confusion.
> 
>> Also waiting for Miquel to ack the patch.
>>
>> Thanks
>>
>> -- Daniel
>>
>>
>> On 05/12/2019 15:19, Zak Hays wrote:
>>> As shown in its device tree, Armada XP has the control1 register at
>>> 0x184d0, not 0x182d0.
>>>
>>> Signed-off-by: Zachary Hays <zhays@lexmark.com>
>>> ---
>>> v2: update commit title and add "Signed-off-by"
>>> ---
>>>  drivers/thermal/armada_thermal.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/thermal/armada_thermal.c b/drivers/thermal/armada_thermal.c
>>> index 709a22f455e9..88363812033c 100644
>>> --- a/drivers/thermal/armada_thermal.c
>>> +++ b/drivers/thermal/armada_thermal.c
>>> @@ -578,7 +578,7 @@ static const struct armada_thermal_data armadaxp_data = {
>>>         .coef_m = 10000000ULL,
>>>         .coef_div = 13825,
>>>         .syscon_status_off = 0xb0,
>>> -       .syscon_control1_off = 0xd0,
>>> +       .syscon_control1_off = 0x2d0,
>>>  };
>>>
>>>  static const struct armada_thermal_data armada370_data = {
>>> --
>>> 2.7.4
>> >


-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2 1/2] thermal: armada: fix register offsets for AXP
  2019-12-09 16:39       ` Daniel Lezcano
@ 2019-12-09 16:58         ` Zak Hays
  2019-12-09 17:11           ` Daniel Lezcano
  0 siblings, 1 reply; 7+ messages in thread
From: Zak Hays @ 2019-12-09 16:58 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: miquel.raynal, rui.zhang, amit.kucheria, linux-pm, linux-kernel

>On 09/12/2019 17:31, Zak Hays wrote:
>> Hi Daniel,
>>
>>> Hi Zak,
>>>
>>> what patch to pick up? V2 or V2?
>> I'm not sure I entirely understand your question. There are two patches:
>>
>> Zachary Hays  thermal: armada: fix register offsets for AXP
>> Zachary Hays  thermal: armada: clear reset in armadaxp_init
>
>I'm seeing multiple posting of the V2 and reply to V2 with I guess a
>changelog added.
>
>It is very difficult to deal with patches when it is unclear, duplicate
>series with different changelog.
>
>In the future, post a resend version explaining what was missing, so it
>is easier to understand what is happening. Or send a V3.

Ah. I understand now. I've been fighting email issues trying to get these patches sent and
inadvertently sent out multiple versions of the same patches. The most recent patches
are correct. Sorry for that spam. Shouldn't happen again.

Would you like me to resend as V3 for clarity or would you prefer to continue with the
V2 patches?

>
>> Version 2 of each patch updates the commit message to add a "Signed-off-by"
>> and cleans up the commit message per Miquèl's earlier comments. Otherwise the
>> patches are the same as they were in version 1. The patch below is version 2 of that first patch.
>>
>> Does that answer your question?
>>>
>>> What email is the correct one?
>>>
>>>        Zachary Hays <zhays@lexmark.com>
>>> or
>>>        Zak Hays <zak.hays@lexmark.com>
>>>
>> Both addresses route to the same place but "Zachary Hays <zhays@lexmark.com>" is preferable. Sorry for the confusion.
>>
>>> Also waiting for Miquel to ack the patch.
>>>
>>> Thanks
>>>
>>> -- Daniel
>>>
>>>
>>> On 05/12/2019 15:19, Zak Hays wrote:
>>>> As shown in its device tree, Armada XP has the control1 register at
>>>> 0x184d0, not 0x182d0.
>>>>
>>>> Signed-off-by: Zachary Hays <zhays@lexmark.com>
>>>> ---
>>>> v2: update commit title and add "Signed-off-by"
>>>> ---
>>>>  drivers/thermal/armada_thermal.c | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/thermal/armada_thermal.c b/drivers/thermal/armada_thermal.c
>>>> index 709a22f455e9..88363812033c 100644
>>>> --- a/drivers/thermal/armada_thermal.c
>>>> +++ b/drivers/thermal/armada_thermal.c
>>>> @@ -578,7 +578,7 @@ static const struct armada_thermal_data armadaxp_data = {
>>>>         .coef_m = 10000000ULL,
>>>>         .coef_div = 13825,
>>>>         .syscon_status_off = 0xb0,
>>>> -       .syscon_control1_off = 0xd0,
>>>> +       .syscon_control1_off = 0x2d0,
>>>>  };
>>>>
>>>>  static const struct armada_thermal_data armada370_data = {
>>>> --
>>>> 2.7.4
>> >>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2 1/2] thermal: armada: fix register offsets for AXP
  2019-12-09 16:58         ` Zak Hays
@ 2019-12-09 17:11           ` Daniel Lezcano
  0 siblings, 0 replies; 7+ messages in thread
From: Daniel Lezcano @ 2019-12-09 17:11 UTC (permalink / raw)
  To: Zak Hays; +Cc: miquel.raynal, rui.zhang, amit.kucheria, linux-pm, linux-kernel

On 09/12/2019 17:58, Zak Hays wrote:
>> On 09/12/2019 17:31, Zak Hays wrote:
>>> Hi Daniel,
>>>
>>>> Hi Zak,
>>>>
>>>> what patch to pick up? V2 or V2?
>>> I'm not sure I entirely understand your question. There are two patches:
>>>
>>> Zachary Hays  thermal: armada: fix register offsets for AXP
>>> Zachary Hays  thermal: armada: clear reset in armadaxp_init
>>
>> I'm seeing multiple posting of the V2 and reply to V2 with I guess a
>> changelog added.
>>
>> It is very difficult to deal with patches when it is unclear, duplicate
>> series with different changelog.
>>
>> In the future, post a resend version explaining what was missing, so it
>> is easier to understand what is happening. Or send a V3.
> 
> Ah. I understand now. I've been fighting email issues trying to get these patches sent and
> inadvertently sent out multiple versions of the same patches. The most recent patches
> are correct. Sorry for that spam. Shouldn't happen again.
> 
> Would you like me to resend as V3 for clarity or would you prefer to continue with the
> V2 patches?

A V3 please, so I can pick them up without ambiguity from patchwork.

Thanks!





-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-12-09 17:11 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1575485367-18262-1-git-send-email-zhays@lexmark.com>
2019-12-05 14:19 ` [PATCH v2 1/2] thermal: armada: fix register offsets for AXP Zak Hays
2019-12-09 15:42   ` Daniel Lezcano
2019-12-09 16:31     ` Zak Hays
2019-12-09 16:39       ` Daniel Lezcano
2019-12-09 16:58         ` Zak Hays
2019-12-09 17:11           ` Daniel Lezcano
     [not found] ` <1575485367-18262-2-git-send-email-zhays@lexmark.com>
2019-12-05 14:19   ` [PATCH v2 2/2] thermal: armada: clear reset in armadaxp_init Zak Hays

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