Hello Pankaj,

I reviewed most of the patches and posted my comments. Honestly, it is very hard to review these large commits.

It would have been very helpful if these were smaller incremental commits.

-Varun

From: Pankaj Gupta <pankaj.gupta@nxp.com>
Sent: Tuesday, March 31, 2020 11:09 AM
To: Varun Wadekar <vwadekar@nvidia.com>; Joanna Farley <Joanna.Farley@arm.com>
Cc: tf-a@lists.trustedfirmware.org
Subject: Re: [TF-A] [EXT] Patch-set review request: New NXP Platform LX2120ARDB support on TFA

External email: Use caution opening links or attachments

Hi Varun,

Thanks for the initial review comments.

Yes, I looking for review comments on entire patch series.

Regards

Pankaj

From: Varun Wadekar

Sent: Tuesday, 31 March, 22:55

Subject: RE: [TF-A] [EXT] Patch-set review request: New NXP Platform LX2120ARDB support on TFA

To: Joanna Farley, Pankaj Gupta

Cc: tf-a@lists.trustedfirmware.org

Caution: EXT Email

Hello Pankaj,

Hope you are doing well.

The initial email point to one change, but I see that as the tip of a patch series. I reviewed https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/3370

and left some comments, but did not review the complete patch series.

Are you requesting a review of the entire patch series?

-Varun

From: TF-A <tf-a-bounces@lists.trustedfirmware.org> On Behalf Of Joanna Farley via TF-A

Sent: Tuesday, March 31, 2020 9:37 AM

To: Pankaj Gupta <pankaj.gupta@nxp.com>; tf-a@lists.trustedfirmware.org

Subject: Re: [TF-A] [EXT] Patch-set review request: New NXP Platform LX2120ARDB support on TFA

External email: Use caution opening links or attachments

Hi Pankaj,

We will try, one of the arm team has done some +1 reviews and I have just kicked of a CI+2 run on the top of the patch stack. As you can imagine the Arm team is pressed for time the closer to the freeze date.

Other TF-A contributors you all have +1 rights so if folks have time assistance with further +1 reviews would be appreciated as that would help in accelerating confidence is getting +2 and merging.

Traditionally the project has relied on Arm folks but for some time now all contributors have had +1 rights on each other patches and the new project maintenance proposal is set up to take us in the direction of enabling all contributors helping each other.

Thanks

Joanna  

From: TF-A <tf-a-bounces@lists.trustedfirmware.org> on behalf of Pankaj Gupta via TF-A <tf-a@lists.trustedfirmware.org>

Reply to: Pankaj Gupta <pankaj.gupta@nxp.com>

Date: Monday, 30 March 2020 at 20:48

To: "tf-a@lists.trustedfirmware.org" <tf-a@lists.trustedfirmware.org>

Subject: Re: [TF-A] [EXT] Patch-set review request: New NXP Platform LX2120ARDB support on TFA

Please find the link to the review request.

https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/3370

Regards

Pankaj

From: Pankaj Gupta via TF-A

Sent: Monday, 30 March, 22:57

Subject: [EXT] [TF-A] Patch-set review request: New NXP Platform LX2120ARDB support on TFA

To: tf-a@lists.trustedfirmware.org

Caution: EXT Email

Hi All,

Please pick this review request so that code changes can be merged before code freeze.

Thanks.

Regards

Pankaj

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

This email message is for the sole use of the intended recipient(s) and may contain confidential information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.