Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967834Ab3E3Hl4 (ORCPT ); Thu, 30 May 2013 03:41:56 -0400 Received: from mail-ie0-f173.google.com ([209.85.223.173]:63870 "EHLO mail-ie0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S967689Ab3E3Hli (ORCPT ); Thu, 30 May 2013 03:41:38 -0400 MIME-Version: 1.0 In-Reply-To: References: Date: Thu, 30 May 2013 09:41:38 +0200 Message-ID: Subject: Re: [PATCH -next v2] dmaengine: ste_dma40: fix error return code in d40_probe() From: Linus Walleij To: Wei Yongjun Cc: Srinidhi KASAGAR , Vinod Koul , Dan Williams , Grant Likely , Rob Herring , Wei Yongjun , "linux-arm-kernel@lists.infradead.org" , "devicetree-discuss@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 855 Lines: 23 On Thu, May 30, 2013 at 6:32 AM, Wei Yongjun wrote: > From: Wei Yongjun > > In many of the error handling case, the return value 'ret' not set > and 0 will be return from d40_probe() even if error, but we should > return a negative error code instead in those error handling case. > This patch fixed them, and also removed useless variable 'err'. > > Signed-off-by: Wei Yongjun > --- > v1 -> v2: rebased on linux-next.git I've tentatively applied this to my dma40 branch, waiting for Vinod to ACK it. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/