Re: [PATCH] dell-smm-hwmon: Detect fan with index=2

From: Pali RohÃr
Date: Sun May 22 2016 - 11:19:37 EST


On Sunday 22 May 2016 17:11:18 Guenter Roeck wrote:
> On 05/21/2016 05:31 PM, Pali RohÃr wrote:
> > On Sunday 22 May 2016 02:21:46 Guenter Roeck wrote:
> >> On 05/21/2016 07:52 AM, Pali RohÃr wrote:
> >>> Some Dell machines (e.g. Dell Precision M3800) have two fans,
> >>> first with index=0 and second with index=2. So export also
> >>> attributes for third fan device with index=2.
> >>>
> >>> Reported-by: Tolga Cakir <cevelnet@xxxxxxxxx>
> >>> Signed-off-by: Pali RohÃr <pali.rohar@xxxxxxxxx>
> >>> ---
> >>>
> >>> drivers/hwmon/dell-smm-hwmon.c | 20 +++++++++++++++++++-
> >>> 1 file changed, 19 insertions(+), 1 deletion(-)
> >>>
> >>> ---
> >>>
> >>> Hi Tolga! Can you test this patch if sensors see fan device
> >>> correctly?
> >>
> >> I assume this means I should wait for test results before applying
> >> the patch ?
> >
> > Yes, testing should be done.
> >
> > Do you have some pending/testing/next tree/branch for such patches?
>
> I have a hwmon-staging branch. I added your patches to it.
>
> Please use "hwmon: (dell-smm)" in the subject line of your patches.
> The subsystem name should come first, the driver name in () is a
> hwmon idiosyncrasy, and we don't need to repeat the "hwmon".

Ok, I use generic "driver_name: description" pattern as each subsystem
use different format. But I can send in above format future hwmon
patches...

--
Pali RohÃr
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.