From patchwork Tue Jul 22 10:44:28 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Heiko Stuebner X-Patchwork-Id: 4600641 Return-Path: X-Original-To: patchwork-linux-samsung-soc@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 40FAEC0514 for ; Tue, 22 Jul 2014 10:43:26 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 45DE1200E5 for ; Tue, 22 Jul 2014 10:43:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 3BB7020145 for ; Tue, 22 Jul 2014 10:43:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752897AbaGVKnI (ORCPT ); Tue, 22 Jul 2014 06:43:08 -0400 Received: from gloria.sntech.de ([95.129.55.99]:34154 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750737AbaGVKnH (ORCPT ); Tue, 22 Jul 2014 06:43:07 -0400 Received: from ip9234425c.dynamic.kabel-deutschland.de ([146.52.66.92] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80) (envelope-from ) id 1X9XX6-0004hR-Di; Tue, 22 Jul 2014 12:42:56 +0200 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Arnd Bergmann Cc: Chanwoo Choi , jic23@kernel.org, ch.naveen@samsung.com, kgene.kim@samsung.com, kyungmin.park@samsung.com, t.figa@samsung.com, linux-iio@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-doc@vger.kernel.org Subject: Re: [PATCH 2/2] iio: adc: exynos_adc: Add support for S3C24xx ADC Date: Tue, 22 Jul 2014 12:44:28 +0200 Message-ID: <175757227.TPQdXAQiJt@diego> User-Agent: KMail/4.11.5 (Linux/3.13-1-amd64; KDE/4.11.3; x86_64; ; ) In-Reply-To: <5200888.DXoC06U953@wuerfel> References: <1405995074-3271-1-git-send-email-cw00.choi@samsung.com> <1405995074-3271-3-git-send-email-cw00.choi@samsung.com> <5200888.DXoC06U953@wuerfel> MIME-Version: 1.0 Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Am Dienstag, 22. Juli 2014, 10:39:38 schrieb Arnd Bergmann: > On Tuesday 22 July 2014 11:11:14 Chanwoo Choi wrote: > > This patch add support for s3c2410/s3c2416/s3c2440/s3c2443 ADC. The > > s3c24xx > > is alomost same as ADCv1. But, There are a little difference as following: > > - ADCMUX register address to select channel > > - ADCDAT mask (10bit or 12bit ADC resolution according to SoC version) > > Very good, thanks for doing this patch! > > (adding Heiko to Cc, he's probably interested in seeing this as well. indeed. Thanks for implementing this. While trying to build a test setup for this, I noticed two points: (1) I'm not sure what the second register (a "phy enable register" according to the binding) is supposed to be. According to binding and adc code it is mandatory, but I didn't find any lone adc register in the s3c2416 manual. (2) You might need something along the lines of: Thanks Heiko > > One comment: > > @@ -101,12 +107,14 @@ struct exynos_adc { > > > > struct completion completion; > > > > u32 value; > > > > + u32 value2; > > > > unsigned int version; > > > > }; > > > > ... > > @@ -365,7 +448,7 @@ static int exynos_read_raw(struct iio_dev *indio_dev, > > > > ret = -ETIMEDOUT; > > > > } else { > > > > *val = info->value; > > > > - *val2 = 0; > > + *val2 = info->value2; > > > > ret = IIO_VAL_INT; > > > > } > > > > @@ -377,9 +460,11 @@ static int exynos_read_raw(struct iio_dev *indio_dev, > > > > static irqreturn_t exynos_adc_isr(int irq, void *dev_id) > > { > > > > struct exynos_adc *info = (struct exynos_adc *)dev_id; > > > > + u32 mask = info->data->mask; > > > > /* Read value */ > > > > - info->value = readl(ADC_V1_DATX(info->regs)) & ADC_DATX_MASK; > > + info->value = readl(ADC_V1_DATX(info->regs)) & mask; > > + info->value2 = readl(ADC_V1_DATY(info->regs)) & mask; > > > > /* clear irq */ > > if (info->data->clear_irq) > > If I understand it right, this would only be necessary if we want > to do the touchscreen driver as a separate iio client using the > in-kernel interfaces. As Jonathan Cameron commented, we probably > don't want to do that though. Even if we do, it should be a separate > patch and not mixed in with the s3c24xx support. > > Aside from this: > > Acked-by: Arnd Bergmann > > Arnd --- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig index 11b048a..088c99a 100644 --- a/drivers/iio/adc/Kconfig +++ b/drivers/iio/adc/Kconfig @@ -129,7 +129,7 @@ config AT91_ADC config EXYNOS_ADC tristate "Exynos ADC driver support" - depends on ARCH_EXYNOS || (OF && COMPILE_TEST) + depends on ARCH_EXYNOS || ARCH_S3C24XX || ARCH_S3C64XX || (OF && COMPILE_TEST) help Core support for the ADC block found in the Samsung EXYNOS series of SoCs for drivers such as the touchscreen and hwmon to use to share