Re: rfc about an uncommented string

[Fuxin Zhang <fxzhang@ict.ac.cn>]

Geert Uytterhoeven wrote:
> On Mon, 10 Oct 2005, Fuxin Zhang wrote:
> 
>>We are using 8172 for 2.4 kernels presently,although may drop it
>>sometime later.
>>
>>I don't read the 2.6 code,but it seems it remains the same as the copy
>>in 2.4(except that #ifdef changes).I can't see why the code is broken?
>>In case you mean the ioport address,mips_io_port_base for that board is
>>0xa0000000, inb(0x14000060) is reading from 0xb4000060, which is correct
>>for it.
> 
> 
> Shouldn't mips_io_port_base be 0xb4000000 for your board, so inb(0x60) looks
> more like a PC-style keyboard controller access?
It is not my code. But anyway the board has more than one discontinuous
io ranges(0xb4000000 is in fact for it8712 superio and legacy ios,
it8172's system registers are located around 0xb8000000, while others
begins at 0xb4010000).
> 
> 
>>Ralf Baechle wrote:
>>
>>>On Sun, Oct 09, 2005 at 03:41:06PM +0200, Carlo Perassi wrote:
>>>
>>>>As suggested (*) by Arthur Othieno on the kernel-janitors mailing list,
>>>>I bounce here this email for collecting comments.
>>>>The old email refers to 2.6.13-rc6 but the code is still the same on
>>>>2.6.14-rc3.
>>>>Thank you.
>>>>
>>>>Hi.
>>>>
>>>>I'd like to collect some comments about the following code
>>>>segment I found in
>>>>linux-2.6.13-rc6/arch/mips/ite-boards/generic/it8172_setup.c
>>>>(the "^^^" sequence is not mine, it's in the code)
>>>
>>>
>>>I know, I put it there.  The code was obviously broken, so I place this
>>>hard to miss not right into the middle of it.  It's there since ages and
>>>nobody did complain.  So unless somebody complains - and that complaint
>>>better include some patches - I will delete support for the IT8172 and
>>>it's eval board.
> 
> 
> Gr{oetje,eeting}s,
> 
> 						Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> 							    -- Linus Torvalds
> 
> 
>