[PATCH v7 03/11] test: Allow connecting to a running board

[Tom Rini]

On Thu, Oct 31, 2024 at 07:00:24PM +0100, Simon Glass wrote:
> Hi Tom,
> 
> On Wed, 30 Oct 2024 at 00:53, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Wed, Oct 09, 2024 at 07:51:37PM -0600, Simon Glass wrote:
> >
> > > Sometimes we know that the board is already running the right software,
> > > so provide an option to allow running of tests directly, without first
> > > resetting the board.
> > >
> > > This saves time when re-running a test where only the Python code is
> > > changing.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> >
> > I'm not NAK'ing this, but this is not a open to errors kind of change
> > that I like. It also feels like an unrelated "I find this change useful"
> > kind of change.
> 
> Are you saying this change is open to errors?
> 
> It just avoids needing to reset the board when I know it is already in
> a good state. So it is a time-saver.

Yes, I'm saying it's open to errors since your assumption is "the board
is running what I want to test". As a human, I've violated that
assumption in the past running tests and short-circuiting the loop to
just re-test what's running.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241031/b863f9e3/attachment.sig>