[PATCH v2 7/7] configs: add new sandbox with crypt-based password
Steffen Jaeckel
jaeckel-floss at eyet-services.de
Mon May 10 19:04:58 CEST 2021
Hi Simon,
On 5/10/21 6:28 PM, Simon Glass wrote:
> Hi Steffen,
>
> On Mon, 10 May 2021 at 00:19, Steffen Jaeckel
> <jaeckel-floss at eyet-services.de> wrote:
>>
>> This is a copy of the regular sandbox with crypt-based password entry
>> enabled.
>>
>> Signed-off-by: Steffen Jaeckel <jaeckel-floss at eyet-services.de>
>> ---
>>
>> (no changes since v1)
>>
>> configs/sandbox_cryptpass_defconfig | 296 ++++++++++++++++++++++++++++
>> 1 file changed, 296 insertions(+)
>> create mode 100644 configs/sandbox_cryptpass_defconfig
>
> Is this so you can add a test? In that case I think you need to adjust things...
>
> For sandbox you need to be able to build in both options, so your
> if/else approach won't work.
>
> You could control it with an env var perhaps, when both are enabled.
> Then it is easy enough to select the required setup with sandbox.
as mentioned in the cover letter, I guess this commit could also be
removed. I'm going to make a v3 without it.
More information about the U-Boot
mailing list