From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753824Ab2BHJ1T (ORCPT ); Wed, 8 Feb 2012 04:27:19 -0500 Received: from acsinet15.oracle.com ([141.146.126.227]:36661 "EHLO acsinet15.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751634Ab2BHJ1Q (ORCPT ); Wed, 8 Feb 2012 04:27:16 -0500 Date: Wed, 8 Feb 2012 12:25:04 +0300 From: Dan Carpenter To: Rodolfo Giometti Cc: linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [patch] pps: checking for NULL instead of IS_ERR() Message-ID: <20120208092504.GD6554@elgon.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: acsinet21.oracle.com [141.146.126.237] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A090202.4F323F79.004C,ss=1,re=0.000,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org class_create() never returns NULLs only ERR_PTRs. Signed-off-by: Dan Carpenter diff --git a/drivers/pps/pps.c b/drivers/pps/pps.c index 2baadd2..98fbe62 100644 --- a/drivers/pps/pps.c +++ b/drivers/pps/pps.c @@ -369,9 +369,9 @@ static int __init pps_init(void) int err; pps_class = class_create(THIS_MODULE, "pps"); - if (!pps_class) { + if (IS_ERR(pps_class)) { pr_err("failed to allocate class\n"); - return -ENOMEM; + return PTR_ERR(pps_class); } pps_class->dev_attrs = pps_attrs;