On 03.08.21 16:28, Pali Rohár wrote:
Obviously it is not possible to send terminate sequence over stream after closing stream.
xyzModem_stream_close() does not close anything; it flushes the stream. xyzModem_stream_terminate() sets xyz.at_eof = true and gives feedback to the user.
So I think this change is incorrect.
What we lack is documentation of the xyzModem functions in include/xyzModem.h.
Cf. https://www.kernel.org/doc/html/latest/doc-guide/kernel-doc.html#function-do...
Best regards
Heinrich
Signed-off-by: Pali Rohár pali@kernel.org
cmd/load.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cmd/load.c b/cmd/load.c index b7894d7db02a..fb8c191fb64f 100644 --- a/cmd/load.c +++ b/cmd/load.c @@ -1006,8 +1006,8 @@ static ulong load_serial_ymodem(ulong offset, int mode) printf("%s\n", xyzModem_error(err)); }
- xyzModem_stream_close(&err); xyzModem_stream_terminate(false, &getcxmodem);
xyzModem_stream_close(&err);
flush_cache(offset, ALIGN(size, ARCH_DMA_MINALIGN));