2007-10-17 02:51:15

by Luming Yu

[permalink] [raw]
Subject: [RFC patch] fallback to "no irq" hack for case of no pnp_irq allocated for 8250_pnp

Hello list,

There is a "ttyS1 irq is -1" problem observed on tiger4 which cause
the serial port broken.
It is because that there is __no__ ACPI IRQ resource assigned for the
serial port. So the value of the IRQ for the port is never changed
since it got initialized to -1. The attached patch falls back to the
"no irq" hack for this case. It works for me. Please review and test.

Thanks,
Luming

Signed-off-by: Yu Luming <[email protected]>

8250_pnp.c | 3 +++
1 files changed, 3 insertions(+)


Attachments:
(No filename) (503.00 B)
8250_pnp.patch (451.00 B)
Download all attachments

2007-10-17 04:40:18

by Matthew Wilcox

[permalink] [raw]
Subject: Re: [RFC patch] fallback to "no irq" hack for case of no pnp_irq allocated for 8250_pnp

On Wed, Oct 17, 2007 at 10:50:53AM +0800, Luming Yu wrote:
> There is a "ttyS1 irq is -1" problem observed on tiger4 which cause
> the serial port broken.
> It is because that there is __no__ ACPI IRQ resource assigned for the
> serial port. So the value of the IRQ for the port is never changed
> since it got initialized to -1. The attached patch falls back to the
> "no irq" hack for this case. It works for me. Please review and test.

Seems reasonable to me. Although arguably pnp_irq is broken, since
Linus has decreed that 0 means no irq on Linux.

--
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."

2007-10-17 10:46:09

by Alan

[permalink] [raw]
Subject: Re: [RFC patch] fallback to "no irq" hack for case of no pnp_irq allocated for 8250_pnp

> Signed-off-by: Yu Luming <[email protected]>
>
> 8250_pnp.c | 3 +++
> 1 files changed, 3 insertions(+)

Should really fix pnp_irq() to use zero to mean none as the kernel
intends. If that causes other complications then this hack will do for
now I guess.

Alan