From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 26 Nov 2009 15:32:12 -0800 (PST) Message-Id: <20091126.153212.142866981.davem@davemloft.net> To: segher@kernel.crashing.org Subject: Re: [PATCH 04/11] of/flattree: eliminate cell_t typedef From: David Miller In-Reply-To: References: <1259213266.16367.274.camel@pasglop> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Cc: sfr@canb.auug.org.au, devicetree-discuss@lists.ozlabs.org, microblaze-uclinux@itee.uq.edu.au, sparclinux@vger.kernel.org, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Segher Boessenkool Date: Thu, 26 Nov 2009 22:36:41 +0100 >>> You're right, it's not, but makes merging less complex, and then I can >>> refactor properly. >> >> Still, make them __be32 at least > > There is no alignment guarantee at all either, better make it all u8 > and use accessor functions everywhere. I think that might be overkill.