From: Lars-Peter Clausen <lars@metafoo.de>
To: Roland Stigge <stigge@antcom.de>
Cc: Jonathan Cameron <jic23@kernel.org>, linux-iio@vger.kernel.org
Subject: Re: [PATCH 2/3] staging:iio:dac:max517: Convert to channel spec
Date: Sun, 13 May 2012 17:03:57 +0200 [thread overview]
Message-ID: <4FAFCD5D.2090805@metafoo.de> (raw)
In-Reply-To: <4FAF81E5.8070700@antcom.de>
On 05/13/2012 11:41 AM, Roland Stigge wrote:
> On 13/05/12 11:20, Lars-Peter Clausen wrote:
> [regarding the "1&2" output]
>> You can only set the same value on both
>> outputs simultaneously, which I think has not so many practical use-cases.
>
> _If_ you want to switch outputs "simultaneously", it's hard to do with
> the max51x chips due to relatively slow I2C communication. Don't have
> timing diagrams right now how close you get if you switch the two
> outputs directly one after another, though.
Hm, I thought the whole purpose of the "1&2" attribute was to update them
simultaneously. According to the datasheet the chip will latch new values on a
stop condition. So if you write both channels in one transfer they will be
updated simultaneously. If you don't care about that you might as well just
update the channels by writing to their respective attribute.
>
> But right, it's not highest priority...
>
>> You'd rather want to set two different values on output 1 and 2.
>
> We have and will have attributes for those anyway, don't we?
>
Yes, sure. But again I meant simultaneously, which will be possible with buffer
output support.
- Lars
next prev parent reply other threads:[~2012-05-13 15:02 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-11 17:04 [PATCH 1/3] staging:iio:dac: Remove unused dac.h includes Lars-Peter Clausen
2012-05-11 17:04 ` [PATCH 2/3] staging:iio:dac:max517: Convert to channel spec Lars-Peter Clausen
2012-05-12 18:51 ` Jonathan Cameron
2012-05-12 21:44 ` Roland Stigge
2012-05-13 7:25 ` Lars-Peter Clausen
2012-05-13 9:06 ` Roland Stigge
2012-05-13 9:20 ` Lars-Peter Clausen
2012-05-13 9:41 ` Roland Stigge
2012-05-13 15:03 ` Lars-Peter Clausen [this message]
2012-05-14 14:31 ` Lars-Peter Clausen
2012-05-14 14:39 ` Jonathan Cameron
2012-05-11 17:04 ` [PATCH 3/3] staging:iio:dac: Remove dac.h Lars-Peter Clausen
2012-05-12 18:52 ` Jonathan Cameron
2012-05-12 18:20 ` [PATCH 1/3] staging:iio:dac: Remove unused dac.h includes 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=4FAFCD5D.2090805@metafoo.de \
--to=lars@metafoo.de \
--cc=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=stigge@antcom.de \
/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).