From mboxrd@z Thu Jan  1 00:00:00 1970
From: Pavan Kondeti <pkondeti@codeaurora.org>
Subject: Re: [PATCH V4 2/5] USB: OTG: msm: vote for dayatona fabric clock
Date: Tue, 03 May 2011 17:59:37 +0530
Message-ID: <4DBFF531.1030208@codeaurora.org>
References: <1303977693-18389-1-git-send-email-pkondeti@codeaurora.org> <1304399633-16120-1-git-send-email-pkondeti@codeaurora.org> <1304399633-16120-3-git-send-email-pkondeti@codeaurora.org> <4DBFE39A.4010804@ru.mvista.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Return-path: <linux-arm-msm-owner@vger.kernel.org>
Received: from wolverine01.qualcomm.com ([199.106.114.254]:60303 "EHLO
	wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751500Ab1ECM3m (ORCPT
	<rfc822;linux-arm-msm@vger.kernel.org>);
	Tue, 3 May 2011 08:29:42 -0400
In-Reply-To: <4DBFE39A.4010804@ru.mvista.com>
Sender: linux-arm-msm-owner@vger.kernel.org
List-Id: linux-arm-msm@vger.kernel.org
To: Sergei Shtylyov <sshtylyov@mvista.com>
Cc: greg@kroah.com, linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org, davidb@codeaurora.org, Anji jonnala <anjir@codeaurora.org>

Hi Sergei,

On 5/3/2011 4:44 PM, Sergei Shtylyov wrote:
> Hello.
> 
> On 03-05-2011 9:13, Pavankumar Kondeti wrote:
> 
>> From: Anji jonnala <anjir@codeaurora.org>
> 
>> HSUSB core clock is derived from daytona fabric clock and for
>> HSUSB operational require minimum core clock at 55MHz. Since, HSUSB
>> cannot tolerate daytona fabric clock change in the middle of HSUSB
>> operational, vote for maximum Daytona fabric clock
>> while usb is operational
> 
>> Signed-off-by: Anji jonnala<anjir@codeaurora.org>
>> Signed-off-by: Pavankumar Kondeti<pkondeti@codeaurora.org>
> [...]
> 
>> diff --git a/drivers/usb/otg/msm_otg.c b/drivers/usb/otg/msm_otg.c
>> index 2965986..cfbb606 100644
>> --- a/drivers/usb/otg/msm_otg.c
>> +++ b/drivers/usb/otg/msm_otg.c
> [...]
>> @@ -954,7 +979,11 @@ free_regs:
>>   put_core_clk:
>>   	if (motg->core_clk)
>>   		clk_put(motg->core_clk);
>> -	clk_put(motg->pclk);
> 
>     Is it really correct to remove this line?
> 
No. It is not correct. I will fix it in next version.

>> +put_pclk_src:
>> +	if (!IS_ERR(motg->pclk_src)) {
>> +		clk_disable(motg->pclk_src);
>> +		clk_put(motg->pclk_src);
>> +	}
>>   put_clk:
>>   	clk_put(motg->clk);
>>   put_phy_reset_clk:
> 
> WBR, Sergei


-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.