Re: [PATCH 7/7] usb: phy: msm: Lindent the code
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 iiva...@mm-sol.com Cc: Felipe Balbi ba...@ti.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: linux-...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com 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-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 7/7] usb: phy: msm: Lindent the code
Hi, On Thu, Jul 25, 2013 at 04:40:51PM +0300, Ivan T. Ivanov wrote: 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. yeah, I don't think it's worth to have this patch. Just looked at the driver and, at least to my eyes, it didn't look all that bad. Matter of taste though -- balbi signature.asc Description: Digital signature
Re: [PATCH 7/7] usb: phy: msm: Lindent the code
On Mon, Jun 24, 2013 at 06:27:44PM +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com Cc: Felipe Balbi ba...@ti.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: linux-...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com I really don't like blind Lindent patches... sometimes it makes things even worse. --- 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 ? -- balbi signature.asc Description: Digital signature
[PATCH 7/7] usb: phy: msm: Lindent the code
From: Ivan T. Ivanov iiva...@mm-sol.com Cc: Felipe Balbi ba...@ti.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: linux-...@vger.kernel.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- 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 @@ -45,18 +45,18 @@ #define ULPI_IO_TIMEOUT_USEC (10 * 1000) -#define USB_PHY_3P3_VOL_MIN305 /* uV */ -#define USB_PHY_3P3_VOL_MAX330 /* uV */ +#define USB_PHY_3P3_VOL_MIN305 /* uV */ +#define USB_PHY_3P3_VOL_MAX330 /* uV */ #define USB_PHY_3P3_HPM_LOAD 5 /* uA */ #define USB_PHY_3P3_LPM_LOAD 4000/* uA */ -#define USB_PHY_1P8_VOL_MIN180 /* uV */ -#define USB_PHY_1P8_VOL_MAX180 /* uV */ +#define USB_PHY_1P8_VOL_MIN180 /* uV */ +#define USB_PHY_1P8_VOL_MAX180 /* uV */ #define USB_PHY_1P8_HPM_LOAD 5 /* uA */ #define USB_PHY_1P8_LPM_LOAD 4000/* uA */ -#define USB_PHY_VDD_DIG_VOL_MIN100 /* uV */ -#define USB_PHY_VDD_DIG_VOL_MAX132 /* uV */ +#define USB_PHY_VDD_DIG_VOL_MIN100 /* uV */ +#define USB_PHY_VDD_DIG_VOL_MAX132 /* uV */ static int msm_hsusb_init_vddcx(struct msm_otg *motg, int init) { @@ -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); if (ret) { dev_err(motg-phy.dev, Cannot set vddcx voltage\n); return ret; @@ -73,15 +73,17 @@ static int msm_hsusb_init_vddcx(struct msm_otg *motg, int init) ret = regulator_enable(motg-vddcx); if (ret) - dev_err(motg-phy.dev, unable to enable hsusb vddcx\n); + dev_err(motg-phy.dev, + unable to enable hsusb vddcx\n); } else { ret = regulator_set_voltage(motg-vddcx, 0, - USB_PHY_VDD_DIG_VOL_MAX); + USB_PHY_VDD_DIG_VOL_MAX); if (ret) dev_err(motg-phy.dev, Cannot set vddcx voltage\n); ret = regulator_disable(motg-vddcx); if (ret) - dev_err(motg-phy.dev, unable to disable hsusb vddcx\n); + dev_err(motg-phy.dev, + unable to disable hsusb vddcx\n); } return ret; @@ -93,26 +95,28 @@ static int msm_hsusb_ldo_init(struct msm_otg *motg, int init) if (init) { rc = regulator_set_voltage(motg-v3p3, USB_PHY_3P3_VOL_MIN, - USB_PHY_3P3_VOL_MAX); + USB_PHY_3P3_VOL_MAX); if (rc) { dev_err(motg-phy.dev, Cannot set v3p3 voltage\n); return rc; } rc = regulator_enable(motg-v3p3); if (rc) { - dev_err(motg-phy.dev, unable to enable the hsusb 3p3\n); + dev_err(motg-phy.dev, + unable to enable the hsusb 3p3\n); return rc; } rc = regulator_set_voltage(motg-v1p8, USB_PHY_1P8_VOL_MIN, - USB_PHY_1P8_VOL_MAX); + USB_PHY_1P8_VOL_MAX); if (rc) { dev_err(motg-phy.dev, Cannot set v1p8 voltage\n); goto disable_3p3; } rc = regulator_enable(motg-v1p8); if (rc) { - dev_err(motg-phy.dev, unable to enable the hsusb 1p8\n); + dev_err(motg-phy.dev, + unable to enable the hsusb 1p8\n); goto disable_3p3; } @@ -156,26 +160,26 @@ static int msm_hsusb_ldo_set_mode(struct msm_otg *motg, int on) if (on) { ret = regulator_set_optimum_mode(motg-v1p8, - USB_PHY_1P8_HPM_LOAD); +USB_PHY_1P8_HPM_LOAD); if (ret 0) { pr_err(Could not set HPM for v1p8\n); return ret; } ret = regulator_set_optimum_mode(motg-v3p3, -