Re: [PATCH v2] hwmon/sht15: Root out platform data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, 10 Sep 2017 11:44:46 +0200
Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:

> After finding out there are active users of this sensor I noticed:

I would like to point out that I'm doing this for learning purposes and that I
don't have any pressing needs for this driver to work. But of course I'm really
grateful for all the replays!

> Cc: arm@xxxxxxxxxx
> Cc: Marco Franchi <marco.franchi@xxxxxxx>
> Cc: Davide Hug <d@xxxxxxxxxx>
> Cc: Jonathan Cameron <jic23@xxxxxxxxx>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> ChangeLog v1->v2:
> - Fix the return value from devm_gpiod_get() with PTR_ERR()
>   so we defer etc correctly.
> 
> ARM SoC folks: please ACK this so the HWMON maintainer can merge
> it when it is in reasonable shape.
> 
> Marco: can you test this patch with your setup?
> 
> Davide: can you test this patch with your setup?

I tested it and I can make it work (as the current driver) by modifying gpiolib
to not test for the FLAG_USED_AS_IRQ in _gpiod_direction_output_raw.
Otherwise again I get:

	gpio-48 (SHT15 data): _gpiod_direction_output_raw: tried to set a GPIO tied to an IRQ as output

as with the current driver. But I guess this is as expected.
(I did this on a beaglebone black with kernel 4.9.10. I'll compile a newer
kernel and try it there to.)

Thanks!
Davide
--
To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux