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@xxxxxxxxxxxxxx> > [...] > >> 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. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html