In the case of x11, I took only gpio.h + few changes in parser.go :)
I don't have this motherboard so that I can use the inteltool
As I said, it is very convenient to have the macto generator, as a separate
utility ...

Point for you but I have the fear that the parser will break because inteltool output changes (see https://review.coreboot.org/c/coreboot/+/35189 for example; it does not break your parser completely but it outputs a bunch of wrong stuff additionally).

Next problem: what happens, when inteltool output changes and your parser gets adapted? Then you would need to use the version of your parser from the time the inteltool.log has been generated. That will get very confusing and complicated I guess.

Something similar already exists https://review.coreboot.org/c/coreboot/+/25145

View Change

To view, visit change 35643. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If3e3b523c4f63dc2f91e9ccd16934e3a1b6e21fa
Gerrit-Change-Number: 35643
Gerrit-PatchSet: 11
Gerrit-Owner: Maxim Polyakov <max.senia.poliak@gmail.com>
Gerrit-Reviewer: Alexander Couzens <lynxis@fe80.eu>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak@gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Christian Walter <christian.walter@9elements.com>
Gerrit-CC: Maxim Polyakov <m.poliakov@yahoo.com>
Gerrit-CC: Michael Niewöhner
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-Comment-Date: Tue, 01 Oct 2019 17:00:22 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment