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

From: Laxman Dewangan
Date: Fri Apr 13 2012 - 06:00:12 EST


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.

+ 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

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