Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36029 )
Change subject: drivers/i2c/rt1011: Add a driver for RT1011 ......................................................................
Patch Set 5:
(4 comments)
If you have not done yet, please run the newly created files through clang-format.
https://review.coreboot.org/c/coreboot/+/36029/5//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36029/5//COMMIT_MSG@11 PS5, Line 11: setup Verb is spelled with a space: set up.
https://review.coreboot.org/c/coreboot/+/36029/5//COMMIT_MSG@9 PS5, Line 9: Set calibration data in device property for RT1011 when config : CHROMEOS_DSM_CALIB is turned on. : Kernel rt1011 codec driver will use these device properties to setup : codec accordingly. Please use the full text width (75 characters) and separate paragraphs by a blank line.
https://review.coreboot.org/c/coreboot/+/36029/5//COMMIT_MSG@13 PS5, Line 13: What datasheet (name, revision) did you use? Is it public?
https://review.coreboot.org/c/coreboot/+/36029/5/src/drivers/i2c/rt1011/rt10... File src/drivers/i2c/rt1011/rt1011.c:
https://review.coreboot.org/c/coreboot/+/36029/5/src/drivers/i2c/rt1011/rt10... PS5, Line 69: printk(BIOS_ERR, "cannot get dsm_calib from VPD\n"); Error messages should be more elaborate.