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 690DDB711E for ; Fri, 21 Jan 2011 21:00:14 +1100 (EST) Subject: Re: [PATCH] powerpc: Fix some 6xx/7xxx CPU setup functions From: Benjamin Herrenschmidt To: Andreas Schwab In-Reply-To: References: <1295591723.2148.281.camel__36384.4125684865$1295591768$gmane$org@pasglop> Content-Type: text/plain; charset="UTF-8" Date: Fri, 21 Jan 2011 21:00:07 +1100 Message-ID: <1295604007.2148.294.camel@pasglop> Mime-Version: 1.0 Cc: linuxppc-dev List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 2011-01-21 at 10:48 +0100, Andreas Schwab wrote: > Benjamin Herrenschmidt writes: > > > In absence of good testing I would appreciate a close inspection of the patch > > by different pairs of eyes :-) > > Looks good to me. That broke apparently in 400d221 ("ppc32: make > cur_cpu_spec a single pointer instead of an array"), which changed the > calling convention in call_setup_cpu. Nice, >5 years ago and it's not even my fault :-) Thanks for the review ! Cheers, Ben.