Message ID | 20210914221122.62315-1-f.fainelli@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | reset: brcmstb-rescal: fix incorrect polarity of status bit | expand |
On Tue, 2021-09-14 at 15:11 -0700, Florian Fainelli wrote: > From: Jim Quinlan <jim2101024@gmail.com> > > The readl_poll_timeout() should complete when the status bit > is a 1, not 0. > > Fixes: 4cf176e52397 ("reset: Add Broadcom STB RESCAL reset controller") > Signed-off-by: Jim Quinlan <jim2101024@gmail.com> > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > --- > drivers/reset/reset-brcmstb-rescal.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/reset/reset-brcmstb-rescal.c b/drivers/reset/reset-brcmstb-rescal.c > index b6f074d6a65f..433fa0c40e47 100644 > --- a/drivers/reset/reset-brcmstb-rescal.c > +++ b/drivers/reset/reset-brcmstb-rescal.c > @@ -38,7 +38,7 @@ static int brcm_rescal_reset_set(struct reset_controller_dev *rcdev, > } > > ret = readl_poll_timeout(base + BRCM_RESCAL_STATUS, reg, > - !(reg & BRCM_RESCAL_STATUS_BIT), 100, 1000); > + (reg & BRCM_RESCAL_STATUS_BIT), 100, 1000); > if (ret) { > dev_err(data->dev, "time out on SATA/PCIe rescal\n"); > return ret; Thank you, applied to reset/fixes. regards Philipp
diff --git a/drivers/reset/reset-brcmstb-rescal.c b/drivers/reset/reset-brcmstb-rescal.c index b6f074d6a65f..433fa0c40e47 100644 --- a/drivers/reset/reset-brcmstb-rescal.c +++ b/drivers/reset/reset-brcmstb-rescal.c @@ -38,7 +38,7 @@ static int brcm_rescal_reset_set(struct reset_controller_dev *rcdev, } ret = readl_poll_timeout(base + BRCM_RESCAL_STATUS, reg, - !(reg & BRCM_RESCAL_STATUS_BIT), 100, 1000); + (reg & BRCM_RESCAL_STATUS_BIT), 100, 1000); if (ret) { dev_err(data->dev, "time out on SATA/PCIe rescal\n"); return ret;