Date: Sat, 26 Apr 2003 09:02:25 -0700 From: "David O'Brien" <obrien@FreeBSD.org> To: Hartmut Brandt <harti@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/modules Makefile src/sys/pci if_en_pci.c src/sys/modules/en Makefile src/sys/dev/en midway.c midwayreg.h midwayvar.h Message-ID: <20030426160225.GA8241@dragon.nuxi.com> In-Reply-To: <200304251614.h3PGE3t5006537@repoman.freebsd.org> References: <200304251614.h3PGE3t5006537@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Apr 25, 2003 at 09:14:03AM -0700, Hartmut Brandt wrote: > Modified files: > sys/modules Makefile > sys/pci if_en_pci.c > sys/dev/en midway.c midwayreg.h midwayvar.h > Added files: > sys/modules/en Makefile > Log: > Convert the midway driver to use busdma. Except for this conversion the > following changes have been done: > > - stylify. The original code was too hard to read. > - get rid of a number of compilation options (Adaptec-only, Eni-only, no-DMA). > - more debugging features. > - locking. This is not correct yet in the absence of interface layer locking, Were you not aware we have rules against exactly this type of commit?? It is totally impossible to tell what the real changes were and which the style changes were. Please don't make a commits like this again.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030426160225.GA8241>