public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
To: Sumit Garg <sumit.garg@linaro.org>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	 "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
	tee-dev@lists.linaro.org,
	 Leif Lindholm <leif.lindholm@linaro.org>,
	Daniel Thompson <daniel.thompson@linaro.org>,
	 Matteo Carlini <Matteo.Carlini@arm.com>,
	Achin Gupta <Achin.Gupta@arm.com>,
	 Udit Kumar <udit.kumar@nxp.com>,
	"Gao, Liming" <liming.gao@intel.com>
Subject: Re: [PATCH v2 1/2] MdePkg/IndustryStandard: Add Global Plaform header file
Date: Tue, 25 Sep 2018 10:31:51 +0200	[thread overview]
Message-ID: <CAKv+Gu_QH_mFxZmhJ7sSdDa58vhOhQzVaMG6m8RaJgon8uQJjA@mail.gmail.com> (raw)
In-Reply-To: <CAFA6WYOwEXchtH5KSZiS01+82O6wWNKDEFutdrkgq6xsbQfwJA@mail.gmail.com>

On Tue, 25 Sep 2018 at 10:27, Sumit Garg <sumit.garg@linaro.org> wrote:
>
> Thanks Mike for your comments.
>
> On Mon, 24 Sep 2018 at 22:20, Kinney, Michael D
> <michael.d.kinney@intel.com> wrote:
> >
> > Hi Ard,
> >
> > The initial content is only from the TEE Client API Specification.
> > I noticed that there is an errata to that spec as well.  Does this
> > content follow the errata?
>
> Yes.
>
> >
> > I also noticed that the specifications require the acceptance of
> > an additional license to view the contents.  Is there a version of
> > the content for this include file available that does not require
> > the acceptance of an additional license?
> >
>
> Yes, following is another version of the content for this include file:
>
> https://github.com/OP-TEE/optee_client/blob/master/public/tee_client_api.h#L163
>

Thanks Sumit.


Mike,

are you saying this content should not be [click through] license
encumbered if we want to put it into MdePkg as an industry standard?


> > > -----Original Message-----
> > > From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> > > Sent: Monday, September 24, 2018 1:26 AM
> > > To: Sumit Garg <sumit.garg@linaro.org>
> > > Cc: edk2-devel@lists.01.org; tee-dev@lists.linaro.org;
> > > Leif Lindholm <leif.lindholm@linaro.org>; Daniel
> > > Thompson <daniel.thompson@linaro.org>; Matteo Carlini
> > > <Matteo.Carlini@arm.com>; Achin Gupta
> > > <Achin.Gupta@arm.com>; Udit Kumar <udit.kumar@nxp.com>;
> > > Kinney, Michael D <michael.d.kinney@intel.com>; Gao,
> > > Liming <liming.gao@intel.com>
> > > Subject: Re: [edk2][PATCH v2 1/2]
> > > MdePkg/IndustryStandard: Add Global Plaform header file
> > >
> > > On Sat, 22 Sep 2018 at 01:15, Ard Biesheuvel
> > > <ard.biesheuvel@linaro.org> wrote:
> > > >
> > > > On 27 August 2018 at 04:50, Sumit Garg
> > > <sumit.garg@linaro.org> wrote:
> > > > > Add Global Plaform header file specific to TEE
> > > Client API Specification v1
> > > > >
> > > <http://www.globalplatform.org/specificationsdevice.asp
> > > >.
> > > > >
> > > > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > > > > Cc: Liming Gao <liming.gao@intel.com>
> > > > > 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>
> > > >
> > > > Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > > >
> > > > Mike,
> > > >
> > > > The header file is a bit light on content at
> > > introduction, but as
> > > > discussed online, we can add stuff as we go for other
> > > use cases.
> > > >
> > >
> > > ... as discussed *offline*
> > >
> > > >
> > > > > ---
> > > > >  MdePkg/Include/IndustryStandard/GlobalPlatform.h |
> > > 27 ++++++++++++++++++++++++
> > > > >  1 file changed, 27 insertions(+)
> > > > >  create mode 100644
> > > MdePkg/Include/IndustryStandard/GlobalPlatform.h
> > > > >
> > > > > diff --git
> > > a/MdePkg/Include/IndustryStandard/GlobalPlatform.h
> > > b/MdePkg/Include/IndustryStandard/GlobalPlatform.h
> > > > > new file mode 100644
> > > > > index 000000000000..72c5af4ef588
> > > > > --- /dev/null
> > > > > +++
> > > b/MdePkg/Include/IndustryStandard/GlobalPlatform.h
> > > > > @@ -0,0 +1,27 @@
> > > > > +/** @file
> > > > > +  Standardized Global Platform header file.
> > > GlobalPlatform TEE Client API
> > > > > +  Specification v1.0:
> > > <http://www.globalplatform.org/specificationsdevice.asp
> > > >
> > > > > +
> > > > > +  Copyright (c) 2018, Linaro Ltd. All rights
> > > reserved.<BR>
> > > > > +
> > > > > +  This program and the accompanying materials
> > > > > +  are licensed and made available under the terms
> > > and conditions of the BSD License
> > > > > +  which accompanies this distribution.  The full
> > > text of the license may be found at
> > > > > +  http://opensource.org/licenses/bsd-license.php
> > > > > +
> > > > > +  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE
> > > ON AN "AS IS" BASIS,
> > > > > +  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> > > KIND, EITHER EXPRESS OR IMPLIED.
> > > > > +
> > > > > +**/
> > > > > +
> > > > > +#ifndef _GLOBAL_PLATFORM_H_
> > > > > +#define _GLOBAL_PLATFORM_H_
> > > > > +
> > > > > +#define TEEC_ORIGIN_COMMS
> > > 0x00000002
> > > > > +
> > > > > +#define TEEC_SUCCESS
> > > 0x00000000
> > > > > +#define TEEC_ERROR_BAD_PARAMETERS
> > > 0xFFFF0006
> > > > > +#define TEEC_ERROR_OUT_OF_MEMORY
> > > 0xFFFF000C
> > > > > +#define TEEC_ERROR_COMMUNICATION
> > > 0xFFFF000E
> > > > > +
> > > > > +#endif
> > > > > --
> > > > > 2.7.4
> > > > >


  reply	other threads:[~2018-09-25  8:32 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-27 11:50 [PATCH v2 0/2] Add ArmPkg/Optee library APIs Sumit Garg
2018-08-27 11:50 ` [PATCH v2 1/2] MdePkg/IndustryStandard: Add Global Plaform header file Sumit Garg
2018-09-21 23:15   ` Ard Biesheuvel
2018-09-24  8:26     ` Ard Biesheuvel
2018-09-24 16:49       ` Kinney, Michael D
2018-09-25  8:27         ` Sumit Garg
2018-09-25  8:31           ` Ard Biesheuvel [this message]
2018-09-26 18:02             ` Kinney, Michael D
2018-09-27 12:10               ` Sumit Garg
     [not found]                 ` <20180927125156.3ylxviy5lnu5pm4w@debby>
2018-09-28  6:29                   ` Sumit Garg
2018-09-28 11:50                     ` Leif Lindholm
2018-08-27 11:50 ` [PATCH v2 2/2] ArmPkg/OpteeLib: Add APIs to communicate with OP-TEE Sumit Garg
2018-09-12  7:28 ` [PATCH v2 0/2] Add ArmPkg/Optee library APIs Sumit Garg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAKv+Gu_QH_mFxZmhJ7sSdDa58vhOhQzVaMG6m8RaJgon8uQJjA@mail.gmail.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox