Subject: Re: CVS commit: src/sys/arch/m68k/include
To: None <tsutsui@ceres.dti.ne.jp>
From: Havard Eidnes <he@NetBSD.org>
List: source-changes
Date: 06/13/2007 14:38:44
> > Modified Files:
> > 	src/sys/arch/m68k/include: cacheops.h cacheops_60.h
> >
> > Log Message:
> > Fix the 060 variants to not have conflicting definitions for the
> > cache ops common to 060 and 040.  Also remove the now-redundant
> > definitions for the 060 variant which simply pointed to the 040
> > variants.
> > This fixes the build of the mvme68k VME172 kernel.
>
> Well, cacheops.h part is my fault, but I think we should
> still keep *_60() macroes in cacheops_60.h for consistency.
> If we remove *_60() ones for those cache ops, we should also
> stop using *_40() ones and just use common names without suffix.

A cursory inspection showed that the specific _060 variants I
removed were no longer being used by any m68k code.  I didn't
check if the same was true for the _040 variants, although I
suspect it is.  Is there still a reason for keeping (or, in the
case of the _060, re-introduce) them?

Regards,

- H=E5vard