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

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

[Guenter Roeck]

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

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

[Guenter Roeck]

On 4/20/21 12:14 AM, Navin Sankar wrote:
> On Tue, Apr 20, 2021 at 11:57 AM Guenter Roeck <linux@roeck-us.net <mailto:linux@roeck-us.net>> 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

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

[Guenter Roeck]

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@roeck-us.net <mailto:linux@roeck-us.net>> wrote:
> 
>     On 4/20/21 12:14 AM, Navin Sankar wrote:
>     > On Tue, Apr 20, 2021 at 11:57 AM Guenter Roeck <linux@roeck-us.net <mailto:linux@roeck-us.net> <mailto:linux@roeck-us.net <mailto:linux@roeck-us.net>>> 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
>