Change in coreboot[master]: util/inteltool: Clean up PCI assignment

[Felix Singer (Code Review)]

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33422 )

Change subject: util/inteltool: Clean up PCI assignment
......................................................................

Patch Set 14:

(5 comments)

https://review.coreboot.org/c/coreboot/+/33422/13/util/inteltool/inteltool.c 
File util/inteltool/inteltool.c:

https://review.coreboot.org/c/coreboot/+/33422/13/util/inteltool/inteltool.c... 
PS13, Line 716: } else {
              : 				fprintf(stderr, "Multiple devices with class ID"
              : 					" 0x0601, using %02x%02x:%02x.%02x\n",
              : 					sb->domain, sb->bus, sb->dev, sb->func);
              : 				break;
> Why drop this?

Done

https://review.coreboot.org/c/coreboot/+/33422/13/util/inteltool/inteltool.c... 
PS13, Line 709: 	for (dev = pacc->devices; dev; dev = dev->next) {
> Please only scan bus 0, e.g. […]

Done

https://review.coreboot.org/c/coreboot/+/33422/13/util/inteltool/inteltool.c... 
PS13, Line 712: 		if (newdev) {
> less indentation: […]

Done

https://review.coreboot.org/c/coreboot/+/33422/13/util/inteltool/inteltool.c... 
PS13, Line 720: 				case PCI_CLASS_BRIDGE_HOST:
> Don't indent `case :`

Done

https://review.coreboot.org/c/coreboot/+/33422/13/util/inteltool/inteltool.c... 
PS13, Line 732: 				default:
> free `newdev`?

Done