<div dir="ltr">Ok, so it sounds that a default value derived from HW strap for LPC/eSPI (see discussion on how to access the registers correctly) + sysfs seems to be ok?<div><br></div><div>Oskar.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jul 28, 2019 at 10:20 PM Benjamin Herrenschmidt <<a href="mailto:benh@kernel.crashing.org">benh@kernel.crashing.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, 2019-07-29 at 08:51 +0800, Jeremy Kerr wrote:<br>
> Hi Oskar,<br>
> <br>
> > For LPC, the polarity should be set to 1, for eSPI the default of 0 is<br>
> > correct.<br>
> <br>
> Would there ever be a case where different SIRQs need different<br>
> polarities? If not, we may want this to be global, rather than for each<br>
> device (the VUART being one...)<br>
<br>
Leave it per device. The way to configure this is completely device<br>
dependent. In fact for most devices it can only be done by the host via<br>
SIO.<br>
<br>
Cheers,<br>
Ben.<br>
<br>
<br>
</blockquote></div>