Hi, Guennadi Sorry to answer the question so later, From: Guennadi Liakhovetski Sent: Thursday, May 12, 2011 5:32 PM > On Thu, 12 May 2011, Wu, Josh wrote: >> Hi, Russell >> >> From: Russell King - ARM Linux [mailto:linux@xxxxxxxxxxxxxxxx] Sent: Thursday, May 12, 2011 3:47 PM >> > On Thu, May 12, 2011 at 03:42:18PM +0800, Josh Wu wrote: >> >> +err_alloc_isi: >> >> + clk_disable(pclk); >> > clk_put() ? >> Ok, will be fixed in V2 patch. Thanks. > You might wait with v2 until I find time to review your patch. Will take a > couple of days, I think. A general question, though: how compatible is > this driver with the AVR32? Now from the information I got from AVR team, The AVR32 is not compatible with ISI driver patch v2. Since AVR32 use the previous version of ISI IP core. (AT91SAM9M10 using ISI_V2 IP, which has different hardware registers from V1 IP core, such like dma control, etc) But I think in future we will add ISI V1 IP support since AT91SAM9263 also using ISI V1 IP core. Best Regards, Josh Wu -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html