
<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">Le 8 févr. 2018 à 17:24, Aaron Durbin <<a href="mailto:adurbin@google.com" class="">adurbin@google.com</a>> a écrit :</div><br class="Apple-interchange-newline"><div class=""><div class="">On Thu, Feb 8, 2018 at 7:20 AM, Julien Viard de Galbert<br class=""><<a href="mailto:jviarddegalbert@online.net" class="">jviarddegalbert@online.net</a>> wrote:<br class=""><blockquote type="cite" class="">Hello all,<br class=""><br class="">First sorry for mailing direclty those of you who are on the coreboot<br class="">mailing list.<br class=""><br class="">I’m currently in the process of upstreaming the changes we have on<br class="">denverton.<br class="">On the ACPI I see a lot in common with the code available in<br class="">soc/intel/common/block/acpi.<br class="">However this depends on PMC which depends on GPIO. And the GPIO code<br class="">conflicts with the code already in denverton so I can’t enable it. (I’ve not<br class="">checked<br class="">In details about the PMC code yet).<br class=""><br class="">What would be the best way to port it ?<br class=""><br class="">1. Stick with what is working, duplicate some code but don’t break other<br class="">platforms.<br class="">2. Try to refactor the GPIO code and denverton code so they are compatible.<br class=""><br class="">For point 2 the thing is that I can test for other platforms and it will<br class="">really add more<br class="">work now. But if the longterm goal is to refactor all intel soc to mostly<br class="">use common/block<br class="">then this might be worth it.<br class=""><br class="">What are your recommendations?<br class=""><br class=""></blockquote><br class="">What are the actual changes in the gpio blocks compared to what's there?<br class=""></div></div></blockquote><div><br class=""></div><div>I’ve not looked into the datasheet of other chips to see the difference.</div><div>But the code is really different.</div><div>The GPIO code in denverton looks like it’s been imported from the EFI code.</div><div><br class=""></div><div>After looking again, it seams that the underlying hardware is similar (I checked</div><div>a few bits and they are at the same place).</div><div>However the ‘user' interface is really different: The denverton uses a configuration</div><div>structure based on enums and bitfields that is not present in common code.</div><div class=""><br class=""></div><blockquote type="cite" class=""><div class=""><div class=""><br class=""><blockquote type="cite" class="">Best Regards<br class=""><br class=""><br class="">--<br class="">Julien Viard de Galbert - <a href="mailto:jviarddegalbert@online.net" class="">jviarddegalbert@online.net</a><br class="">Online / Scaleway<br class="">Looking for an amazing job? Join us NOW ! <a href="https://careers.scaleway.com/" class="">https://careers.scaleway.com/</a><br class=""><br class=""><br class=""><br class=""><br class=""><br class="">--<br class="">coreboot mailing list: <a href="mailto:coreboot@coreboot.org" class="">coreboot@coreboot.org</a><br class=""><a href="https://mail.coreboot.org/mailman/listinfo/coreboot" class="">https://mail.coreboot.org/mailman/listinfo/coreboot</a><br class=""></blockquote></div></div></blockquote></div><br class=""><div class="">

<div dir="auto" style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;">--</div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Julien Viard de Galbert - <a href="mailto:jviarddegalbert@online.net" class="">jviarddegalbert@online.net</a></div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Online / Scaleway<br class="">Looking for an amazing job? Join us NOW ! <a href="https://careers.scaleway.com/" class="">https://careers.scaleway.com/</a></div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br class=""></div><br class="Apple-interchange-newline"></div><br class="Apple-interchange-newline">

</div>

<br class=""></body></html>