Peter Lemenkov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38707 )
Change subject: mb/lenovo/*/devicetree: Trim DID entries to actual size ......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38707/5//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38707/5//COMMIT_MSG@7 PS5, Line 7: DID entries
The real question is: do they make any sense, or can they be removed?
Looks like they can be replaced with some default values at least. I've found just a few boards which use different DID list.
On the other hand it's a part of board technical specification so maybe it's better to declare it in a devicetree explicitly anyway.
I believe it's better to discuss it separately (in a different commit).