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 0C8B92C00BC for ; Wed, 9 Oct 2013 17:52:03 +1100 (EST) Message-ID: <1381301512.645.272.camel@pasglop> Subject: Re: [PATCH] powerpc/powernv: Add a debugfs file to read the firmware console From: Benjamin Herrenschmidt To: Michael Ellerman Date: Wed, 09 Oct 2013 17:51:52 +1100 In-Reply-To: <20131009060613.GC28160@concordia> References: <1381218400.645.204.camel@pasglop> <20131009032314.GD23780@concordia> <1381292601.645.258.camel@pasglop> <20131009060613.GC28160@concordia> Content-Type: text/plain; charset="UTF-8" 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 Wed, 2013-10-09 at 17:06 +1100, Michael Ellerman wrote: > Call it twice. > > And you wonder why no one reviews your patches? Not that easy :-) I had a look at using it and unless I did something stupid, it wasn't actually that trivial to figure out what arg to pass it for calling it twice, ie, it didn't simplify the code significantly ... I had another problem too but I don't remember what it was. But it could be that I had a very dumb moment... Cheers, Ben.