From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH 2/6]: powerpc/cell spidernet low watermark patch. Date: Wed, 23 Aug 2006 15:03:14 -0700 (PDT) Message-ID: <20060823.150314.41637066.davem@davemloft.net> References: <1156055509.5803.77.camel@localhost.localdomain> <200608201203.15645.arnd@arndb.de> <20060823213642.GG4401@austin.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: arnd@arndb.de, jklewis@us.ibm.com, linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org, netdev@vger.kernel.org Return-path: To: linas@austin.ibm.com In-Reply-To: <20060823213642.GG4401@austin.ibm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linuxppc-dev-bounces+glppd-linuxppc64-dev=m.gmane.org@ozlabs.org Errors-To: linuxppc-dev-bounces+glppd-linuxppc64-dev=m.gmane.org@ozlabs.org List-Id: netdev.vger.kernel.org From: linas@austin.ibm.com (Linas Vepstas) Date: Wed, 23 Aug 2006 16:36:42 -0500 > I could create a searate patch to change struct descr {} to split > the u32 into several u8's; there's a dozen spots that get touched. > > Alternatel, I could do a cheesy cast to char[4] and access that way. > Opinions? The most portable scheme would be a "u32/u8[4]" union with appropriate endianness checks when determining which byte to access in the u8[] view.