Kyösti Mälkki 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:
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.
With PSP doing raminit, SMBus on amd/picasso may remain unused for a long time. Not sure if your designs have some other SMBus components that use it, and even if they did, existing code only has reference to PCI_DEVICE_ID_AMD_CZ_SMBUS. Is that the correct PCI ID? If not, there is no functionality dropped with the source removal.
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.
Fix, as in introduce new SOC_AMD_COMMON_SMBUS kconfig and add/replace necessary PCI IDs and move the ST files.
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.
I would need to rebase the entire Mandolin CRB board and the dependencies to get a build-tested result for the common SMBUS approach. It's just less hassle for you to incorporate that on yout patcht-train once/if/where you want it.