From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Samuel Thibault <samuel.thibault@ens-lyon.org>,
okash.khawaja@gmail.com, Jiri Slaby <jslaby@suse.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
devel@driverdev.osuosl.org, Kirk Reiser <kirk@reisers.ca>,
speakup@linux-speakup.org, Chris Brannon <chris@the-brannons.com>
Subject: Re: [patch 1/2] staging: speakup: add function to convert dev name to number
Date: Wed, 14 Jun 2017 15:04:18 +0200 [thread overview]
Message-ID: <20170614130418.GA1184@kroah.com> (raw)
In-Reply-To: <CAHp75VfxxZKPboXwP49F_mJAYDK+fVWKpgZXfj-x+YmZ1wqMfg@mail.gmail.com>
On Wed, Jun 14, 2017 at 03:36:30PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 14, 2017 at 3:18 PM, Samuel Thibault
> <samuel.thibault@ens-lyon.org> wrote:
> > Greg KH, on mer. 14 juin 2017 13:48:02 +0200, wrote:
> >> On Wed, Jun 14, 2017 at 01:26:09PM +0200, Samuel Thibault wrote:
> >> > Greg KH, on mer. 14 juin 2017 12:15:41 +0200, wrote:
>
> >> And trying to have the kernel do the mapping based on strings like this
> >> is not ok, sorry.
> >
> > So what is the solution? Users would find it completely crazy to have
> > to provide major/minor numbers.
>
> Shouldn't udev take care of major/minor and alike stuff in user space?
No, that's all handled by the kernel now, in devtmpfs.
The console stuff is odd though, but that is each driver defining the
name for itself, major/minor does not apply. Also, a separate driver is
not having to figure this all out. I wonder if we could just add a tty
core function for this as it does know the name of everything that has
been registered with it, right?
thanks,
greg k-h
next prev parent reply other threads:[~2017-06-14 13:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-13 22:37 [patch 0/2] staging: speakup: support more than ttyS* okash.khawaja
2017-06-13 22:37 ` [patch 1/2] staging: speakup: add function to convert dev name to number okash.khawaja
2017-06-14 8:23 ` Dan Carpenter
2017-06-15 8:17 ` Okash Khawaja
2017-06-14 10:13 ` Greg Kroah-Hartman
2017-06-14 10:15 ` Greg Kroah-Hartman
2017-06-14 11:26 ` Samuel Thibault
2017-06-14 11:48 ` Greg Kroah-Hartman
2017-06-14 12:18 ` Samuel Thibault
2017-06-14 12:36 ` Andy Shevchenko
2017-06-14 12:46 ` Samuel Thibault
2017-06-14 13:04 ` Greg Kroah-Hartman [this message]
2017-06-15 6:52 ` Okash Khawaja
2017-06-17 10:16 ` Okash Khawaja
2017-06-17 10:25 ` Greg Kroah-Hartman
2017-06-17 10:57 ` Okash Khawaja
2017-06-13 22:37 ` [patch 2/2] staging: speakup: make ttyio synths use device name okash.khawaja
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=20170614130418.GA1184@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=andy.shevchenko@gmail.com \
--cc=chris@the-brannons.com \
--cc=devel@driverdev.osuosl.org \
--cc=jslaby@suse.com \
--cc=kirk@reisers.ca \
--cc=linux-kernel@vger.kernel.org \
--cc=okash.khawaja@gmail.com \
--cc=samuel.thibault@ens-lyon.org \
--cc=speakup@linux-speakup.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;
as well as URLs for NNTP newsgroup(s).