[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080610144154.GA29042@sgi.com>
Date: Tue, 10 Jun 2008 09:41:54 -0500
From: Jack Steiner <steiner@....com>
To: Roland Dreier <rdreier@...co.com>
Cc: akpm@...l.org, linux-kernel@...r.kernel.org, mingo@...e.hu,
tglx@...utronix.de, holt@....com, andrea@...ranet.com
Subject: Re: [patch 11/11] GRU Driver - makefile & Kconfig file changes
On Mon, Jun 09, 2008 at 02:35:09PM -0700, Roland Dreier wrote:
> > +EXTRA_CFLAGS += -DDEBUG -Werror -Wall
>
> Not sure if it's appropriate to set these flags for your driver...
Deleted. This was primarily for debugging. I moved the DEBUG option to
Kconfig as a driver DEBUG option that can be selected.
>
> > + ---help---
> > + This option enables basic support for the SGI UV GRU driver. If
> > + you are not running on a SGI UV system, say N.
>
> "...enables basic support for the SGI UV GRU." And it would be a good
> idea to include some info about what UV GRU does -- your 0/11
> introduction mail had a good description that would be nice to have here
> as well.
Done
--- jack
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists