linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: dwmw2@infradead.org
Cc: linuxppc-dev@ozlabs.org, paulus@samba.org,
	linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk
Subject: Re: [PATCH] Stop pmac_zilog from abusing 8250's device numbers.
Date: Wed, 04 Apr 2007 01:12:08 -0700 (PDT)	[thread overview]
Message-ID: <20070404.011208.120446173.davem@davemloft.net> (raw)
In-Reply-To: <1175666635.2932.17.camel@shinybook.infradead.org>

From: David Woodhouse <dwmw2@infradead.org>
Date: Wed, 04 Apr 2007 02:03:54 -0400

> On Wed, 2007-04-04 at 15:53 +1000, Paul Mackerras wrote:
> > It 'should' be the case because that is what is easiest for users and
> > makes most sense from a user's point of view.
> 
> I really don't buy that argument.

I do.

David, do this.  Fly to Portland, drive over to Linus's house, once he
lets you in, tell him to walk up to his computer and run 'cat' on the
primary serial port.  Do all of this before he reads this thread. :-)

What device name do you think he will type in?

Will he type a different device name on his PowerPC box than he will
on his x86 systems?

I think he will type /dev/ttyS0 in both cases, and it won't be because
he knows what kind of serial port controller is in either machine.

It's that simple.

Your ARM example holds zero water because that platform has all kinds
of weird devices so people there are used to all kinds of non-standard
conventions and naming.

Try desktop systems that lots of users actually use.  Ask them what
the serial device file is.  Ask them if they had to know what
serial controller they have in their computer in order to figure
that out.

> That's what you get when you abuse device numbers belonging to
> another driver.

No, it's what you get whan drivers of the same major number do
not cooperate with minor number allocations.

Add a notion of onboard serial controllers and a properly coordinated
minor number allocation system for /dev/ttyS0 if you want to solve this.

As it stands your patch is a non-starter because it breaks a user
visible API, and people's machines won't bootup or have a console
any longer.  Or, if the powerpc machine is at Alan's house his
model trains might stop working :-)

  parent reply	other threads:[~2007-04-04  8:12 UTC|newest]

Thread overview: 99+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-03 14:25 [PATCH] Stop pmac_zilog from abusing 8250's device numbers David Woodhouse
2007-04-03 20:37 ` Brad Boyer
2007-04-03 22:58   ` David Woodhouse
2007-04-03 23:14 ` Paul Mackerras
2007-04-03 23:28   ` David Woodhouse
2007-04-03 21:29     ` Brad Boyer
2007-04-03 23:57       ` David Woodhouse
     [not found]         ` <200704032210 02.GA13210@cynthia.pants.nu>
2007-04-03 22:10         ` Brad Boyer
2007-04-04  0:54           ` David Woodhouse
2007-04-03 23:09             ` Brad Boyer
2007-04-04  1:38               ` Alan Cox
2007-04-04  2:10               ` Valdis.Kletnieks
2007-04-04  1:57                 ` David Lang
2007-04-04  8:51               ` Russell King
2007-04-04  9:48                 ` Geert Uytterhoeven
2007-04-04 16:09                   ` H. Peter Anvin
2007-04-04 16:22               ` Linas Vepstas
2007-04-04 16:34                 ` David Woodhouse
2007-04-04 17:38                   ` Linas Vepstas
2007-04-04  1:16             ` David Miller
2007-04-04  1:28               ` Alan Cox
2007-04-04  1:07                 ` David Lang
2007-04-04  1:44                   ` Alan Cox
2007-04-04  1:23                     ` David Lang
2007-04-04  1:32                       ` David Lang
2007-04-04 16:07                       ` H. Peter Anvin
2007-04-04 17:50                       ` Linas Vepstas
2007-04-04  1:50                     ` David Miller
2007-04-04 11:14                       ` Alan Cox
2007-04-04 12:20                         ` Olivier Galibert
2007-04-04  1:48                   ` David Miller
2007-04-04  3:07                 ` Gerhard Mack
2007-04-11 16:30                   ` Benny Amorsen
2007-04-12 15:34                     ` Roland Dreier
2007-04-13  2:45                       ` Gerhard Mack
2007-04-13  2:24                         ` David Lang
2007-04-13  8:09                           ` Geert Uytterhoeven
2007-04-04  0:02         ` David Miller
2007-04-04  0:12           ` David Woodhouse
2007-04-03 22:14             ` Brad Boyer
2007-04-04  1:13             ` Benjamin Herrenschmidt
2007-04-04  1:17               ` David Miller
2007-04-04  1:32                 ` Benjamin Herrenschmidt
2007-04-04  0:01       ` [PATCH v2] " David Woodhouse
2007-04-04  1:25       ` [PATCH] " Alan Cox
2007-04-04  4:20         ` Paul Mackerras
2007-04-04  5:19           ` David Woodhouse
2007-04-04  5:50             ` David Miller
2007-04-04  7:04               ` Geert Uytterhoeven
2007-04-04 13:43                 ` Maciej W. Rozycki
2007-04-04  7:52               ` Russell King
2007-04-04  8:41                 ` David Miller
2007-04-04  5:53             ` Paul Mackerras
2007-04-04  6:03               ` David Woodhouse
2007-04-04  6:31                 ` Paul Mackerras
2007-04-04 14:19                   ` [PATCH v3] Stop pmac_zilog from abusing 8250's device numbers; optionally David Woodhouse
2007-04-04 23:48                     ` Paul Mackerras
2007-04-05 12:45                       ` David Woodhouse
2007-04-05 22:53                         ` Paul Mackerras
2007-04-05 22:59                           ` David Woodhouse
2007-04-05 14:31                     ` Atsushi Nemoto
2007-04-05 14:47                       ` David Woodhouse
2007-04-05 15:09                         ` Atsushi Nemoto
2007-04-05 15:18                           ` David Woodhouse
2007-04-05 15:49                             ` Atsushi Nemoto
2007-04-05 15:56                         ` Geert Uytterhoeven
2007-04-04  8:12                 ` David Miller [this message]
2007-04-04  8:38                   ` [PATCH] Stop pmac_zilog from abusing 8250's device numbers Russell King
2007-04-04  8:43                     ` David Miller
2007-04-04  9:00                       ` Russell King
2007-04-04 19:58                         ` David Miller
2007-04-04 12:21                     ` Theodore Tso
2007-04-04 16:10                       ` Russell King
2007-04-04 16:16                       ` H. Peter Anvin
2007-04-04 16:25                         ` Alan Cox
2007-04-04 17:41                         ` Theodore Tso
2007-04-04 18:15                           ` Russell King
2007-04-04 19:50                             ` David Woodhouse
2007-04-05  6:05                               ` Paul Mackerras
2007-04-05  8:50                                 ` Russell King
2007-04-05 16:15                             ` Olivier Galibert
2007-04-05 21:41                 ` Pavel Machek
2007-04-03 23:56     ` David Miller
2007-04-04  1:13       ` Benjamin Herrenschmidt
2007-04-04  6:58     ` Paul Mackerras
2007-04-04 14:16       ` David Woodhouse
2007-04-03 23:55   ` David Miller
2007-04-04  1:19     ` Alan Cox
2007-04-04  1:21       ` David Miller
2007-04-04  1:36         ` Alan Cox
2007-04-04  1:47           ` David Miller
2007-04-04  7:47         ` Russell King
2007-04-04  9:55           ` Paul Mackerras
2007-04-04 16:06             ` Russell King
2007-04-04  6:58     ` Geert Uytterhoeven
2007-08-14 11:49   ` Olaf Hering
2007-08-14 11:53     ` David Woodhouse
2007-08-14 16:24       ` Olaf Hering
2007-04-04  7:55 ` Olaf Hering

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=20070404.011208.120446173.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.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).