linux-iio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@kernel.org>
To: Alessandro Rubini <rubini@gnudd.com>
Cc: christophe.leroy@c-s.fr, linux-kernel@vger.kernel.org,
	linux-iio@vger.kernel.org, patrick.vasseur@c-s.fr
Subject: Re: [PATCH] IIO ADC support for AD7923
Date: Sat, 12 Jan 2013 10:39:39 +0000	[thread overview]
Message-ID: <50F13D6B.4030703@kernel.org> (raw)
In-Reply-To: <20130108091414.GA28404@mail.gnudd.com>

On 01/08/2013 09:14 AM, Alessandro Rubini wrote:
>> +config AD7923
> 
> I wonder if IIO config symbols should have IIO_ in their name, so
> people looking at config files knows what they actually are.
> Actually, all USB drivers have USB in their config name, which is
> useful even if e.g. "PL2303" cannot be but USB.
> 
> On the other hand, AD7923 and all the others can well be driven by
> something else than IIO, even if currently this is the only mainstream
> option. Here comedi made the right choice, and all their symbols are
> in the CONFIG_COMEDI_ name space.
> 
> Thanks
> /alessandro
I have no particular problem with this for new drivers, but
obviously it is an interface change for older ones that may
cause some issues.  Anyone else care either way?

Various patches to at least kill off missleading config symbol
names (SENSOR_ which is used hwmon) have been kicking around but
I admit I keep putting them on the back burner on the basis there
are better things to be getting on with and it will cause a fair bit
of churn.

  reply	other threads:[~2013-01-12 10:39 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-08  8:42 [PATCH] IIO ADC support for AD7923 Christophe Leroy
2013-01-08  9:14 ` Alessandro Rubini
2013-01-12 10:39   ` Jonathan Cameron [this message]
2013-01-12 17:14   ` Lars-Peter Clausen
2013-01-17 16:32   ` Alessandro Rubini
2013-01-17 17:11   ` Alessandro Rubini
2013-01-17 17:36     ` Lars-Peter Clausen
2013-01-18 23:02       ` Getz, Robin
2013-01-19 12:51         ` Jonathan Cameron
2013-01-17 18:26     ` Alessandro Rubini
2013-01-08 10:27 ` Lars-Peter Clausen
2013-01-19 15:05   ` christophe leroy
2013-01-19 19:59     ` Lars-Peter Clausen

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=50F13D6B.4030703@kernel.org \
    --to=jic23@kernel.org \
    --cc=christophe.leroy@c-s.fr \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=patrick.vasseur@c-s.fr \
    --cc=rubini@gnudd.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 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).