Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752728Ab3F0K3s (ORCPT ); Thu, 27 Jun 2013 06:29:48 -0400 Received: from mail-ob0-f173.google.com ([209.85.214.173]:55279 "EHLO mail-ob0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751927Ab3F0K3r (ORCPT ); Thu, 27 Jun 2013 06:29:47 -0400 MIME-Version: 1.0 In-Reply-To: References: Date: Thu, 27 Jun 2013 12:29:46 +0200 Message-ID: Subject: Re: [PATCH -next] pinctrl: st: fix return value check From: Linus Walleij To: Wei Yongjun , Mark Brown Cc: Grant Likely , Rob Herring , Wei Yongjun , "linux-kernel@vger.kernel.org" , "devicetree-discuss@lists.ozlabs.org" , Srinivas Kandagatla 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: 975 Lines: 27 On Wed, Jun 26, 2013 at 6:30 PM, Wei Yongjun wrote: > From: Wei Yongjun > > In case of error, the function pinctrl_register() returns > NULL not ERR_PTR(). The IS_ERR() test in the return value > check should be replaced with NULL test. > The function syscon_regmap_lookup_by_phandle() returns ERR_PTR() > and never returns NULL. The NULL test in the return value check > should be replaced with IS_ERR(). > > Signed-off-by: Wei Yongjun This driver is merged into Mark Brown's regmap tree and need to be applied as a fix there. Pls resend it if Mark has not recieved it. Include Srinivas on patches so he can ACK them. 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/