Thanks a lot Manish. I have addressed the comments and uploaded the coverity patch. I hope we can conclude the review and merge the changes.
________________________________ From: Manish Badarkhe Manish.Badarkhe@arm.com Sent: Friday, December 6, 2024 7:11 PM To: tf-a@lists.trustedfirmware.org tf-a@lists.trustedfirmware.org; Amarinder Singh Sethi (QUIC) quic_assethi@quicinc.com Subject: Re: Unable to merge gerrit#33536
WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros.
Hi Amarinder
After rebasing your patch, the CIs look good. However, I’ve added a comment on Gerrit suggesting coverage for this platform’s build and run tests in the TF-A CI framework. Please take a look.
Thanks, Manish Badarkhe ________________________________ From: Amarinder Singh Sethi (QUIC) via TF-A tf-a@lists.trustedfirmware.org Sent: 06 December 2024 12:30 To: tf-a@lists.trustedfirmware.org tf-a@lists.trustedfirmware.org Subject: [TF-A] Unable to merge gerrit#33536
Hi,
I recently uploaded the patch (https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/33536) to add qti platform support for qcs615. The patch is fully isolated and doesn't touch any common code and doesn't get included in any other platform compilation.
On enabling basic CI, I see some failures which are unrelated to my patch.
I wonder what the next step should be. If somebody can help me, it will be greatly appreciated.
Thanks
Amarinder