]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
usb: phy: tegra: Perform general clean up of the code
authorDmitry Osipenko <digetx@gmail.com>
Mon, 6 Jan 2020 01:34:02 +0000 (04:34 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 9 Jan 2020 10:15:31 +0000 (11:15 +0100)
This patch fixes few dozens of legit checkpatch warnings, adds missed
handling of potential error-cases and prettifies code where makes sense.
All these clean-up changes are quite minor and do not fix any real
problems.

Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Link: https://lore.kernel.org/r/20200106013416.9604-7-digetx@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/phy/phy-tegra-usb.c

index 664259d746580d3a74718c6df23b4d9a7b1e89fd..c045f44ea96425c7be60c45d6f9e73dbbab95eb7 100644 (file)
 #include <linux/usb/tegra_usb_phy.h>
 #include <linux/regulator/consumer.h>
 
-#define ULPI_VIEWPORT          0x170
+#define ULPI_VIEWPORT                          0x170
 
 /* PORTSC PTS/PHCD bits, Tegra20 only */
-#define TEGRA_USB_PORTSC1                              0x184
-#define TEGRA_USB_PORTSC1_PTS(x)                       (((x) & 0x3) << 30)
-#define TEGRA_USB_PORTSC1_PHCD                         (1 << 23)
+#define TEGRA_USB_PORTSC1                      0x184
+#define TEGRA_USB_PORTSC1_PTS(x)               (((x) & 0x3) << 30)
+#define TEGRA_USB_PORTSC1_PHCD                 BIT(23)
 
 /* HOSTPC1 PTS/PHCD bits, Tegra30 and above */
-#define TEGRA_USB_HOSTPC1_DEVLC                0x1b4
-#define TEGRA_USB_HOSTPC1_DEVLC_PTS(x) (((x) & 0x7) << 29)
-#define TEGRA_USB_HOSTPC1_DEVLC_PHCD   (1 << 22)
+#define TEGRA_USB_HOSTPC1_DEVLC                        0x1b4
+#define TEGRA_USB_HOSTPC1_DEVLC_PTS(x)         (((x) & 0x7) << 29)
+#define TEGRA_USB_HOSTPC1_DEVLC_PHCD           BIT(22)
 
 /* Bits of PORTSC1, which will get cleared by writing 1 into them */
 #define TEGRA_PORTSC1_RWC_BITS (PORT_CSC | PORT_PEC | PORT_OCC)
 
-#define USB_SUSP_CTRL          0x400
-#define   USB_WAKE_ON_CNNT_EN_DEV      (1 << 3)
-#define   USB_WAKE_ON_DISCON_EN_DEV    (1 << 4)
-#define   USB_SUSP_CLR         (1 << 5)
-#define   USB_PHY_CLK_VALID    (1 << 7)
-#define   UTMIP_RESET                  (1 << 11)
-#define   UHSIC_RESET                  (1 << 11)
-#define   UTMIP_PHY_ENABLE             (1 << 12)
-#define   ULPI_PHY_ENABLE      (1 << 13)
-#define   USB_SUSP_SET         (1 << 14)
-#define   USB_WAKEUP_DEBOUNCE_COUNT(x) (((x) & 0x7) << 16)
-
-#define USB1_LEGACY_CTRL       0x410
-#define   USB1_NO_LEGACY_MODE                  (1 << 0)
+#define USB_SUSP_CTRL                          0x400
+#define   USB_WAKE_ON_CNNT_EN_DEV              BIT(3)
+#define   USB_WAKE_ON_DISCON_EN_DEV            BIT(4)
+#define   USB_SUSP_CLR                         BIT(5)
+#define   USB_PHY_CLK_VALID                    BIT(7)
+#define   UTMIP_RESET                          BIT(11)
+#define   UHSIC_RESET                          BIT(11)
+#define   UTMIP_PHY_ENABLE                     BIT(12)
+#define   ULPI_PHY_ENABLE                      BIT(13)
+#define   USB_SUSP_SET                         BIT(14)
+#define   USB_WAKEUP_DEBOUNCE_COUNT(x)         (((x) & 0x7) << 16)
+
+#define USB1_LEGACY_CTRL                       0x410
+#define   USB1_NO_LEGACY_MODE                  BIT(0)
 #define   USB1_VBUS_SENSE_CTL_MASK             (3 << 1)
 #define   USB1_VBUS_SENSE_CTL_VBUS_WAKEUP      (0 << 1)
 #define   USB1_VBUS_SENSE_CTL_AB_SESS_VLD_OR_VBUS_WAKEUP \
 #define   USB1_VBUS_SENSE_CTL_AB_SESS_VLD      (2 << 1)
 #define   USB1_VBUS_SENSE_CTL_A_SESS_VLD       (3 << 1)
 
-#define ULPI_TIMING_CTRL_0     0x424
-#define   ULPI_OUTPUT_PINMUX_BYP       (1 << 10)
-#define   ULPI_CLKOUT_PINMUX_BYP       (1 << 11)
+#define ULPI_TIMING_CTRL_0                     0x424
+#define   ULPI_OUTPUT_PINMUX_BYP               BIT(10)
+#define   ULPI_CLKOUT_PINMUX_BYP               BIT(11)
 
-#define ULPI_TIMING_CTRL_1     0x428
-#define   ULPI_DATA_TRIMMER_LOAD       (1 << 0)
-#define   ULPI_DATA_TRIMMER_SEL(x)     (((x) & 0x7) << 1)
-#define   ULPI_STPDIRNXT_TRIMMER_LOAD  (1 << 16)
-#define   ULPI_STPDIRNXT_TRIMMER_SEL(x)        (((x) & 0x7) << 17)
-#define   ULPI_DIR_TRIMMER_LOAD                (1 << 24)
-#define   ULPI_DIR_TRIMMER_SEL(x)      (((x) & 0x7) << 25)
+#define ULPI_TIMING_CTRL_1                     0x428
+#define   ULPI_DATA_TRIMMER_LOAD               BIT(0)
+#define   ULPI_DATA_TRIMMER_SEL(x)             (((x) & 0x7) << 1)
+#define   ULPI_STPDIRNXT_TRIMMER_LOAD          BIT(16)
+#define   ULPI_STPDIRNXT_TRIMMER_SEL(x)                (((x) & 0x7) << 17)
+#define   ULPI_DIR_TRIMMER_LOAD                        BIT(24)
+#define   ULPI_DIR_TRIMMER_SEL(x)              (((x) & 0x7) << 25)
 
-#define UTMIP_PLL_CFG1         0x804
+#define UTMIP_PLL_CFG1                         0x804
 #define   UTMIP_XTAL_FREQ_COUNT(x)             (((x) & 0xfff) << 0)
 #define   UTMIP_PLLU_ENABLE_DLY_COUNT(x)       (((x) & 0x1f) << 27)
 
-#define UTMIP_XCVR_CFG0                0x808
+#define UTMIP_XCVR_CFG0                                0x808
 #define   UTMIP_XCVR_SETUP(x)                  (((x) & 0xf) << 0)
 #define   UTMIP_XCVR_SETUP_MSB(x)              ((((x) & 0x70) >> 4) << 22)
 #define   UTMIP_XCVR_LSRSLEW(x)                        (((x) & 0x3) << 8)
 #define   UTMIP_XCVR_LSFSLEW(x)                        (((x) & 0x3) << 10)
-#define   UTMIP_FORCE_PD_POWERDOWN             (1 << 14)
-#define   UTMIP_FORCE_PD2_POWERDOWN            (1 << 16)
-#define   UTMIP_FORCE_PDZI_POWERDOWN           (1 << 18)
-#define   UTMIP_XCVR_LSBIAS_SEL                        (1 << 21)
+#define   UTMIP_FORCE_PD_POWERDOWN             BIT(14)
+#define   UTMIP_FORCE_PD2_POWERDOWN            BIT(16)
+#define   UTMIP_FORCE_PDZI_POWERDOWN           BIT(18)
+#define   UTMIP_XCVR_LSBIAS_SEL                        BIT(21)
 #define   UTMIP_XCVR_HSSLEW(x)                 (((x) & 0x3) << 4)
 #define   UTMIP_XCVR_HSSLEW_MSB(x)             ((((x) & 0x1fc) >> 2) << 25)
 
-#define UTMIP_BIAS_CFG0                0x80c
-#define   UTMIP_OTGPD                  (1 << 11)
-#define   UTMIP_BIASPD                 (1 << 10)
-#define   UTMIP_HSSQUELCH_LEVEL(x)     (((x) & 0x3) << 0)
-#define   UTMIP_HSDISCON_LEVEL(x)      (((x) & 0x3) << 2)
-#define   UTMIP_HSDISCON_LEVEL_MSB(x)  ((((x) & 0x4) >> 2) << 24)
+#define UTMIP_BIAS_CFG0                                0x80c
+#define   UTMIP_OTGPD                          BIT(11)
+#define   UTMIP_BIASPD                         BIT(10)
+#define   UTMIP_HSSQUELCH_LEVEL(x)             (((x) & 0x3) << 0)
+#define   UTMIP_HSDISCON_LEVEL(x)              (((x) & 0x3) << 2)
+#define   UTMIP_HSDISCON_LEVEL_MSB(x)          ((((x) & 0x4) >> 2) << 24)
 
-#define UTMIP_HSRX_CFG0                0x810
-#define   UTMIP_ELASTIC_LIMIT(x)       (((x) & 0x1f) << 10)
-#define   UTMIP_IDLE_WAIT(x)           (((x) & 0x1f) << 15)
+#define UTMIP_HSRX_CFG0                                0x810
+#define   UTMIP_ELASTIC_LIMIT(x)               (((x) & 0x1f) << 10)
+#define   UTMIP_IDLE_WAIT(x)                   (((x) & 0x1f) << 15)
 
-#define UTMIP_HSRX_CFG1                0x814
-#define   UTMIP_HS_SYNC_START_DLY(x)   (((x) & 0x1f) << 1)
+#define UTMIP_HSRX_CFG1                                0x814
+#define   UTMIP_HS_SYNC_START_DLY(x)           (((x) & 0x1f) << 1)
 
-#define UTMIP_TX_CFG0          0x820
-#define   UTMIP_FS_PREABMLE_J          (1 << 19)
-#define   UTMIP_HS_DISCON_DISABLE      (1 << 8)
+#define UTMIP_TX_CFG0                          0x820
+#define   UTMIP_FS_PREABMLE_J                  BIT(19)
+#define   UTMIP_HS_DISCON_DISABLE              BIT(8)
 
-#define UTMIP_MISC_CFG0                0x824
-#define   UTMIP_DPDM_OBSERVE           (1 << 26)
-#define   UTMIP_DPDM_OBSERVE_SEL(x)    (((x) & 0xf) << 27)
-#define   UTMIP_DPDM_OBSERVE_SEL_FS_J  UTMIP_DPDM_OBSERVE_SEL(0xf)
-#define   UTMIP_DPDM_OBSERVE_SEL_FS_K  UTMIP_DPDM_OBSERVE_SEL(0xe)
-#define   UTMIP_DPDM_OBSERVE_SEL_FS_SE1 UTMIP_DPDM_OBSERVE_SEL(0xd)
-#define   UTMIP_DPDM_OBSERVE_SEL_FS_SE0 UTMIP_DPDM_OBSERVE_SEL(0xc)
-#define   UTMIP_SUSPEND_EXIT_ON_EDGE   (1 << 22)
+#define UTMIP_MISC_CFG0                                0x824
+#define   UTMIP_DPDM_OBSERVE                   BIT(26)
+#define   UTMIP_DPDM_OBSERVE_SEL(x)            (((x) & 0xf) << 27)
+#define   UTMIP_DPDM_OBSERVE_SEL_FS_J          UTMIP_DPDM_OBSERVE_SEL(0xf)
+#define   UTMIP_DPDM_OBSERVE_SEL_FS_K          UTMIP_DPDM_OBSERVE_SEL(0xe)
+#define   UTMIP_DPDM_OBSERVE_SEL_FS_SE1                UTMIP_DPDM_OBSERVE_SEL(0xd)
+#define   UTMIP_DPDM_OBSERVE_SEL_FS_SE0                UTMIP_DPDM_OBSERVE_SEL(0xc)
+#define   UTMIP_SUSPEND_EXIT_ON_EDGE           BIT(22)
 
-#define UTMIP_MISC_CFG1                0x828
-#define   UTMIP_PLL_ACTIVE_DLY_COUNT(x)        (((x) & 0x1f) << 18)
-#define   UTMIP_PLLU_STABLE_COUNT(x)   (((x) & 0xfff) << 6)
+#define UTMIP_MISC_CFG1                                0x828
+#define   UTMIP_PLL_ACTIVE_DLY_COUNT(x)                (((x) & 0x1f) << 18)
+#define   UTMIP_PLLU_STABLE_COUNT(x)           (((x) & 0xfff) << 6)
 
-#define UTMIP_DEBOUNCE_CFG0    0x82c
-#define   UTMIP_BIAS_DEBOUNCE_A(x)     (((x) & 0xffff) << 0)
+#define UTMIP_DEBOUNCE_CFG0                    0x82c
+#define   UTMIP_BIAS_DEBOUNCE_A(x)             (((x) & 0xffff) << 0)
 
-#define UTMIP_BAT_CHRG_CFG0    0x830
-#define   UTMIP_PD_CHRG                        (1 << 0)
+#define UTMIP_BAT_CHRG_CFG0                    0x830
+#define   UTMIP_PD_CHRG                                BIT(0)
 
-#define UTMIP_SPARE_CFG0       0x834
-#define   FUSE_SETUP_SEL               (1 << 3)
+#define UTMIP_SPARE_CFG0                       0x834
+#define   FUSE_SETUP_SEL                       BIT(3)
 
-#define UTMIP_XCVR_CFG1                0x838
-#define   UTMIP_FORCE_PDDISC_POWERDOWN (1 << 0)
-#define   UTMIP_FORCE_PDCHRP_POWERDOWN (1 << 2)
-#define   UTMIP_FORCE_PDDR_POWERDOWN   (1 << 4)
-#define   UTMIP_XCVR_TERM_RANGE_ADJ(x) (((x) & 0xf) << 18)
+#define UTMIP_XCVR_CFG1                                0x838
+#define   UTMIP_FORCE_PDDISC_POWERDOWN         BIT(0)
+#define   UTMIP_FORCE_PDCHRP_POWERDOWN         BIT(2)
+#define   UTMIP_FORCE_PDDR_POWERDOWN           BIT(4)
+#define   UTMIP_XCVR_TERM_RANGE_ADJ(x)         (((x) & 0xf) << 18)
 
-#define UTMIP_BIAS_CFG1                0x83c
-#define   UTMIP_BIAS_PDTRK_COUNT(x)    (((x) & 0x1f) << 3)
+#define UTMIP_BIAS_CFG1                                0x83c
+#define   UTMIP_BIAS_PDTRK_COUNT(x)            (((x) & 0x1f) << 3)
 
 /* For Tegra30 and above only, the address is different in Tegra20 */
-#define USB_USBMODE            0x1f8
-#define   USB_USBMODE_MASK             (3 << 0)
-#define   USB_USBMODE_HOST             (3 << 0)
-#define   USB_USBMODE_DEVICE           (2 << 0)
+#define USB_USBMODE                            0x1f8
+#define   USB_USBMODE_MASK                     (3 << 0)
+#define   USB_USBMODE_HOST                     (3 << 0)
+#define   USB_USBMODE_DEVICE                   (2 << 0)
 
 static DEFINE_SPINLOCK(utmip_pad_lock);
-static int utmip_pad_count;
+static unsigned int utmip_pad_count;
 
 struct tegra_xtal_freq {
-       int freq;
+       unsigned int freq;
        u8 enable_delay;
        u8 stable_count;
        u8 active_delay;
@@ -194,6 +194,11 @@ static const struct tegra_xtal_freq tegra_freq_table[] = {
        },
 };
 
+static inline struct tegra_usb_phy *to_tegra_usb_phy(struct usb_phy *u_phy)
+{
+       return container_of(u_phy, struct tegra_usb_phy, u_phy);
+}
+
 static void set_pts(struct tegra_usb_phy *phy, u8 pts_val)
 {
        void __iomem *base = phy->regs;
@@ -298,13 +303,16 @@ static int utmip_pad_close(struct tegra_usb_phy *phy)
        return ret;
 }
 
-static void utmip_pad_power_on(struct tegra_usb_phy *phy)
+static int utmip_pad_power_on(struct tegra_usb_phy *phy)
 {
-       unsigned long val, flags;
-       void __iomem *base = phy->pad_regs;
        struct tegra_utmip_config *config = phy->config;
+       void __iomem *base = phy->pad_regs;
+       unsigned long val, flags;
+       int err;
 
-       clk_prepare_enable(phy->pad_clk);
+       err = clk_prepare_enable(phy->pad_clk);
+       if (err)
+               return err;
 
        spin_lock_irqsave(&utmip_pad_lock, flags);
 
@@ -327,19 +335,24 @@ static void utmip_pad_power_on(struct tegra_usb_phy *phy)
        spin_unlock_irqrestore(&utmip_pad_lock, flags);
 
        clk_disable_unprepare(phy->pad_clk);
+
+       return 0;
 }
 
 static int utmip_pad_power_off(struct tegra_usb_phy *phy)
 {
-       unsigned long val, flags;
        void __iomem *base = phy->pad_regs;
+       unsigned long val, flags;
+       int err;
 
        if (!utmip_pad_count) {
                dev_err(phy->u_phy.dev, "UTMIP pad already powered off\n");
                return -EINVAL;
        }
 
-       clk_prepare_enable(phy->pad_clk);
+       err = clk_prepare_enable(phy->pad_clk);
+       if (err)
+               return err;
 
        spin_lock_irqsave(&utmip_pad_lock, flags);
 
@@ -366,8 +379,8 @@ static int utmi_wait_register(void __iomem *reg, u32 mask, u32 result)
 
 static void utmi_phy_clk_disable(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        /*
         * The USB driver may have already initiated the phy clock
@@ -382,23 +395,24 @@ static void utmi_phy_clk_disable(struct tegra_usb_phy *phy)
                val |= USB_SUSP_SET;
                writel(val, base + USB_SUSP_CTRL);
 
-               udelay(10);
+               usleep_range(10, 100);
 
                val = readl(base + USB_SUSP_CTRL);
                val &= ~USB_SUSP_SET;
                writel(val, base + USB_SUSP_CTRL);
-       } else
+       } else {
                set_phcd(phy, true);
+       }
 
-       if (utmi_wait_register(base + USB_SUSP_CTRL, USB_PHY_CLK_VALID, 0) < 0)
+       if (utmi_wait_register(base + USB_SUSP_CTRL, USB_PHY_CLK_VALID, 0))
                dev_err(phy->u_phy.dev,
                        "Timeout waiting for PHY to stabilize on disable\n");
 }
 
 static void utmi_phy_clk_enable(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        /*
         * The USB driver may have already initiated the phy clock
@@ -414,25 +428,27 @@ static void utmi_phy_clk_enable(struct tegra_usb_phy *phy)
                val |= USB_SUSP_CLR;
                writel(val, base + USB_SUSP_CTRL);
 
-               udelay(10);
+               usleep_range(10, 100);
 
                val = readl(base + USB_SUSP_CTRL);
                val &= ~USB_SUSP_CLR;
                writel(val, base + USB_SUSP_CTRL);
-       } else
+       } else {
                set_phcd(phy, false);
+       }
 
        if (utmi_wait_register(base + USB_SUSP_CTRL, USB_PHY_CLK_VALID,
-                                                    USB_PHY_CLK_VALID))
+                              USB_PHY_CLK_VALID))
                dev_err(phy->u_phy.dev,
                        "Timeout waiting for PHY to stabilize on enable\n");
 }
 
 static int utmi_phy_power_on(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
-       void __iomem *base = phy->regs;
        struct tegra_utmip_config *config = phy->config;
+       void __iomem *base = phy->regs;
+       unsigned long val;
+       int err;
 
        val = readl(base + USB_SUSP_CTRL);
        val |= UTMIP_RESET;
@@ -498,7 +514,9 @@ static int utmi_phy_power_on(struct tegra_usb_phy *phy)
                writel(val, base + UTMIP_BAT_CHRG_CFG0);
        }
 
-       utmip_pad_power_on(phy);
+       err = utmip_pad_power_on(phy);
+       if (err)
+               return err;
 
        val = readl(base + UTMIP_XCVR_CFG0);
        val &= ~(UTMIP_FORCE_PD_POWERDOWN | UTMIP_FORCE_PD2_POWERDOWN |
@@ -579,8 +597,8 @@ static int utmi_phy_power_on(struct tegra_usb_phy *phy)
 
 static int utmi_phy_power_off(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        utmi_phy_clk_disable(phy);
 
@@ -614,8 +632,8 @@ static int utmi_phy_power_off(struct tegra_usb_phy *phy)
 
 static void utmi_phy_preresume(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        val = readl(base + UTMIP_TX_CFG0);
        val |= UTMIP_HS_DISCON_DISABLE;
@@ -624,8 +642,8 @@ static void utmi_phy_preresume(struct tegra_usb_phy *phy)
 
 static void utmi_phy_postresume(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        val = readl(base + UTMIP_TX_CFG0);
        val &= ~UTMIP_HS_DISCON_DISABLE;
@@ -635,8 +653,8 @@ static void utmi_phy_postresume(struct tegra_usb_phy *phy)
 static void utmi_phy_restore_start(struct tegra_usb_phy *phy,
                                   enum tegra_usb_phy_port_speed port_speed)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        val = readl(base + UTMIP_MISC_CFG0);
        val &= ~UTMIP_DPDM_OBSERVE_SEL(~0);
@@ -645,47 +663,52 @@ static void utmi_phy_restore_start(struct tegra_usb_phy *phy,
        else
                val |= UTMIP_DPDM_OBSERVE_SEL_FS_J;
        writel(val, base + UTMIP_MISC_CFG0);
-       udelay(1);
+       usleep_range(1, 10);
 
        val = readl(base + UTMIP_MISC_CFG0);
        val |= UTMIP_DPDM_OBSERVE;
        writel(val, base + UTMIP_MISC_CFG0);
-       udelay(10);
+       usleep_range(10, 100);
 }
 
 static void utmi_phy_restore_end(struct tegra_usb_phy *phy)
 {
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
 
        val = readl(base + UTMIP_MISC_CFG0);
        val &= ~UTMIP_DPDM_OBSERVE;
        writel(val, base + UTMIP_MISC_CFG0);
-       udelay(10);
+       usleep_range(10, 100);
 }
 
 static int ulpi_phy_power_on(struct tegra_usb_phy *phy)
 {
-       int ret;
-       unsigned long val;
        void __iomem *base = phy->regs;
+       unsigned long val;
+       int err;
 
-       ret = gpio_direction_output(phy->reset_gpio, 0);
-       if (ret < 0) {
+       err = gpio_direction_output(phy->reset_gpio, 0);
+       if (err) {
                dev_err(phy->u_phy.dev, "GPIO %d not set to 0: %d\n",
-                       phy->reset_gpio, ret);
-               return ret;
+                       phy->reset_gpio, err);
+               return err;
        }
-       msleep(5);
-       ret = gpio_direction_output(phy->reset_gpio, 1);
-       if (ret < 0) {
+
+       err = clk_prepare_enable(phy->clk);
+       if (err)
+               return err;
+
+       usleep_range(5000, 6000);
+
+       err = gpio_direction_output(phy->reset_gpio, 1);
+       if (err) {
                dev_err(phy->u_phy.dev, "GPIO %d not set to 1: %d\n",
-                       phy->reset_gpio, ret);
-               return ret;
+                       phy->reset_gpio, err);
+               goto disable_clk;
        }
 
-       clk_prepare_enable(phy->clk);
-       msleep(1);
+       usleep_range(1000, 2000);
 
        val = readl(base + USB_SUSP_CTRL);
        val |= UHSIC_RESET;
@@ -706,7 +729,7 @@ static int ulpi_phy_power_on(struct tegra_usb_phy *phy)
        val |= ULPI_STPDIRNXT_TRIMMER_SEL(4);
        val |= ULPI_DIR_TRIMMER_SEL(4);
        writel(val, base + ULPI_TIMING_CTRL_1);
-       udelay(10);
+       usleep_range(10, 100);
 
        val |= ULPI_DATA_TRIMMER_LOAD;
        val |= ULPI_STPDIRNXT_TRIMMER_LOAD;
@@ -714,28 +737,33 @@ static int ulpi_phy_power_on(struct tegra_usb_phy *phy)
        writel(val, base + ULPI_TIMING_CTRL_1);
 
        /* Fix VbusInvalid due to floating VBUS */
-       ret = usb_phy_io_write(phy->ulpi, 0x40, 0x08);
-       if (ret) {
-               dev_err(phy->u_phy.dev, "ULPI write failed: %d\n", ret);
-               return ret;
+       err = usb_phy_io_write(phy->ulpi, 0x40, 0x08);
+       if (err) {
+               dev_err(phy->u_phy.dev, "ULPI write failed: %d\n", err);
+               goto disable_clk;
        }
 
-       ret = usb_phy_io_write(phy->ulpi, 0x80, 0x0B);
-       if (ret) {
-               dev_err(phy->u_phy.dev, "ULPI write failed: %d\n", ret);
-               return ret;
+       err = usb_phy_io_write(phy->ulpi, 0x80, 0x0B);
+       if (err) {
+               dev_err(phy->u_phy.dev, "ULPI write failed: %d\n", err);
+               goto disable_clk;
        }
 
        val = readl(base + USB_SUSP_CTRL);
        val |= USB_SUSP_CLR;
        writel(val, base + USB_SUSP_CTRL);
-       udelay(100);
+       usleep_range(100, 1000);
 
        val = readl(base + USB_SUSP_CTRL);
        val &= ~USB_SUSP_CLR;
        writel(val, base + USB_SUSP_CTRL);
 
        return 0;
+
+disable_clk:
+       clk_disable_unprepare(phy->clk);
+
+       return err;
 }
 
 static int ulpi_phy_power_off(struct tegra_usb_phy *phy)
@@ -795,8 +823,7 @@ static int tegra_usb_phy_power_off(struct tegra_usb_phy *phy)
 
 static void tegra_usb_phy_shutdown(struct usb_phy *u_phy)
 {
-       struct tegra_usb_phy *phy = container_of(u_phy, struct tegra_usb_phy,
-                                                u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
 
        if (WARN_ON(!phy->freq))
                return;
@@ -814,9 +841,9 @@ static void tegra_usb_phy_shutdown(struct usb_phy *u_phy)
        phy->freq = NULL;
 }
 
-static int tegra_usb_phy_set_suspend(struct usb_phy *x, int suspend)
+static int tegra_usb_phy_set_suspend(struct usb_phy *u_phy, int suspend)
 {
-       struct tegra_usb_phy *phy = container_of(x, struct tegra_usb_phy, u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
 
        if (WARN_ON(!phy->freq))
                return -EINVAL;
@@ -832,7 +859,7 @@ static int ulpi_open(struct tegra_usb_phy *phy)
        int err;
 
        err = gpio_direction_output(phy->reset_gpio, 0);
-       if (err < 0) {
+       if (err) {
                dev_err(phy->u_phy.dev,
                        "ULPI reset GPIO %d direction not asserted: %d\n",
                        phy->reset_gpio, err);
@@ -844,10 +871,9 @@ static int ulpi_open(struct tegra_usb_phy *phy)
 
 static int tegra_usb_phy_init(struct usb_phy *u_phy)
 {
-       struct tegra_usb_phy *phy = container_of(u_phy, struct tegra_usb_phy,
-                                                u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
        unsigned long parent_rate;
-       int i;
+       unsigned int i;
        int err;
 
        if (WARN_ON(phy->freq))
@@ -885,7 +911,7 @@ static int tegra_usb_phy_init(struct usb_phy *u_phy)
                err = ulpi_open(phy);
        else
                err = utmip_pad_open(phy);
-       if (err < 0)
+       if (err)
                goto fail;
 
        err = tegra_usb_phy_power_on(phy);
@@ -905,37 +931,37 @@ static int tegra_usb_phy_init(struct usb_phy *u_phy)
        return err;
 }
 
-void tegra_usb_phy_preresume(struct usb_phy *x)
+void tegra_usb_phy_preresume(struct usb_phy *u_phy)
 {
-       struct tegra_usb_phy *phy = container_of(x, struct tegra_usb_phy, u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
 
        if (!phy->is_ulpi_phy)
                utmi_phy_preresume(phy);
 }
 EXPORT_SYMBOL_GPL(tegra_usb_phy_preresume);
 
-void tegra_usb_phy_postresume(struct usb_phy *x)
+void tegra_usb_phy_postresume(struct usb_phy *u_phy)
 {
-       struct tegra_usb_phy *phy = container_of(x, struct tegra_usb_phy, u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
 
        if (!phy->is_ulpi_phy)
                utmi_phy_postresume(phy);
 }
 EXPORT_SYMBOL_GPL(tegra_usb_phy_postresume);
 
-void tegra_ehci_phy_restore_start(struct usb_phy *x,
-                                enum tegra_usb_phy_port_speed port_speed)
+void tegra_ehci_phy_restore_start(struct usb_phy *u_phy,
+                                 enum tegra_usb_phy_port_speed port_speed)
 {
-       struct tegra_usb_phy *phy = container_of(x, struct tegra_usb_phy, u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
 
        if (!phy->is_ulpi_phy)
                utmi_phy_restore_start(phy, port_speed);
 }
 EXPORT_SYMBOL_GPL(tegra_ehci_phy_restore_start);
 
-void tegra_ehci_phy_restore_end(struct usb_phy *x)
+void tegra_ehci_phy_restore_end(struct usb_phy *u_phy)
 {
-       struct tegra_usb_phy *phy = container_of(x, struct tegra_usb_phy, u_phy);
+       struct tegra_usb_phy *phy = to_tegra_usb_phy(u_phy);
 
        if (!phy->is_ulpi_phy)
                utmi_phy_restore_end(phy);
@@ -946,21 +972,25 @@ static int read_utmi_param(struct platform_device *pdev, const char *param,
                           u8 *dest)
 {
        u32 value;
-       int err = of_property_read_u32(pdev->dev.of_node, param, &value);
-       *dest = (u8)value;
-       if (err < 0)
+       int err;
+
+       err = of_property_read_u32(pdev->dev.of_node, param, &value);
+       if (err)
                dev_err(&pdev->dev,
                        "Failed to read USB UTMI parameter %s: %d\n",
                        param, err);
+       else
+               *dest = value;
+
        return err;
 }
 
 static int utmi_phy_probe(struct tegra_usb_phy *tegra_phy,
                          struct platform_device *pdev)
 {
+       struct tegra_utmip_config *config;
        struct resource *res;
        int err;
-       struct tegra_utmip_config *config;
 
        tegra_phy->is_ulpi_phy = false;
 
@@ -971,7 +1001,7 @@ static int utmi_phy_probe(struct tegra_usb_phy *tegra_phy,
        }
 
        tegra_phy->pad_regs = devm_ioremap(&pdev->dev, res->start,
-               resource_size(res));
+                                          resource_size(res));
        if (!tegra_phy->pad_regs) {
                dev_err(&pdev->dev, "Failed to remap UTMI pad regs\n");
                return -ENOMEM;
@@ -985,49 +1015,49 @@ static int utmi_phy_probe(struct tegra_usb_phy *tegra_phy,
        config = tegra_phy->config;
 
        err = read_utmi_param(pdev, "nvidia,hssync-start-delay",
-               &config->hssync_start_delay);
-       if (err < 0)
+                             &config->hssync_start_delay);
+       if (err)
                return err;
 
        err = read_utmi_param(pdev, "nvidia,elastic-limit",
-               &config->elastic_limit);
-       if (err < 0)
+                             &config->elastic_limit);
+       if (err)
                return err;
 
        err = read_utmi_param(pdev, "nvidia,idle-wait-delay",
-               &config->idle_wait_delay);
-       if (err < 0)
+                             &config->idle_wait_delay);
+       if (err)
                return err;
 
        err = read_utmi_param(pdev, "nvidia,term-range-adj",
-               &config->term_range_adj);
-       if (err < 0)
+                             &config->term_range_adj);
+       if (err)
                return err;
 
        err = read_utmi_param(pdev, "nvidia,xcvr-lsfslew",
-               &config->xcvr_lsfslew);
-       if (err < 0)
+                             &config->xcvr_lsfslew);
+       if (err)
                return err;
 
        err = read_utmi_param(pdev, "nvidia,xcvr-lsrslew",
-               &config->xcvr_lsrslew);
-       if (err < 0)
+                             &config->xcvr_lsrslew);
+       if (err)
                return err;
 
        if (tegra_phy->soc_config->requires_extra_tuning_parameters) {
                err = read_utmi_param(pdev, "nvidia,xcvr-hsslew",
-                       &config->xcvr_hsslew);
-               if (err < 0)
+                                     &config->xcvr_hsslew);
+               if (err)
                        return err;
 
                err = read_utmi_param(pdev, "nvidia,hssquelch-level",
-                       &config->hssquelch_level);
-               if (err < 0)
+                                     &config->hssquelch_level);
+               if (err)
                        return err;
 
                err = read_utmi_param(pdev, "nvidia,hsdiscon-level",
-                       &config->hsdiscon_level);
-               if (err < 0)
+                                     &config->hsdiscon_level);
+               if (err)
                        return err;
        }
 
@@ -1036,8 +1066,8 @@ static int utmi_phy_probe(struct tegra_usb_phy *tegra_phy,
 
        if (!config->xcvr_setup_use_fuses) {
                err = read_utmi_param(pdev, "nvidia,xcvr-setup",
-                       &config->xcvr_setup);
-               if (err < 0)
+                                     &config->xcvr_setup);
+               if (err)
                        return err;
        }
 
@@ -1067,23 +1097,18 @@ MODULE_DEVICE_TABLE(of, tegra_usb_phy_id_table);
 
 static int tegra_usb_phy_probe(struct platform_device *pdev)
 {
-       const struct of_device_id *match;
-       struct resource *res;
-       struct tegra_usb_phy *tegra_phy = NULL;
        struct device_node *np = pdev->dev.of_node;
+       struct tegra_usb_phy *tegra_phy;
        enum usb_phy_interface phy_type;
+       struct reset_control *reset;
+       struct resource *res;
        int err;
 
        tegra_phy = devm_kzalloc(&pdev->dev, sizeof(*tegra_phy), GFP_KERNEL);
        if (!tegra_phy)
                return -ENOMEM;
 
-       match = of_match_device(tegra_usb_phy_id_table, &pdev->dev);
-       if (!match) {
-               dev_err(&pdev->dev, "Error: No device match found\n");
-               return -ENODEV;
-       }
-       tegra_phy->soc_config = match->data;
+       tegra_phy->soc_config = of_device_get_match_data(&pdev->dev);
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        if (!res) {
@@ -1092,7 +1117,7 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
        }
 
        tegra_phy->regs = devm_ioremap(&pdev->dev, res->start,
-               resource_size(res));
+                                      resource_size(res));
        if (!tegra_phy->regs) {
                dev_err(&pdev->dev, "Failed to remap I/O memory\n");
                return -ENOMEM;
@@ -1101,33 +1126,6 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
        tegra_phy->is_legacy_phy =
                of_property_read_bool(np, "nvidia,has-legacy-mode");
 
-       phy_type = of_usb_get_phy_mode(np);
-       switch (phy_type) {
-       case USBPHY_INTERFACE_MODE_UTMI:
-               err = utmi_phy_probe(tegra_phy, pdev);
-               if (err < 0)
-                       return err;
-               break;
-
-       case USBPHY_INTERFACE_MODE_ULPI:
-               tegra_phy->is_ulpi_phy = true;
-
-               tegra_phy->reset_gpio =
-                       of_get_named_gpio(np, "nvidia,phy-reset-gpio", 0);
-               if (!gpio_is_valid(tegra_phy->reset_gpio)) {
-                       dev_err(&pdev->dev,
-                               "Invalid GPIO: %d\n", tegra_phy->reset_gpio);
-                       return tegra_phy->reset_gpio;
-               }
-               tegra_phy->config = NULL;
-               break;
-
-       default:
-               dev_err(&pdev->dev, "phy_type %u is invalid or unsupported\n",
-                       phy_type);
-               return -EINVAL;
-       }
-
        if (of_find_property(np, "dr_mode", NULL))
                tegra_phy->mode = usb_get_dr_mode(&pdev->dev);
        else
@@ -1155,7 +1153,44 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
                return err;
        }
 
-       if (tegra_phy->is_ulpi_phy) {
+       phy_type = of_usb_get_phy_mode(np);
+       switch (phy_type) {
+       case USBPHY_INTERFACE_MODE_UTMI:
+               err = utmi_phy_probe(tegra_phy, pdev);
+               if (err)
+                       return err;
+
+               tegra_phy->pad_clk = devm_clk_get(&pdev->dev, "utmi-pads");
+               err = PTR_ERR_OR_ZERO(tegra_phy->pad_clk);
+               if (err) {
+                       dev_err(&pdev->dev,
+                               "Failed to get UTMIP pad clock: %d\n", err);
+                       return err;
+               }
+
+               reset = devm_reset_control_get_optional_shared(&pdev->dev,
+                                                              "utmi-pads");
+               err = PTR_ERR_OR_ZERO(reset);
+               if (err) {
+                       dev_err(&pdev->dev,
+                               "Failed to get UTMI-pads reset: %d\n", err);
+                       return err;
+               }
+               tegra_phy->pad_rst = reset;
+               break;
+
+       case USBPHY_INTERFACE_MODE_ULPI:
+               tegra_phy->is_ulpi_phy = true;
+
+               tegra_phy->reset_gpio =
+                       of_get_named_gpio(np, "nvidia,phy-reset-gpio", 0);
+
+               if (!gpio_is_valid(tegra_phy->reset_gpio)) {
+                       dev_err(&pdev->dev,
+                               "Invalid GPIO: %d\n", tegra_phy->reset_gpio);
+                       return tegra_phy->reset_gpio;
+               }
+
                tegra_phy->clk = devm_clk_get(&pdev->dev, "ulpi-link");
                err = PTR_ERR_OR_ZERO(tegra_phy->clk);
                if (err) {
@@ -1165,8 +1200,8 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
                }
 
                err = devm_gpio_request(&pdev->dev, tegra_phy->reset_gpio,
-                       "ulpi_phy_reset_b");
-               if (err < 0) {
+                                       "ulpi_phy_reset_b");
+               if (err) {
                        dev_err(&pdev->dev, "Request failed for GPIO %d: %d\n",
                                tegra_phy->reset_gpio, err);
                        return err;
@@ -1175,28 +1210,16 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
                tegra_phy->ulpi = otg_ulpi_create(&ulpi_viewport_access_ops, 0);
                if (!tegra_phy->ulpi) {
                        dev_err(&pdev->dev, "Failed to create ULPI OTG\n");
-                       err = -ENOMEM;
-                       return err;
+                       return -ENOMEM;
                }
 
                tegra_phy->ulpi->io_priv = tegra_phy->regs + ULPI_VIEWPORT;
-       } else {
-               tegra_phy->pad_clk = devm_clk_get(&pdev->dev, "utmi-pads");
-               err = PTR_ERR_OR_ZERO(tegra_phy->pad_clk);
-               if (err) {
-                       dev_err(&pdev->dev,
-                               "Failed to get UTMIP pad clock: %d\n", err);
-                       return err;
-               }
+               break;
 
-               tegra_phy->pad_rst = devm_reset_control_get_optional_shared(
-                                               &pdev->dev, "utmi-pads");
-               err = PTR_ERR_OR_ZERO(tegra_phy->pad_rst);
-               if (err) {
-                       dev_err(&pdev->dev,
-                               "Failed to get UTMI-pads reset: %d\n", err);
-                       return err;
-               }
+       default:
+               dev_err(&pdev->dev, "phy_type %u is invalid or unsupported\n",
+                       phy_type);
+               return -EINVAL;
        }
 
        tegra_phy->u_phy.dev = &pdev->dev;
@@ -1207,7 +1230,7 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
        platform_set_drvdata(pdev, tegra_phy);
 
        err = usb_add_phy_dev(&tegra_phy->u_phy);
-       if (err < 0)
+       if (err)
                goto free_ulpi;
 
        return 0;