Attention is currently required from: Felix Singer, Alan Green, Angel Pons, Michael Niewöhner. Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/57810 )
Change subject: [RFC] [WIP] ft2232_spi: reintroduce generic GPIOL control ......................................................................
Patch Set 6:
(1 comment)
Patchset:
PS6:
Oh, I just had this idea. Wdyt? […]
Hmmm, actually, it took me too long to understand what you mean ;)
It's more explicit, but seems inconvenient, both for the user who has to encode it and the program that has to decode it.
Let's compare these notations directly
-p ft2232_spi:gpiol0=C,gpiol3=H
-p ft2232_spi:gpiol=0C3H
The first is more typing effort, but easier to read. The other way around for the second. In both cases it doesn't look very nice, as the user has to specify everything in a single string without spaces. *shrug*
I prefer the first as it should be obvious to any reader (at least for H/L). Especially somebody used to the old `gpiol` option could get confused by a notation that is incompatible.