From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 1/1] iio: stx104: Add IIO support for the ADC channels
Date: Mon, 22 Aug 2016 09:24:09 -0400 [thread overview]
Message-ID: <20160822132409.GA9041@sophia> (raw)
In-Reply-To: <99c36f16-9f2c-383d-9f44-badeee523a5a@kernel.org>
On Sun, Aug 21, 2016 at 08:39:01PM +0100, Jonathan Cameron wrote:
>On 25/07/16 18:54, William Breathitt Gray wrote:
>> The Apex Embedded Systems STX104 features 16 channels of single-ended (8
>> channels of true differential) 16-bit analog input. Differential input
>> configuration may be selected via a physical jumper on the device.
>> Similarly, input polarity (unipolar/bipolar) is configured via a
>> physical jumper on the device.
>>
>> Input gain selection is available to the user via software, thus
>> allowing eight possible input ranges: +-10V, +-5V, +-2.5V, +-1.25V,
>> 0 to 10V, 0 to 5V, 0 to 2.5V, and 0 to 1.25V. Four input gain
>> configurations are supported: x1, x2, x4, and x8.
>>
>> This ADC resolution is 16-bits (1/65536 of full scale). Analog input
>> samples are taken on software trigger; neither FIFO sampling nor
>> interrupt triggering is supported by this driver.
>>
>> The Apex Embedded Systems STX104 is primarily an analog-to-digital
>> converter device. The STX104 IIO driver was initially placed in the DAC
>> directory because only the DAC portion of the STX104 was supported at
>> the time. Now that ADC support has been added to the STX104 IIO driver,
>> the driver should be moved to the more appropriate ADC directory.
>>
>> Signed-off-by: William Breathitt Gray <vilhelm.gray@gmail.com>
>Hi William,
>
>I'd missed the odd use of an output channel to control scale.
>I'm guessing this is because we are looking at actual control of a front
>end separate amplifier on the board? However as far as the driver is
>confirmed this is no different from any multi range ADC and so
>should be done with IIO_INFO_SCALE.
>
>Sorry for taking so long to get back to you. Holiday season I'm afraid.
>
>Jonathan
>> +#define STX104_GAIN_CHAN { \
>> + .type = IIO_VOLTAGE, \
>> + .info_mask_separate = BIT(IIO_CHAN_INFO_HARDWAREGAIN), \
>> + .output = 1 \
>> }
>
>I'm confused. You have a separate output channel to control gain?
>Why... Variable gain devices are normally just represented via the
>info_mask element IIO_CHAN_INFO_SCALE as an info_mask_shared_by_type.
Hey Jonathan,
Thanks for getting back to me. I did have trouble figuring out the
proper way to expose the gain control for this device. Perhaps if I
describe the register interface it may make more sense.
The STX104 ADC input range may be probed with a read to register offset
11; bit 2 represents the ADC bipolar/unipolar configuration (ADBU),
while bits 1 and 0 represent the ADC gain setting. ADBU is configured by
physical jumper on the device, while the ADC gain setting is configured
in software via a register write.
ADC gain may be set with a write of bits 1 and 0 to register offset 11;
00 = gain of x1, 01 = gain of x2, 10 = gain of x4, 11 = gain of x8.
Writing to this register sets the analog input gain for all 8/16 analog
inputs.
Since the ADC gain setting applies to all analog inputs rather than
individual inputs separately, what is the proper way to expose this
option? Should I remove the STX104_GAIN_CHAN #define entirely and use
something like the following change instead:
#define STX104_IN_CHAN(chan, diff) { \
.type = IIO_VOLTAGE, \
.channel = chan, \
.channel2 = chan, \
.info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE), \
.info_mask_separate = BIT(IIO_CHAN_INFO_RAW), \
.indexed = 1, \
.output = 0, \
.differential = diff \
}
Where the IIO_CHAN_INFO_SCALE now exposes the gain setting affecting all
analog inputs.
Thanks,
William Breathitt Gray
next prev parent reply other threads:[~2016-08-22 13:24 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-25 17:53 [PATCH v4 0/1] iio: stx104: Add IIO support for the ADC channels William Breathitt Gray
2016-07-25 17:54 ` [PATCH v4 1/1] " William Breathitt Gray
2016-08-21 19:39 ` Jonathan Cameron
2016-08-22 13:24 ` William Breathitt Gray [this message]
2016-08-23 18:12 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160822132409.GA9041@sophia \
--to=vilhelm.gray@gmail.com \
--cc=jic23@kernel.org \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pmeerw@pmeerw.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).