Hi Sughosh,
On Sat, 5 Aug 2023 at 11:50, Sughosh Ganu <sughosh.g...@linaro.org> wrote:
>
> hi Simon,
>
> On Sat, 5 Aug 2023 at 20:34, Simon Glass <s...@chromium.org> wrote:
> >
> > Hi Sughosh,
> >
> > On Sat, 5 Aug 2023 at 05:35, Sughosh Ganu <sughosh.g...@linaro.org> wrote:
> > >
> > > Add the private keys and public key certificates which are to be used
> > > for capsule authentication while testing the EFI capsule update
> > > functionality. There are two pairs of private and public keys. The
> > > SIGNER.{key,crt} pair will be used for signing capsules, whilst the
> > > SIGNER2.{key,crt} pair is to be used as malicious keys for testing
> > > authentication failure cases. The SIGNER.crt is also converted to an
> > > EFI Signature List(ESL) file, SIGNER.esl, which is embedded in the
> > > platform's device-tree for capsule authentication.
> > >
> > > Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org>
> > > ---
> > > Changes since V6:
> > > * New patch that puts the keys and cert files under board/sandbox/
> > > directory as suggested Simon Glass.
> > >
> > > board/sandbox/SIGNER.crt | 19 +++++++++++++++++++
> > > board/sandbox/SIGNER.esl | Bin 0 -> 831 bytes
> > > board/sandbox/SIGNER.key | 28 ++++++++++++++++++++++++++++
> > > board/sandbox/SIGNER2.crt | 19 +++++++++++++++++++
> > > board/sandbox/SIGNER2.key | 28 ++++++++++++++++++++++++++++
> > > 5 files changed, 94 insertions(+)
> > > create mode 100644 board/sandbox/SIGNER.crt
> > > create mode 100644 board/sandbox/SIGNER.esl
> > > create mode 100644 board/sandbox/SIGNER.key
> > > create mode 100644 board/sandbox/SIGNER2.crt
> > > create mode 100644 board/sandbox/SIGNER2.key
> >
> > Can we call these good.* and bad.* so it is clear what they are for?
> > Also, please avoid capital letters in filenames.
>
> I was using the same nomenclature that was being used currently by the
> efi capsule update tests. But I guess I can change this.
Yes please. You could use a patch at the start of your series, perhaps?
Regards,
Simon
