Date: Thu, 17 May 2007 20:12:23 +0200
From: Christoph Hellwig <hch@lst.de>
To: David Woodhouse <dwmw2@infradead.org>
Cc: Christoph Hellwig <hch@lst.de>, dedekind@infradead.org, akpm@osdl.org,
       linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ubi: kill homegrown endian macros
Message-ID: <20070517181223.GA17765@lst.de>
References: <20070517143200.GA30850@lst.de> <1179417055.2859.506.camel@shinybook.infradead.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1179417055.2859.506.camel@shinybook.infradead.org>
User-Agent: Mutt/1.3.28i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 866
Lines: 18

On Thu, May 17, 2007 at 11:50:54PM +0800, David Woodhouse wrote:
> On Thu, 2007-05-17 at 16:32 +0200, Christoph Hellwig wrote:
> > Kill ubis homegrown endianess handling crap and replace it with
> > the normal kernel endianess handling. 
> 
> NAK. The 'normal' kernel stuff doesn't work in GCC; only in sparse.
> 
> If you want to go removing the ones which _do_ work in GCC, then fix the
> 'normal' one first. _Then_ go about converting others over to it.

Nope.  gcc checking is crap and we need to run sparse anyway.  It's what
everyone in the kernel use so it's perfect for some random code almost
no one uses aswell.
-
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/