Re: [coreboot] Missing ACPI ASL code for Denverton GPIO controller

[Julien Viard de Galbert]

> Le 26 mars 2018 à 21:24, Sumo  a écrit :
>
> Hi all,

Hi Sumo,
> We have a kernel patch which adds pinctrl/GPIO support for Intel Denverton SoC (https://patchwork.kernel.org/patch/9879473/ https://patchwork.kernel.org/patch/9879473/) to make possible to access the GPIO from user space (via sysfs, i.e. /sys/kernel/debug/pinctrl/), however this patch is expects a "INTC3000" GPIO controller definition in the ACPI tables.
> I want to add/implement the INTC3000 in the ASL code, but since I don´t want to reinvent the wheel I´m asking if such ASL code is already available somewhere (at least in the Intel Pine Lake CRB there´s no such reference to INTC3000 in the ACPI tables).

I have a set of patches that still need some work (https://review.coreboot.org/c/coreboot/+/24928) that enable some shared code for GPIO; following I also have more patches to use the ACPI code from common block too (not published yet). I didn’t check but probably apollolake or cannonlake has the ACPI code for GPIO (using common block).
If you are interested I can probably work on adding the ACPI implementation to Gerrit sooner.

Best Regards,

Julien
> Thanks,
> Sumo
> -- 
> coreboot mailing list: [e-mail]
> https://mail.coreboot.org/mailman/listinfo/coreboot

Missing ACPI ASL code for Denverton GPIO controller

[Sumo]

Hi Julien,

Yes, I'm interested. But no need to hurry up, do it in your own time. ;)

Thanks,
Sumo

2018-03-27 6:54 GMT-03:00 Julien Viard de Galbert <
[e-mail]>:
> Le 26 mars 2018 à 21:24, Sumo  a écrit :
>
> Hi all,
>
> Hi Sumo,
>
> We have a kernel patch which adds pinctrl/GPIO support for Intel Denverton
> SoC (https://patchwork.kernel.org/patch/9879473/) to make possible to
> access the GPIO from user space (via sysfs, i.e.
> /sys/kernel/debug/pinctrl/), however this patch is expects a "INTC3000"
> GPIO controller definition in the ACPI tables.
> I want to add/implement the INTC3000 in the ASL code, but since I don´t
> want to reinvent the wheel I´m asking if such ASL code is already available
> somewhere (at least in the Intel Pine Lake CRB there´s no such reference to
> INTC3000 in the ACPI tables).
>
> I have a set of patches that still need some work (
> https://review.coreboot.org/c/coreboot/+/24928) that enable some shared
> code for GPIO; following I also have more patches to use the ACPI code from
> common block too (not published yet). I didn’t check but probably
> apollolake or cannonlake has the ACPI code for GPIO (using common block).
> If you are interested I can probably work on adding the ACPI
> implementation to Gerrit sooner.
>
> Best Regards,
>
> Julien
>
> Thanks,
> Sumo
> --
> coreboot mailing list: [e-mail]
> https://mail.coreboot.org/mailman/listinfo/coreboot