"Karicheri, Muralidharan" <m-karicheri2@xxxxxx> writes: > Can I remove it through a separate patch? This patch is already merged in Hans tree. Hmm, arch patches should not be merged yet as I have not ack'd them. Kevin >>-----Original Message----- >>From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx] >>Sent: Thursday, January 07, 2010 2:44 PM >>To: Karicheri, Muralidharan >>Cc: linux-media@xxxxxxxxxxxxxxx; hverkuil@xxxxxxxxx; davinci-linux-open- >>source@xxxxxxxxxxxxxxxxxxxx >>Subject: Re: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting ccdc >>drivers to platform driver >> >>"Karicheri, Muralidharan" <m-karicheri2@xxxxxx> writes: >> >>> Kevin, >>> >>>> >>>>OK, I'm not extremely familar with the whole video architecture here, >>>>but are all of these drivers expected to be doing clk_get() and >>>>clk_enable()? >>>> >>> >>> [MK]Many IPs on DaVinci VPFE would require vpss master clock. So >>> it is better to do the way I have done in my patch. So it is expected >>> that clk_get, clk_enable etc are called from other drivers as well. >> >>OK, then you are expecting to add clkdev nodes for the other devices >>as well. That's ok. >> >>However, you still haven't answered my original question. AFAICT, >>there are no users of the clkdev nodes "vpss_master" and "vpss_slave". >>Why not remove those and replace them with your new nodes instead of >>leaving them and adding new ones? >> >>Kevin -- 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