On Sun, Jan 26, 2025 at 12:59:52PM +0530, Manivannan Sadhasivam wrote:

On January 25, 2025 11:00:23 PM GMT+05:30, Dmitry Baryshkov [off-list ref] wrote:

quoted

On Sat, Jan 25, 2025 at 04:31:19AM +0100, Konrad Dybcio wrote:

quoted

From: Konrad Dybcio <redacted>

Add a new, common configuration for Gen4x4 V6 PHYs without an init
sequence.

The bootloader configures the hardware once and the OS retains that
configuration by using the NOCSR reset line (which doesn't drop
register state on assert) in place of the "full reset" one.

I know your opinion, but my 2c would still be for not depending on the
bootloader. I think that was the rule for ages for many possible
reasons.

But if Linux or other OS can trust the bootloader, then it makes perfect sense to rely on them. Obviously, the question here is that on which platforms this level of trust should be established. And the answer I got was starting from the compute platforms (atleast X1E).

Is there any way how those values can be lost that we still might want
to support ? The GDSC going to the OFF state? Some deep sleep state or a
power collapse? Actual suspend to RAM (instead of current S2Idle)?

So let's take it on an experimental basis and see how it goes? If at all any problem arises, we can always resort to in kernel sequences.

Sounds like a good proposal. Can possibly have a corresponding 'do not
merge' patch with actual init tables?

- Mani

quoted

quoted

Use this new configuration for X1P42100's Gen4x4 PHY.

Signed-off-by: Konrad Dybcio <redacted>
---
 drivers/phy/qualcomm/phy-qcom-qmp-pcie.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

மணிவண்ணன் சதாசிவம்

-- 
With best wishes
Dmitry

-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy