From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753421AbcHPMpj (ORCPT ); Tue, 16 Aug 2016 08:45:39 -0400 Received: from mail-wm0-f48.google.com ([74.125.82.48]:35628 "EHLO mail-wm0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753339AbcHPMpi (ORCPT ); Tue, 16 Aug 2016 08:45:38 -0400 Date: Tue, 16 Aug 2016 13:47:06 +0100 From: Lee Jones To: Jonathan Cameron Cc: Alexander Stein , linux-kernel@vger.kernel.org, Ksenija Stanojevic , dmitry.torokhov@gmail.com, linux-input@vger.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: <20160816124706.GJ25844@dell> References: <0a60a10b249062d305237f04fdb9aea05b128b85.1470316956.git.ksenija.stanojevic@gmail.com> <2721285.1CS7CJCgc7@ws-stein> <20160805070224.GH5243@dell> <22e16c7d-4ad7-6432-11e7-30b6b6194573@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <22e16c7d-4ad7-6432-11e7-30b6b6194573@kernel.org> 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 Mon, 15 Aug 2016, Jonathan Cameron wrote: > On 05/08/16 08:02, Lee Jones wrote: > > 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, you've confused me with this answer. Yes, it is common or Yes it > should have been done the way Alexander suggested? The remapping and IRQ obtainment should be done in the leaf driver. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog