On 27/05/14 16:34, Srinivas Pandruvada wrote:
Hi Reyad,
On 05/26/2014 08:20 PM, Reyad Attiyat wrote:
Hey Jonathan and Srinivas,
I have added in the needed magn_north iio sysfs attributes to the documentation.
I hope I did everything correct.
I noticed that the line length was very long as compensated is a fairly long
word.
I'm thinking I could shorten some of the sysfs paths or shorten the name.
For updating the hid-sensor-magn-3d do you have any tips on this? I'm thinking
I'll need a
dynamic iio_chan array that is passed to iio_register function. It would need to
be dynamic in the sense it would only add channels if it found x, y, z, or one
of
the north attributes. I'll start working on this soon.
I think you mean dynamic " iio_chan_spec magn_3d_channels[]". Currently this is
a
constant. So we can change this to add only channel, which are available.
Also your other changes must be submitted as series of patches, including the
drivers using them.
I specifically requested that Reyad get the proposal out asap so that we could
discuss it
'before' the patch series using it. Obviously it won't get merged until there
is
a driver using this interface, but often the interface discussion takes longer
than the review of the code so best to get it going asap!
Thanks,
Srinivas
Thank You,
Reyad Attiyat
Reyad Attiyat (1):
IIO: Added iio magn_north ABI documentation
Documentation/ABI/testing/sysfs-bus-iio | 76 +++++++++++++++++++++++++++++++++
1 file changed, 76 insertions(+)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
