Alexander Couzens has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41390 )
Change subject: mb/lenovo/x230: Fix issues detected after 6b95507e is merged ......................................................................
Patch Set 11: -Code-Review
I can only accept to split "disabling superspeed capabilities" stuff into a separate commit. It is not my fault that my commit trying to introducing x230s (CB:41168) got merged semi-finished.
I would rather completely revert CB:41168 and commit it again in a more elegant way, with "disabling superspeed capabilities" splitted out, as you wish.
Sure, you can revert the original commit, but I don't think it's necessary.
I don't think who's fault is important here. The x230s is part of the tree and you like to improve it. Great! Just imagine the original x230s commit came from me. Why do you want to improve everything in one commit?
Usually smaller commits are much faster reviewed and most times people don't overseen stuff. If you like to have only one commit (except the disable superseed), please rework the commit message, since it doesn't really describe what you do. Only the reason what lead to the commit.
(removed -1)