Date: Fri, 21 Oct 2022 20:08:01 +0000 (UTC) From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: Andrew Turner <andrew@fubar.geek.nz> Cc: 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: <1pr758pp-3640-43ss-451-nqnoqs248s5p@SerrOFQ.bet> In-Reply-To: <90B2F15A-3102-4771-BD46-C5304091569C@fubar.geek.nz> References: <202210180540.29I5eNBZ068087@gitrepo.freebsd.org> <8sps479n-9961-rq8o-52op-po77rpo9p377@mnoonqbm.arg> <90B2F15A-3102-4771-BD46-C5304091569C@fubar.geek.nz>
next in thread | previous in thread | raw e-mail | index | archive | help
This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --1098556516-637080106-1666382883=:2458 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8BIT 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=2782ed8f6cd3d7f59219a783bc7fa7bbfb1fe26f >>> >>> 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+= memac_mdio_common.c memac_mdio_if.c memac_mdio_if.h >>> >>> SRCS+= bus_if.h device_if.h miibus_if.h >>> SRCS+= pcib_if.h pci_if.h >>> +SRCS+= mdio_if.h >> >> What do you need this for? >> >>> +SRCS+= 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’t exist as a make variable so the ".if !empty(OPT_ACPI)” check won’t work as it is expected to work here so opt_acpi.h won’t 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? -- Bjoern A. Zeeb r15:7 --1098556516-637080106-1666382883=:2458--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1pr758pp-3640-43ss-451-nqnoqs248s5p>