Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40256 )
Change subject: src/ec: Add missing "set_resources = noop_set_resources" ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/40256/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40256/1//COMMIT_MSG@7 PS1, Line 7: missing IMO, not "missing" but "warranted". It's not that it's always missing a no-op, no-op could also be an error. But in this case (read_resources being a no-op too) we can be rather sure that it's ok.