Re: [PATCH] HPE BMC GXP SUPPORT
From: Joe Perches <joe@perches.com>
Date: 2022-02-04 14:02:17
Also in:
dri-devel, linux-devicetree, linux-gpio, linux-i2c, linux-pm, linux-pwm, linux-serial, linux-spi, linux-usb, linux-watchdog, lkml
Possibly related (same subject, not in this thread)
- 2022-02-04 · Re: [PATCH] HPE BMC GXP SUPPORT · Krzysztof Kozlowski <hidden>
- 2022-02-03 · Re: [PATCH] HPE BMC GXP SUPPORT · "Verdun, Jean-Marie" <verdun@hpe.com>
- 2022-02-03 · Re: [PATCH] HPE BMC GXP SUPPORT · Krzysztof Kozlowski <hidden>
On Fri, 2022-02-04 at 12:31 +0000, Russell King (Oracle) wrote:
On Fri, Feb 04, 2022 at 04:18:24AM -0800, Joe Perches wrote:quoted
On Fri, 2022-02-04 at 12:05 +0000, Russell King (Oracle) wrote:quoted
On Wed, Feb 02, 2022 at 10:52:50AM -0600, nick.hawkins@hpe.com wrote:quoted
+ if (readb_relaxed(timer->control) & MASK_TCS_TC) { + writeb_relaxed(MASK_TCS_TC, timer->control); + + event_handler = READ_ONCE(timer->evt.event_handler); + if (event_handler) + event_handler(&timer->evt); + return IRQ_HANDLED; + } else { + return IRQ_NONE; + } +}It's also less indented code and perhaps clearer to reverse the test if (!readb_relaxed(timer->control) & MASK_TCS_TC)This will need to be: if (!(readb_relaxed(timer->control) & MASK_TCS_TC))
right, thanks.