phy: ti: tusb1210: Define device IDs

Define device IDs since some of them are being used more than once.

Signed-off-by: Andy Shevchenko <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Vinod Koul <[email protected]>

diff --git a/drivers/phy/ti/phy-tusb1210.c b/drivers/phy/ti/phy-tusb1210.c
index 7232b104a62dc93c10dfae6c24f8d36e8870020e..13cd614e12a1d3c8ad98de07a40f63091409d228 100644 (file)
--- a/drivers/phy/ti/phy-tusb1210.c
+++ b/drivers/phy/ti/phy-tusb1210.c
@@ -17,6 +17,10 @@
 #include <linux/property.h>
 #include <linux/workqueue.h>
 
+#define TI_VENDOR_ID           0x0451
+#define TI_DEVICE_TUSB1210     0x1507
+#define TI_DEVICE_TUSB1211     0x1508
+
 #define TUSB1211_POWER_CONTROL                         0x3d
 #define TUSB1211_POWER_CONTROL_SET                     0x3e
 #define TUSB1211_POWER_CONTROL_CLEAR                   0x3f
@@ -436,7 +440,7 @@ static void tusb1210_probe_charger_detect(struct tusb1210 *tusb)
        if (!device_property_read_bool(dev->parent, "linux,phy_charger_detect"))
                return;
 
-       if (ulpi->id.product != 0x1508) {
+       if (ulpi->id.product != TI_DEVICE_TUSB1211) {
                dev_err(dev, "error charger detection is only supported on the TUSB1211\n");
                return;
        }
@@ -562,11 +566,9 @@ static void tusb1210_remove(struct ulpi *ulpi)
        tusb1210_remove_charger_detect(tusb);
 }
 
-#define TI_VENDOR_ID 0x0451
-
 static const struct ulpi_device_id tusb1210_ulpi_id[] = {
-       { TI_VENDOR_ID, 0x1507, },  /* TUSB1210 */
-       { TI_VENDOR_ID, 0x1508, },  /* TUSB1211 */
+       { TI_VENDOR_ID, TI_DEVICE_TUSB1210 },
+       { TI_VENDOR_ID, TI_DEVICE_TUSB1211 },
        { },
 };
 MODULE_DEVICE_TABLE(ulpi, tusb1210_ulpi_id);