Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38443 )
Change subject: mb/google/kahlee/treeya: Update STAPM parameters for Treeya ......................................................................
Patch Set 4: Verified+1 Code-Review+2
(2 comments)
https://review.coreboot.org/c/coreboot/+/38443/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38443/2//COMMIT_MSG@10 PS2, Line 10: it meet
I am not really sure, how such a reliable check would look like. […]
This is really trivial. The information is there, it's readable, it's understandable. So what if the line isn't QUITE as long as it could be. It all needs to go onto 2 lines anyway.
I could understand a complaint if text that would fit on 2 lines were wrapped to fit on 3.
This though, seems like a complaint in search of a problem. How nitpicky can we get?
https://review.coreboot.org/c/coreboot/+/38443/2//COMMIT_MSG@10 PS2, Line 10: it meet
I am not really sure, how such a reliable check would look like. […]
This is really trivial. The information is there, it's readable, it's understandable. So what if the line isn't QUITE as long as it could be. It all needs to go onto 2 lines anyway.
I could understand a complaint if text that would fit on 2 lines were wrapped to fit on 3.
This though, seems like a complaint in search of a problem. How nitpicky can we get?