Hi Joel, > A feature was added to the aspeed vuart driver to configure the vuart > interrupt (sirq) polarity according to the LPC/eSPI strapping register. > > Systems that depend on a active low behaviour (sirq_polarity set to 0) > such as OpenPower boxes also use LPC, so this relationship does not > hold. > > The property was added for a Tyan S7106 system which is not supported > in the kernel tree. Should this or other systems wish to use this > feature of the driver they should add it to the machine specific device > tree. > > Fixes: c791fc76bc72 ("arm: dts: aspeed: Add vuart aspeed,sirq-polarity-sense...") > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Joel Stanley <joel@xxxxxxxxx> LGTM. I've tested this on the s2600st, which is strapped for eSPI. All good there too, as expected. Tested-by: Jeremy Kerr <jk@xxxxxxxxxx> and/or: Reviewed-by: Jeremy Kerr <jk@xxxxxxxxxx> Cheers, Jeremy