Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38259 )
Change subject: mb/intel/jasperlake_rvp: Comment out sd card detect gpio ......................................................................
Patch Set 4:
(4 comments)
https://review.coreboot.org/c/coreboot/+/38259/4//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38259/4//COMMIT_MSG@7 PS4, Line 7: Commen I'm asking that we remove the commented-out code before merge, so how about we change this to something like "Removed declaration of sdcard_cd_gpio" ?
https://review.coreboot.org/c/coreboot/+/38259/4//COMMIT_MSG@11 PS4, Line 11: TODO: We still need to update gpio definition for Jasper Lake soc. : We will update correct gpio for sd card detect once we have Jasper Lake : soc gpio patch. : Please remove TODO and create a bug to track this work and add the bug number to the comments in this CL for future reference. Thanks.
https://review.coreboot.org/c/coreboot/+/38259/4/src/mainboard/intel/jasperl... File src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/38259/4/src/mainboard/intel/jasperl... PS4, Line 162: # TODO: Replace with correct GPIO definition once gpio_soc_defs.h is Please remove TODO and create a bug for tracking that work and add the bug number to the CL comments for future tracking purposes. Thanks.
https://review.coreboot.org/c/coreboot/+/38259/4/src/mainboard/intel/jasperl... PS4, Line 164: # register "sdcard_cd_gpio" = "GPP_G5" Please remove commented-out code.