From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 863C1C43334 for ; Fri, 3 Jun 2022 17:24:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=j/EdBdpbKF7kmSrIcoVeHgNPaeaIbJoMF+eX+H8dqXg=; b=esa0KulduotD63 suS0D71ozSoRY0Rb8v574CZdPGpWCnkN+YORQOd/eMhdkMcZmNFKbvpEMSK0NgYOypTKPYGRtm6a0 eYAV0qfR7ELDA1+DHq40RCzFHBqmhzre2+knEWAeidplIRpc0FoQjUWiImBY+NoumpWcUgdb+4RFp Cho6ZKKMEK70XGtzv8GEzkaYUAnzr6Mp9Cx3cQS41BOtNZE3+DSMzmZILHgKtoCq11su09gfExPlj iLEMJoQOXeIOxfm8BhMpCajmrnYlpntr+pi8W9t2yXL8KWzbN7QPW0jmDoup1uewodP980S3Qk6NR 0yWCcZ00Roxms0qYD8iQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nxB1F-008IQw-Ri; Fri, 03 Jun 2022 17:23:29 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nxB1C-008IQK-9W for linux-arm-kernel@lists.infradead.org; Fri, 03 Jun 2022 17:23:27 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id EEBF661A9D; Fri, 3 Jun 2022 17:23:24 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A8D75C385A9; Fri, 3 Jun 2022 17:23:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1654277004; bh=IC0b4j9tRA7J2kGa80DzPEvWME4w+UNjiMU0VqZZJJc=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Wlep1SZv7MhpG4ewG6PDQwTxXox7R54NkMKwVTHUs9F/YpWxQSMrNcBhb5qR7fWKi tC/KpeNV2nL1DA91vvJra1NnfUUa0wTqIa6K1nfkChUrcsybLGFucsvmPt57rNNxTE l2j73b8UPWVfT1KJz6Npk7ZlnR2NhOE9qz56PzoGgWr77t+sKWNObUpHzK0piB1JpF ubYhfiNTfoSafXa3wU9dJWfYK40KR8iYrXMMOKOvaNrpC2bo5oj5zzll0isNkCzgqx 3T1t6C6Hyj8YipCj69a/hCr7TWaFxaXQE46FIBwKJ3VUBKLbWH1nB8nAGhq/jN961Z P06aZ/rk3o+Fw== Date: Fri, 3 Jun 2022 18:32:24 +0100 From: Jonathan Cameron To: Andy Shevchenko Cc: linux-iio@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Lars-Peter Clausen , Michal Simek Subject: Re: [PATCH v1 1/2] iio: adc: xilinx-xadc: Make use of device properties Message-ID: <20220603183224.540b3808@jic23-huawei> In-Reply-To: <20220531141118.64540-1-andriy.shevchenko@linux.intel.com> References: <20220531141118.64540-1-andriy.shevchenko@linux.intel.com> X-Mailer: Claws Mail 4.1.0 (GTK 3.24.34; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220603_102326_458729_2338D56B X-CRM114-Status: GOOD ( 31.12 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, 31 May 2022 17:11:17 +0300 Andy Shevchenko wrote: > Convert the module to be property provider agnostic and allow > it to be used on non-OF platforms. > > Add mod_devicetable.h include. > > Signed-off-by: Andy Shevchenko These both seem good to me, but as the driver is fairly actively maintained, I'll let this one sit on the list for a while so others can take a look. Whilst it 'seems' unlikely anyone will ever use this driver with other firmware I am keen to reduce the number of of-specific drivers in IIO just to avoid any chance of cut and paste. Who knows, I'm trying to run an aspeed-i2c driver with ACPI at the moment because it's handy for a emulated setup, so maybe the same will one day happen with this device :) Weirder things have happened. Jonathan > --- > drivers/iio/adc/xilinx-xadc-core.c | 39 ++++++++++++------------------ > 1 file changed, 16 insertions(+), 23 deletions(-) > > diff --git a/drivers/iio/adc/xilinx-xadc-core.c b/drivers/iio/adc/xilinx-xadc-core.c > index 823c8e5f9809..e883f95f0cda 100644 > --- a/drivers/iio/adc/xilinx-xadc-core.c > +++ b/drivers/iio/adc/xilinx-xadc-core.c > @@ -17,10 +17,11 @@ > #include > #include > #include > +#include > #include > -#include > #include > #include > +#include > #include > #include > > @@ -1182,14 +1183,13 @@ static const struct of_device_id xadc_of_match_table[] = { > }; > MODULE_DEVICE_TABLE(of, xadc_of_match_table); > > -static int xadc_parse_dt(struct iio_dev *indio_dev, struct device_node *np, > - unsigned int *conf, int irq) > +static int xadc_parse_dt(struct iio_dev *indio_dev, unsigned int *conf, int irq) > { > struct device *dev = indio_dev->dev.parent; > struct xadc *xadc = iio_priv(indio_dev); > const struct iio_chan_spec *channel_templates; > struct iio_chan_spec *channels, *chan; > - struct device_node *chan_node, *child; > + struct fwnode_handle *chan_node, *child; > unsigned int max_channels; > unsigned int num_channels; > const char *external_mux; > @@ -1200,7 +1200,7 @@ static int xadc_parse_dt(struct iio_dev *indio_dev, struct device_node *np, > > *conf = 0; > > - ret = of_property_read_string(np, "xlnx,external-mux", &external_mux); > + ret = device_property_read_string(dev, "xlnx,external-mux", &external_mux); > if (ret < 0 || strcasecmp(external_mux, "none") == 0) > xadc->external_mux_mode = XADC_EXTERNAL_MUX_NONE; > else if (strcasecmp(external_mux, "single") == 0) > @@ -1211,8 +1211,7 @@ static int xadc_parse_dt(struct iio_dev *indio_dev, struct device_node *np, > return -EINVAL; > > if (xadc->external_mux_mode != XADC_EXTERNAL_MUX_NONE) { > - ret = of_property_read_u32(np, "xlnx,external-mux-channel", > - &ext_mux_chan); > + ret = device_property_read_u32(dev, "xlnx,external-mux-channel", &ext_mux_chan); > if (ret < 0) > return ret; > > @@ -1247,19 +1246,19 @@ static int xadc_parse_dt(struct iio_dev *indio_dev, struct device_node *np, > num_channels = 9; > chan = &channels[9]; > > - chan_node = of_get_child_by_name(np, "xlnx,channels"); > + chan_node = device_get_named_child_node(dev, "xlnx,channels"); > if (chan_node) { > - for_each_child_of_node(chan_node, child) { > + fwnode_for_each_child_node(chan_node, child) { > if (num_channels >= max_channels) { > - of_node_put(child); > + fwnode_handle_put(child); > break; > } > > - ret = of_property_read_u32(child, "reg", ®); > + ret = fwnode_property_read_u32(child, "reg", ®); > if (ret || reg > 16) > continue; > > - if (of_property_read_bool(child, "xlnx,bipolar")) > + if (fwnode_property_read_bool(child, "xlnx,bipolar")) > chan->scan_type.sign = 's'; > > if (reg == 0) { > @@ -1273,7 +1272,7 @@ static int xadc_parse_dt(struct iio_dev *indio_dev, struct device_node *np, > chan++; > } > } > - of_node_put(chan_node); > + fwnode_handle_put(chan_node); > > /* No IRQ => no events */ > if (irq <= 0) { > @@ -1316,7 +1315,6 @@ static void xadc_cancel_delayed_work(void *data) > static int xadc_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > - const struct of_device_id *id; > const struct xadc_ops *ops; > struct iio_dev *indio_dev; > unsigned int bipolar_mask; > @@ -1326,15 +1324,10 @@ static int xadc_probe(struct platform_device *pdev) > int irq; > int i; > > - if (!dev->of_node) > - return -ENODEV; > - > - id = of_match_node(xadc_of_match_table, dev->of_node); > - if (!id) > + ops = device_get_match_data(dev); > + if (!ops) > return -EINVAL; > > - ops = id->data; > - > irq = platform_get_irq_optional(pdev, 0); > if (irq < 0 && > (irq != -ENXIO || !(ops->flags & XADC_FLAGS_IRQ_OPTIONAL))) > @@ -1345,7 +1338,7 @@ static int xadc_probe(struct platform_device *pdev) > return -ENOMEM; > > xadc = iio_priv(indio_dev); > - xadc->ops = id->data; > + xadc->ops = ops; > init_completion(&xadc->completion); > mutex_init(&xadc->mutex); > spin_lock_init(&xadc->lock); > @@ -1359,7 +1352,7 @@ static int xadc_probe(struct platform_device *pdev) > indio_dev->modes = INDIO_DIRECT_MODE; > indio_dev->info = &xadc_info; > > - ret = xadc_parse_dt(indio_dev, dev->of_node, &conf0, irq); > + ret = xadc_parse_dt(indio_dev, &conf0, irq); > if (ret) > return ret; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel