Kevin, Can I remove it through a separate patch? This patch is already merged in Hans tree. Murali Karicheri Software Design Engineer Texas Instruments Inc. Germantown, MD 20874 phone: 301-407-9583 email: m-karicheri2@xxxxxx >-----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
