linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] powerpc/t1024rdb: add ina220 current sense node
@ 2015-07-03  9:50 Shengzhou Liu
  2015-07-03  9:50 ` [PATCH 2/2] powerpc/t1023rdb: " Shengzhou Liu
  2015-07-06 21:48 ` [PATCH 1/2] powerpc/t1024rdb: " Scott Wood
  0 siblings, 2 replies; 3+ messages in thread
From: Shengzhou Liu @ 2015-07-03  9:50 UTC (permalink / raw)
  To: linuxppc-dev, scottwood; +Cc: Shengzhou Liu

Add support for INA220 current sense.

Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
---
 arch/powerpc/boot/dts/t1024rdb.dts | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/powerpc/boot/dts/t1024rdb.dts b/arch/powerpc/boot/dts/t1024rdb.dts
index 733e723..43474d9 100644
--- a/arch/powerpc/boot/dts/t1024rdb.dts
+++ b/arch/powerpc/boot/dts/t1024rdb.dts
@@ -114,6 +114,13 @@
 				reg = <0x4c>;
 			};
 
+			/* INA220 Current Sense */
+			ina220@40 {
+				compatible = "ti,ina220";
+				reg = <0x40>;
+				shunt-resistor = <1000>;
+			};
+
 			eeprom@50 {
 				compatible = "atmel,24c256";
 				reg = <0x50>;
-- 
2.1.0.27.g96db324

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

* [PATCH 2/2] powerpc/t1023rdb: add ina220 current sense node
  2015-07-03  9:50 [PATCH 1/2] powerpc/t1024rdb: add ina220 current sense node Shengzhou Liu
@ 2015-07-03  9:50 ` Shengzhou Liu
  2015-07-06 21:48 ` [PATCH 1/2] powerpc/t1024rdb: " Scott Wood
  1 sibling, 0 replies; 3+ messages in thread
From: Shengzhou Liu @ 2015-07-03  9:50 UTC (permalink / raw)
  To: linuxppc-dev, scottwood; +Cc: Shengzhou Liu

Add support for INA220 current sense.

Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
---
 arch/powerpc/boot/dts/t1023rdb.dts | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/powerpc/boot/dts/t1023rdb.dts b/arch/powerpc/boot/dts/t1023rdb.dts
index 06b090a..a1c647b 100644
--- a/arch/powerpc/boot/dts/t1023rdb.dts
+++ b/arch/powerpc/boot/dts/t1023rdb.dts
@@ -99,6 +99,17 @@
 		};
 
 		i2c@118100 {
+			ina220@40 {
+				compatible = "ti,ina220";
+				reg = <0x40>;
+				shunt-resistor = <1000>;
+			};
+
+			ina220@41 {
+				compatible = "ti,ina220";
+				reg = <0x41>;
+				shunt-resistor = <1000>;
+			};
 		};
 	};
 
-- 
2.1.0.27.g96db324

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

* Re: [PATCH 1/2] powerpc/t1024rdb: add ina220 current sense node
  2015-07-03  9:50 [PATCH 1/2] powerpc/t1024rdb: add ina220 current sense node Shengzhou Liu
  2015-07-03  9:50 ` [PATCH 2/2] powerpc/t1023rdb: " Shengzhou Liu
@ 2015-07-06 21:48 ` Scott Wood
  1 sibling, 0 replies; 3+ messages in thread
From: Scott Wood @ 2015-07-06 21:48 UTC (permalink / raw)
  To: Shengzhou Liu; +Cc: linuxppc-dev

On Fri, 2015-07-03 at 17:50 +0800, Shengzhou Liu wrote:
> Add support for INA220 current sense.
> 
> Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
> ---
>  arch/powerpc/boot/dts/t1024rdb.dts | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/powerpc/boot/dts/t1024rdb.dts 
> b/arch/powerpc/boot/dts/t1024rdb.dts
> index 733e723..43474d9 100644
> --- a/arch/powerpc/boot/dts/t1024rdb.dts
> +++ b/arch/powerpc/boot/dts/t1024rdb.dts
> @@ -114,6 +114,13 @@
>                               reg = <0x4c>;
>                       };
>  
> +                     /* INA220 Current Sense */

Do you mean "current sensor"?

Why is this comment needed here, but not in the t1023 file?

> +                      ina220@40{

Node names should be generic (which would also eliminate any reason for the 
above comment).

-Scott

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

end of thread, other threads:[~2015-07-06 21:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-03  9:50 [PATCH 1/2] powerpc/t1024rdb: add ina220 current sense node Shengzhou Liu
2015-07-03  9:50 ` [PATCH 2/2] powerpc/t1023rdb: " Shengzhou Liu
2015-07-06 21:48 ` [PATCH 1/2] powerpc/t1024rdb: " Scott Wood

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