Received: by 2002:ac0:bc90:0:0:0:0:0 with SMTP id a16csp1326059img; Tue, 19 Mar 2019 05:27:52 -0700 (PDT) X-Google-Smtp-Source: APXvYqxkHXE11wgI7+avHWq4nKl18rM+QaGpuoFSFaO3SyHwQ1zi1PtorqMzXG1WTxGsBdnZZ4Q+ X-Received: by 2002:aa7:8b03:: with SMTP id f3mr1678701pfd.208.1552998472375; Tue, 19 Mar 2019 05:27:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1552998472; cv=none; d=google.com; s=arc-20160816; b=xbkrSkejThBmHjKfvCEADvyHJ2acdzCl5QlbpLDh8vmgVXYtFpn46esAJCilGNJcRI PbYO7HpMDKhH/ScjuNcanBoISJ6SwYXT02Nkf4z8pNDDVKR7caXZNd58xjk4Z0hunhzS 3j9KJai/F7Aw41mZWWhUA4NNokv6gtvyo+JB3aJdV5kGDcXKwuDuJ4HjdQRm5VMec2jN +LSjWFzXf81aOHu99CkRysHniB8a3BmQ67+ypsWb9jFXA/Ztdz1op/37c0AdscpKtSJM dkMwOyFbj2/nZMQtcNPR8XxQE5QkOd8d1liBBA/YeSTcruI0A/gnsPy9V7rEIR46be8m uL2w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:organisation:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=piOyFBwHJUdFJMcSB+q5jITz96GW9DeenQoWFiKcnP0=; b=kr1a2LlrbR4H9kPUsx2sRqJbv6Mw2r5y0U9XFASfNj30VuIrleaFVxxN1wRcAdJYbt 7pQ11yXdiEW00JrechPJcknKd5+QxR4K/PfZSOSa2N/hFerRaZF4VI0VXR08a+BI+Ueg j9sRZgRUZiSLkSptpAP43ZibU4QSwJizBAvqXgiGNbDGDce9ldWuC5c0wGP46wPCPEMD 4g1Tkxp0LKgYkaj/cBImsLimEDghXrw6HMFUk5TkBC3J2Fo1NO7tF0K652zjLv9asEUK kn950LXft3/VtEKkllmgutWyaeHDYZiH3f/WfCjOsCFyiZ+WCtqtjSGjdCnNXUp8vgvJ uoiQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@verge.net.au header.s=mail header.b=ho4oKh0C; 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 cm8si12567807plb.47.2019.03.19.05.27.36; Tue, 19 Mar 2019 05:27:52 -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; dkim=fail header.i=@verge.net.au header.s=mail header.b=ho4oKh0C; 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 S1727591AbfCSM0C (ORCPT + 99 others); Tue, 19 Mar 2019 08:26:02 -0400 Received: from kirsty.vergenet.net ([202.4.237.240]:47013 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726876AbfCSM0C (ORCPT ); Tue, 19 Mar 2019 08:26:02 -0400 Received: from reginn.horms.nl (watermunt.horms.nl [80.127.179.77]) by kirsty.vergenet.net (Postfix) with ESMTPA id E72B225B76D; Tue, 19 Mar 2019 23:25:58 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=verge.net.au; s=mail; t=1552998359; bh=W13YZhaBJ5eucexeaUcxCJszOyKnR0lgBq5HCNITuBE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ho4oKh0COPLG40kwmANq3K+879KolZ1z43eIO2M7k7VZ7gwHJHWkDnpDf2qlVjRku pNHAx50aXvfBff3DNhkB3P50pzI3J7BbKvAbseiWizPVVCD8o3thhHNU4ymC7SzOqj QRQ8oMnR7fv5s8upm8IeBOCQUiqKNWvmxWRN8fQs= Received: by reginn.horms.nl (Postfix, from userid 7100) id 414FB940663; Tue, 19 Mar 2019 13:25:57 +0100 (CET) Date: Tue, 19 Mar 2019 13:25:57 +0100 From: Simon Horman To: Wolfram Sang Cc: linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Lee Jones , Mark Brown , Support Opensource Subject: Re: [PATCH 3/6] mfd: da9063: remove platform_data support Message-ID: <20190319122556.7wpabcnwklv3fg3v@verge.net.au> References: <20190318154759.21978-1-wsa+renesas@sang-engineering.com> <20190318154759.21978-4-wsa+renesas@sang-engineering.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190318154759.21978-4-wsa+renesas@sang-engineering.com> Organisation: Horms Solutions BV User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 18, 2019 at 04:47:55PM +0100, Wolfram Sang wrote: > There are no in-kernel users anymore, so remove this outdated interface. > > Signed-off-by: Wolfram Sang > --- > drivers/mfd/da9063-core.c | 20 ++------------------ > drivers/mfd/da9063-i2c.c | 1 - > drivers/mfd/da9063-irq.c | 1 - > 3 files changed, 2 insertions(+), 20 deletions(-) > > diff --git a/drivers/mfd/da9063-core.c b/drivers/mfd/da9063-core.c > index 6e4ce49b4405..65ca288b0c90 100644 > --- a/drivers/mfd/da9063-core.c > +++ b/drivers/mfd/da9063-core.c > @@ -26,7 +26,6 @@ > #include > > #include > -#include > #include > > #include > @@ -165,7 +164,6 @@ static int da9063_clear_fault_log(struct da9063 *da9063) > > int da9063_device_init(struct da9063 *da9063, unsigned int irq) > { > - struct da9063_pdata *pdata = da9063->dev->platform_data; > int model, variant_id, variant_code; > int ret; > > @@ -173,24 +171,10 @@ int da9063_device_init(struct da9063 *da9063, unsigned int irq) > if (ret < 0) > dev_err(da9063->dev, "Cannot clear fault log\n"); > > - if (pdata) { > - da9063->flags = pdata->flags; > - da9063->irq_base = pdata->irq_base; > - } else { > - da9063->flags = 0; > - da9063->irq_base = -1; > - } > + da9063->flags = 0; > + da9063->irq_base = -1; If the flag and irq_base fields have fixed values can they be removed? As a follow-up? > da9063->chip_irq = irq; > > - if (pdata && pdata->init != NULL) { > - ret = pdata->init(da9063); > - if (ret != 0) { > - dev_err(da9063->dev, > - "Platform initialization failed.\n"); > - return ret; > - } > - } > - > ret = regmap_read(da9063->regmap, DA9063_REG_CHIP_ID, &model); > if (ret < 0) { > dev_err(da9063->dev, "Cannot read chip model id.\n"); > diff --git a/drivers/mfd/da9063-i2c.c b/drivers/mfd/da9063-i2c.c > index 50a24b1921d0..5f707b100e69 100644 > --- a/drivers/mfd/da9063-i2c.c > +++ b/drivers/mfd/da9063-i2c.c > @@ -22,7 +22,6 @@ > > #include > #include > -#include > #include > > #include > diff --git a/drivers/mfd/da9063-irq.c b/drivers/mfd/da9063-irq.c > index ecc0c8ce6c58..ac02c31c5802 100644 > --- a/drivers/mfd/da9063-irq.c > +++ b/drivers/mfd/da9063-irq.c > @@ -19,7 +19,6 @@ > #include > #include > #include > -#include > > #define DA9063_REG_EVENT_A_OFFSET 0 > #define DA9063_REG_EVENT_B_OFFSET 1 > -- > 2.11.0 >