| Message ID | 20220720192807.130098-3-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Accepted |
| Commit | 6356c7bb5383c1e65cd718d94392fdc8cfa79fc8 |
| Headers | show |
| Series | None | expand |
On 7/21/22 12:57 AM, Krzysztof Kozlowski wrote: > The interrupt enable, status and clear registers have the same layout, > so BWMON_GLOBAL_IRQ_ENABLE_ENABLE define can be used also for clearing > IRQ, instead of hard-coded value. > > Cc: Rajendra Nayak <quic_rjendra@quicinc.com> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Sibi Sankar <quic_sibis@quicinc.com> > --- > drivers/soc/qcom/icc-bwmon.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/soc/qcom/icc-bwmon.c b/drivers/soc/qcom/icc-bwmon.c > index bea3ea1de7a4..3415f42523cd 100644 > --- a/drivers/soc/qcom/icc-bwmon.c > +++ b/drivers/soc/qcom/icc-bwmon.c > @@ -162,7 +162,8 @@ static void bwmon_clear_irq(struct icc_bwmon *bwmon) > * interrupt is cleared. > */ > writel(BWMON_IRQ_ENABLE_MASK, bwmon->base + BWMON_IRQ_CLEAR); > - writel(BIT(0), bwmon->base + BWMON_GLOBAL_IRQ_CLEAR); > + writel(BWMON_GLOBAL_IRQ_ENABLE_ENABLE, > + bwmon->base + BWMON_GLOBAL_IRQ_CLEAR); > } > > static void bwmon_disable(struct icc_bwmon *bwmon) >
diff --git a/drivers/soc/qcom/icc-bwmon.c b/drivers/soc/qcom/icc-bwmon.c index bea3ea1de7a4..3415f42523cd 100644 --- a/drivers/soc/qcom/icc-bwmon.c +++ b/drivers/soc/qcom/icc-bwmon.c @@ -162,7 +162,8 @@ static void bwmon_clear_irq(struct icc_bwmon *bwmon) * interrupt is cleared. */ writel(BWMON_IRQ_ENABLE_MASK, bwmon->base + BWMON_IRQ_CLEAR); - writel(BIT(0), bwmon->base + BWMON_GLOBAL_IRQ_CLEAR); + writel(BWMON_GLOBAL_IRQ_ENABLE_ENABLE, + bwmon->base + BWMON_GLOBAL_IRQ_CLEAR); } static void bwmon_disable(struct icc_bwmon *bwmon)
The interrupt enable, status and clear registers have the same layout, so BWMON_GLOBAL_IRQ_ENABLE_ENABLE define can be used also for clearing IRQ, instead of hard-coded value. Cc: Rajendra Nayak <quic_rjendra@quicinc.com> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- drivers/soc/qcom/icc-bwmon.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)