Re: [PATCH V2 5/5] USB: OTG: msm: Add PHY suspend support for MSM8960

[Pavan Kondeti <pkondeti@codeaurora.org>]

On 5/2/2011 5:52 PM, Sergei Shtylyov wrote:
> Hello.
> 
> On 02-05-2011 10:36, Pavankumar Kondeti wrote:
> 
>> 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 66a1b7c..52853fd 100644
>> --- a/drivers/usb/otg/msm_otg.c
>> +++ b/drivers/usb/otg/msm_otg.c
>> @@ -163,6 +163,32 @@ put_3p3:
>>   	return rc;
>>   }
>>
>> +#ifdef CONFIG_PM_SLEEP
>> +#define USB_PHY_SUSP_DIG_VOL  500000
>> +static int msm_hsusb_config_vddcx(int high)
>> +{
>> +	int max_vol = USB_PHY_VDD_DIG_VOL_MAX;
>> +	int min_vol;
>> +	int ret;
>> +
>> +	if (high)
>> +		min_vol = USB_PHY_VDD_DIG_VOL_MIN;
>> +	else
>> +		min_vol = USB_PHY_SUSP_DIG_VOL;
>> +
>> +	ret = regulator_set_voltage(hsusb_vddcx, min_vol, max_vol);
>> +	if (ret) {
>> +		pr_err("%s: unable to set the voltage for regulator"
> 
>     You missed space after "regulator"...

Thanks for pointing this out. The other patch (USB: OTG: msm: Configure
PHY Analog and Digital voltage domains) also has missed spaces. I will
fix it in next patch set.

> 
>> +			"HSUSB_VDDCX\n", __func__);
>> +		return ret;
>> +	}
>> +
> [...]
> 
> WBR, Sergei

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