Re: rfc about an uncommented string

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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?

> 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@xxxxxxxxxxxxxx

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


[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux