<div dir="ltr"><div dir="ltr">Hey Joel<div><br></div><div>Thank you so much for the super fast review! So many things have changed a bit since I worked on this DTS originally (years ago) - thanks for pointing me to the updates! I'll make the changes as recommended.</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">> +&gpio {<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +       status = "okay";<br>
> +       gpio-line-names =<br>
> +       /*A0-A7*/       "","","IDLED_N","","","","","",<br>
> +       /*B0-B7*/       "","","","","","","","",<br>
> +       /*C0-C7*/       "","","","","ID_BUTTON","POST_COMPLETE","","",<br>
> +       /*D0-D7*/       "","","PS_PWROK","PLTRST_N","","","","",<br>
> +       /*E0-E7*/       "POWER_BUTTON","POWER_OUT","RESET_BUTTON","RESET_OUT",<br>
> +                                       "NMI_BUTTON","NMI_OUT","","HEARTBEAT_LED_N",<br>
<br>
We have a document that contains names for common BMC GPIOs:<br>
<br>
<a href="https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md" rel="noreferrer" target="_blank">https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md</a><br>
<br>
Ideally your device tree would use them, but given this is an old<br>
platform then I understand if you want to maintain compatibility with<br>
existing userspace.<br><br></blockquote><div>This is good to know! This is a "new" platform in the sense that we're finally upstreaming (even if we have been running it for 2 years on an older internal build). Also, this is going to be the basis for at least one additional TYAN board, so I'd like to get it "as right as possible" to avoid bad copy&paste. I was following the "standard" names from x86-power-control (<a href="https://github.com/openbmc/x86-power-control">https://github.com/openbmc/x86-power-control</a>) so I had already renamed some of TYAN's original net names. With that, I'd be happy to rename all of the signals to follow the OpenBMC standard.</div><div><br></div><div>Some questions on that, given that there are a few signals that don't have an obvious equivalent in the GPIO naming doc:</div><div><ul><li>Some LEDs and outputs are _N-ed, i.e. active low. Is there a good way to indicate that? This is important, e.g. for "ALERT".</li><li>There are some signals that are inputs but are not buttons, e.g. "PLTRST_N". Also, this particular example is active low.</li><li>There are a bunch of output signals that control muxes, e.g. BMC_PE_SMB_EN_1_N. Is there a naming convention for those?</li></ul><div>Does it make sense for each OpenBMC signal to also indicate the original net name from the schematics. I realize that not many people have access to that, but it would be good for those who do. I'd rather document "too much" than too little.</div></div><div><br></div><div>Thanks</div><div>Oskar.</div><div><br></div></div></div>