Re: Trying to fix ITE-887x parallel/serial driver bugs (including unhandled IRQs)

[Joseph Krahn]

On Mon, Apr 12, 2010 at 5:05 PM, Joseph Krahn <joseph.krahn@xxxxxxxxx> wrote:
> On Mon, Apr 12, 2010 at 12:43 PM, Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:
...
>> I don't think your description is accurate entirely. The device is picked
>> up by PCi scans and the INTCBAR is then set up by Linux having checked
>> for free address ranges. It's hardly 'random probing' and it isn't a
>> normal BAR or guaranteed to have been configured by anything beforehand.
> Yes, but the drivers don't use INTCBAR. They scan through a herd-coded
> list of possible port ranges:
>
> static int __devinit sio_ite_8872_probe(struct pci_dev *pdev, int autoirq,
>                                         int autodma,
>                                         const struct parport_pc_via_data *via)
> {
>        short inta_addr[6] = { 0x2A0, 0x2C0, 0x220, 0x240, 0x1E0 };
> ...
>        for (i = 0; i < 5; i++) {
>                base_res = request_region(inta_addr[i], 32, "it887x");
>                if (base_res) {
> ...
Oops. I meant that INTCBAR should just be mapped to PCI BAR 0, which
has already been mapped to an available range. The problem with the
recent 8250_pci driver is that it repeats the above process,
requesting an I/O range while the existing range was never released.
This is how I think it should be done:

        /* Map INTC control to BAR 0 */
        intc_base = pci_resource_start(pdev,0);
        intc_res = request_region(intc_base, ITE_887x_POSIO0_IOSIZE,
"it887x_intc");

...
> One possibility is to install custom io_serial_in/out functions. When
> the value of IIR is requested, the INTC can be checked, and modify the
> result to what it should be for a normal UART. That is a bit of a
> hack, but makes it much easier to fit into the existing code.
I am testing a custom io_serial_in() to translate IRQ info. It looks
like a good solution. Based on the current parport_pc driver, the
partport I/O may work like a standard port, and not need any extra IRQ
code.

Instead of adding another UPIO_ definition, it should be possible to
define serial_in and serial_out in the 8250_pci.c quirks init
function. In some cases, but not all, the result from
set_io_from_upio() is allowed to be overridden, so this may be the
intended design. It appears that UPIO_TSI exists only to map
serial_in/out functions.

Thanks,
Joe
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/