Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38221 )
Change subject: soc/amd/picasso: Drop forked copy of SMBus source ......................................................................
Patch Set 4: Code-Review-2
Patch Set 4:
Patch Set 4:
Patch Set 3: Code-Review+2
It's not used at all?
What am I missing and why is this being dropped? Is there a switch to common code somewhere that was omitted from the patch stack?
That copy-paste should not have been approved in the first place.
Well, that's a fair enough argument, but assuming the preferred way to go is to use common code instead, then let's create the common code first before dropping PCO functionality.
And no, I don't plan to fix it for you as none of soc/amd/picasso is build-tested on master.
I'm not sure what you mean by "fix". If it's keeping PCO in sync with ST until we have common code, then I have no problem doing it myself.
As far as "none of soc/amd/picasso is build-tested on master", that's not a valid argument. By that same logic, you could justify removing the entire picasso directory right now. There can't/won't be a mainboard to test until the bootblock/romstage changes are updated and land. I presume you feel the Family 17h development is going too slow, but I'm sorry that you have very limited visibility into the other design, work, and negotiations that are actually happening.