From mboxrd@z Thu Jan  1 00:00:00 1970
From: Miquel Raynal <miquel.raynal@bootlin.com>
Date: Fri, 27 Apr 2018 15:45:36 +0200
Subject: [U-Boot] [PATCH v2 09/19] tpm: add TPM2_Startup command support
In-Reply-To: <CAPnjgZ1q8MbrP31Uf9zTzh-+yicK3JBRBbAA2Jvj5fR1x6ekig@mail.gmail.com>
References: <20180329074401.8691-1-miquel.raynal@bootlin.com>
 <20180329074401.8691-10-miquel.raynal@bootlin.com>
 <CAPnjgZ1q8MbrP31Uf9zTzh-+yicK3JBRBbAA2Jvj5fR1x6ekig@mail.gmail.com>
Message-ID: <20180427154536.1e483904@xps13>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: u-boot@lists.denx.de

Hi Simon,

> >  enum tpm_physical_presence {
> >         TPM_PHYSICAL_PRESENCE_HW_DISABLE        =3D 0x0200,
> >         TPM_PHYSICAL_PRESENCE_CMD_DISABLE       =3D 0x0100,
> > @@ -445,7 +469,7 @@ int tpm_get_specification(void);
> >   * @param mode         TPM startup mode
> >   * @return return code of the operation
> >   */
> > -uint32_t tpm_startup(enum tpm_startup_type mode);
> > +int tpm_startup(enum tpm_startup_type mode); =20
>=20
> How come the change to an int? It's fine, I just want to understand it.
>=20

Good catch, that is a mistake and has no interest. In the new version I
moved all the functions to be aligned by returning an u32.

Thanks,
Miqu=C3=A8l

--=20
Miquel Raynal, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com