[coreboot-gerrit] Change in coreboot[master]: util/inteltool: gpio: initialize register size variables

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39514 )

Change subject: util/inteltool: gpio: initialize register size variables
......................................................................

Patch Set 6: Code-Review-1

(2 comments)

What does this fix? Isn't it just hiding potential bugs?

https://review.coreboot.org/c/coreboot/+/39514/6//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/39514/6//COMMIT_MSG@9 
PS6, Line 9: Initialize register size variables to prevent segfaults.
If the program would read those variables on a broken path (only chance
for a segfault), the only thing we achieve with these initializations is
hiding bugs.

https://review.coreboot.org/c/coreboot/+/39514/6/util/inteltool/gpio_groups.... 
File util/inteltool/gpio_groups.c:

https://review.coreboot.org/c/coreboot/+/39514/6/util/inteltool/gpio_groups.... 
PS6, Line 185: 	communities = get_gpio_communities(sb, &community_count, &pad_stepping);
why initialize one arg but not the other?