On Fri, Dec 16, 2011 at 10:03 PM, Ming Lei <tom.leiming@xxxxxxxxx> wrote: > On Sat, Dec 17, 2011 at 12:09 AM, Shilimkar, Santosh > <santosh.shilimkar@xxxxxx> wrote: >> No. The table reflects the actual possible combination on hardware. You can't >> add anything there arbitrary if that combination is not supported. >> >> But that makes me wonder how the patch worked. I will look at this in detail >> next week. > > Thanks, I use the below hwmod data patch[1] and omap device patch[2] > to bring up fdif module, and Benoit has ACKed on patch [1]. > > > [1], http://marc.info/?l=linux-kernel&m=132387140703874&w=2 > [2], http://marc.info/?l=linux-kernel&m=132387155403929&w=2 > The patches are fine but I am surprised that 'dyndep.patch' is helping you. L3_2 dependency is always enabled with ISS or Camera and it should have worked. clkdm_add_wkdep(iss_clkdm, l3_2_clkdm), shoudn't do anything since the bit is read-only. Can you check value of CM_CAM_STATICDEP [ 0x4A009004], after your patch. I am suspecting that for some reason l3_1 dep. is getting enabled which might be helping your case. Regards Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
