lkml.org 
[lkml]   [2008]   [Apr]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 5/6] Clocklib: Use correct clock for IrDA on pxa
    On Tue, Apr 08, 2008 at 01:47:35PM +0400, Dmitry wrote:
    > Yes, it works currently. But there are a few problems: we declare
    > STUART's UARTCLK with dev=NULL (all other UARTCLKs are declared with
    > proper devices). Therefore, I consider it as a hack and would like
    > to remove it.

    I don't consider it a hack at all - it's a work around for the fact
    that the PXA FIR driver shares the UART, but the FIR driver doesn't
    bind to the UART itself.

    The _real_ issue is with IrDA itself, and is larger than just the
    clock library. Any serial port which supports IrDA, even on x86,
    has to be shared between the serial driver and the IrDA driver -
    there's no way for them to quietly co-exist and "just work" as
    requested.

    So, let's not work around the short comings of Serial/IrDA interactions
    by adding additional complexity to random other layers which _shouldn't_
    even be seeing the issue.

    In addition, the point of the clock framework is that you ask for the
    device plus clock NAME on _that_ device. Inventing random other names
    for the same physical clock on the same physical device is just nonsense -
    even more so than the existing workaround.

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


    \
     
     \ /
      Last update: 2008-04-08 21:37    [W:0.021 / U:61.840 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site