Understood. Thanks.
-----Original Message----- From: Soby Mathew Soby.Mathew@arm.com Sent: Friday, September 6, 2019 1:56 AM To: Varun Wadekar vwadekar@nvidia.com; Joanna Farley Joanna.Farley@arm.com Cc: tf-a@lists.trustedfirmware.org Subject: Re: Merge Tegra platform changes
On 05/09/2019 17:08, Varun Wadekar wrote:
Thanks Soby.
I suggest you to push only a subset of the patches initially for review (say around 50)
Just so that I understand, you suggest pushing individual patches to gerrit for review, instead of pushing a branch and then reviewing the merge commit (?)
Hi Varun, Please push a topic branch to gerrit, but restrict the number of patches for review to < 50 on that branch. The branch should compile and all the new files added should be part of the build.
git push origin <branch-name>:refs/for/integration/<branch-name>
Once that branch is merged, you can sent the next batch of patches for review.
Best Regards Soby Mathew 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. -----------------------------------------------------------------------------------