Date: Fri, 21 Oct 2022 14:13:28 -0600 From: Warner Losh <imp@bsdimp.com> To: "Bjoern A. Zeeb" <bz@freebsd.org> Cc: Andrew Turner <andrew@fubar.geek.nz>, Gleb Smirnoff <glebius@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.org> Subject: Re: git: 2782ed8f6cd3 - main - dpaa2: fix standalone module build Message-ID: <CANCZdfrn2HPBY1THTF2GKX1cDnEuSMJYjDaJJ4Es_8KBzO4bJQ@mail.gmail.com> In-Reply-To: <1pr758pp-3640-43ss-451-nqnoqs248s5p@SerrOFQ.bet> References: <202210180540.29I5eNBZ068087@gitrepo.freebsd.org> <8sps479n-9961-rq8o-52op-po77rpo9p377@mnoonqbm.arg> <90B2F15A-3102-4771-BD46-C5304091569C@fubar.geek.nz> <1pr758pp-3640-43ss-451-nqnoqs248s5p@SerrOFQ.bet>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000daa8e505eb91137b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hey Bjoern, On Fri, Oct 21, 2022 at 2:08 PM Bjoern A. Zeeb <bz@freebsd.org> wrote: > On Fri, 21 Oct 2022, Andrew Turner wrote: > > > > > > >> On 21 Oct 2022, at 16:17, Bjoern A. Zeeb <bz@FreeBSD.org> wrote: > >> > >> On Tue, 18 Oct 2022, Gleb Smirnoff wrote: > >> > >>> The branch main has been updated by glebius: > >>> > >>> URL: > https://cgit.FreeBSD.org/src/commit/?id=3D2782ed8f6cd3d7f59219a783bc7fa7b= bfb1fe26f > >>> > >>> commit 2782ed8f6cd3d7f59219a783bc7fa7bbfb1fe26f > >>> Author: Gleb Smirnoff <glebius@FreeBSD.org> > >>> AuthorDate: 2022-10-18 05:38:24 +0000 > >>> Commit: Gleb Smirnoff <glebius@FreeBSD.org> > >>> CommitDate: 2022-10-18 05:38:24 +0000 > >>> > >>> dpaa2: fix standalone module build > >>> --- > >>> sys/modules/dpaa2/Makefile | 2 ++ > >>> 1 file changed, 2 insertions(+) > >>> > >>> diff --git a/sys/modules/dpaa2/Makefile b/sys/modules/dpaa2/Makefile > >>> index 556a1c531669..fa280452bcfa 100644 > >>> --- a/sys/modules/dpaa2/Makefile > >>> +++ b/sys/modules/dpaa2/Makefile > >>> @@ -19,6 +19,8 @@ SRCS+=3D memac_mdio_common.c memac_mdio_if.c > memac_mdio_if.h > >>> > >>> SRCS+=3D bus_if.h device_if.h miibus_if.h > >>> SRCS+=3D pcib_if.h pci_if.h > >>> +SRCS+=3D mdio_if.h > >> > >> What do you need this for? > >> > >>> +SRCS+=3D opt_acpi.h opt_platform.h > >> > >> opt_acpi gets inlcuded lines below and so does opt_platform under the > >> respective OPT_ checks. Now they are there twice. > > > > OPT_ACPI doesn=E2=80=99t exist as a make variable so the ".if !empty(OP= T_ACPI)=E2=80=9D > check won=E2=80=99t work as it is expected to work here so opt_acpi.h won= =E2=80=99t be > built. The second problem is the opt_*.h files need to be unconditionally > created so they can be included in the relevant code. > > Then they should probably have been removed from under the .ifdefs > in the same commit? > > > We do use OPT_ACPI elsewhere in the tree as well.. hwpmc? Can't > remember. How's it working there? > > What is currently then including these files then? Hmm I haven't used > the module in a while... > > Should we duplicate the FDT logic for ACPI in sys/conf/kern.opts.mk > so LINT-ACPI can be a better thing long-term as well? > Yes. We likely should clean this up... But the appetite to polish the currently way over polished and under powered config(8) program is low. However, having it create a .mk file that can be included that summarizes all the options in a kernel config would be the least amount of effort relative to the crazy greps we have going on now. Warner --000000000000daa8e505eb91137b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr">Hey Bjoern,<br></div><br><div class=3D"gm= ail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Fri, Oct 21, 2022 at 2:= 08 PM Bjoern A. Zeeb <<a href=3D"mailto:bz@freebsd.org">bz@freebsd.org</= a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0p= x 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On= Fri, 21 Oct 2022, Andrew Turner wrote:<br> <br> ><br> ><br> >> On 21 Oct 2022, at 16:17, Bjoern A. Zeeb <bz@FreeBSD.org> wr= ote:<br> >><br> >> On Tue, 18 Oct 2022, Gleb Smirnoff wrote:<br> >><br> >>> The branch main has been updated by glebius:<br> >>><br> >>> URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D2782= ed8f6cd3d7f59219a783bc7fa7bbfb1fe26f" rel=3D"noreferrer" target=3D"_blank">= https://cgit.FreeBSD.org/src/commit/?id=3D2782ed8f6cd3d7f59219a783bc7fa7bbf= b1fe26f</a><br> >>><br> >>> commit 2782ed8f6cd3d7f59219a783bc7fa7bbfb1fe26f<br> >>> Author:=C2=A0 =C2=A0 =C2=A0Gleb Smirnoff <glebius@FreeBSD.o= rg><br> >>> AuthorDate: 2022-10-18 05:38:24 +0000<br> >>> Commit:=C2=A0 =C2=A0 =C2=A0Gleb Smirnoff <glebius@FreeBSD.o= rg><br> >>> CommitDate: 2022-10-18 05:38:24 +0000<br> >>><br> >>>=C2=A0 =C2=A0dpaa2: fix standalone module build<br> >>> ---<br> >>> sys/modules/dpaa2/Makefile | 2 ++<br> >>> 1 file changed, 2 insertions(+)<br> >>><br> >>> diff --git a/sys/modules/dpaa2/Makefile b/sys/modules/dpaa2/Ma= kefile<br> >>> index 556a1c531669..fa280452bcfa 100644<br> >>> --- a/sys/modules/dpaa2/Makefile<br> >>> +++ b/sys/modules/dpaa2/Makefile<br> >>> @@ -19,6 +19,8 @@ SRCS+=3D=C2=A0 =C2=A0 memac_mdio_common.c me= mac_mdio_if.c memac_mdio_if.h<br> >>><br> >>> SRCS+=3D=C2=A0 =C2=A0 =C2=A0 bus_if.h device_if.h miibus_if.h<= br> >>> SRCS+=3D=C2=A0 =C2=A0 =C2=A0 pcib_if.h pci_if.h<br> >>> +SRCS+=3D=C2=A0 =C2=A0 =C2=A0mdio_if.h<br> >><br> >> What do you need this for?<br> >><br> >>> +SRCS+=3D=C2=A0 =C2=A0 =C2=A0opt_acpi.h opt_platform.h<br> >><br> >> opt_acpi gets inlcuded lines below and so does opt_platform under = the<br> >> respective OPT_ checks.=C2=A0 Now they are there twice.<br> ><br> > OPT_ACPI doesn=E2=80=99t exist as a make variable so the ".if !em= pty(OPT_ACPI)=E2=80=9D check won=E2=80=99t work as it is expected to work h= ere so opt_acpi.h won=E2=80=99t be built. The second problem is the opt_*.h= files need to be unconditionally created so they can be included in the re= levant code.<br> <br> Then they should probably have been removed from under the .ifdefs<br> in the same commit?<br> <br> <br> We do use OPT_ACPI elsewhere in the tree as well.. hwpmc?=C2=A0 Can't<b= r> remember.=C2=A0 How's it working there?<br> <br> What is currently then including these files then?=C2=A0 Hmm I haven't = used<br> the module in a while...<br> <br> Should we duplicate the FDT logic for ACPI in sys/conf/<a href=3D"http://ke= rn.opts.mk" rel=3D"noreferrer" target=3D"_blank">kern.opts.mk</a><br> so LINT-ACPI can be a better thing long-term as well?<br></blockquote><div>= <br></div><div>Yes. We likely should clean this up... But the appetite to p= olish the currently</div><div>way over polished and under powered config(8)= program is low. However,</div><div>having it create a .mk file that can be= included that summarizes all the</div><div>options in a kernel config woul= d be the least amount of effort relative to the</div><div>crazy greps we ha= ve going on now.</div><div><br></div><div>Warner</div></div></div> --000000000000daa8e505eb91137b--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrn2HPBY1THTF2GKX1cDnEuSMJYjDaJJ4Es_8KBzO4bJQ>