[LTP] [PATCH v5 1/2] firmware/fw_load: rewrite firmware loading test using new LTP API

Andrea Cervesato andrea.cervesato@suse.com
Thu Jun 11 11:43:58 CEST 2026


Hi Cyril,

> Hi!
> > ---
> >  runtest/kernel_misc                                |   2 +-
> >  .../{fw_load_kernel => fw_load}/.gitignore         |   1 +
> >  .../firmware/{fw_load_kernel => fw_load}/Makefile  |   8 +-
> >  testcases/kernel/firmware/fw_load/fw_load.h        |  18 ++
> >  testcases/kernel/firmware/fw_load/fw_load01.c      | 165 ++++++++++++++++
> >  .../{fw_load_kernel => fw_load}/ltp_fw_load.c      |   0
> >  testcases/kernel/firmware/fw_load_user/.gitignore  |   1 -
> >  testcases/kernel/firmware/fw_load_user/Makefile    |  20 --
> >  testcases/kernel/firmware/fw_load_user/README      |  11 --
> >  testcases/kernel/firmware/fw_load_user/fw_load.c   | 213 ---------------------
> 
> This is impossible to review. If you move/rename files it's usually done
> in a separate patch that does not change the content.

I usually review with double column editor left=old, right=new :-)
But I agree with you that diff is messed up. Should I send a new
version?

--
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com


More information about the ltp mailing list