diff options
author | Hans de Goede <hdegoede@redhat.com> | 2022-02-13 14:05:22 +0100 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2022-02-25 09:58:07 +0100 |
commit | cb7d21b44f250a97660b904c43440f74e318aa82 (patch) | |
tree | 6bd5858a36e29e93bcaf78bfe0bcd28827b0d2d4 /drivers/phy | |
parent | phy: ti: tusb1210: Improve ulpi_read()/_write() error checking (diff) | |
download | linux-cb7d21b44f250a97660b904c43440f74e318aa82.tar.xz linux-cb7d21b44f250a97660b904c43440f74e318aa82.zip |
phy: ti: tusb1210: Drop tusb->vendor_specific2 != 0 check from tusb1210_power_on()
Since commit eb445a15fa69 ("phy: tusb1210: use bitmasks to set
VENDOR_SPECIFIC2") tusb->vendor_specific2 always contains a valid value
so there no need to check that it is set.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20220213130524.18748-8-hdegoede@redhat.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/phy')
-rw-r--r-- | drivers/phy/ti/phy-tusb1210.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/phy/ti/phy-tusb1210.c b/drivers/phy/ti/phy-tusb1210.c index bf7793afdc84..04baed24469f 100644 --- a/drivers/phy/ti/phy-tusb1210.c +++ b/drivers/phy/ti/phy-tusb1210.c @@ -61,11 +61,8 @@ static int tusb1210_power_on(struct phy *phy) gpiod_set_value_cansleep(tusb->gpio_cs, 1); /* Restore the optional eye diagram optimization value */ - if (tusb->vendor_specific2) - return tusb1210_ulpi_write(tusb, TUSB1210_VENDOR_SPECIFIC2, - tusb->vendor_specific2); - - return 0; + return tusb1210_ulpi_write(tusb, TUSB1210_VENDOR_SPECIFIC2, + tusb->vendor_specific2); } static int tusb1210_power_off(struct phy *phy) |