On Wed, 11 Dec 2024 22:17:49 -0800
Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
Hi,New type of sensor, but sure could be in IIO.
On 12/11/24 20:24, Hermes Zhang wrote:
Add support for Nicera D3-323-AA Pyroelectric IR sensor. The sensor...
support to config the threshold/filter_type/filter_step and return the
detect result in sysfs attribute.
Signed-off-by: Hermes Zhang <Hermes.Zhang@xxxxxxxx>
---
+I don't know what this is, but it is most definitely not a hardware
+static DEVICE_ATTR_WO(pir_threshold);
+static DEVICE_ATTR_WO(pir_filter_step);
+static DEVICE_ATTR_WO(pir_filter_type);
+static DEVICE_ATTR_RO(pir_detector);
+
+static struct attribute *d3323aa_attrs[] = {
+ &dev_attr_pir_threshold.attr,
+ &dev_attr_pir_filter_step.attr,
+ &dev_attr_pir_filter_type.attr,
+ &dev_attr_pir_detector.attr,
+ NULL,
+};
+
+ATTRIBUTE_GROUPS(d3323aa);
+
monitoring device. I don't see a definition of those attributes,
so I have no idea what they represent.
Maybe this is an iio device, but given the unusual attributes
I am not even sure about that. Jonathan, any thoughts ?
Seems mostly a human presence sensor. Not that different from some
types of proximity sensor and indeed that might be the path to take
here.
Taking a quick look at the driver suggests there is lots more information
needed to understand the ABI. At very least needs ABI docs so we can
discuss how that is generalized. So if submitting an IIO driver
I want to see
Documenation/ABI/testing/sysfs-bus-iio-xxxx
with significant detail. The datasheet provides no where near enough
info.
Jonathan