Re: [PATCH 1/2] serial: samsung: Move uart_register_driver call todevice probe

From: Nicolas Pitre
Date: Sun Jan 26 2014 - 23:30:14 EST


On Sun, 26 Jan 2014, Russell King - ARM Linux wrote:

> On Tue, Jan 21, 2014 at 12:45:05AM +0000, Alan Cox wrote:
> > Peter handed it on. Try using git log on Documentation/devices.txt. It
> > still gets updates.
> >
> > Perhaps you'd care to stick to reality and fix the tree instead of trying
> > to excuse the mess ?
>
> Perhaps returning to reality might be advantageous rather than trying
> to repeat statements which can't have any bearing on this - especially
> as the git history which you're referring to only goes back to 2.6.12-rc2,
> and this predates 2.6.12-rc2 by a long shot.
>
> > More importantly certain folks need to stop abusing static numbers
> > allocated properly. Repeating it having made a total hash of it before
> > is dismal.
>
> And if you continue these stupid accusations which have no basis at all,
> we're going to get into a real big argument, because you are soo _wrong_
> on that point. I was always the one arguing /against/ the re-use of
> existing major/minor numbers. I was the one arguing /against/ Nicolas'
> patches to make every serial port appear in the 4,64 ttyS namespace.

If you remember correctly, that was my attempt at making serial port
minor assignment to be dynamic... just like everything else is today.
And it seemed to me that you thought this was a good idea.

But that was objected by the X86 folks who insisted on always having
COM1 == ttyS0 and COM2 == ttyS1, whether or not COM1 was present.

Nowdays serial ports have pretty much disappeared from the X86 world.
But the problem they've created is still with us.

Either everything is dynamic, or everything follows proper minor
assignment process.


Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/