Hi Heinrich, On Sat, 19 Sep 2020 at 12:07, Heinrich Schuchardt <[email protected]> wrote: > > On 9/19/20 3:26 PM, Simon Glass wrote: > > Hi Heinrich, > > > > On Fri, 18 Sep 2020 at 22:19, Heinrich Schuchardt <[email protected]> > > wrote: > >> > >> Am 19. September 2020 04:59:46 MESZ schrieb Simon Glass > >> <[email protected]>: > >>> Hi Heinrich, > >>> > >>> On Thu, 17 Sep 2020 at 12:04, Heinrich Schuchardt <[email protected]> > >>> wrote: > >>>> > >>>> Currently if SIGINT is received, it terminates U-Boot. This does not > >>> allow > >>>> testing the handling of CTRL-C in U-Boot and in UEFI applications. > >>>> It is especially annoying when working with the 'mm' command. > >>>> > >>>> Let the serial console driver provide character 0x03 if SIGINT > >>> occurs. We > >>>> can still exit U-Boot using the reset or poweroff command. > >>>> > >>>> Adjust the sandbox test_ctrl_c() Python test accordingly. > >>>> > >>>> Signed-off-by: Heinrich Schuchardt <[email protected]> > >>>> --- > >>>> arch/sandbox/cpu/os.c | 15 ++++++++++++--- > >>>> drivers/serial/sandbox.c | 11 ++++++++--- > >>>> include/os.h | 14 ++++++++++++++ > >>>> test/py/tests/test_sandbox_exit.py | 10 +++++++++- > >>>> 4 files changed, 43 insertions(+), 7 deletions(-) > >>> > >>> I'm sorry but I still do want Ctrl-C to work. How about adding a short > >>> option to change the terminal mode? > >>> > >>> Regards, > >>> Simon > >> > >> > >> On QEMU you quit via <ctrl+A>X. Can we use the same for the sandbox? > > > > I just don't like programs that refused to quit when asked I use> Ctrl-C > > all the time. Can you not use the -t options to get what you > > want? Perhaps we could have an environment variable too? > > Sorry, I was not aware of the -t option. > > ./u-boot -t raw -d arch/sandbox/dts/test.dtb > > behaves according to my preferences.
OK great! If you want an environment variable we could add that too. Regards, Simon
