On 5/22/2026 6:13 PM, Graham O'Connor wrote:
Thank you to Konrad, Neil, and others for the very prompt and helpful reviews.
Based on the feedback received, I think it best to withdraw this series to
address the issues raised, most notobly:
1. The DP_TRAIN_LEVEL_MAX change (patch 3) is too broad - it affects all
devices using the Qualcomm MSM DP driver rather than being specific to
the RA620 bridge. A proper fix should be implemented at the bridge
level. This is going to take more investigation.
2. The display DT nodes (patch 5) should use the radxa,ra620 compatible
string per the existing Radxa upstream series from Xilin Wu, and the
lane ordering needs correcting.
3. The rpmh-rsc early return (patch 1) needs further review regarding
the implications of returning before full driver initialization.
4. DTS patch label dependency issue (sorry)
I'll integrate with the Radxa upstream effort, re-evaluate the other areas
and resubmit corrected patches addressing these concerns at a later date.
Thanks for the feedback.
Cheers
Graham
Hi Graham,
Thanks for working on the additional hardware bring-up for Q6A upstream
support. I'm really glad to see someone pushing this forward.
I actually posted a patch series [1] last month, and it is generally in
good shape. Unfortunately, I got busy with other work and haven't had a
chance to send v2 yet.
If it helps, please feel free to take over that series and fold your
additional patches into it. I think that would be a good way to move
things forward.
Also, I believe patches 10-12 in my series are no longer needed and can
be dropped.
Thank you again for the work you have done.
[1]
https://patchwork.kernel.org/project/linux-arm-msm/list/?series=1078242&state=%2A&archive=both
--
Best regards,
Xilin Wu <[email protected]>
