[coreboot-gerrit] Change in coreboot[master]: util/inteltool: powermgt: add code for dumping config registers

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39509 )

Change subject: util/inteltool: powermgt: add code for dumping config registers
......................................................................

Patch Set 4: Code-Review+1

(3 comments)

https://review.coreboot.org/c/coreboot/+/39509/4/util/inteltool/powermgt.c 
File util/inteltool/powermgt.c:

https://review.coreboot.org/c/coreboot/+/39509/4/util/inteltool/powermgt.c@6... 
PS4, Line 674: pm_cfg_registers_size
Are you initializing this anywhere?

https://review.coreboot.org/c/coreboot/+/39509/4/util/inteltool/powermgt.c@6... 
PS4, Line 680: 	printf("\n========== ACPI/PMC =========\n\n");
Why the rename?

https://review.coreboot.org/c/coreboot/+/39509/4/util/inteltool/powermgt.c@8... 
PS4, Line 871: 	for (i = 0; i < pm_cfg_registers_size; i++) {
Why not use a separate function?