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