2006-08-17 14:29:52

by Vitaly Wool

[permalink] [raw]
Subject: ip3106_uart oddity

Hello Russell,

it looks like drivers/serial/ip3106_uart.c was dropped from the mainline at some point I couldn't identify. Can you please confirm that?
I'd like to take the burden of restoring the UART functionality for PNX8550 boards in the mainline. This very UART HW is very weird and doesn't fit well into 8250 model, even with fixups like those that were introduced for Alchemy. It also differs from the IP_3106-based UARTs used on Philips ARM targets in registers layout so I'm not sure it's correct to call it ip3106_uart.
So, given the above, does it make sense to try make it fir into standard 8250 driver model or restore/rework the custom driver?

Thanks,
Vitaly


2006-08-17 20:30:07

by Russell King

[permalink] [raw]
Subject: Re: ip3106_uart oddity

On Thu, Aug 17, 2006 at 06:29:48PM +0400, Vitaly Wool wrote:
> it looks like drivers/serial/ip3106_uart.c was dropped from the
> mainline at some point I couldn't identify. Can you please confirm
> that?

I am not aware of its addition nor removal of this file. There was
au1x00_uart.c at one time.

> I'd like to take the burden of restoring the UART functionality for
> PNX8550 boards in the mainline. This very UART HW is very weird and
> doesn't fit well into 8250 model, even with fixups like those that
> were introduced for Alchemy. It also differs from the IP_3106-based
> UARTs used on Philips ARM targets in registers layout so I'm not
> sure it's correct to call it ip3106_uart.
> So, given the above, does it make sense to try make it fir into
> standard 8250 driver model or restore/rework the custom driver?

No real clue. Is it similar to any other drivers?

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core

2006-08-18 07:46:37

by Jean-Paul Saman

[permalink] [raw]
Subject: Re: ip3106_uart oddity

[email protected] wrote on 17-08-2006 22:29:54:

> On Thu, Aug 17, 2006 at 06:29:48PM +0400, Vitaly Wool wrote:
> > it looks like drivers/serial/ip3106_uart.c was dropped from the
> > mainline at some point I couldn't identify. Can you please confirm
> > that?
>
Looks like someone wanted to rename it, but forgot to include the new
file.

> I am not aware of its addition nor removal of this file. There was
> au1x00_uart.c at one time.
>
> > I'd like to take the burden of restoring the UART functionality for
> > PNX8550 boards in the mainline. This very UART HW is very weird and
> > doesn't fit well into 8250 model, even with fixups like those that
> > were introduced for Alchemy. It also differs from the IP_3106-based
> > UARTs used on Philips ARM targets in registers layout so I'm not
> > sure it's correct to call it ip3106_uart.
> > So, given the above, does it make sense to try make it fir into
> > standard 8250 driver model or restore/rework the custom driver?
>
> No real clue. Is it similar to any other drivers?

The ip3106_uart.c file that was used in the PNX8550 boards is wrongly
named. The uart just isn't an ip3106, because those are used in philips
ARM based devices.

If you restore the ip3106T_uart.c, then please rename it pnx8550_uart.c
(or pnx8xxxx_uart.c).

Kind greetings,

Jean-Paul Saman

Philips Semiconductors

2006-08-18 08:28:08

by Russell King

[permalink] [raw]
Subject: Re: ip3106_uart oddity

On Fri, Aug 18, 2006 at 09:46:01AM +0200, Jean-Paul Saman wrote:
> [email protected] wrote on 17-08-2006 22:29:54:
> > On Thu, Aug 17, 2006 at 06:29:48PM +0400, Vitaly Wool wrote:
> > > I'd like to take the burden of restoring the UART functionality for
> > > PNX8550 boards in the mainline. This very UART HW is very weird and
> > > doesn't fit well into 8250 model, even with fixups like those that
> > > were introduced for Alchemy. It also differs from the IP_3106-based
> > > UARTs used on Philips ARM targets in registers layout so I'm not
> > > sure it's correct to call it ip3106_uart.
> > > So, given the above, does it make sense to try make it fir into
> > > standard 8250 driver model or restore/rework the custom driver?
> >
> > No real clue. Is it similar to any other drivers?
>
> The ip3106_uart.c file that was used in the PNX8550 boards is wrongly
> named. The uart just isn't an ip3106, because those are used in philips
> ARM based devices.
>
> If you restore the ip3106T_uart.c, then please rename it pnx8550_uart.c
> (or pnx8xxxx_uart.c).

There are no plans what so ever to "restore" what was never even there.
Searching around, there was a pnx0105 driver submitted but needed some
additional work which was never done.

The same situation seems to apply to this driver. Ralf submitted a
driver called ip3106_uart.c which claimed to be a rewrite of
pnx8550_uart.c. Comments were given at that time, no real feedback
came of that.

I would suggest you read the "Serial driver for the Philips PNX8550
SOC." thread, but it was a private one between Ralf, me and an
embeddedalley.com employee (who allegedly was the author of the driver
and who volunteered to do the work to fix the comments, but seemingly
never actually did anything.)

I suggest you ask embeddedalley.com...

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core

2006-08-18 11:24:12

by Vitaly Wool

[permalink] [raw]
Subject: Re: ip3106_uart oddity

On Fri, 18 Aug 2006 09:27:56 +0100
Russell King <[email protected]> wrote:

> There are no plans what so ever to "restore" what was never even there.
> Searching around, there was a pnx0105 driver submitted but needed some
> additional work which was never done.

pnx0105's uart fits well into 8250 concept, so I don't think it will ever show up.

> The same situation seems to apply to this driver. Ralf submitted a
> driver called ip3106_uart.c which claimed to be a rewrite of
> pnx8550_uart.c. Comments were given at that time, no real feedback
> came of that.

Is it possbile that you include comments in the reply to this message? TIA! :)

Vitaly

2006-08-18 12:56:58

by Russell King

[permalink] [raw]
Subject: Re: ip3106_uart oddity

On Fri, Aug 18, 2006 at 03:23:46PM +0400, Vitaly Wool wrote:
> On Fri, 18 Aug 2006 09:27:56 +0100
> Russell King <[email protected]> wrote:
> > There are no plans what so ever to "restore" what was never even there.
> > Searching around, there was a pnx0105 driver submitted but needed some
> > additional work which was never done.
>
> pnx0105's uart fits well into 8250 concept, so I don't think it will ever show up.
>
> > The same situation seems to apply to this driver. Ralf submitted a
> > driver called ip3106_uart.c which claimed to be a rewrite of
> > pnx8550_uart.c. Comments were given at that time, no real feedback
> > came of that.
>
> Is it possbile that you include comments in the reply to this message? TIA! :)

Attached.

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core


Attachments:
(No filename) (870.00 B)
(No filename) (4.62 kB)
Download all attachments