Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38631 )
Change subject: util/inteltool: Split GPIO community switch-case into its own function ......................................................................
Patch Set 4:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38631/2/util/inteltool/gpio_groups.... File util/inteltool/gpio_groups.c:
https://review.coreboot.org/c/coreboot/+/38631/2/util/inteltool/gpio_groups.... PS2, Line 99: get_gpio_communitys
Don't forget! get_gpio_communities
Done
https://review.coreboot.org/c/coreboot/+/38631/2/util/inteltool/gpio_groups.... PS2, Line 180: get_gpio_communitys
here too
Done
https://review.coreboot.org/c/coreboot/+/38631/2/util/inteltool/inteltool.h File util/inteltool/inteltool.h:
https://review.coreboot.org/c/coreboot/+/38631/2/util/inteltool/inteltool.h@... PS2, Line 396: get_gpio_communitys
communities
Done