RE: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
> On Thu, Oct 03, 2019 at 06:35:51PM +, 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 > >
Re: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
On Thu, Oct 03, 2019 at 06:35:51PM +, 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 >
RE: [PATCH v3 0/2] hwmon: Add driver for Texas Instruments TMP512/513 sensor chips
> 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
