Re: Question: Driver support for SHT4X temperature and humidity sensor

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

 



On 4/20/21 6:15 AM, Navin Sankar wrote:
> I thought this way, by looking at temp & humidity drivers in iio.
>  Sorry for the noise. Since I am new to kernel contribution.
> 

Normally it is a question of use cases. If the sensor is
typically used for hardware monitoring of the system
it is built into, it belongs into hwmon. If it is typically
used for other purposes, it should reside in iio.

Guenter

> Thanks & Regards,
> Navin Sankar N V.
> 
> On Tue, Apr 20, 2021 at 4:18 PM Guenter Roeck <linux@xxxxxxxxxxxx <mailto:linux@xxxxxxxxxxxx>> wrote:
> 
>     On 4/20/21 12:14 AM, Navin Sankar wrote:
>     > On Tue, Apr 20, 2021 at 11:57 AM Guenter Roeck <linux@xxxxxxxxxxxx <mailto:linux@xxxxxxxxxxxx> <mailto:linux@xxxxxxxxxxxx <mailto:linux@xxxxxxxxxxxx>>> wrote:
>     >
>     >     On 4/19/21 9:43 PM, Navin Sankar wrote:
>     >     > Dear Maintainer,
>     >     >
>     >     >     I am trying to add driver support for sensirion based temperature & humidity sensor sht4x.
>     >     >
>     >     > I am bit confused, where to add support for this driver. Either in iio subsys or hwmon subsys?
>     >     >
>     >
>     >     The sht[23] sensor drivers are in hwmon, so I'd normally suggest
>     >     a hwmon driver.
>     >
>     >     Having said that: What is the cause of your confusion ?
>     >
>     >     Guenter
>     >
>     >
>     > Thank you for the clarification.
>     >
>     > I thought the convention is to use iio for new temperature and humidity sensor.
>     >
> 
>     I didn't know that; that decision was made without consulting hardware
>     monitoring maintainers. Can you point me to the relevant exchanges or
>     documentation ?
> 
>     Thanks,
>     Guenter
> 





[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