lkml.org 
[lkml]   [2010]   [Nov]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] Fix Winbond CIR driver initialisation
    On Sun, Nov 07, 2010 at 03:46:48PM +0000, Alan Cox wrote:
    > > A workaround is to boot with argument 8250.nr_uarts=1. This is not really
    > > desirable, so this patch makes ttyS1 go away.
    >
    > setserial can also be used for this surely ?
    >
    > > + dev_info(device, "Unregistering phony serial port ttyS1\n");
    > > + serial8250_unregister_port(1);
    > > + ok = request_region(data->sbase, SP_IOMEM_LEN, DRVNAME);
    >
    > That's a hack that is only going to work on specific systems where it is
    > mapped the way you expect and doing stuff behind the back of the serial
    > driver.
    >
    > I'm not averse to a better solution but it needs to be general and
    > maintainable. Is there a way to identify the presence of the windbond CIR
    > device as opposed to an 8250 ?

    The past couple of days I've been trying to find a way of identifying the
    winbond cir device correctly in the 8250 driver by its registers, but so
    far I've had no luck. It is a device which can operate in CEIR, UART and
    one the irda modes, but all I can find for identifying what is physically
    connected is querying the Super I/O model or the PNP identifier for CIR.

    The Super I/O model might not be enough in cases where different things
    can be connected to it physically.

    The hardware can be flipped between various IR (SIR/CIR/etc) and normal
    uart modes. We could have an new call which represents this a bit
    more closely.

    To ensure it is mapped as expected, the call should include the I/O
    port range and irq.

    bool serial8250_port_non_uart(int ioport, int ioportlen, int irq);

    The 8250 serial driver would unregister the port which matches this, and
    unregister the corresponding port, returning true if such a port can be
    found.

    This problem here is also one which is encountered by some of the irda
    drivers, so it would be nice to solve as well for those too.

    Would that make more sense?

    Regards,
    Sean


    \
     
     \ /
      Last update: 2010-11-10 17:43    [W:0.040 / U:6.180 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site