From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from az33egw01.freescale.net (az33egw01.freescale.net [192.88.158.102]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "az33egw01.freescale.net", Issuer "Thawte Premium Server CA" (verified OK)) by ozlabs.org (Postfix) with ESMTP id CEBE3DDEC4 for ; Wed, 12 Sep 2007 03:09:19 +1000 (EST) Message-ID: <46E6CBB9.90008@freescale.com> Date: Tue, 11 Sep 2007 12:09:13 -0500 From: Timur Tabi MIME-Version: 1.0 To: linuxppc-dev@ozlabs.org, Paul Mackerras , Gala Kumar-B11780 Subject: Re: [PATCH] [POWERPC] QE: simplify GUEMR register initialization References: <11895258991411-git-send-email-timur@freescale.com> In-Reply-To: <11895258991411-git-send-email-timur@freescale.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Paul and Kumar, It turns out that I have a bunch more QE library code clean-up, so please ignore this patch. I'll submit a more comprehensive one later this week. Timur Tabi wrote: > The initialization of the QE's GUEMR register is overly complicated, involving > multiple functions and a redundant ucc_common structure. This patch removes > struct ucc_common, merges ucc_init_guemr() into ucc_set_type() (every caller > of ucc_init_guemr() also calls ucc_set_type() immediately afterward), and > removes the unused enum ucc_pram_initial_offset. -- Timur Tabi Linux Kernel Developer @ Freescale