On 5/28/2026 4:43 AM, Gekko wrote:
Hi Xilin,
Thank you for the response.
I appreciate the offer to take over your patch set to move things
forward. However, I feel I must respectfully decline. This is mainly due
to my lack of experience with both upstream kernel development on this
hardware and the patch management process itself. It would be
irresponsible of me to take on something that I'm not knowledgeable
enough to ensure successful completion.
I'm happy to work with you to help in any other way that I can such as
testing patches, but I think you should maintain ownership of the patch-
set and submission process, at least for now.
One of the reasons I withdrew my own patch-set is that from what I can
see your series covers everything I was trying to achieve, and does it
correctly :-)
The only one that isn't covered is the patch for early TCS
initialisation that I'm discussing with Konrad which may be unnecessary
anyway.
Graham
Hi Graham,
Thanks, I understand your point.
I appreciate your offer to help with testing. I will do my best to move
this series forward once I get the current work off my plate.
Thanks again for your help.
On 24/05/2026 13:58, Xilin Wu wrote:
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]>
