On 30/10/2022 15:23, Bjorn Helgaas wrote:
On Sun, Oct 30, 2022 at 12:13:05AM +0300, Dmitry Baryshkov wrote:
SM8350 is one of the recent Qualcomm platforms which lacks PCIe support.
I guess the "platform" (the hardware) has PCIe, but the current driver
doesn't support it?
Yes.
Use sm8450 PHY tables to add support for the PCIe hosts on Qualcomm SM8350 platform.
Note: the PCIe0 table is based on the v2.1 tables, so it might work
incorrectly on earlier platforms.
I'm not sure what this means in terms of applying this series. It
sounds like "this series might break earlier platforms". That
wouldn't be good, so I assume it's more subtle than that.
I guess "v2.1 tables" refers to "PHY config tables"? "PCIe0" appears
mostly in [6/7] as a 1-lane Gen3 host. "v2.1" and "v2_1" don't appear
at all. I can't quite figure out what symbols in the patches these
refer to.
Oh, excuse me. There were several revisions of sm8350 SoC (1.0, 2.0,
2.1), with slight differences in the PHY programming. Usually we support
only the latest version, which is the version going into
mass-production. I'll expand the description in the v2.
--
With best wishes
Dmitry
