git: dfcaa2c18bf9 - main - enetc_mdio: Support building the driver as a loadable module.

[Marcin Wojtas]

Hi Ian,

wt., 3 sie 2021 o 18:31 Ian Lepore <ian at freebsd.org> napisał(a):
>
> On Tue, 2021-08-03 at 10:08 +0000, Marcin Wojtas wrote:
> > The branch main has been updated by mw:
> >
> > URL:
> > https://cgit.FreeBSD.org/src/commit/?id=dfcaa2c18bf9c69b94d531364f8913b23f19505f
> >
> > commit dfcaa2c18bf9c69b94d531364f8913b23f19505f
> > Author:     Kornel Duleba <mindal at semihalf.com>
> > AuthorDate: 2021-07-28 11:38:53 +0000
> > Commit:     Marcin Wojtas <mw at FreeBSD.org>
> > CommitDate: 2021-08-03 10:07:49 +0000
> >
> >     enetc_mdio: Support building the driver as a loadable module.
> >
> > [...]
> > diff --git a/sys/modules/enetc_mdio/Makefile
> > b/sys/modules/enetc_mdio/Makefile
> > new file mode 100644
> > index 000000000000..f448fc526705
> > --- /dev/null
> > +++ b/sys/modules/enetc_mdio/Makefile
> > @@ -0,0 +1,8 @@
> > +#$FreeBSD$
> > +
> > +.PATH: ${SRCTOP}/sys/dev/enetc
> > +
> > +KMOD   = enetc_mdio
> > +SRCS   = enetc_mdio_pci.c enetc_mdio.c
> > +
> > +.include <bsd.kmod.mk>
>
> A module makefile must list all the header files it uses which are
> generated at build time from interface definitions (.m files) in its
> SRCS list.  For this driver, I think that means ifdi_if.h and
> miibus_if.h.
>
> -- Ian
>

Agreed. Part of it was already fixed by manu, I'm about to add
remaining dependencies.

Thanks,
Marcin