Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31083 )
Change subject: cheza: configure gpios and gcc clocks for sound ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/#/c/31083/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/31083/1//COMMIT_MSG@2 PS1, Line 2: RajendraBabu Add a space between the names?
https://review.coreboot.org/#/c/31083/1//COMMIT_MSG@7 PS1, Line 7: cheza: configure gpios and gcc clocks for sound Use the SoC name as prefix?
https://review.coreboot.org/#/c/31083/1//COMMIT_MSG@9 PS1, Line 9: Configure mi2s, codec reset gpios and enable : GCC LPASS clocks required for audio support. Please use the fully allowed text width.
https://review.coreboot.org/#/c/31083/1//COMMIT_MSG@11 PS1, Line 11: Tested how?
https://review.coreboot.org/#/c/31083/1//COMMIT_MSG@14 PS1, Line 14: kumar Capital K?