[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