On 27/11/18 8:22, Thomas Huth wrote: > On 2018-11-25 21:49, Philippe Mathieu-Daudé wrote: >> The CHRP NVRAM has his own section, but since the New World >> section contains the mac_nvram.c, it makes sens to also monitor >> the corresponding header. >> >> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> >> --- >> MAINTAINERS | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index 92691589b1..a248c8c564 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -930,6 +930,7 @@ F: hw/nvram/mac_nvram.c >> F: include/hw/misc/macio/ >> F: include/hw/misc/mos6522.h >> F: include/hw/ppc/mac_dbdma.h >> +F: include/hw/nvram/chrp_nvram.h > > I don't see the point why we should add duplicated entries here? The > header is also used by the Sparc machines, so why should this be added > additionally to the Mac machines only here, too? I suggest to drop this > patch.
OK, will drop.