2010/7/30 Kevin Wells [off-list ref]:

quoted

- ? ? id = amba_lookup(pcdrv->id_table, pcdev);
+ ? ? do {
+ ? ? ? ? ? ? ret = amba_get_enable_pclk(pcdev);
+ ? ? ? ? ? ? if (ret)
+ ? ? ? ? ? ? ? ? ? ? break;
+
+ ? ? ? ? ? ? ret = pcdrv->probe(pcdev, id);
+ ? ? ? ? ? ? if (ret == 0)
+ ? ? ? ? ? ? ? ? ? ? break;

- ? ? return pcdrv->probe(pcdev, id);
+ ? ? ? ? ? ? amba_put_disable_pclk(pcdev);

Should the AMBA clock be disabled if the probe() fails?

Yes. This is the pclk for this one PrimeCell. It is first turned on,
then the driver attempts to probe and in case that fails there
obviously is no driver for this piece of silicon so why should it
be clocked?

Yours,
Linus Walleij