From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Ivan T. Ivanov" Subject: Re: [PATCH 7/7] usb: phy: msm: Lindent the code Date: Thu, 25 Jul 2013 16:40:51 +0300 Message-ID: <1374759651.1956.29.camel@iivanov-dev.int.mm-sol.com> References: <1372087664-26396-1-git-send-email-iivanov@mm-sol.com> <1372087664-26396-8-git-send-email-iivanov@mm-sol.com> <20130724122253.GC17962@radagast> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130724122253.GC17962@radagast> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: balbi-l0cyMroinI0@public.gmane.org Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Greg Kroah-Hartman , linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-arm-msm@vger.kernel.org Hi, On Wed, 2013-07-24 at 15:22 +0300, Felipe Balbi wrote: > On Mon, Jun 24, 2013 at 06:27:44PM +0300, Ivan T. Ivanov wrote: > > From: "Ivan T. Ivanov" > > > > Cc: Felipe Balbi > > Cc: Greg Kroah-Hartman > > Cc: linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > > > Signed-off-by: Ivan T. Ivanov > > I really don't like blind Lindent patches... sometimes it makes things > even worse. It is not exactly blindly. I have removed some of the changes in the platform_driver structure. > > > --- > > drivers/usb/phy/phy-msm-usb.c | 99 ++++++++++++++++++++++------------------- > > 1 file changed, 52 insertions(+), 47 deletions(-) > > > > diff --git a/drivers/usb/phy/phy-msm-usb.c b/drivers/usb/phy/phy-msm-usb.c > > index 6d05085..111f454 100644 > > --- a/drivers/usb/phy/phy-msm-usb.c > > +++ b/drivers/usb/phy/phy-msm-usb.c > > @@ -64,8 +64,8 @@ static int msm_hsusb_init_vddcx(struct msm_otg *motg, int init) > > > > if (init) { > > ret = regulator_set_voltage(motg->vddcx, > > - USB_PHY_VDD_DIG_VOL_MIN, > > - USB_PHY_VDD_DIG_VOL_MAX); > > + USB_PHY_VDD_DIG_VOL_MIN, > > + USB_PHY_VDD_DIG_VOL_MAX); > > like here, what's the point ? It makes indentation the same like the majority of the code. I could drop this patch if you like. Thanks, Ivan > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html