On Thu, Feb 29, 2024 at 02:52:40PM -0600, Bjorn Helgaas wrote: > On Thu, Feb 29, 2024 at 02:10:21PM +0100, Johan Hovold wrote: > > It also depends on the severity of the issue and to some extent the > > complexity of the fix. These binding fixes are certainly low risk. > > :) > > IIUC we're talking about: > > arm64: dts: qcom: sc8280xp: add missing PCIe minimum OPP > dt-bindings: PCI: qcom: Allow 'required-opps' Right. > These don't look like a regression fix (correct me if I'm wrong), and > I can't tell whether they fix a user-visible problem, since > sc8280xp.dtsi does already contain 'required-opps' for ufs_mem_hc, > usb_0, and usb_1, which are mentioned in the commit log as covering up > the issue. The issue has been there since PCIe support was added for this platform and does not cause any issues until the USB and UFS controllers are runtime suspended. When that happens nothing is currently making sure that we have enough power to run PCIe at gen3 speeds, something which can potentially result in system instability (e.g. resets). > If these patches wait until v6.9, what badness ensues? We'd have a few more weeks where users enabling runtime PM for USB on the X13s could hit this before we can get the fix backported to stable. I could have put some more details in the commit message for the DT patch, but I did not think that amending the PCIe binding would be controversial. (I guess we can also take the DT fix without waiting for the binding update as it has been acked by a DT maintainer even if that would result in some DT checker warnings until things are aligned again.) Let me know what you decide regarding getting the whole series into 6.8, and then I can spend some more time on rewording, splitting and rebasing this series if needed. Johan
