Re: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting ccdc drivers to platform driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



"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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux