Re: DAC960 in 2.5.38, with new changes

Dave Olien (dmo@osdl.org)
Mon, 23 Sep 2002 15:03:02 -0700


By the way, I don't know how extensive your code reviewing
of my changes has been.

This morning I sent you the complete driver patch from 2.5.38
to the latest version of the driver. That contains all of
the changes I sent you last week, plus new ones.

Would it be better for me to send you incremental patches
in the future? I could instead generate a patch relative to the
previous version of the driver I sent you.

Let me know what works better for you, based on where you are in code
reviewing.

Thanks!

On Mon, Sep 23, 2002 at 11:53:47PM +0200, Daniel Phillips wrote:
> On Monday 23 September 2002 23:41, Dave Olien wrote:
> > On Mon, Sep 23, 2002 at 10:39:00PM +0200, Daniel Phillips wrote:
> > > Minor whitespace suggestion: don't worry too much about breaking up
> > > lines to fit in 80 columns. It's nice where it works, but where it
> > > just makes more lines, don't bother. We are going to go do spelling
> > > patches to shorten a lot of those names anyway.
> >
> > thanks.
> > I'd been wondering whether there was a guidline for this.
> > I'll relax my 80 column constraints.
>
> It's a lot more important for core kernel, and ever there, a few lines
> tend to break loose here and there.
>
> Thanks for the test code. Using your recipe, I confirmed I have the
> same controller as you. Running on a dual PIII in my case.
>
> Apparently Jens has a DAC as well. Jens, is it the same?
>
> --
> Daniel
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/