Re: [PATCH] HPE BMC GXP SUPPORT
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
Date: 2022-02-04 12:31:57
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, Feb 04, 2022 at 04:18:24AM -0800, Joe Perches wrote:
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))
return IRQ_NONE; 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;
-- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!