All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick.
@ 2017-03-23  1:19 Peter Hanson
  2017-03-24 20:09 ` Xo Wang
  0 siblings, 1 reply; 6+ messages in thread
From: Peter Hanson @ 2017-03-23  1:19 UTC (permalink / raw)
  To: Joel Stanley; +Cc: OpenBMC, Xo Wang, Peter Hanson

Signed-off-by: Peter Hanson <peterh@google.com>
---
 arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
index 3bd70ed810af..dfd0dd7c16e4 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
@@ -269,7 +269,13 @@
 	/* Master selector PCA9541A @70h (other master: CPU0)
 	 *   LM5066I PMBUS @10h
 	 */
-	/* 12V SMPS Q54SH12050NNDH @61h */
+
+	/* 12V Quarter Brick DC/DC Converter Q54SJ12050 @61h */
+	power-brick@61 {
+		compatible = "delta,dps800";
+		reg = <0x61>;
+	};
+
 	/* CPU0 VR ISL68137 0.7V, 0.96V PMBUS @64h */
 	/* CPU0 VR ISL68137 1.2V CH03 PMBUS @40h */
 	/* CPU0 VR ISL68137 0.8V PMBUS @60h */
-- 
2.12.1.500.gab5fba24ee-goog

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

* Re: [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick.
  2017-03-23  1:19 [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick Peter Hanson
@ 2017-03-24 20:09 ` Xo Wang
  2017-03-26 23:09   ` Joel Stanley
  0 siblings, 1 reply; 6+ messages in thread
From: Xo Wang @ 2017-03-24 20:09 UTC (permalink / raw)
  To: Peter Hanson; +Cc: Joel Stanley, OpenBMC

On Wed, Mar 22, 2017 at 6:19 PM, Peter Hanson <peterh@google.com> wrote:
> Signed-off-by: Peter Hanson <peterh@google.com>
> ---
>  arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> index 3bd70ed810af..dfd0dd7c16e4 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> @@ -269,7 +269,13 @@
>         /* Master selector PCA9541A @70h (other master: CPU0)
>          *   LM5066I PMBUS @10h
>          */
> -       /* 12V SMPS Q54SH12050NNDH @61h */
> +
> +       /* 12V Quarter Brick DC/DC Converter Q54SJ12050 @61h */
> +       power-brick@61 {
> +               compatible = "delta,dps800";
> +               reg = <0x61>;
> +       };
> +
>         /* CPU0 VR ISL68137 0.7V, 0.96V PMBUS @64h */
>         /* CPU0 VR ISL68137 1.2V CH03 PMBUS @40h */
>         /* CPU0 VR ISL68137 0.8V PMBUS @60h */
> --
> 2.12.1.500.gab5fba24ee-goog
>

Reviewed-by: Xo Wang <xow@google.com>

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

* Re: [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick.
  2017-03-24 20:09 ` Xo Wang
@ 2017-03-26 23:09   ` Joel Stanley
  0 siblings, 0 replies; 6+ messages in thread
From: Joel Stanley @ 2017-03-26 23:09 UTC (permalink / raw)
  To: Xo Wang; +Cc: Peter Hanson, OpenBMC

On Sat, Mar 25, 2017 at 6:39 AM, Xo Wang <xow@google.com> wrote:
> On Wed, Mar 22, 2017 at 6:19 PM, Peter Hanson <peterh@google.com> wrote:
>> Signed-off-by: Peter Hanson <peterh@google.com>
>
> Reviewed-by: Xo Wang <xow@google.com>

Thanks. Applied to dev-4.7.

Cheers,

Joel

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

* Re: [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick.
  2017-03-23  0:41 ` Joel Stanley
@ 2017-03-23  0:55   ` Peter Hanson
  0 siblings, 0 replies; 6+ messages in thread
From: Peter Hanson @ 2017-03-23  0:55 UTC (permalink / raw)
  To: Joel Stanley; +Cc: OpenBMC, Xo Wang

On Wed, Mar 22, 2017 at 5:41 PM, Joel Stanley <joel@jms.id.au> wrote:
> Hello Peter,
>
> Welcome to the project. Congrats on your success with the git tools,
> v2 looks much better. There's some tips and tricks on the wiki for
> next time:
>
>  https://github.com/openbmc/linux/wiki/SubmittingPatches#use-the-git-tools
>
> On Thu, Mar 23, 2017 at 10:57 AM, Peter Hanson <peterh@google.com> wrote:
>> Signed-off-by: Peter Hanson <peterh@google.com>
>> ---
>>  arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +++++++-
>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
>> index 3bd70ed810af..4fd27617bb3a 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
>> @@ -269,7 +269,13 @@
>>         /* Master selector PCA9541A @70h (other master: CPU0)
>>          *   LM5066I PMBUS @10h
>>          */
>> -       /* 12V SMPS Q54SH12050NNDH @61h */
>> +
>> +       /* 12V SMPS Q54SJ12050 @61h */
>> +       q54sj12050@61 {
>
> This is an interesting name. I assume it means something to you? My
> favourite search engine came up empty.
>
> You're free to use whatever name you think is appropriate for the
> node, so if you want to make it name like "power-brick" you could.

It is (allegedly) the part number I tested with.

I prefer your idea. The Delta site references to quite a few related parts,
any of which should work the same, as far as pmbus is concerned.
Incoming!

> If you're happy with this name then that's fine too. I'll wait for
> Xo's ack and apply it.
>
> Cheers,
>
> Joel
>
>> +               compatible = "delta,dps800";
>> +               reg = <0x61>;
>> +       };
>> +
>>         /* CPU0 VR ISL68137 0.7V, 0.96V PMBUS @64h */
>>         /* CPU0 VR ISL68137 1.2V CH03 PMBUS @40h */
>>         /* CPU0 VR ISL68137 0.8V PMBUS @60h */
>> --
>> 2.12.1.500.gab5fba24ee-goog
>>

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

* Re: [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick.
  2017-03-23  0:27 Peter Hanson
@ 2017-03-23  0:41 ` Joel Stanley
  2017-03-23  0:55   ` Peter Hanson
  0 siblings, 1 reply; 6+ messages in thread
From: Joel Stanley @ 2017-03-23  0:41 UTC (permalink / raw)
  To: Peter Hanson; +Cc: OpenBMC, Xo Wang

Hello Peter,

Welcome to the project. Congrats on your success with the git tools,
v2 looks much better. There's some tips and tricks on the wiki for
next time:

 https://github.com/openbmc/linux/wiki/SubmittingPatches#use-the-git-tools

On Thu, Mar 23, 2017 at 10:57 AM, Peter Hanson <peterh@google.com> wrote:
> Signed-off-by: Peter Hanson <peterh@google.com>
> ---
>  arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> index 3bd70ed810af..4fd27617bb3a 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> @@ -269,7 +269,13 @@
>         /* Master selector PCA9541A @70h (other master: CPU0)
>          *   LM5066I PMBUS @10h
>          */
> -       /* 12V SMPS Q54SH12050NNDH @61h */
> +
> +       /* 12V SMPS Q54SJ12050 @61h */
> +       q54sj12050@61 {

This is an interesting name. I assume it means something to you? My
favourite search engine came up empty.

You're free to use whatever name you think is appropriate for the
node, so if you want to make it name like "power-brick" you could.

If you're happy with this name then that's fine too. I'll wait for
Xo's ack and apply it.

Cheers,

Joel

> +               compatible = "delta,dps800";
> +               reg = <0x61>;
> +       };
> +
>         /* CPU0 VR ISL68137 0.7V, 0.96V PMBUS @64h */
>         /* CPU0 VR ISL68137 1.2V CH03 PMBUS @40h */
>         /* CPU0 VR ISL68137 0.8V PMBUS @60h */
> --
> 2.12.1.500.gab5fba24ee-goog
>

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

* [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick.
@ 2017-03-23  0:27 Peter Hanson
  2017-03-23  0:41 ` Joel Stanley
  0 siblings, 1 reply; 6+ messages in thread
From: Peter Hanson @ 2017-03-23  0:27 UTC (permalink / raw)
  To: Joel Stanley; +Cc: OpenBMC, Xo Wang, Peter Hanson

Signed-off-by: Peter Hanson <peterh@google.com>
---
 arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
index 3bd70ed810af..4fd27617bb3a 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
@@ -269,7 +269,13 @@
 	/* Master selector PCA9541A @70h (other master: CPU0)
 	 *   LM5066I PMBUS @10h
 	 */
-	/* 12V SMPS Q54SH12050NNDH @61h */
+
+	/* 12V SMPS Q54SJ12050 @61h */
+	q54sj12050@61 {
+		compatible = "delta,dps800";
+		reg = <0x61>;
+	};
+
 	/* CPU0 VR ISL68137 0.7V, 0.96V PMBUS @64h */
 	/* CPU0 VR ISL68137 1.2V CH03 PMBUS @40h */
 	/* CPU0 VR ISL68137 0.8V PMBUS @60h */
-- 
2.12.1.500.gab5fba24ee-goog

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

end of thread, other threads:[~2017-03-26 23:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-23  1:19 [PATCH] ARM: dts: aspeed: zaius: Enable 12V Delta Power Brick Peter Hanson
2017-03-24 20:09 ` Xo Wang
2017-03-26 23:09   ` Joel Stanley
  -- strict thread matches above, loose matches on Subject: below --
2017-03-23  0:27 Peter Hanson
2017-03-23  0:41 ` Joel Stanley
2017-03-23  0:55   ` Peter Hanson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.