Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41159 )
Change subject: mainboard: Add HP EliteBook 2560p ......................................................................
Patch Set 4:
(15 comments)
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... File Documentation/mainboard/hp/2560p.md:
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 8: secondhand second-hand
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 37: two empty lines
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 64: integrated Integrated
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 66: audio Audio
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 70: bluetooth BlueTooth
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 76: keyboard Keyboard
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 78: internal Internal
https://review.coreboot.org/c/coreboot/+/41159/4/Documentation/mainboard/hp/... PS4, Line 79: using Using
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... File src/mainboard/hp/snb_ivb_laptops/variants/2560p/early_init.c:
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 2: /* This file is part of the coreboot project. */ Remove this
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... File src/mainboard/hp/snb_ivb_laptops/variants/2560p/gma-mainboard.ads:
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 2: -- This file is part of the coreboot project. Remove this
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... File src/mainboard/hp/snb_ivb_laptops/variants/2560p/gpio.c:
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 2: /* This file is part of the coreboot project. */ Remove this
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... File src/mainboard/hp/snb_ivb_laptops/variants/2560p/hda_verb.c:
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 2: /* This file is part of the coreboot project. */ Remove this
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... File src/mainboard/hp/snb_ivb_laptops/variants/2560p/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 2: # This file is part of the coreboot project. Remove this
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 22: # HDD(0), ODD(1), eSATA(4) : register "sata_port_map" = "0x33" This is not consistent. That port map enables ports 0, 1, 4 and 5
https://review.coreboot.org/c/coreboot/+/41159/4/src/mainboard/hp/snb_ivb_la... PS4, Line 25: device pci 1c.0 on end # PCIe Port #1 Does this need to be enabled?