Attention is currently required from: Angel Pons, Anastasia Klimchuk, Peter Marheine.

View Change

2 comments:

• Commit Message:

  • Patch Set #2, Line 7: interal

    typo: inter*n*al

    Done

• File board_enable.c:

  • Patch Set #2, Line 1553:

    /* First, look for a known LPC bridge */
    	dev = pcidev_find_vendorclass(0x8086, 0x0601); /* ISA bridge */
    	if (!dev) {
    		msg_perr("\nERROR: No known Intel LPC bridge found.\n");
    		return -1;
    	}
    	/* Is this device in our list? */
    	for (i = 0; intel_ich_gpio_table[i].id; i++)
    		if (dev->device_id == intel_ich_gpio_table[i].id)
    			break;
    
    	if (!intel_ich_gpio_table[i].id) {
    		msg_perr("\nERROR: No known Intel LPC bridge found.\n");
    		return -1;
    	}
    

    This part belongs to a separate commit, it has nothing to do with renaming and moving `pci_dev_find_ […]

    https://review.coreboot.org/c/flashrom/+/62405

To view, visit change 59277. To unsubscribe, or for help writing mail filters, visit settings.