From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ard.biesheuvel@linaro.org>
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
 client-ip=2607:f8b0:4864:20::144; helo=mail-it1-x144.google.com;
 envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org 
Received: from mail-it1-x144.google.com (mail-it1-x144.google.com
 [IPv6:2607:f8b0:4864:20::144])
 (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 135FE21198CCC
 for <edk2-devel@lists.01.org>; Wed, 19 Dec 2018 07:07:33 -0800 (PST)
Received: by mail-it1-x144.google.com with SMTP id m62so10095117ith.5
 for <edk2-devel@lists.01.org>; Wed, 19 Dec 2018 07:07:33 -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=CGsEXWUC1jYtdD3xMwvqTOtzJZV4vJylJqx2Xa9ZGO4=;
 b=Uc0Kh7ax6dREzzhlmCPtc38TEX11fA0gJqm5OQsbbncfbBb5TOZxjTBlVLfxyXhziu
 mmkMQe/0f4mbGz3xfZAAKe4luZdieRRiRX/C9k3UyhuA6T/1g0uhG5vVeygE8bpktthM
 70XaL7urQlhaUp1RTXXXWnvKSIE2Zmi7iNIsM=
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=CGsEXWUC1jYtdD3xMwvqTOtzJZV4vJylJqx2Xa9ZGO4=;
 b=UK75GR5wMqhwBO5cq2YnEn+DPRW77q954jhQMlWINxlSeGW1V/XJIaRLEKGc96o17D
 bWz3wF6z/l1ygbmyiNrk7zA+35NQsUH+7oqWKTdyydkGpsTmooSiNvvckTvrw3LirvmY
 M4ZTv9dSf2VqiaorawJLtXKsKgOcKVyniR0KsuE/yq55EE8ph3MVF6WlQNwDSZhvlUJc
 dvX52gg00p1V0psrPtLtzM+Eq+vYWhiEElTjYamDBgSpEyZUYQhqa0FclzAxw3QnBzK6
 vMJZ/Kf2DJYR1PVV/6WXGpqDYXXUQiL7KU8zXwBnROJ7+Z/SY/xW3uwj9chZ8cT9vk7y
 oPpw==
X-Gm-Message-State: AA+aEWbsnbTAS2foMmOnDxdZr8dZ4lLKLfRWkIRGHFnut50AxNos0f9I
 cvQDpt1N7+LU7OSKKNlZY6BrVFzmtbWH/pHANKRomw==
X-Google-Smtp-Source: AFSGD/XkWwShYuHx1zCpK5wH5S0fwwAYc9V/pn9YgO7DEwS/4U4K3tzvkuwrgIbIb0CXhEmgJmcvDK0Jedt1ef/j7YE=
X-Received: by 2002:a24:710:: with SMTP id f16mr6100778itf.121.1545232052978; 
 Wed, 19 Dec 2018 07:07:32 -0800 (PST)
MIME-Version: 1.0
References: <1545134943-16346-1-git-send-email-sumit.garg@linaro.org>
 <CAKv+Gu9OzyvMXkhuZ9232AEbRWA1eivSKwTgaBWUSN=vyWpb1g@mail.gmail.com>
 <CAFA6WYMvgEZFWfdULi-HqGpitM-Jn374Y-uHnGY6UWXfaux00w@mail.gmail.com>
 <CAKv+Gu-1Ja78BK9_OEkJTyn=U-tss8rRNryvH6kRUfmgNcwv8A@mail.gmail.com>
 <CAFA6WYMP9FdoE6RS5r6Top6v78cMp9npkCMDr8nqFFVZPq1X9A@mail.gmail.com>
 <CAKv+Gu8qSAVZ2O_k_ZZiyaLJ1QJSR_1GM4QQv=hF3wX_WO4=qg@mail.gmail.com>
In-Reply-To: <CAKv+Gu8qSAVZ2O_k_ZZiyaLJ1QJSR_1GM4QQv=hF3wX_WO4=qg@mail.gmail.com>
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Date: Wed, 19 Dec 2018 16:07:20 +0100
Message-ID: <CAKv+Gu_WJiFBE-h=DamQTa+Zkmy7sgySJ0V60S7xZMWwG2UmLw@mail.gmail.com>
To: Sumit Garg <sumit.garg@linaro.org>
Cc: "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
 Daniel Thompson <daniel.thompson@linaro.org>, 
 Leif Lindholm <leif.lindholm@linaro.org>
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  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Dec 2018 15:07:34 -0000
Content-Type: text/plain; charset="UTF-8"

On Tue, 18 Dec 2018 at 14:05, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
>
> On Tue, 18 Dec 2018 at 14:04, Sumit Garg <sumit.garg@linaro.org> wrote:
> >
> > On Tue, 18 Dec 2018 at 18:30, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
> > >
> > > On Tue, 18 Dec 2018 at 13:58, Sumit Garg <sumit.garg@linaro.org> wrote:
> > > >
> > > > On Tue, 18 Dec 2018 at 18:16, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
> > > > >
> > > > > On Tue, 18 Dec 2018 at 13:09, Sumit Garg <sumit.garg@linaro.org> 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 <ard.biesheuvel@linaro.org>
> > > > > > Cc: Leif Lindholm <leif.lindholm@linaro.org>
> > > > > > Contributed-under: TianoCore Contribution Agreement 1.1
> > > > > > Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> > > > > > ---
> > > > > >  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?