From: Guenter Roeck <linux@roeck-us.net>
To: "Tremblay, Eric" <etremblay@distech-controls.com>
Cc: "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>
Subject: Re: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
Date: Thu, 3 Oct 2019 13:33:58 -0700 [thread overview]
Message-ID: <20191003203358.GA16786@roeck-us.net> (raw)
In-Reply-To: <BL0PR01MB48354C0BF0D094921233611A959F0@BL0PR01MB4835.prod.exchangelabs.com>
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
>
next prev parent reply other threads:[~2019-10-03 20:34 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
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 [this message]
2019-10-04 14:45 ` Tremblay, Eric
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191003203358.GA16786@roeck-us.net \
--to=linux@roeck-us.net \
--cc=etremblay@distech-controls.com \
--cc=linux-hwmon@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).