Re: svn commit: r289710 - in head/sys/modules: i2c/iicbb uart
> On Oct 21, 2015, at 1:16 PM, Ed Maste <[hidden email]> wrote:
> On 21 October 2015 at 14:30, Ed Maste <[hidden email]> wrote:
>> Author: emaste
>> Date: Wed Oct 21 18:30:42 2015
>> New Revision: 289710
>> URL: https://svnweb.freebsd.org/changeset/base/289710 >>
>> Build ofw_bus_if.h for modules that need it on arm64
> Module Makefile build rules for ofw_bus_if.h are a bit of a mess; some
> modules build it unconditionally while others build it only on certain
> archs. There are a large number of different combinations of sparc64,
> mips, powerpc, arm, and aarch64. This is true for a number of other
> arch-specific module Makefile support. Should we have MK_* knobs to
> control these cases -- e.g. MK_OFW (MK_FDT?) here? That would at least
> move the arch-specific tests to a common location.
How much of the swamp do you want to drain?
None: Create MK_OFW (not MK_FDT: it already means something else). Add
a few lines to src.opts.mk. Watch me complain :)
A little: Add DEV_OFW to options. Add DEV_OFW to the relevant KERN_OPTS
clauses in sys/conf/config.mk and then test for it like so in the modules makefiles:
Maybe add it in config.mk if MK_FDT is defined, but I’d prefer not
That’s what we’re really testing for here: is this kernel likely to have the OFW
routines being called.
A lot: rewrite config(8).
I’m hoping you’ll pick “a little” to not make things worse.