Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35643 )
Change subject: util/intelp2m: add Intel Pad to Macro util ......................................................................
Patch Set 11:
- inteltool
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.
- C-syntax.
Something similar already exists https://review.coreboot.org/c/coreboot/+/25145