From: Paul Mundt <lethal@linux-sh.org>
To: Joakim Tjernlund <joakim.tjernlund@transmode.se>
Cc: Ben Nizette <bn@niasdigital.com>,
"Hans J. Koch" <hjk@linutronix.de>,
gregkh@suse.de, Linux-Kernel <linux-kernel@vger.kernel.org>
Subject: Re: UIO device name
Date: Thu, 25 Sep 2008 20:53:38 +0900 [thread overview]
Message-ID: <20080925115337.GA9334@linux-sh.org> (raw)
In-Reply-To: <1222342868.12624.304.camel@gentoo-jocke.transmode.se>
On Thu, Sep 25, 2008 at 01:41:08PM +0200, Joakim Tjernlund wrote:
> On Thu, 2008-09-25 at 20:48 +1000, Ben Nizette wrote:
> > UIO is an interface type, not a bus type. UIO isn't a subsystem as
> > such, it's a user interface. If the interface is consistent (even if
> > the backing device is different) I don't see the problem with consistent
> > naming.
>
> Do you see a problem with letting the protocol driver choose another
> one? Why not offer the user the chance to let the name mean something?
>
This thread is still going? Amazing. Anyways, your protocol driver
argument doesn't make any sense. Take the case of uio_pdrv or the genirq
variant. This is the name it hands off to the core, while the devices
that register underneath it all have their own names set. Go grep for all
instances of uio_pdrv platform data in the architecture code. I'm getting
the impression you haven't actually even bothered to look at the name
entries.
Breaking the uio%d stuff is unacceptable, and if you need explanations
for why, then you really shouldn't be touching subsystem code in the
first place. Additionally, so far you haven't been able to show a single
example of something you can't already do with the information in
userspace today. If you want more descriptive names, use udev or symlinks
you kick off from the script.
next prev parent reply other threads:[~2008-09-25 11:54 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-24 9:21 UIO device name Joakim Tjernlund
2008-09-24 9:35 ` Paul Mundt
2008-09-24 9:57 ` Joakim Tjernlund
2008-09-24 10:14 ` Paul Mundt
2008-09-24 10:22 ` Hans J. Koch
2008-09-24 11:33 ` Joakim Tjernlund
2008-09-24 11:47 ` Paul Mundt
2008-09-24 12:38 ` Joakim Tjernlund
2008-09-24 22:57 ` Ben Nizette
2008-09-25 10:05 ` Joakim Tjernlund
2008-09-25 10:48 ` Ben Nizette
2008-09-25 11:41 ` Joakim Tjernlund
2008-09-25 11:53 ` Paul Mundt [this message]
2008-09-25 12:36 ` Joakim Tjernlund
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=20080925115337.GA9334@linux-sh.org \
--to=lethal@linux-sh.org \
--cc=bn@niasdigital.com \
--cc=gregkh@suse.de \
--cc=hjk@linutronix.de \
--cc=joakim.tjernlund@transmode.se \
--cc=linux-kernel@vger.kernel.org \
/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