Received: by 2002:a4a:311b:0:0:0:0:0 with SMTP id k27-v6csp3917741ooa; Mon, 13 Aug 2018 21:19:11 -0700 (PDT) X-Google-Smtp-Source: AA+uWPz+aPUa7FCLlJON+XJA5Q/MWL11jNy0V3fL9KUGA66ZmC745Vj3iEKAClpn/D1hcs28Stwl X-Received: by 2002:a63:f043:: with SMTP id s3-v6mr18617046pgj.94.1534220351451; Mon, 13 Aug 2018 21:19:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1534220351; cv=none; d=google.com; s=arc-20160816; b=jYNLEN4yO+JfWggTyQ96iatUJPzgGnB6xEJtUGJg6opIrmPWXU0E6/TkPU3TkFy7m3 SV7z+dhTdF4haTwdfyhIS8bVW69TXJBvDFi8CFnMTnX85EAXvVdFAsCithEVunWNhXQX Mktq4dSg4t6UH+RuPvtDITakWk6xskwvDIQvu7BKKaD8VtUCaHGfH9IRiiwBR8KpKHMZ KrfuPnCFYBpNVzsa9HAILN/t6kRf3QOa1p1iQt3xOFaW+U9B3HQb/awsg65MlGoZCH7g 3SgzJT98YHPRSt7EBeKT/nx8bpMC8bgC6hgCbdO17ayeWjaNvHmDrZ+h/ontyVyL+vl6 O4PA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=DUrEfb8SB8YRx874uEsofxoobjhh06DoUSseomykCDE=; b=RXR0o0AH4fx3OVnxj5bQtJAvDAyepujeCpACwgPGNsHeRYMzHsFHc8P5U0aTDHKaSw uZWRHYsPcWbIgbe99t+ZPEtCbGG6G6rf6VgmEN3DrVEoB2GsQSsjX3qIZF8Wzj1koPsc xVC9aZ3GZzujIKYHkiJQBORgEGKco4ghqV7ACfN18WP3DPJM7DoKkQCbxVeM2xoxUv/Q nzDZe60xSl7UHMpju9GvevFdaiGvjxyFykkcpkihkw782orH/r+c90DvKCM11JNmZasz k6j7ytgTDSEGEyb0yCGjydk9LkKiSjkFJ1FrldoBssdPo88ptTxGYQVpQ/wICmpg49TF wOjw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 184-v6si19806259pgj.421.2018.08.13.21.18.56; Mon, 13 Aug 2018 21:19:11 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731253AbeHNHD3 (ORCPT + 99 others); Tue, 14 Aug 2018 03:03:29 -0400 Received: from guitar.tcltek.co.il ([192.115.133.116]:38309 "EHLO mx.tkos.co.il" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729443AbeHNHD3 (ORCPT ); Tue, 14 Aug 2018 03:03:29 -0400 Received: from sapphire.tkos.co.il (unknown [10.0.4.3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx.tkos.co.il (Postfix) with ESMTPS id CE823440524; Tue, 14 Aug 2018 07:18:06 +0300 (IDT) Date: Tue, 14 Aug 2018 07:18:05 +0300 From: Baruch Siach To: zhong jiang Cc: a.zummo@towertech.it, alexandre.belloni@bootlin.com, linux-rtc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] rtc:rtc-digicolor: Use PTR_ERR_OR_ZERO to replace the open code Message-ID: <20180814041805.666uaxe2gf75r3wr@sapphire.tkos.co.il> References: <1534159885-25364-1-git-send-email-zhongjiang@huawei.com> <1534159885-25364-2-git-send-email-zhongjiang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1534159885-25364-2-git-send-email-zhongjiang@huawei.com> User-Agent: NeoMutt/20180716 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi zhong, On Mon, Aug 13, 2018 at 07:31:24PM +0800, zhong jiang wrote: > PTR_ERR_OR_ZERO has implemented the if(IS_ERR(...)) + PTR_ERR, So > just replace them rather than duplicating its implement. > > Signed-off-by: zhong jiang Acked-by: Baruch Siach Thanks, baruch > --- > drivers/rtc/rtc-digicolor.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/rtc/rtc-digicolor.c b/drivers/rtc/rtc-digicolor.c > index b253bf1..fd6850c 100644 > --- a/drivers/rtc/rtc-digicolor.c > +++ b/drivers/rtc/rtc-digicolor.c > @@ -202,10 +202,8 @@ static int __init dc_rtc_probe(struct platform_device *pdev) > platform_set_drvdata(pdev, rtc); > rtc->rtc_dev = devm_rtc_device_register(&pdev->dev, pdev->name, > &dc_rtc_ops, THIS_MODULE); > - if (IS_ERR(rtc->rtc_dev)) > - return PTR_ERR(rtc->rtc_dev); > > - return 0; > + return PTR_ERR_OR_ZERO(rtc->rtc_dev); > } > > static const struct of_device_id dc_dt_ids[] = { -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -