Re: [PATCH] powerpc: add new required termio functions

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Michael Neuling <mikey@...>
Cc: <akpm@...>, <paulus@...>, Alan Cox <alan@...>, David S. Miller <davem@...>, <linux-kernel@...>, <linuxppc-dev@...>
Date: Tuesday, September 11, 2007 - 10:17 pm

On Wed, 12 Sep 2007, Michael Neuling wrote:

Really?

It shouldn't. The use of kernel_termios_to_user_termios_1() is conditional 
on the architecture having a define for TCGETS2, and I think they match 
up. I see:

	[torvalds@woody linux]$ git grep -l kernel_termios_to_user_termios_1 include | wc -l
	10
	[torvalds@woody linux]$ git grep -l TCGETS2 include | wc -l
	10

and in neither case is ppc in that list of architecures.

So maybe you just read the patch without actually testing whether it 
actually broke powerpc?

Or is something subtler going on?

			Linus
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] powerpc: add new required termio functions, Michael Neuling, (Tue Sep 11, 10:04 pm)
Re: [PATCH] powerpc: add new required termio functions, Heiko Carstens, (Wed Sep 12, 6:20 am)
Re: [PATCH] powerpc: add new required termio functions, Andrew Morton, (Wed Sep 12, 7:01 am)
Re: [PATCH] powerpc: add new required termio functions, Christoph Hellwig, (Wed Sep 12, 7:34 am)
Re: [PATCH] powerpc: add new required termio functions, Heiko Carstens, (Wed Sep 12, 7:52 am)
Re: [PATCH] powerpc: add new required termio functions, Christoph Hellwig, (Wed Sep 12, 9:49 am)
Re: [PATCH] powerpc: add new required termio functions , Michael Neuling, (Wed Sep 12, 7:16 am)
Re: [PATCH] powerpc: add new required termio functions, Geoff Levand, (Tue Sep 11, 10:19 pm)
Re: [PATCH] powerpc: add new required termio functions, Linus Torvalds, (Tue Sep 11, 10:17 pm)
Re: [PATCH] powerpc: add new required termio functions , Michael Neuling, (Tue Sep 11, 10:34 pm)
Re: [PATCH] powerpc: add new required termio functions, Tony Breeds, (Tue Sep 11, 10:33 pm)