Re: svn commit: r289710 - in head/sys/modules: i2c/iicbb uart

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

Re: svn commit: r289710 - in head/sys/modules: i2c/iicbb uart

Warner Losh

> 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:
>> Log:
>>  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 Watch me complain :)

A little: Add DEV_OFW to options. Add DEV_OFW to the relevant KERN_OPTS
clauses in sys/conf/ and then test for it like so in the modules makefiles:

Maybe add it in 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.


signature.asc (859 bytes) Download Attachment