On Sat, Feb 22, 2025 at 04:53:47PM +0900, Krzysztof Wilczyński wrote:
> Hello,
>
> > > > This patch series add separate MHI host configuration to enable
> > > > only IP_SW channel for SA8775P target.
> > > >
> > > > And also update the proper device id for SA8775P endpoint.
> > >
> > > Applied to epf-mhi, thank you!
> >
> > I see "[2/2] PCI: epf-mhi: Update device id for SA8775P" on
> > pci/epf-mhi, but I don't see patch [1/2]. Where did that go?
> > They seem related, so I would think we'd want to merge them together.
>
> I asked Mani whether he would prefer for me to take the entire series via
> the PCI tree, but he said that the first patch should go via the MHI tree.
>
> So, I assume Mani will take it, then. Mani, thoughts?
>
Yeah. Patch 1/2 should go via MHI tree as it is purely an MHI controller patch
(a.k.a host side patch). Whereas patch 2/2 is an endpoint side patch.
> > Also, in [2/2], I guess the .deviceid change is known not to break
> > anything that's already in the field?
>
> Mrinmay, are you expecting any issues with this change?
>
> Per the c670e29f5bfe ("PCI: epf-mhi: Add support for SA8775P SoC"):
>
> Add support for Qualcomm Snapdragon SA8775P SoC to the EPF driver.
> SA8775P is currently reusing the PID 0x0306 (the default one hardcoded
> in the config space header) as the unique PID is not yet allocated.
>
> I think, we should be fine. But would be best to confirm that.
>
SA8775P with upstream endpoint is not used by anyone outside Qcom till now. So
changing the PID is safe.
- Mani
--
மணிவண்ணன் சதாசிவம்
