mbox series

[v2,0/4] Add support for variable sample time in lm75 driver

Message ID 20190808080246.8371-1-iker.perez@codethink.co.uk (mailing list archive)
Headers show
Series Add support for variable sample time in lm75 driver | expand

Message

Iker Perez Aug. 8, 2019, 8:02 a.m. UTC
From: Iker Perez del Palomar Sustatxa <iker.perez@codethink.co.uk>

Hello,

The objective of following patch series is to add support to lm75 driver
to be able to configure the sample time of it's supported devices,
particularly the tmp75b.

The applied changes involve:
	* Replace the current switch-case method for configuration
	  parameters selection to a structure storing them. This method
	  allows easier management of the parameters.
	* Split the writing of configuration registers into a separate
	  function. This method saves code in later patches.
	* Include new fields in lm75_params to add support for multiple
	  sample times.
	* Split the lm75_write functionality into separate, simpler,
	  functions.
	* Add support for configuring the devices via their sysfs nodes.

The patch series was based on linux-next's master branch.

Thank you Guenter Roeck, Michael Drake, Thomas Preston and Tom Eccles for
your time, help and feedback.

Regards,

Iker Perez del Palomar Sustatxa

Iker Perez del Palomar Sustatxa (4):
  hwmon: (lm75) Create structure to save all the configuration
    parameters.
  hwmon: (lm75) Create function from code to write into registers
  hwmon: (lm75) Add new fields into lm75_params_
  hwmon: (lm75) Modularize lm75_write and make hwmon_chip writable

 drivers/hwmon/lm75.c | 390 ++++++++++++++++++++++++++++++++++++---------------
 1 file changed, 274 insertions(+), 116 deletions(-)

Comments

Guenter Roeck Aug. 8, 2019, 2:14 p.m. UTC | #1
On Thu, Aug 08, 2019 at 09:02:42AM +0100, Iker Perez wrote:
> From: Iker Perez del Palomar Sustatxa <iker.perez@codethink.co.uk>
> 
> Hello,
> 
> The objective of following patch series is to add support to lm75 driver
> to be able to configure the sample time of it's supported devices,
> particularly the tmp75b.
> 
> The applied changes involve:
> 	* Replace the current switch-case method for configuration
> 	  parameters selection to a structure storing them. This method
> 	  allows easier management of the parameters.
> 	* Split the writing of configuration registers into a separate
> 	  function. This method saves code in later patches.
> 	* Include new fields in lm75_params to add support for multiple
> 	  sample times.
> 	* Split the lm75_write functionality into separate, simpler,
> 	  functions.
> 	* Add support for configuring the devices via their sysfs nodes.
> 
> The patch series was based on linux-next's master branch.
> 
> Thank you Guenter Roeck, Michael Drake, Thomas Preston and Tom Eccles for
> your time, help and feedback.
> 

Series applied to hwmon-next.

Thanks,
Guenter