Skip to content

Commit

Permalink
phy: ti: tusb1210: Drop tusb->vendor_specific2 != 0 check from tusb12…
Browse files Browse the repository at this point in the history
…10_power_on()

Since commit eb445a1 ("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>
  • Loading branch information
Hans de Goede authored and Vinod Koul committed Feb 25, 2022
1 parent 09a3512 commit cb7d21b
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions drivers/phy/ti/phy-tusb1210.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit cb7d21b

Please sign in to comment.