On Thu, Jan 27, 2011 at 11:16 PM, Kevin Hilman <khilman@xxxxxx> wrote:
> Sumit Semwal <sumit.semwal@xxxxxx> writes:
>
>> DSS code uses ick as one of the clocks in clk_get/clk_put. OMAP4 clock database
>> doesn't have ick for DSS, so adding ick as dummy clock.
>>
>> Once pm_runtime* APIs get introduced in DSS, this will be revisited.
>>
>> Signed-off-by: Sumit Semwal <sumit.semwal@xxxxxx>
>> ---
>> arch/arm/mach-omap2/clock44xx_data.c | 2 ++
>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c
>> index 74db324..8603b1f 100644
>> --- a/arch/arm/mach-omap2/clock44xx_data.c
>> +++ b/arch/arm/mach-omap2/clock44xx_data.c
>> @@ -3109,6 +3109,8 @@ static struct omap_clk omap44xx_clks[] = {
>> CLK(NULL, "dsp_fck", &dsp_fck, CK_443X),
>> /* dss clocks */
>> CLK(NULL, "fck", &dss_fck, CK_443X),
>> + CLK("omap_dss", "ick", &dummy_ck, CK_443X),
>> + /* on OMAP4, ick is a dummy clock */
>
> minor nit: the comment should go above the code, and should also state
> it's needed for OMAP2/3 compatibility.
Sure, thanks! Will update in next version.
~Sumit.
>
> Kevin
>
>> /*
>> * clocks handled via hwmod opt_clk mechanism need dev=NULL,
>> * con=clock name as per actual clk structure, NOT role
>
--
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
