linux-kernel - Re: [PATCHv5 0/4] iio: adc: exynos_adc: Support Exynos3250 ADC and code clean

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 09 Jul 2014 16:27:29 +0900
From:	Chanwoo Choi <cw00.choi@...sung.com>
To:	kgene.kim@...sung.com
Cc:	Jonathan Cameron <jic23@...nel.org>, ch.naveen@...sung.com,
	t.figa@...sung.com, robh+dt@...nel.org, pawel.moll@....com,
	mark.rutland@....com, ijc+devicetree@...lion.org.uk,
	galak@...eaurora.org, rdunlap@...radead.org,
	sachin.kamat@...aro.org, kyungmin.park@...sung.com,
	linux-iio@...r.kernel.org, linux-samsung-soc@...r.kernel.org,
	linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCHv5 0/4] iio: adc: exynos_adc: Support Exynos3250 ADC and
 code clean

Dear Kukjin Kim,

This patch modified Exynos3250 dtsi to support ADC with IIO.
Could you please reivew this patchset about exynos3250.dtsi?

Best Regards,
Chanwoo Choi

On 06/30/2014 10:51 AM, Chanwoo Choi wrote:
> Dear Kukjin,
> 
> On 06/29/2014 08:50 PM, Jonathan Cameron wrote:
>> On 27/06/14 05:30, Chanwoo Choi wrote:
>>> Changes from v4:
>>> - Use 'exynos_adc_data' structure instead of 'exynos_adc_ops' structure
>>>    and remove enum variable of ADC version
>>> - Fix wrong name of special clock (sclk_tsadc -> sclk_adc)
>>> - Add reviewed message by Naveen Krishna Chatradhi
>>> - Add functions for ADC clock control
>>>
>>> Changes from v3:
>>> - Add new 'exynos_adc_ops' structure to improve readability according to
>>>   Tomasz Figa comment[1]
>>>   [1] https://lkml.org/lkml/2014/4/16/238
>>> - Add new 'exynos3250-adc-v2' compatible string to support Exynos3250 ADC
>>> - Fix wrong compaitlbe string of ADC in Exynos3250 dtsi file
>>>
>>> Changes from v2:
>>> - Check return value of clock function to deal with error exception
>>> - Fix minor coding style to improve readability
>>>
>>> Changes from v1:
>>> - Add new "samsung,exynos-adc-v3" compatible to support Exynos3250 ADC
>>> - Add a patch about DT binding documentation
>>>
>>> Chanwoo Choi (4):
>>>    iio: adc: exynos_adc: Add exynos_adc_data structure to improve readability
>>>    iio: adc: exynos_adc: Control special clock of ADC to support Exynos3250 ADC
>>>    iio: devicetree: Add DT binding documentation for Exynos3250 ADC
>>>    ARM: dts: Fix wrong compatible string for Exynos3250 ADC
>>>
>>>   .../devicetree/bindings/arm/samsung/exynos-adc.txt |  26 +-
>>>   arch/arm/boot/dts/exynos3250.dtsi                  |   4 +-
>>>   drivers/iio/adc/exynos_adc.c                       | 326 +++++++++++++++------
>>>   3 files changed, 268 insertions(+), 88 deletions(-)
>>>
>> I am happy with this series, but given it touches some exynos bindings, I would
>> like an ack from Kukjin Kim (or according to MAINTAINERS Ben Dooks) before taking it
>> all through IIO.
> 
> Could you please review this patchset about exynos dtsi and bindings?
> 
> Best Regards,
> Chanwoo Choi
> 
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/