Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48122 )
Change subject: mb/intel/ehlcrb: Remove IPU & MIPI related support from EHL CRB ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/48122/2/src/mainboard/intel/elkhart... File src/mainboard/intel/elkhartlake_crb/variants/ehlcrb/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/48122/2/src/mainboard/intel/elkhart... PS2, Line 31: end Would you mind to add a comment here which describes this PCI device? Just like the other already do.
https://review.coreboot.org/c/coreboot/+/48122/2/src/mainboard/intel/elkhart... PS2, Line 167: I2C A I2C controller number would be nice here. Should be I2C #4 and I2C #5 if I am not mistaken.