Messages in this thread | | | Date | Tue, 8 Apr 2008 20:33:09 +0100 | From | Russell King <> | Subject | Re: [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:
| |