Arch patches are not usually merged in Hans tree. Murali Karicheri Software Design Engineer Texas Instruments Inc. Germantown, MD 20874 phone: 301-407-9583 email: m-kariche...@ti.com
>-----Original Message----- >From: Kevin Hilman [mailto:khil...@deeprootsystems.com] >Sent: Thursday, January 07, 2010 4:50 PM >To: Karicheri, Muralidharan >Cc: linux-media@vger.kernel.org; hverk...@xs4all.nl; davinci-linux-open- >sou...@linux.davincidsp.com >Subject: Re: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting ccdc >drivers to platform driver > >"Karicheri, Muralidharan" <m-kariche...@ti.com> 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:khil...@deeprootsystems.com] >>>Sent: Thursday, January 07, 2010 2:44 PM >>>To: Karicheri, Muralidharan >>>Cc: linux-media@vger.kernel.org; hverk...@xs4all.nl; davinci-linux-open- >>>sou...@linux.davincidsp.com >>>Subject: Re: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting ccdc >>>drivers to platform driver >>> >>>"Karicheri, Muralidharan" <m-kariche...@ti.com> 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
