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