hi Simon, On Thu, 18 Aug 2022 at 08:51, Simon Glass <s...@chromium.org> wrote: > > Hi Sugosh, > > On Wed, 17 Aug 2022 at 06:44, Sughosh Ganu <sughosh.g...@linaro.org> wrote: > > > > Add a command to read the metadata as specified in the FWU > > specification and print the fields of the metadata. > > > > Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org> > > Reviewed-by: Ilias Apalodimas <ilias.apalodi...@linaro.org> > > --- > > Changes since V7: None > > > > cmd/Kconfig | 7 +++++ > > cmd/Makefile | 1 + > > cmd/fwu_mdata.c | 80 +++++++++++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 88 insertions(+) > > create mode 100644 cmd/fwu_mdata.c > > This needs docs and a test. > > BTW I forgot to mention that the uclass needs a simple test of some sort. > > https://u-boot.readthedocs.io/en/latest/develop/tests_writing.html
Okay. I will check how this can be done. Btw, there are plans to add a test for the feature once support for the feature has been added on the Synquacer platform. That test will exercise the above command as well as the driver code. Do we still need a standalone test for the uclass? -sughosh
