From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id C9327B6FBD for ; Sat, 9 Jun 2012 08:43:07 +1000 (EST) Message-ID: <1339195362.24838.50.camel@pasglop> Subject: Re: [PATCH 10/15] powerpc: fixes for instructions not using correct register naming From: Benjamin Herrenschmidt To: Andreas Schwab Date: Sat, 09 Jun 2012 08:42:42 +1000 In-Reply-To: References: <1339155365.316308.981577666228.qpush@ale> <20120608113605.91B88D43B2F__24549.446340143$1339156128$gmane$org@localhost.localdomain> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Cc: Michael Neuling , Michael Ellerman , linuxppc-dev@ozlabs.org, Paul Mackerras , Anton Blanchard , Olof Johannsson List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 2012-06-08 at 14:15 +0200, Andreas Schwab wrote: > Michael Neuling writes: > > > /* Invalidate all TLBs */ > > - PPC_TLBILX_ALL(0,0) > > + PPC_TLBILX_ALL(R0,R0) > > The first argument is (RA|0), so 0 is zero, not r0. The macro system we use cannot do that (it will prefix with REG_), since both arguments are registers we must use R0 in this case. Cheers, Ben.