From: Lars-Peter Clausen <lars@metafoo.de>
To: Jonathan Cameron <jic23@cam.ac.uk>
Cc: linux-iio@vger.kernel.org, michael.hennerich@analog.com,
device-drivers-devel@blackfin.uclinux.org
Subject: Re: [PATCH] staging:iio:treewide only use shared to decide on interfaces
Date: Mon, 24 Oct 2011 10:58:30 +0200 [thread overview]
Message-ID: <4EA528B6.1030300@metafoo.de> (raw)
In-Reply-To: <4EA522D3.9000906@cam.ac.uk>
On 10/24/2011 10:33 AM, Jonathan Cameron wrote:
> On 10/24/11 09:30, Lars-Peter Clausen wrote:
>> On 10/21/2011 05:39 PM, Jonathan Cameron wrote:
>>> Internally the fact that say scale is shared across channels is
>>> actually of remarkably little interest. Hence lets not store it.
>>> Numerous devices have weird combinations of channels sharing
>>> scale anyway so it is not as though this was really telling
>>> us much. Note however that we do still use the shared sysfs
>>> attrs thus massively reducing the number of attrs in complex
>>> drivers.
>>>
>>> Side effect is that certain drivers that were abusing this
>>> (mostly my work) needed to do a few more checks on what the
>>> channel they are being queried on actually is.
>>>
>>> This is also helpful for in kernel interfaces where we
>>> just want to query the scale and don't care whether it
>>> is shared with other channels or not.
>>>
>>> Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
>>
>> Hi
>>
>> Seems to work. I think it makes sense to submit this one and the other,
>> which removes the bitmask addressing, together. Will you take care of this?
> Yes. Might take a few days to hit though as I would guess Greg is going
> to be busy at the kernel summit.
>
> Are you happy to Ack this patch or if not give a tested-by.
> Whilst I can push stuff on without review to Greg, I always prefer not to!
>
> Jonathan
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
prev parent reply other threads:[~2011-10-24 8:58 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-21 15:39 [PATCH] IIO: Why maintain separate and shared knowledge internally? Jonathan Cameron
2011-10-21 15:39 ` [PATCH] staging:iio:treewide only use shared to decide on interfaces Jonathan Cameron
2011-10-24 8:30 ` Lars-Peter Clausen
2011-10-24 8:33 ` Jonathan Cameron
2011-10-24 8:58 ` Lars-Peter Clausen [this message]
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=4EA528B6.1030300@metafoo.de \
--to=lars@metafoo.de \
--cc=device-drivers-devel@blackfin.uclinux.org \
--cc=jic23@cam.ac.uk \
--cc=linux-iio@vger.kernel.org \
--cc=michael.hennerich@analog.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.