Messages in this thread | | | Subject | Re: [PATCH] reset: brcmstb-rescal: fix incorrect polarity of status bit | From | Philipp Zabel <> | Date | Tue, 05 Oct 2021 11:54:47 +0200 |
| |
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
| |