Hi, Jean Do you have any more suggestions on this series? Thanks. Wei. On 08/07/2013 02:18 PM, Wei Ni wrote: > This patch set enhance the lm90 driver, > it make the driver more readable and easier to use thermal framework. > > This series is v4, previous version patches: > [RFC]: http://thread.gmane.org/gmane.linux.power-management.general/31056 > [v1]: http://thread.gmane.org/gmane.linux.ports.tegra/11710/ > [v2]: http://www.mail-archive.com/linux-kernel@xxxxxxxxxxxxxxx/msg465555.html > [v3]: http://www.mail-archive.com/linux-kernel@xxxxxxxxxxxxxxx/msg466772.html > > Changes from v3: > 1. remove the patch "hwmon: (lm90) split set&show temp as common codes", > and sent it as a separated one. > 2. fix the bug of second read on STATUS register. > 3. fix some code style issue according to Jean's comments. > > Changes from v2: > 1. update the defines for status bit, and go into a separate patch. > 2. introduce the new lm90_is_tripped() for lm90_irq_thread and lm90_alert. > > Changes from v1: > 1. change the string "irq" to "IRQ" > 2. add macro defines for the alarm status > 3. consider the shared IRQ. > > Changes from RFC: > 1. change _show_temp() to read_temp(), _set_temp() to write_temp(). > 2. simply return value for the read_temp(), not use pointer. > 3. use devm_request_threaded_irq() to request irq and set flag IRQF_ONESHOT. > > Wei Ni (3): > hwmon: (lm90) Define status bits > hwmon: (lm90) add support to handle IRQ > hwmon: (lm90) use enums for the indexes of temp8 and temp11 > > drivers/hwmon/lm90.c | 252 ++++++++++++++++++++++++++++++++------------------ > 1 file changed, 163 insertions(+), 89 deletions(-) > -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
