Linux-Hwmon Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
@ 2019-10-03 18:32 Tremblay, Eric
  2019-10-03 18:35 ` Tremblay, Eric
  0 siblings, 1 reply; 4+ messages in thread
From: Tremblay, Eric @ 2019-10-03 18:32 UTC (permalink / raw)
  To: linux-hwmon

Version three of the driver for Texas Instruments TMP512/513 sensors.

Thanks again for the review, I think we are getting close. I'm not in an easy
position for now to test the driver without device tree. I can put the required
effort to

Main changes from version 2:
	- Remove the neccessity to provide a shunt-resistor value
	- Remove max-expected-current-ma configuration and always
	  use max possible current for calibration
	- Make sure calculation during calibration can't overflow
	- Add value clamping for DT value and runtime parameters
	- Support non DT system
	- Move hysteresis from DT to standard attribute

Main changes from version 1:
	- Use the with_info API instead of sysfs attributes.
	- Remove non-standard attributes and raw value.
	- Move settings that were non-standard attributes to
	  device tree, update documentation as well.
	- Fix coding style issues

Eric Tremblay (2):
  Add driver for Texas Instruments TMP512/513 sensor chips.
  Add DT bindings for TMP513 driver

 .../devicetree/bindings/hwmon/tmp513.txt      |  33 +
 Documentation/hwmon/tmp513.rst                | 102 ++
 MAINTAINERS                                   |   7 +
 drivers/hwmon/Kconfig                         |  10 +
 drivers/hwmon/Makefile                        |   1 +
 drivers/hwmon/tmp513.c                        | 901 ++++++++++++++++++
 include/linux/platform_data/tmp513.h          |  28 +
 7 files changed, 1082 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/hwmon/tmp513.txt
 create mode 100644 Documentation/hwmon/tmp513.rst
 create mode 100644 drivers/hwmon/tmp513.c
 create mode 100644 include/linux/platform_data/tmp513.h

-- 
2.17.1


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

* RE: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
  2019-10-03 18:32 [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips Tremblay, Eric
@ 2019-10-03 18:35 ` Tremblay, Eric
  2019-10-03 20:33   ` Guenter Roeck
  0 siblings, 1 reply; 4+ messages in thread
From: Tremblay, Eric @ 2019-10-03 18:35 UTC (permalink / raw)
  To: Tremblay, Eric; +Cc: linux-hwmon

> Version three of the driver for Texas Instruments TMP512/513 sensors.
> 
> Thanks again for the review, I think we are getting close. I'm not in an easy position for now to test the driver without device tree. I
> can put the required effort to

Sorry, I mean that I'm not in a good position to test it, but I can put the required effort to if needed.
> 
> Main changes from version 2:
> 	- Remove the neccessity to provide a shunt-resistor value
> 	- Remove max-expected-current-ma configuration and always
> 	  use max possible current for calibration
> 	- Make sure calculation during calibration can't overflow
> 	- Add value clamping for DT value and runtime parameters
> 	- Support non DT system
> 	- Move hysteresis from DT to standard attribute
> 
> Main changes from version 1:
> 	- Use the with_info API instead of sysfs attributes.
> 	- Remove non-standard attributes and raw value.
> 	- Move settings that were non-standard attributes to
> 	  device tree, update documentation as well.
> 	- Fix coding style issues
> 
> Eric Tremblay (2):
>   Add driver for Texas Instruments TMP512/513 sensor chips.
>   Add DT bindings for TMP513 driver
> 
>  .../devicetree/bindings/hwmon/tmp513.txt      |  33 +
>  Documentation/hwmon/tmp513.rst                | 102 ++
>  MAINTAINERS                                   |   7 +
>  drivers/hwmon/Kconfig                         |  10 +
>  drivers/hwmon/Makefile                        |   1 +
>  drivers/hwmon/tmp513.c                        | 901 ++++++++++++++++++
>  include/linux/platform_data/tmp513.h          |  28 +
>  7 files changed, 1082 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/hwmon/tmp513.txt
>  create mode 100644 Documentation/hwmon/tmp513.rst  create mode 100644 drivers/hwmon/tmp513.c  create mode 100644
> include/linux/platform_data/tmp513.h
> 
> --
> 2.17.1


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

* Re: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
  2019-10-03 18:35 ` Tremblay, Eric
@ 2019-10-03 20:33   ` Guenter Roeck
  2019-10-04 14:45     ` Tremblay, Eric
  0 siblings, 1 reply; 4+ messages in thread
From: Guenter Roeck @ 2019-10-03 20:33 UTC (permalink / raw)
  To: Tremblay, Eric; +Cc: linux-hwmon

On Thu, Oct 03, 2019 at 06:35:51PM +0000, Tremblay, Eric wrote:
> > Version three of the driver for Texas Instruments TMP512/513 sensors.
> > 
> > Thanks again for the review, I think we are getting close. I'm not in an easy position for now to test the driver without device tree. I
> > can put the required effort to
> 
> Sorry, I mean that I'm not in a good position to test it, but I can put the required effort to if needed.

All you would need to do is to use device_ API functions instead of
of_device_ API functions, so I can not entirely follow your logic.

Guenter

> > 
> > Main changes from version 2:
> > 	- Remove the neccessity to provide a shunt-resistor value
> > 	- Remove max-expected-current-ma configuration and always
> > 	  use max possible current for calibration
> > 	- Make sure calculation during calibration can't overflow
> > 	- Add value clamping for DT value and runtime parameters
> > 	- Support non DT system
> > 	- Move hysteresis from DT to standard attribute
> > 
> > Main changes from version 1:
> > 	- Use the with_info API instead of sysfs attributes.
> > 	- Remove non-standard attributes and raw value.
> > 	- Move settings that were non-standard attributes to
> > 	  device tree, update documentation as well.
> > 	- Fix coding style issues
> > 
> > Eric Tremblay (2):
> >   Add driver for Texas Instruments TMP512/513 sensor chips.
> >   Add DT bindings for TMP513 driver
> > 
> >  .../devicetree/bindings/hwmon/tmp513.txt      |  33 +
> >  Documentation/hwmon/tmp513.rst                | 102 ++
> >  MAINTAINERS                                   |   7 +
> >  drivers/hwmon/Kconfig                         |  10 +
> >  drivers/hwmon/Makefile                        |   1 +
> >  drivers/hwmon/tmp513.c                        | 901 ++++++++++++++++++
> >  include/linux/platform_data/tmp513.h          |  28 +
> >  7 files changed, 1082 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/hwmon/tmp513.txt
> >  create mode 100644 Documentation/hwmon/tmp513.rst  create mode 100644 drivers/hwmon/tmp513.c  create mode 100644
> > include/linux/platform_data/tmp513.h
> > 
> > --
> > 2.17.1
> 

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

* RE: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
  2019-10-03 20:33   ` Guenter Roeck
@ 2019-10-04 14:45     ` Tremblay, Eric
  0 siblings, 0 replies; 4+ messages in thread
From: Tremblay, Eric @ 2019-10-04 14:45 UTC (permalink / raw)
  To: Guenter Roeck; +Cc: linux-hwmon

> On Thu, Oct 03, 2019 at 06:35:51PM +0000, Tremblay, Eric wrote:
> > > Version three of the driver for Texas Instruments TMP512/513 sensors.
> > >
> > > Thanks again for the review, I think we are getting close. I'm not
> > > in an easy position for now to test the driver without device tree.
> > > I can put the required effort to
> >
> > Sorry, I mean that I'm not in a good position to test it, but I can put the required effort to if needed.
> 
> All you would need to do is to use device_ API functions instead of of_device_ API functions, so I can not entirely follow your logic.

I was clearly missing some piece of knowledge about the device API... I did some research and I was able to instantiate and test
the driver without using the of_device API. The version 3 work as expected with or without platform_data provided.

Eric 
> 
> Guenter
> 
> > >
> > > Main changes from version 2:
> > > 	- Remove the neccessity to provide a shunt-resistor value
> > > 	- Remove max-expected-current-ma configuration and always
> > > 	  use max possible current for calibration
> > > 	- Make sure calculation during calibration can't overflow
> > > 	- Add value clamping for DT value and runtime parameters
> > > 	- Support non DT system
> > > 	- Move hysteresis from DT to standard attribute
> > >
> > > Main changes from version 1:
> > > 	- Use the with_info API instead of sysfs attributes.
> > > 	- Remove non-standard attributes and raw value.
> > > 	- Move settings that were non-standard attributes to
> > > 	  device tree, update documentation as well.
> > > 	- Fix coding style issues
> > >
> > > Eric Tremblay (2):
> > >   Add driver for Texas Instruments TMP512/513 sensor chips.
> > >   Add DT bindings for TMP513 driver
> > >
> > >  .../devicetree/bindings/hwmon/tmp513.txt      |  33 +
> > >  Documentation/hwmon/tmp513.rst                | 102 ++
> > >  MAINTAINERS                                   |   7 +
> > >  drivers/hwmon/Kconfig                         |  10 +
> > >  drivers/hwmon/Makefile                        |   1 +
> > >  drivers/hwmon/tmp513.c                        | 901 ++++++++++++++++++
> > >  include/linux/platform_data/tmp513.h          |  28 +
> > >  7 files changed, 1082 insertions(+)  create mode 100644
> > > Documentation/devicetree/bindings/hwmon/tmp513.txt
> > >  create mode 100644 Documentation/hwmon/tmp513.rst  create mode
> > > 100644 drivers/hwmon/tmp513.c  create mode 100644
> > > include/linux/platform_data/tmp513.h
> > >
> > > --
> > > 2.17.1
> >

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

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-03 18:32 [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips Tremblay, Eric
2019-10-03 18:35 ` Tremblay, Eric
2019-10-03 20:33   ` Guenter Roeck
2019-10-04 14:45     ` Tremblay, Eric

Linux-Hwmon Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-hwmon/0 linux-hwmon/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-hwmon linux-hwmon/ https://lore.kernel.org/linux-hwmon \
		linux-hwmon@vger.kernel.org
	public-inbox-index linux-hwmon

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-hwmon


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git