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-fl...@eyet-services.de> wrote: >> >> This is a copy of the regular sandbox with crypt-based password entry >> enabled. >> >> Signed-off-by: Steffen Jaeckel <jaeckel-fl...@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.