From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from az33egw02.freescale.net (az33egw02.freescale.net [192.88.158.103]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "az33egw02.freescale.net", Issuer "Thawte Premium Server CA" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id 68301DDE11 for ; Sat, 28 Mar 2009 00:26:06 +1100 (EST) Message-ID: <49CCD3E8.9040007@freescale.com> Date: Fri, 27 Mar 2009 08:26:00 -0500 From: Scott Wood MIME-Version: 1.0 To: Joakim Tjernlund Subject: Re: [PATCH] ucc_geth: Rework the TX logic. References: <1238089445-28396-1-git-send-email-Joakim.Tjernlund@transmode.se> <2a27d3730903270245k6e8633eehfb5cd3fcebd36240@mail.gmail.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: linuxppc-dev@ozlabs.org, Li Yang , pku.leo@gmail.com, netdev@vger.kernel.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Joakim Tjernlund wrote: > This is a bit better than the current type casts. Moving over to qe_bd > accesses is a bigger cleanup that will have to be a seperate patch. > I am not sure it is an all win as you probably loose the ability > to read both status and len in one access. Not if you define the struct to have one status_len field (or a union of that with separate fields), as gianfar does. -Scott