Messages in this thread | | | Subject | Re: [PATCH 4/4] hwmon: vcnl3020: add hwmon driver for intrusion sensor | From | Ivan Mikhaylov <> | Date | Tue, 4 May 2021 22:46:53 +0300 |
| |
On Fri, 2021-04-30 at 09:38 -0700, Guenter Roeck wrote: > On Fri, Apr 30, 2021 at 06:24:19PM +0300, Ivan Mikhaylov wrote: > > Intrusion status detection via Interrupt Status Register. > > > > Signed-off-by: Ivan Mikhaylov <i.mikhaylov@yadro.com> > > I think this should, if at all, be handled using the > iio->hwmon bridge (or, in other words, require a solution > which is not chip specific).
Thanks a lot for suggestion, it's actually looks what's needed here instead of this driver. Anyways, there is no IIO_PROXIMITY support inside supported types in iio_hwmon.c. Should I add additional case inside this driver for IIO_PROXIMITY type?
> I am also not sure if "proximity" is really appropriate to use > for intrusion detection in the sense of hardware monitoring. > This would require a proximity sensor within a chassis, which > would be both overkill and unlikely to happen in the real world. > "Intrusion", in hardware monitoring context, means "someone > opened the chassis", not "someone got [too] close". >
I'm not sure either but it exists :) And it's exactly for this purpose: "someone opened the chassis", "how near/far is cover?".
| |