<div dir="ltr"><div dir="ltr">Hi Andrew and Jeremy<div><br></div><div>Thanks for both your input, that was helpful. I'm sorry it took me so long to get back to this. I hope it's in better shape now.</div><div><br></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Given it's ASPEED-specific I expect you should use a vendor prefix for the<br>
> property, e.g. aspeed,espi-enabled.<br></blockquote><div>That was a very good point.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> However, as I understand it you want to determine what polarity the SIRQ<br>
> should be regardless of which of eSPI or LPC are enabled, so I don't think<br>
> the property name should be an explicit statement about eSPI. Maybe<br>
> "aspeed,sirq-polarity-sense"?<br>
<br>
Yep, +1 on Andrew's comments here. This property isn't an indication on<br>
whether espi is enabled, but a method to detect it.<br></blockquote><div>I agree. I was so focused on functionality that I didn't look at this with a wider view.</div><div><br></div><div>I'll send v3 of the patch that contains appropriate changes. I'll also include further lists and individuals to get OKs as needed.</div><div><br></div><div>Thanks</div><div>Oskar.</div></div></div>