From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 1/3] spi/s3c64xx: Log error interrupts Date: Sat, 21 Jan 2012 15:29:48 +0000 Message-ID: <20120121152947.GD10751@opensource.wolfsonmicro.com> References: <20120121132329.GA10751@opensource.wolfsonmicro.com> <1327152265-10789-1-git-send-email-broonie@opensource.wolfsonmicro.com> <4F1AD954.9030303@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, Linus Walleij , linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sylwester Nawrocki Return-path: Content-Disposition: inline In-Reply-To: <4F1AD954.9030303-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-spi.vger.kernel.org On Sat, Jan 21, 2012 at 04:27:16PM +0100, Sylwester Nawrocki wrote: > On 01/21/2012 02:24 PM, Mark Brown wrote: > > - int ret; > > + int ret, irq; > > + char clk_name[16]; > clk_name seems to be unused, otherwise looks good. That's git cherry-pick being too clever for me when I pulled this out of my -next branch for Grant. clk_name is used in -next by some updates which I guess Kukjin is carrying, I hadn't noticed they weren't in the SPI tree. I don't know if it's better to just leave things as they are for now given that it'll all come out in the merge window? ------------------------------------------------------------------------------ Try before you buy = See our experts in action! The most comprehensive online learning library for Microsoft developers is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, Metro Style Apps, more. Free future releases when you subscribe now! http://p.sf.net/sfu/learndevnow-dev2