Sure, I will list other involved files. Thanks for you suggestion.
SU Hang
"Eric Blake" wrote:
> On 05/24/2018 06:29 AM, Su Hang wrote:
> > 'test.hex' file is a bare metal ARM software stored in Hexadecimal
> > Object Format. When it's loaded by QEMU, it will print "Hello world!\n"
> > on console.
> >
> > `pre_store` array in 'hexloader-test.c' file, stores the binary format
> > of 'test.hex' file, which is used to verify correctness.
> >
> > Reviewed-by: Stefan Hajnoczi
> > Suggested-by: Steffen Gortz
> > Suggested-by: Stefan Hajnoczi
> > Signed-off-by: Su Hang
> > ---
> > MAINTAINERS | 6
> > configure| 4 +++
> > tests/Makefile.include | 2 ++
> > tests/hex-loader-check-data/test.hex | 12
> > tests/hexloader-test.c | 56
> >
>
> The previous patch also touched:
>
> hw/arm/boot.c | 7 +-
> hw/core/loader.c| 246
>
> include/hw/loader.h | 12 +++
>
> > +++ b/MAINTAINERS
> > @@ -1291,6 +1291,12 @@ F: hw/core/generic-loader.c
> > F: include/hw/core/generic-loader.h
> > F: docs/generic-loader.txt
> >
> > +Intel Hexadecimal Object File Loader
> > +M: Su Hang
> > +S: Maintained
> > +F: tests/hexloader-test.c
> > +F: tests/hex-loader-check-data/test.hex
> > +
>
> It looks odd having a maintainer that claims only test files; do you
> want to also list some of the other files touched by this patch so that
> you get notification if one of the implementation files has subsequent
> patches (rather than just the test files)?
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org
