Re: [PATCH] phy: brcm-sata: fix a timeout test in init

2017-06-20 Thread Vivek Gautam
On 06/20/2017 02:12 PM, Dan Carpenter wrote: On Tue, Jun 20, 2017 at 01:56:35PM +0530, Vivek Gautam wrote: On 06/19/2017 04:26 PM, Dan Carpenter wrote: We want to timeout with try set to zero so this should be a pre-op instead of post-op. Signed-off-by: Dan Carpenter diff --git a/drivers/

Re: [PATCH] phy: brcm-sata: fix a timeout test in init

2017-06-20 Thread Dan Carpenter
On Tue, Jun 20, 2017 at 01:56:35PM +0530, Vivek Gautam wrote: > > > On 06/19/2017 04:26 PM, Dan Carpenter wrote: > > We want to timeout with try set to zero so this should be a pre-op > > instead of post-op. > > > > Signed-off-by: Dan Carpenter > > > > diff --git a/drivers/phy/broadcom/phy-brc

Re: [PATCH] phy: brcm-sata: fix a timeout test in init

2017-06-20 Thread Vivek Gautam
On 06/19/2017 04:26 PM, Dan Carpenter wrote: We want to timeout with try set to zero so this should be a pre-op instead of post-op. Signed-off-by: Dan Carpenter diff --git a/drivers/phy/broadcom/phy-brcm-sata.c b/drivers/phy/broadcom/phy-brcm-sata.c index ccbc3d994998..48fb016ce689 100644 -

[PATCH] phy: brcm-sata: fix a timeout test in init

2017-06-19 Thread Dan Carpenter
We want to timeout with try set to zero so this should be a pre-op instead of post-op. Signed-off-by: Dan Carpenter diff --git a/drivers/phy/broadcom/phy-brcm-sata.c b/drivers/phy/broadcom/phy-brcm-sata.c index ccbc3d994998..48fb016ce689 100644 --- a/drivers/phy/broadcom/phy-brcm-sata.c +++ b/d