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 X-Spam-Level: X-Spam-Status: No, score=-7.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9D166C282C4 for ; Tue, 12 Feb 2019 20:57:10 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6BB1B222C1 for ; Tue, 12 Feb 2019 20:57:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1550005030; bh=98rjWiStFkBn6SS6Uevu+Z3ZheXFAbV83L6MrRwEkEU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=kT45JPzRb6aV1bYE2y/mXjY7gnHnLrkeGwL9h85zry38b6HnllhktIng5XpeC8VJd JS7rUdKf1sXnpgr8Wh0SGUTJcLkhpMp6ab0uz/Sy0JCsSlw86BNPjNBO+wvhxRSivX Nzei/KYbu8QZdVLSKZ1mxheHf/70nIPgAB3jlcDs= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732256AbfBLU5I (ORCPT ); Tue, 12 Feb 2019 15:57:08 -0500 Received: from mail.kernel.org ([198.145.29.99]:38220 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727977AbfBLU5I (ORCPT ); Tue, 12 Feb 2019 15:57:08 -0500 Received: from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net [81.96.234.148]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id EB1CA222C0; Tue, 12 Feb 2019 20:57:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1550005027; bh=98rjWiStFkBn6SS6Uevu+Z3ZheXFAbV83L6MrRwEkEU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=PdXACQ23/cNTpis18t5R7I0ewz21izMr4S3znikocQp7cLXeklxqkasXtjxOi+nkk +ny/KVyuBcZ6TjGFX10uTgruzBBMNEmWPeUx6GyI23GQNe16t7MIf5LLn+key9mvg5 5wZd0SVGYMn5PXBZ3l/qhxdzV5Z6vAxBMdzbxlJ0= Date: Tue, 12 Feb 2019 20:57:01 +0000 From: Jonathan Cameron To: Krzysztof Kozlowski Cc: Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Rob Herring , Mark Rutland , Kukjin Kim , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] iio: adc: exynos-adc: Use proper number of channels for Exynos4x12 Message-ID: <20190212205701.4cf66d68@archlinux> In-Reply-To: <20190212174550.4383-2-krzk@kernel.org> References: <20190212174550.4383-1-krzk@kernel.org> <20190212174550.4383-2-krzk@kernel.org> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 12 Feb 2019 18:45:49 +0100 Krzysztof Kozlowski wrote: > Exynos4212 and Exynos4412 have only four ADC channels so using > "samsung,exynos-adc-v1" compatible (for eight channels ADCv1) on them is > wrong. Add a new compatible for Exynos4x12. > > Signed-off-by: Krzysztof Kozlowski I'll sneak this in as my last patch for this cycle (lucky timing ;). It could have done with a fixes tag. I'll mark it for stable as sounds and looks safe and simple. Looks like it goes all the way back so I'll leave it without one. Applied to the togreg branch of iio.git. This will be queued for the next merge window given how late we are in the cycle. Thanks, Jonathan > --- > .../bindings/iio/adc/samsung,exynos-adc.txt | 4 +++- > drivers/iio/adc/exynos_adc.c | 17 +++++++++++++++++ > 2 files changed, 20 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.txt b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.txt > index a10c1f89037d..e1fe02f3e3e9 100644 > --- a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.txt > +++ b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.txt > @@ -11,11 +11,13 @@ New driver handles the following > > Required properties: > - compatible: Must be "samsung,exynos-adc-v1" > - for exynos4412/5250 controllers. > + for Exynos5250 controllers. > Must be "samsung,exynos-adc-v2" for > future controllers. > Must be "samsung,exynos3250-adc" for > controllers compatible with ADC of Exynos3250. > + Must be "samsung,exynos4212-adc" for > + controllers compatible with ADC of Exynos4212 and Exynos4412. > Must be "samsung,exynos7-adc" for > the ADC in Exynos7 and compatibles > Must be "samsung,s3c2410-adc" for > diff --git a/drivers/iio/adc/exynos_adc.c b/drivers/iio/adc/exynos_adc.c > index fa2d2b5767f3..01f7c2570a1e 100644 > --- a/drivers/iio/adc/exynos_adc.c > +++ b/drivers/iio/adc/exynos_adc.c > @@ -115,6 +115,7 @@ > #define MAX_ADC_V2_CHANNELS 10 > #define MAX_ADC_V1_CHANNELS 8 > #define MAX_EXYNOS3250_ADC_CHANNELS 2 > +#define MAX_EXYNOS4212_ADC_CHANNELS 4 > #define MAX_S5PV210_ADC_CHANNELS 10 > > /* Bit definitions common for ADC_V1 and ADC_V2 */ > @@ -271,6 +272,19 @@ static void exynos_adc_v1_start_conv(struct exynos_adc *info, > writel(con1 | ADC_CON_EN_START, ADC_V1_CON(info->regs)); > } > > +/* Exynos4212 and 4412 is like ADCv1 but with four channels only */ > +static const struct exynos_adc_data exynos4212_adc_data = { > + .num_channels = MAX_EXYNOS4212_ADC_CHANNELS, > + .mask = ADC_DATX_MASK, /* 12 bit ADC resolution */ > + .needs_adc_phy = true, > + .phy_offset = EXYNOS_ADCV1_PHY_OFFSET, > + > + .init_hw = exynos_adc_v1_init_hw, > + .exit_hw = exynos_adc_v1_exit_hw, > + .clear_irq = exynos_adc_v1_clear_irq, > + .start_conv = exynos_adc_v1_start_conv, > +}; > + > static const struct exynos_adc_data exynos_adc_v1_data = { > .num_channels = MAX_ADC_V1_CHANNELS, > .mask = ADC_DATX_MASK, /* 12 bit ADC resolution */ > @@ -492,6 +506,9 @@ static const struct of_device_id exynos_adc_match[] = { > }, { > .compatible = "samsung,s5pv210-adc", > .data = &exynos_adc_s5pv210_data, > + }, { > + .compatible = "samsung,exynos4212-adc", > + .data = &exynos4212_adc_data, > }, { > .compatible = "samsung,exynos-adc-v1", > .data = &exynos_adc_v1_data,