[lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Guenter Roeck]

From: Guenter Roeck <linux@roeck-us.net>

This patch series builds on top of v4.5-rc1. The series was tested on
a system with IT8728F. Out-of-kernel versions were tested by various
people using a github repository.

Most of the patches in this series were previously submitted as part of
earlier patch series. No substantial changes were made to those patches
since the previous submission.

----------------------------------------------------------------
Guenter Roeck (25):
      hwmon: (it87) Add feature flag for AVCC3 support
      hwmon: (it87) Add support for all pwm channels on IT8620E
      hwmon: (it87) Add support for second pwm frequency register
      hwmon: (it87) Simplify error return in it87_device_add
      hwmon: (it87) Don't use pdev as static driver variable
      hwmon: (it87) Pass SIO base address as parameter to superio functions
      hwmon: (it87) Add support for second Super-IO chip
      hwmon: (it87) Rearrange code to avoid forward declarations
      hwmon: (it87) Use is_visible for voltage sensors
      hwmon: (it87) Use is_visible for temperature sensors
      hwmon: (it87) Use is_visible for fan attributes
      hwmon: (it87) Use is_visible for pwm attributes
      hwmon: (it87) Use single group and is_visible for miscellaneous attributes
      hwmon: (it87) Convert to use new hwmon API
      hwmon: (it87) Support up to 6 temperature sensors on IT8620E
      hwmon: (it87) Simplify reading voltage registers
      hwmon: (it87) Add support for VIN7 to VIN10 on IT8620E
      hwmon: (it87) Use BIT macro
      hwmon: (it87) Use defines for array sizes and sensor counts
      hwmon: (it87) Formatting cleanup
      hwmon: (it87) Support disabling fan control for all pwm control and chips
      hwmon: (it87) Enhance validation for fan4 and fan5
      hwmon: (it87) Support automatic pwm control on newer chips
      hwmon: (it87) Fix pwm_temp_map for system with 6 pwm channels
      hwmon: (it87) Add support for IT8628E

 Documentation/hwmon/it87 |   15 +-
 drivers/hwmon/Kconfig    |    3 +-
 drivers/hwmon/it87.c     | 2281 ++++++++++++++++++++++++++--------------------
 3 files changed, 1300 insertions(+), 999 deletions(-)

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Guenter Roeck]

On 01/26/2016 06:03 PM, Guenter Roeck wrote:
> From: Guenter Roeck <linux@roeck-us.net>
>
> This patch series builds on top of v4.5-rc1. The series was tested on
> a system with IT8728F. Out-of-kernel versions were tested by various
> people using a github repository.
>
> Most of the patches in this series were previously submitted as part of
> earlier patch series. No substantial changes were made to those patches
> since the previous submission.
>

Jean,

Any idea what to do with this series ?

Guenter

> ----------------------------------------------------------------
> Guenter Roeck (25):
>        hwmon: (it87) Add feature flag for AVCC3 support
>        hwmon: (it87) Add support for all pwm channels on IT8620E
>        hwmon: (it87) Add support for second pwm frequency register
>        hwmon: (it87) Simplify error return in it87_device_add
>        hwmon: (it87) Don't use pdev as static driver variable
>        hwmon: (it87) Pass SIO base address as parameter to superio functions
>        hwmon: (it87) Add support for second Super-IO chip
>        hwmon: (it87) Rearrange code to avoid forward declarations
>        hwmon: (it87) Use is_visible for voltage sensors
>        hwmon: (it87) Use is_visible for temperature sensors
>        hwmon: (it87) Use is_visible for fan attributes
>        hwmon: (it87) Use is_visible for pwm attributes
>        hwmon: (it87) Use single group and is_visible for miscellaneous attributes
>        hwmon: (it87) Convert to use new hwmon API
>        hwmon: (it87) Support up to 6 temperature sensors on IT8620E
>        hwmon: (it87) Simplify reading voltage registers
>        hwmon: (it87) Add support for VIN7 to VIN10 on IT8620E
>        hwmon: (it87) Use BIT macro
>        hwmon: (it87) Use defines for array sizes and sensor counts
>        hwmon: (it87) Formatting cleanup
>        hwmon: (it87) Support disabling fan control for all pwm control and chips
>        hwmon: (it87) Enhance validation for fan4 and fan5
>        hwmon: (it87) Support automatic pwm control on newer chips
>        hwmon: (it87) Fix pwm_temp_map for system with 6 pwm channels
>        hwmon: (it87) Add support for IT8628E
>
>   Documentation/hwmon/it87 |   15 +-
>   drivers/hwmon/Kconfig    |    3 +-
>   drivers/hwmon/it87.c     | 2281 ++++++++++++++++++++++++++--------------------
>   3 files changed, 1300 insertions(+), 999 deletions(-)
>


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Guenter Roeck]

On 01/26/2016 06:03 PM, Guenter Roeck wrote:
> From: Guenter Roeck <linux@roeck-us.net>
>
> This patch series builds on top of v4.5-rc1. The series was tested on
> a system with IT8728F. Out-of-kernel versions were tested by various
> people using a github repository.
>
> Most of the patches in this series were previously submitted as part of
> earlier patch series. No substantial changes were made to those patches
> since the previous submission.
>
Jean,

any idea what to do with this patch series ?

Guenter

> ----------------------------------------------------------------
> Guenter Roeck (25):
>        hwmon: (it87) Add feature flag for AVCC3 support
>        hwmon: (it87) Add support for all pwm channels on IT8620E
>        hwmon: (it87) Add support for second pwm frequency register
>        hwmon: (it87) Simplify error return in it87_device_add
>        hwmon: (it87) Don't use pdev as static driver variable
>        hwmon: (it87) Pass SIO base address as parameter to superio functions
>        hwmon: (it87) Add support for second Super-IO chip
>        hwmon: (it87) Rearrange code to avoid forward declarations
>        hwmon: (it87) Use is_visible for voltage sensors
>        hwmon: (it87) Use is_visible for temperature sensors
>        hwmon: (it87) Use is_visible for fan attributes
>        hwmon: (it87) Use is_visible for pwm attributes
>        hwmon: (it87) Use single group and is_visible for miscellaneous attributes
>        hwmon: (it87) Convert to use new hwmon API
>        hwmon: (it87) Support up to 6 temperature sensors on IT8620E
>        hwmon: (it87) Simplify reading voltage registers
>        hwmon: (it87) Add support for VIN7 to VIN10 on IT8620E
>        hwmon: (it87) Use BIT macro
>        hwmon: (it87) Use defines for array sizes and sensor counts
>        hwmon: (it87) Formatting cleanup
>        hwmon: (it87) Support disabling fan control for all pwm control and chips
>        hwmon: (it87) Enhance validation for fan4 and fan5
>        hwmon: (it87) Support automatic pwm control on newer chips
>        hwmon: (it87) Fix pwm_temp_map for system with 6 pwm channels
>        hwmon: (it87) Add support for IT8628E
>
>   Documentation/hwmon/it87 |   15 +-
>   drivers/hwmon/Kconfig    |    3 +-
>   drivers/hwmon/it87.c     | 2281 ++++++++++++++++++++++++++--------------------
>   3 files changed, 1300 insertions(+), 999 deletions(-)
>


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) V

[Martin Blumenstingl]

Guenter Roeck <patchwork <at> patchwork.roeck-us.net> writes:
> From: Guenter Roeck <linux <at> roeck-us.net>
> 
> This patch series builds on top of v4.5-rc1. The series was tested on
> a system with IT8728F. Out-of-kernel versions were tested by various
> people using a github repository.
I've been running the whole series on top of v4.5 for the past two
weeks on a system with IT8628E. I did not find any problems so far.



_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Guenter Roeck]

Hi Martin,

On 04/09/2016 02:33 AM, Martin Blumenstingl wrote:
> Guenter Roeck <patchwork <at> patchwork.roeck-us.net> writes:
>> From: Guenter Roeck <linux <at> roeck-us.net>
>>
>> This patch series builds on top of v4.5-rc1. The series was tested on
>> a system with IT8728F. Out-of-kernel versions were tested by various
>> people using a github repository.
> I've been running the whole series on top of v4.5 for the past two
> weeks on a system with IT8628E. I did not find any problems so far.
>

thanks a lot for the feedback. It would be great if you can send me
an official Tested-by: that I can add into the series (or a permission
to do so).

With the experience gained with the off-list driver, I think this series
is good to go for upstream, even without code review feedback. Otherwise
I fear we'll never get there. Jean, any objections ?

Thanks,
Guenter


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Guenter Roeck]

Hi Martin,

On 04/09/2016 02:33 AM, Martin Blumenstingl wrote:
> Guenter Roeck <patchwork <at> patchwork.roeck-us.net> writes:
>> From: Guenter Roeck <linux <at> roeck-us.net>
>>
>> This patch series builds on top of v4.5-rc1. The series was tested on
>> a system with IT8728F. Out-of-kernel versions were tested by various
>> people using a github repository.
> I've been running the whole series on top of v4.5 for the past two
> weeks on a system with IT8628E. I did not find any problems so far.
>

thanks a lot for the feedback. It would be great if you can send me
an official Tested-by: that I can add into the series (or a permission
to do so).

With the experience gained with the off-list driver, I think this series
is good to go for upstream, even without code review feedback. Otherwise
I fear we'll never get there. Jean, any objections ?

Thanks,
Guenter


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Martin Blumenstingl]

On Sat, Apr 9, 2016 at 5:07 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>>> This patch series builds on top of v4.5-rc1. The series was tested on
>>> a system with IT8728F. Out-of-kernel versions were tested by various
>>> people using a github repository.
>>
>> I've been running the whole series on top of v4.5 for the past two
>> weeks on a system with IT8628E. I did not find any problems so far.
>>
>
> thanks a lot for the feedback. It would be great if you can send me
> an official Tested-by: that I can add into the series (or a permission
> to do so).
sure, feel free to add this to the whole series:
Tested-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

PS: Thank you Guenter for your hard work on this!

Re: [lm-sensors] [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Martin Blumenstingl]

On Sat, Apr 9, 2016 at 5:07 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>>> This patch series builds on top of v4.5-rc1. The series was tested on
>>> a system with IT8728F. Out-of-kernel versions were tested by various
>>> people using a github repository.
>>
>> I've been running the whole series on top of v4.5 for the past two
>> weeks on a system with IT8628E. I did not find any problems so far.
>>
>
> thanks a lot for the feedback. It would be great if you can send me
> an official Tested-by: that I can add into the series (or a permission
> to do so).
sure, feel free to add this to the whole series:
Tested-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

PS: Thank you Guenter for your hard work on this!

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Jean Delvare]

Hi Guenter,

On Sat, 9 Apr 2016 08:07:27 -0700, Guenter Roeck wrote:
> Hi Martin,
> 
> On 04/09/2016 02:33 AM, Martin Blumenstingl wrote:
> > Guenter Roeck <patchwork <at> patchwork.roeck-us.net> writes:
> >> From: Guenter Roeck <linux <at> roeck-us.net>
> >>
> >> This patch series builds on top of v4.5-rc1. The series was tested on
> >> a system with IT8728F. Out-of-kernel versions were tested by various
> >> people using a github repository.
> > I've been running the whole series on top of v4.5 for the past two
> > weeks on a system with IT8628E. I did not find any problems so far.
> 
> thanks a lot for the feedback. It would be great if you can send me
> an official Tested-by: that I can add into the series (or a permission
> to do so).
> 
> With the experience gained with the off-list driver, I think this series
> is good to go for upstream, even without code review feedback. Otherwise
> I fear we'll never get there. Jean, any objections ?

I am sorry that I do not have the time to review this patch series. But
I trust your judgment and I have no objection, if you think the series
is good to go upstream then please push it there.

Thanks,
-- 
Jean Delvare
SUSE L3 Support

Re: [PATCH v3 00/25] hwmon: (it87) Various enhancements and fixes

[Guenter Roeck]

On 04/12/2016 12:21 AM, Jean Delvare wrote:
> Hi Guenter,
>
> On Sat, 9 Apr 2016 08:07:27 -0700, Guenter Roeck wrote:
>> Hi Martin,
>>
>> On 04/09/2016 02:33 AM, Martin Blumenstingl wrote:
>>> Guenter Roeck <patchwork <at> patchwork.roeck-us.net> writes:
>>>> From: Guenter Roeck <linux <at> roeck-us.net>
>>>>
>>>> This patch series builds on top of v4.5-rc1. The series was tested on
>>>> a system with IT8728F. Out-of-kernel versions were tested by various
>>>> people using a github repository.
>>> I've been running the whole series on top of v4.5 for the past two
>>> weeks on a system with IT8628E. I did not find any problems so far.
>>
>> thanks a lot for the feedback. It would be great if you can send me
>> an official Tested-by: that I can add into the series (or a permission
>> to do so).
>>
>> With the experience gained with the off-list driver, I think this series
>> is good to go for upstream, even without code review feedback. Otherwise
>> I fear we'll never get there. Jean, any objections ?
>
> I am sorry that I do not have the time to review this patch series. But
> I trust your judgment and I have no objection, if you think the series
> is good to go upstream then please push it there.
>
No need to be sorry. We are all busy.

Thanks!
Guenter