[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250302111858.343dd4d2@jic23-huawei>
Date: Sun, 2 Mar 2025 11:18:58 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Marcelo Schmitt <marcelo.schmitt1@...il.com>
Cc: Bagas Sanjaya <bagasdotme@...il.com>, Marcelo Schmitt
<marcelo.schmitt@...log.com>, linux-iio@...r.kernel.org,
linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org, lars@...afoo.de,
corbet@....net, dlechner@...libre.com
Subject: Re: [PATCH v4 1/1] Documentation: iio: Add ADC documentation
On Sat, 22 Feb 2025 13:43:31 -0300
Marcelo Schmitt <marcelo.schmitt1@...il.com> wrote:
> On 02/20, Bagas Sanjaya wrote:
> > On Wed, Feb 19, 2025 at 06:00:56PM -0300, Marcelo Schmitt wrote:
> > > +For a **single-ended bipolar** channel, the analog voltage input can go from
> > > +-VREF to +VREF (where -VREF is the voltage reference that has the lower
> > > +electrical potential while +VREF is the reference with the higher one). Some ADC
> > > +chips derive the lower reference from +VREF, others get it from a separate
> > "... obtain it ..."
> Ack. Maybe, if there are no additional suggestions to the docs, Jonathan can
> tweak this while applying the patch? Otherwise I'll change accordingly on v5.
That one is marginal. Formal vs less formal language with slightly different
emphasis. I think either works here.
>
> > > +input. Often, +VREF and -VREF are symmetric but they don't need to be so. When
> > > +-VREF is lower than system ground, these inputs are also called single-ended
> > > +true bipolar. Also, while there is a relevant difference between bipolar and
> > > +true bipolar from the electrical perspective, IIO makes no explicit distinction
> > > +between them.
> > > +
> > > <snipped>...
> > > +In the ADC driver, ``differential = 1`` is set into ``struct iio_chan_spec`` for
> > > +the channel. Even though, there are three general input types, ``differential``
> > > +is only used to distinguish between differential and not differential (either
> > "... non-differential ..."
> Ack. Same as the other suggestion.
I agree with this change - tweaked and patch applied.
Thanks,
Jonathan
>
> > > +single-ended or pseudo-differential) input types. See
> > > +``include/linux/iio/iio.h`` for more information.
> >
> > Thanks.
> >
> > --
> > An old man doll... just what I always wanted! - Clara
>
>
> Thanks,
>
> Marcelo
Powered by blists - more mailing lists