Messages in this thread | | | Date | Wed, 29 Jul 2009 21:27:54 +0900 | Subject | Re: MMC: Make the configuration memory resource optional | From | Magnus Damm <> |
| |
On Wed, Jul 29, 2009 at 8:58 PM, Mark Brown<broonie@opensource.wolfsonmicro.com> wrote: > On Wed, Jul 29, 2009 at 11:48:51AM +0900, Magnus Damm wrote: > >> I understand that you want to have clocklib so you can manage clocks >> dynamically for your mfd drivers, but in our use case we already have >> working clock framework implementations withing our SoC. There is no >> need for any dynamic clock registration and unregistration. With that >> in mind it can't be very difficult to understand that there is no >> point for SoC vendors to work on clocklib. If's mainly an issue for >> mfd. > > While it's true that this doesn't bother SoCs the fact that most clock > API implementations don't allow any off-chip drivers to register clocks > renders the clock API essentially unusable for fairly large parts of the > kernel. This is especially problematic where the clocks cross from > the SoC domain to the off-SoC domain and clocks from each domain may be > used interchangably.
Yeah, clocks outside the SoC are not well supported today. From what I've seen, most embedded boards come with external chips for cameras, audio codecs and/or phy devices. These devices often get their clocks from the main SoC. Allowing the drivers for such chips to use the clock framework to register clocks for internal divisors would allow driver writers to write better code which in turn would make life easier for most people hacking on embedded kernels.
So I'm all for working towards allowing off-chip drivers to register and unregister clocks.
The problem with the clock framework API is that the data structures varies depending on implementation. So the ops callback structure on SuperH is different compared to ARM. I suspect that adding generic clocklib support across the architectures will take quite a bit of time to implement propely.
> Looking at the original patch I'm not sure exactly why it runs into > clock API issues so I'm not sure if this is a relevant concern or not > here but I'm mentioning it since I'd kind of expect an impact on the > SoCs from addressing it due to the way the clock API functions are > currently provided.
In my opinion this patch has nothing to do with the clock framework.
But fixing up clocklib properly would certainly be beneficial for everyone. Holding the driver hostage until clocklib is upstream however, that's just silly.
Cheers,
/ magnus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |