Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44000 )
Change subject: soc/intel/jasperlake: Add new IGD Device ID ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG@7 PS2, Line 7: new don't need this word here, its enough to say Add IGD device id
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG@9 PS2, Line 9: new same
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG@9 PS2, Line 9: jasperlake Jasperlake
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG@10 PS2, Line 10: Reference is taken from jasperlake EDS volume 1 chapter 13.3 possible to give doc number is enough than chapter
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG@12 PS2, Line 12: BRANCH=None : BUG=None don't need
https://review.coreboot.org/c/coreboot/+/44000/2//COMMIT_MSG@14 PS2, Line 14: jasperlake Jasperlake