From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161064AbcHEHBn (ORCPT ); Fri, 5 Aug 2016 03:01:43 -0400 Received: from mail-wm0-f44.google.com ([74.125.82.44]:36371 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966198AbcHEHBg (ORCPT ); Fri, 5 Aug 2016 03:01:36 -0400 Date: Fri, 5 Aug 2016 08:02:24 +0100 From: Lee Jones To: Alexander Stein Cc: linux-kernel@vger.kernel.org, Ksenija Stanojevic , dmitry.torokhov@gmail.com, linux-input@vger.kernel.org, jic23@kernel.org, knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, marex@denx.de, linux-iio@vger.kernel.org, harald@ccbib.org, stefan.wahren@i2se.com, fabio.estevam@freescale.com Subject: Re: [PATCH v4 1/4] mfd: mxs-lradc: Add support for mxs-lradc MFD Message-ID: <20160805070224.GH5243@dell> References: <0a60a10b249062d305237f04fdb9aea05b128b85.1470316956.git.ksenija.stanojevic@gmail.com> <2721285.1CS7CJCgc7@ws-stein> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <2721285.1CS7CJCgc7@ws-stein> User-Agent: Mutt/1.6.2 (2016-07-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 04 Aug 2016, Alexander Stein wrote: > On Thursday 04 August 2016 15:28:18, Ksenija Stanojevic wrote: > > Add core files for mxs-lradc MFD driver. > > > > Note: this patch won't compile in iio/testing without this patch: > > a8f447be8056 ("mfd: Add resource managed APIs for mfd_add_devices") > > > > Signed-off-by: Ksenija Stanojevic > > --- > > [...] > > +static int mxs_lradc_probe(struct platform_device *pdev) > > +{ > > + const struct of_device_id *of_id; > > + struct device *dev = &pdev->dev; > > + struct device_node *node = dev->of_node; > > + struct mxs_lradc *lradc; > > + struct resource *iores; > > + struct mfd_cell *cells = NULL; > > + int ret = 0; > > + u32 ts_wires = 0; > > + > > + lradc = devm_kzalloc(&pdev->dev, sizeof(*lradc), GFP_KERNEL); > > + if (!lradc) > > + return -ENOMEM; > > + > > + of_id = of_match_device(mxs_lradc_dt_ids, &pdev->dev); > > + lradc->soc = (enum mxs_lradc_id)of_id->data; > > + > > + iores = platform_get_resource(pdev, IORESOURCE_MEM, 0); > > + lradc->base = devm_ioremap_resource(dev, iores); > > Out of couriosity: Is it common to pass already ioremapped addresses to (MFD > cell) platform drivers? I would have expected, and done myself, that ioremap > is done in the driver itself, same as the IRQ. Yes, this is exactly what should happen. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog