[coreboot-gerrit] Change in coreboot[master]: mb/purism/librem_skl: use SOC_INTEL_COMMON_BLOCK_HDA_VERB

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34531 )

Change subject: mb/purism/librem_skl: use SOC_INTEL_COMMON_BLOCK_HDA_VERB
......................................................................

Patch Set 1: Code-Review+1

(2 comments)

https://review.coreboot.org/c/coreboot/+/34531/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34531/1//COMMIT_MSG@14 
PS1, Line 14: 
Could you please add the Coverity error line as a *Found-by* tag? Jacob, do you have it?

Tested how?

https://review.coreboot.org/c/coreboot/+/34531/1//COMMIT_MSG@16 
PS1, Line 16: Change-Id: I2fd5363aad027f215f93964bc6a85f00fea86c88
Don’t the hooks add the Signed-off-by line to the end?