Re: [PATCH V4 2/2] staging: iio: add driver for isl29028

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

 



On 4/13/2012 10:48 AM, Laxman Dewangan wrote:
On Thursday 12 April 2012 08:04 PM, Jonathan Cameron wrote:
On 4/12/2012 3:01 PM, Laxman Dewangan wrote:
Intersil's ISL29028 is concurrent Ambient Light and
Proximity Sensor device.
Add driver to access the light and IR intensity and
proximity value via iio interface.
Very nearly there.  The available attributes need to match naming
of what they are providing values for. (we need a better way of handling
these but that's a job for another day).

Otherwise, all good.

Fine, I will do change.

+     mutex_unlock(&chip->lock);
+     return ret;
+}
+
+static IIO_CONST_ATTR(proximity_sampling_frequency_available,
+                             "1, 3, 5, 10, 13, 20, 83, 100");
+static IIO_CONST_ATTR(illuminance_scale_available, "125, 2000");
+
+#define ISL29028_DEV_ATTR(name) (&iio_dev_attr_##name.dev_attr.attr)
+#define ISL29028_CONST_ATTR(name) (&iio_const_attr_##name.dev_attr.attr)
+static struct attribute *isl29028_attributes[] = {
+     ISL29028_CONST_ATTR(proximity_sampling_frequency_available),
tiny disconnect now that you have in_proximity0_sampling frequency, this
should be in_proximity0_sampling_frequency_available


I saw the attribute appear as
in_illuminance_input
in_illuminance_scale
in_intensity_raw
in_proximity_raw
in_proximity_sampling_frequency


As I have not set the .indexed to non-zero, I will go as non-indexd name like
in_proximity_sampling_frequency_available.
Fair enough.

+     ISL29028_CONST_ATTR(illuminance_scale_available),
snap here (sorry, missed that last time).

in_illuminance0_scale_available


I think should be in_illuminance_scale_available
yup

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

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux