Re: [RFC 2/5] i3c: Add core I3C infrastructure

2017-08-01 Thread Andrew F. Davis
On 07/31/2017 04:42 PM, Wolfram Sang wrote: > >> Actually, that's the first option I considered, but I3C and I2C are >> really different. I'm not talking about the physical layer here, but >> the way the bus has to be handled by the software layer. Actually, I >> thing the I3C bus is philosophical

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-24 Thread Andrew F. Davis
On 06/24/2016 11:46 AM, Guenter Roeck wrote: > On Fri, Jun 24, 2016 at 10:02:51AM -0500, Andrew F. Davis wrote: >> On 06/18/2016 10:16 AM, Guenter Roeck wrote: >>> >>> The chip registers are 16 bit. Can you repeat the command using the "w" >>> option

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-24 Thread Andrew F. Davis
On 06/18/2016 10:16 AM, Guenter Roeck wrote: > > The chip registers are 16 bit. Can you repeat the command using the "w" > option ? > # i2cdump -y 2 0x40 w 0,8 1,9 2,a 3,b 4,c 5,d 6,e 7,f 00: 2771 f87f 08: f87f f87f f87f f87f f87f fe7f 0300 10: 1027

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-17 Thread Andrew F. Davis
On 06/10/2016 11:44 AM, Guenter Roeck wrote: > On Fri, Jun 10, 2016 at 10:32:33AM -0500, Andrew F. Davis wrote: >> Add support for the the INA3221 26v capable, Triple channel, >> Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor >> with I2C interface. >&g

[PATCH v5 0/1] Add support for INA3221 Triple Current/Voltage Monitors

2016-06-10 Thread Andrew F. Davis
- added of_match_table - other small fixups/typos Changes from v1: - rearranged and renumbered sysfs enteries - added reading alert bits to sysfs - removed internal power calculation - added DT setting of shunt resistors - various other minor fixups Thanks, Andrew Andrew F. Davis (1): hwmon

[PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-10 Thread Andrew F. Davis
Add support for the the INA3221 26v capable, Triple channel, Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor with I2C interface. Signed-off-by: Andrew F. Davis --- Documentation/hwmon/ina3221 | 35 drivers/hwmon/Kconfig | 11 ++ drivers/hwmon/Makefile | 1

[PATCH v2 1/1] hwmon: (tmp401) Add support for TI TMP461

2016-06-08 Thread Andrew F. Davis
Signed-off-by: Andrew F. Davis --- Changes from v1: - Dropped n-factor correction until a generic solution is developed - Removed double empty line Documentation/hwmon/tmp401 | 14 -- drivers/hwmon/Kconfig | 2 +- drivers/hwmon/tmp401.c | 35

Re: [PATCH] hwmon: (tmp401) Add support for TI TMP461

2016-06-08 Thread Andrew F. Davis
On 06/03/2016 11:41 PM, Guenter Roeck wrote: > On 05/31/2016 09:27 AM, Andrew F. Davis wrote: >> Signed-off-by: Andrew F. Davis >> --- >> Documentation/hwmon/tmp401 | 18 +-- >> drivers/hwmon/Kconfig | 2 +- >>

Re: [PATCH 2/3] iio: adc: ina3221: Add support for IIO ADC driver for TI INA3221

2016-06-08 Thread Andrew F. Davis
On 06/07/2016 05:30 PM, Guenter Roeck wrote: > On Fri, Jun 03, 2016 at 10:17:55AM -0500, Andrew F. Davis wrote: >> On 06/03/2016 09:14 AM, Laxman Dewangan wrote: >>> >>> On Friday 03 June 2016 06:59 PM, Guenter Roeck wrote: >>>> On 06/03/2016 03:06 AM, Jonat

Re: [PATCH 2/3] iio: adc: ina3221: Add support for IIO ADC driver for TI INA3221

2016-06-03 Thread Andrew F. Davis
On 06/03/2016 09:14 AM, Laxman Dewangan wrote: > > On Friday 03 June 2016 06:59 PM, Guenter Roeck wrote: >> On 06/03/2016 03:06 AM, Jonathan Cameron wrote: >>> On 01/06/16 13:34, Laxman Dewangan wrote: The INA3221 is a three-channel, high-side current and bus voltage monitor with an

[PATCH] hwmon: (tmp401) Add support for TI TMP461

2016-05-31 Thread Andrew F. Davis
Signed-off-by: Andrew F. Davis --- Documentation/hwmon/tmp401 | 18 +-- drivers/hwmon/Kconfig | 2 +- drivers/hwmon/tmp401.c | 81 ++ 3 files changed, 92 insertions(+), 9 deletions(-) diff --git a/Documentation/hwmon/tmp401 b

[PATCH v2 1/4] rpmsg: add THIS_MODULE to rpmsg_driver in rpmsg core

2016-05-04 Thread Andrew F. Davis
Add register_rpmsg_driver helper macro that adds THIS_MODULE to rpmsg_driver for the registering driver. We rename and modify the existing register_rpmsg_driver to enable this. Signed-off-by: Andrew F. Davis Acked-by: Suman Anna --- drivers/rpmsg/virtio_rpmsg_bus.c | 8 +--- include/linux

[PATCH v2 4/4] rpmsg: use module_rpmsg_driver in existing drivers and examples

2016-05-04 Thread Andrew F. Davis
Existing drivers and examples are updated to use the module_rpmsg_driver helper macro. Signed-off-by: Andrew F. Davis --- Documentation/rpmsg.txt | 13 + samples/rpmsg/rpmsg_client_sample.c | 13 + 2 files changed, 2 insertions(+), 24 deletions(-) diff --git

[PATCH v2 2/4] rpmsg: drop owner assignment from rpmsg_drivers

2016-05-04 Thread Andrew F. Davis
An rpmsg_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Andrew F. Davis --- Documentation/rpmsg.txt | 1 - samples/rpmsg/rpmsg_client_sample.c | 1 - 2 files changed, 2 deletions(-) diff --git a/Documentation/rpmsg.txt b/Documentation

[PATCH v2 3/4] rpmsg: add helper macro module_rpmsg_driver

2016-05-04 Thread Andrew F. Davis
eliminate a few lines of boilerplate code per rpmsg driver. Signed-off-by: Andrew F. Davis --- include/linux/rpmsg.h | 12 1 file changed, 12 insertions(+) diff --git a/include/linux/rpmsg.h b/include/linux/rpmsg.h index 7ff5790..ada50ff 100644 --- a/include/linux/rpmsg.h +++ b/include

Re: [PATCH 3/4] rpmsg: add helper macro module_rpmsg_driver

2016-05-04 Thread Andrew F. Davis
On 05/04/2016 01:54 PM, Suman Anna wrote: > Hi Andrew, > > On 05/04/2016 01:34 PM, Andrew F. Davis wrote: >> This patch introduces the module_rpmsg_driver macro which is a >> convenience macro for rpmsg driver modules similar to >> module_platform_driver. It is int

Re: [PATCH 2/4] rpmsg: drop owner assignment from spi_drivers

2016-05-04 Thread Andrew F. Davis
On 05/04/2016 01:55 PM, Suman Anna wrote: > On 05/04/2016 01:34 PM, Andrew F. Davis wrote: >> An rpmsg_driver does not need to set an owner, it will be populated by >> the driver core. > > spi_drivers in patch subject?? > copy/paste error, this is all based on my SPI p

[PATCH 2/4] rpmsg: drop owner assignment from spi_drivers

2016-05-04 Thread Andrew F. Davis
An rpmsg_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Andrew F. Davis --- Documentation/rpmsg.txt | 1 - samples/rpmsg/rpmsg_client_sample.c | 1 - 2 files changed, 2 deletions(-) diff --git a/Documentation/rpmsg.txt b/Documentation

[PATCH 4/4] rpmsg: use module_rpmsg_driver in existing drivers and examples

2016-05-04 Thread Andrew F. Davis
Existing drivers and examples are updated to use the module_rpmsg_driver helper macro. Signed-off-by: Andrew F. Davis --- Documentation/rpmsg.txt | 14 +- samples/rpmsg/rpmsg_client_sample.c | 13 + 2 files changed, 2 insertions(+), 25 deletions(-) diff

[PATCH 3/4] rpmsg: add helper macro module_rpmsg_driver

2016-05-04 Thread Andrew F. Davis
eliminate a few lines of boilerplate code per rpmsg driver. Signed-off-by: Andrew F. Davis --- include/linux/rpmsg.h | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/include/linux/rpmsg.h b/include/linux/rpmsg.h index 78e45ce..ada50ff 100644 --- a/include/linux

[PATCH 1/4] rpmsg: add THIS_MODULE to rpmsg_driver in rpmsg core

2016-05-04 Thread Andrew F. Davis
Add register_rpmsg_driver helper macro that adds THIS_MODULE to rpmsg_driver for the registering driver. We rename and modify the existing register_rpmsg_driver to enable this. Signed-off-by: Andrew F. Davis --- drivers/rpmsg/virtio_rpmsg_bus.c | 8 +--- include/linux/rpmsg.h| 8

[PATCH v3 2/2] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-04-25 Thread Andrew F. Davis
Add support for the the INA3221 26v capable, Triple channel, Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor with I2C interface. Signed-off-by: Andrew F. Davis --- Documentation/hwmon/ina3221 | 35 drivers/hwmon/Kconfig | 11 ++ drivers/hwmon/Makefile | 1

[PATCH v3 0/2] Add support for INA3221 Triple Current/Voltage Monitors

2016-04-25 Thread Andrew F. Davis
- removed internal power calculation - added DT setting of shunt resistors - various other minor fixups Thanks, Andrew Andrew F. Davis (2): hwmon: Define binding for the INA3221 hwmon driver hwmon: Add support for INA3221 Triple Current/Voltage Monitors .../devicetree/bindings/hwmon/ina3221.txt

[PATCH v3 1/2] hwmon: Define binding for the INA3221 hwmon driver

2016-04-25 Thread Andrew F. Davis
Define a binding for the INA3221 Triple Current/Voltage Monitor. Signed-off-by: Andrew F. Davis --- Documentation/devicetree/bindings/hwmon/ina3221.txt | 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/hwmon/ina3221.txt diff --git

Re: [PATCH v2 2/2] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-04-25 Thread Andrew F. Davis
On 04/23/2016 11:53 AM, Guenter Roeck wrote: > On 04/19/2016 11:28 AM, Andrew F. Davis wrote: >> Add support for the the INA3221 26v capable, Triple channel, >> Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor >> with I2C interface. >> >&g

[PATCH v2 2/2] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-04-19 Thread Andrew F. Davis
Add support for the the INA3221 26v capable, Triple channel, Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor with I2C interface. Signed-off-by: Andrew F. Davis --- Documentation/hwmon/ina3221 | 35 drivers/hwmon/Kconfig | 11 ++ drivers/hwmon/Makefile | 1

[PATCH v2 1/2] hwmon: Define binding for the INA3221 hwmon driver

2016-04-19 Thread Andrew F. Davis
Define a binding for the INA3221 Triple Current/Voltage Monitor. Signed-off-by: Andrew F. Davis --- Documentation/devicetree/bindings/hwmon/ina3221.txt | 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/hwmon/ina3221.txt diff --git

[PATCH v2 0/2] Add support for INA3221 Triple Current/Voltage Monitors

2016-04-19 Thread Andrew F. Davis
, Andrew Andrew F. Davis (2): hwmon: Define binding for the INA3221 hwmon driver hwmon: Add support for INA3221 Triple Current/Voltage Monitors .../devicetree/bindings/hwmon/ina3221.txt | 19 + Documentation/hwmon/ina3221| 35 ++ drivers/hwmon/Kconfig

Re: [PATCH 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-04-19 Thread Andrew F. Davis
On 04/19/2016 09:09 AM, Guenter Roeck wrote: > On 04/11/2016 01:50 PM, Andrew F. Davis wrote: >> Add support for the the INA3221 26v capable, Triple channel, >> Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor >> with I2C interface. >> >&g

[PATCH 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-04-11 Thread Andrew F. Davis
Add support for the the INA3221 26v capable, Triple channel, Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor with I2C interface. Signed-off-by: Andrew F. Davis --- Documentation/hwmon/ina3221 | 32 drivers/hwmon/Kconfig | 11 ++ drivers/hwmon/Makefile | 1