From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4864:20::e42; helo=mail-vs1-xe42.google.com; envelope-from=sumit.garg@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-vs1-xe42.google.com (mail-vs1-xe42.google.com [IPv6:2607:f8b0:4864:20::e42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 43E47211A2D98 for ; Wed, 19 Dec 2018 21:34:35 -0800 (PST) Received: by mail-vs1-xe42.google.com with SMTP id n10so306270vso.13 for ; Wed, 19 Dec 2018 21:34:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=7lBZ3xwur5kUBkW1Ph/W7JzAQxrVMQHiI+PnadzlQE4=; b=NRMWo75jITFHpjxPpHz+4fnShTibfVwtEP7vccMh73moBX0DMvC9BXQ4sVriXRaObp 8G6IbYjpR6g16d5j0LKMy30RmMih57EQcrSCJham5s2xXJdkdeg4f+/ejgs8wsGoeh4i AGqLXal2TFiS8Gir0A2XTvsm1HBa5XCdaYKSA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=7lBZ3xwur5kUBkW1Ph/W7JzAQxrVMQHiI+PnadzlQE4=; b=bnrNYxeAibHOeZK1Fk6BaxhH+mz2Lw1zfdVRaeZ0mWN6bQHtP1DYk7Q3djddCsZ4H7 C9KmjAJX0C5AnszBuDN9+jYutPmwrJam6zjSRNupjtwIoAybBBRumDme5W5o+PgWAwwq tLf4jSv0omw5yzVguyRCMxnrdeBB7iOeOJJmlIq37CDdePf5wjl9VX1PlU6Y4nsK92D0 3uerjUHPY6FnyRd/pwmnwVDQ2vJ+SCic7HONw12BdEfB5o4Ua7gEOhONhB/TkCCi6UJl UaOpNTKlBP36nivylFgE91z5mWcpN9byjop77Jjhiem0AAdzJH0Ho76T+TOfRPcZJFY9 2F/Q== X-Gm-Message-State: AA+aEWZUirgP10UXDpmWohglMKxhCqQ+pHBgiSSMUaPVlEiN9dNBXr0C lwTNC9RnQeUqynVmTeFROjLXNBsfJNusg5V2l2Im5w== X-Google-Smtp-Source: AFSGD/W6FYorj0SjlLMm0EgZ3wLdORSBjl1pVOPGv3dVXogfWmddsySUby1NLBM1OixbRwtbQYoDJf84ppz4r1yt/+s= X-Received: by 2002:a67:8cc2:: with SMTP id o185mr11364408vsd.55.1545284074509; Wed, 19 Dec 2018 21:34:34 -0800 (PST) MIME-Version: 1.0 References: <1545134943-16346-1-git-send-email-sumit.garg@linaro.org> In-Reply-To: From: Sumit Garg Date: Thu, 20 Dec 2018 11:04:22 +0530 Message-ID: To: Ard Biesheuvel Cc: edk2-devel@lists.01.org, Daniel Thompson , Leif Lindholm Subject: Re: [PATCH edk2-non-osi 1/1] Platform/Socionext: update ARM-TF binary to include OP-TEE X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Dec 2018 05:34:36 -0000 Content-Type: text/plain; charset="UTF-8" On Wed, 19 Dec 2018 at 20:37, Ard Biesheuvel wrote: > > On Tue, 18 Dec 2018 at 14:05, Ard Biesheuvel wrote: > > > > On Tue, 18 Dec 2018 at 14:04, Sumit Garg wrote: > > > > > > On Tue, 18 Dec 2018 at 18:30, Ard Biesheuvel wrote: > > > > > > > > On Tue, 18 Dec 2018 at 13:58, Sumit Garg wrote: > > > > > > > > > > On Tue, 18 Dec 2018 at 18:16, Ard Biesheuvel wrote: > > > > > > > > > > > > On Tue, 18 Dec 2018 at 13:09, Sumit Garg wrote: > > > > > > > > > > > > > > Include a prebuilt binary of OP-TEE OS built from commit > > > > > > > a5d528c7e54fd7726230483bd4cd5c4786d7703f. > > > > > > > (https://github.com/OP-TEE/optee_os.git master) > > > > > > > > > > > > > > Also update ARM-TF RELEASE build to commit 47577cbaaf4b. > > > > > > > > > > > > > > Cc: Ard Biesheuvel > > > > > > > Cc: Leif Lindholm > > > > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > > > > > > Signed-off-by: Sumit Garg > > > > > > > --- > > > > > > > Platform/Socionext/DeveloperBox/fip_all_arm_tf.bin | Bin 46824 -> 415251 bytes > > > > > > > > > > > > Thanks Sumit! > > > > > > > > > > > > In the future, please use --no-binary when sending patches like this one. > > > > > > > > > > > > > > > > Sure will take care of this in future. > > > > > > > > > > > However, do you have any explanation why the size increases 10x? This > > > > > > patch adds the pseudo-TA, but OP-TEE was already included in the > > > > > > previous build, so this is a bit unexpected. > > > > > > > > > > > > > > > > Actually I think ARM-TF was not updated earlier to include OP-TEE in > > > > > upstream (tee.bin size: 316392 bytes). > > > > > > > > > > > > > Ah, right. So we added the OP-TEE aware ATF build but not OP-TEE > > > > itself? That would indeed explain it. > > > > > > We didn't updated ATF build at all in upstream. I think you have > > > updated ATF build in local repo [1] only. > > > > > > [1] git://git.linaro.org/leg/noupstream/edk2-non-osi.git > > > > > > > Indeed. > > I cannot apply this patch. Could you please rebase it to latest > upstream edk2-non-osi and send me a link to the repo? Here is the link [1] to upstream repo with this patch applied. [1] ssh://git@git.linaro.org/people/sumit.garg/edk2-non-osi.git -Sumit