Re: [PATCH 2/2] iio: adc: xilinx-xadc: Add xlnx,extend-name as an optional argument for aux channels

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

 



On 5/26/2015 4:38 AM, Lars-Peter Clausen wrote:
On 05/23/2015 01:23 PM, Jonathan Cameron wrote:
On 19/05/15 18:53, Lars-Peter Clausen wrote:
On 05/08/2015 12:44 AM, Xander Huff wrote:
To better facilitate user-mode access to optional aux channels, allow
device trees to specify a custom extended name for defined channels.

Hi,

'extend-name' is kind of a IIO specific term. I think a better name in the
devicetree context would be 'label'. That's used everywhere else when giving
a short description string to a node.

Hi Lars,

What do you think of the general idea?  I'm unconvinced we want to do this in
channel naming.
Perhaps we want an additional attribute giving access to this sort of
'semantic' information?

I'm not convinced about it either. My gut feeling is that this is not the right
approach, but I don't really have any better ideas at the moment.

Xander can you give a short description of how this information will be used by
an application? Would having a label property for the channel also work for you
(e.g. in_voltage0_label)?

- Lars

We use 10 of the optional aux channels available in the XADC and have them wired up to specific power supplies. For example, channel 9 is connected to the current of a 6V power supply and channel 12 is connected to the voltage of that same supply, channel 4 is connected to the current of a 3.3V power supply and channel 5 is connected to its voltage.

Without adding extended names to distinguish these aux channels, the handles available inside /sys/bus/iio/devices/[device] are generic: in_voltage10_raw, in_voltage10_scale, for example. If I added the label 'user5v_v', then these handles would be in_voltage10_user5v_v_raw and in_voltage10_user5v_v_scale. This makes it a lot easier to get started and let the handles describe themselves, rather than having to look up which channels are connected to which things.

As far as using 'label' instead of 'extend-name', that's fine with me. I've already sent out a v2 using 'label' instead.

--
Xander Huff
Staff Software Engineer
National Instruments
--
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




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux