[PATCH v4 1/8] lib: add crypt subsystem

Heiko Schocher hs at denx.de
Thu Jul 8 13:50:40 CEST 2021


Hello Steffen,

On 08.07.21 13:43, Steffen Jaeckel wrote:
> Hi Heiko,
> 
> thanks for the review!
> 
> On 7/8/21 5:56 AM, Heiko Schocher wrote:
>> [...]
>>
>>> diff --git a/lib/crypt/Makefile b/lib/crypt/Makefile
>>> new file mode 100644
>>> index 0000000000..290231064c
>>> --- /dev/null
>>> +++ b/lib/crypt/Makefile
>>> @@ -0,0 +1,10 @@
>>> +# SPDX-License-Identifier: GPL-2.0+
>>> +#
>>> +# Copyright (c) 2013, Google Inc.
>>> +#
>>> +# (C) Copyright 2000-2007
>>> +# Wolfgang Denk, DENX Software Engineering, wd at denx.de.
>>
>> Hmm... this is a new file ... I think you should add
>> only your Copyright and drop the others... ?
> 
> thx, done
> 
> 
>> [...]
>>
>> I miss here the SPDX license identifier... also some hint, from which
>> exact version this code is from ...
> 
> I've added this info in the next commit, this contains the vanilla
> version. Is this ok?

Yes from my side, thanks!

bye,
Heiko
> 
> 
> Cheers,
> Steffen
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de


More information about the U-Boot mailing list