From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Subject: Re: [PATCH] ucc_geth: Rework the TX logic. Date: Fri, 27 Mar 2009 08:26:00 -0500 Message-ID: <49CCD3E8.9040007@freescale.com> References: <1238089445-28396-1-git-send-email-Joakim.Tjernlund@transmode.se> <2a27d3730903270245k6e8633eehfb5cd3fcebd36240@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Cc: linuxppc-dev@ozlabs.org, Li Yang , pku.leo@gmail.com, netdev@vger.kernel.org To: Joakim Tjernlund Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@ozlabs.org Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@ozlabs.org List-Id: netdev.vger.kernel.org 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