Re: [PATCH 04/14] hw/i386: Restrict fw_cfg to the PC machines

[Philippe Mathieu-Daudé]

On 1/7/20 11:01 AM, Paolo Bonzini wrote:
> On 31/12/19 19:32, Philippe Mathieu-Daudé wrote:
> > Only the PC-based machines use the fw_cfg device. In particular,
> > the MicroVM machine does not use it. Only compile/link it when
> > machines require it.
> >
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > ---
> > Cc: Sergio Lopez <address@hidden>
> > ---
> >   hw/i386/Makefile.objs | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/hw/i386/Makefile.objs b/hw/i386/Makefile.objs
> > index 6ebb6d0cf0..48f2693546 100644
> > --- a/hw/i386/Makefile.objs
> > +++ b/hw/i386/Makefile.objs
> > @@ -5,7 +5,7 @@ obj-$(CONFIG_PC) += pc.o pc_sysfw.o
> >   obj-$(CONFIG_I440FX) += pc_piix.o
> >   obj-$(CONFIG_Q35) += pc_q35.o
> >   obj-$(CONFIG_MICROVM) += microvm.o
> > -obj-y += fw_cfg.o
> > +obj-$(CONFIG_PC) += fw_cfg.o
> >   obj-$(CONFIG_X86_IOMMU) += x86-iommu.o
> >   obj-$(CONFIG_VTD) += intel_iommu.o
> >   obj-$(CONFIG_AMD_IOMMU) += amd_iommu.o
>
> Ah no, this is hw/i386/fw_cfg.c; of course hw/nvram/fw_cfg.c has its own
> Kconfig symbol.  Can you rename the file to pc-fwcfg.c and adjust the
> commit message?

Yes, will do (I'll try to post a new series with only patches you 
haven't picked already).