> Looking at clock.c, the code in oclockattach() is always compiled if SUN4 is
> defined [*].  Should there be some extra glue that says only pull in oclock*
> if we've defined that in the kernel config?  Presumably that would be wrapping
> all the references to oclock* with #if INTERSIL7170 > 0, cf. the references

Yes, clock.c is a bit of a mess; it should be thoroughly reorganised.

-pk