On 03/03/15 14:04, Axel Lin wrote: > Code simplification. No functional change. > > Signed-off-by: Axel Lin <[email protected]>
Acked-by: Roger Quadros <[email protected]> cheers, -roger > --- > drivers/phy/phy-ti-pipe3.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/drivers/phy/phy-ti-pipe3.c b/drivers/phy/phy-ti-pipe3.c > index 95c88f9..ed72b0d 100644 > --- a/drivers/phy/phy-ti-pipe3.c > +++ b/drivers/phy/phy-ti-pipe3.c > @@ -165,15 +165,11 @@ static int ti_pipe3_dpll_wait_lock(struct ti_pipe3 *phy) > cpu_relax(); > val = ti_pipe3_readl(phy->pll_ctrl_base, PLL_STATUS); > if (val & PLL_LOCK) > - break; > + return 0; > } while (!time_after(jiffies, timeout)); > > - if (!(val & PLL_LOCK)) { > - dev_err(phy->dev, "DPLL failed to lock\n"); > - return -EBUSY; > - } > - > - return 0; > + dev_err(phy->dev, "DPLL failed to lock\n"); > + return -EBUSY; > } > > static int ti_pipe3_dpll_program(struct ti_pipe3 *phy) > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
