Re: [PATCH 7/7] serial/8250: make PIO support optional

[Alan Cox]

> Where I failed so far is the dynamic configuration of ports using
> setserial. This intentionally allows changing the io_type setting
> as well as the actual resources (ioport, mapbase, irq, ...).
> Changing the io_type is not supported by /bin/setserial, but
> other tools might be doing it.
> My question is whether we should still care about those. If
> we can remove the reconfiguration of existing ports or move
> it to one of the more obscure parts of the driver, it's possible
> to confine the dependencies on the ioport_ops to the front-end
> drivers, while the core 8250 library driver would not need it
> any more.

otherwise you need a table that drivers register their port types
in and to take module references on the table entry to pin the relevant
driver code ?
 
> Today, most ports don't set the UPF_FIXED flag, even though
> the ports definitely have fixed resources, e.g. all of the
> 8250-platform drivers in arch/ or the 8250_pnp and 8250_cs
> front-ends. Do you think it would be reasonable to mark all
> 8250 ports except the ISA ones as UPF_FIXED, and move the
> reconfiguration logic into the 8250_isa driver along with
> old_serial_port, serial8250_isa_devs,  and
> serial8250_isa_init_ports?

I suspect one or two people will scream about some peculiar configuration
that should be handled automatically anyway and those are best fixed
properly if so rather than by allowing setserial incantations to work
around stuff like unknown PCI idents

--
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/