Message-ID: <532D828A.8000108@kernel.org>
Date:	Sat, 22 Mar 2014 12:31:06 +0000
From:	Jonathan Cameron <jic23@...nel.org>
To:	Nicolas Ferre <nicolas.ferre@...el.com>,
	Alexandre Belloni <alexandre.belloni@...e-electrons.com>,
	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	ARM Maintainers <arm@...nel.org>
CC:	linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-input@...r.kernel.org,
	Gregory Clement <gregory.clement@...e-electrons.com>,
	Maxime Ripard <maxime.ripard@...e-electrons.com>,
	Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
	linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 00/15] iio: adc: at91 cleanups and atmel_tsadcc removal

On 19/03/14 10:19, Nicolas Ferre wrote:
> On 16/03/2014 19:30, Jonathan Cameron :
>> On 05/03/14 17:35, Alexandre Belloni wrote:
>>> This patch set is a followup of my previous series: iio: adc: at91 fixes
>>>
>>> I'm sorry it is so long, I made sure this is bissectable.
>> It's a nice series, don't apologise for doing things right!
>>
>> Clearly there are a few outstanding snippets from the reviews so far, but we
>> are getting to the point where we need to agree on a path for this.  I'm
>> happy to take the whole series through the IIO tree if Dmitry and the at91
>> maintainers are happy with that.  If it is considered two invasive form the
>> atmel side then I'm perfectly happy with someone to take it through the relevant
>> platform tree instead.
>
> Jonathan, Dmitry,
>
> Considering the amount of AT91 files touched by this series, I would
> like to take everything through Atmel's git tree and later on arm-soc.
Sure. Just as a warning, there are one or two patches queued up already that this
sits on top of.  Give a spot of bad timing these are too late to go in as fixes
before the release. Might be worth waiting on those hitting Linus' tree sometime
after rc1 before taking this series.
>
>
>> I'm obviously not going to pick it up at all until it has a complete set
>> of at91 maintainer acks though!
>>
>> Realistically it's missed the coming merge window anyway, so we have a fair
>> bit of time.
>
> Yes, I will review Alexandre's v2 series.
>
> Thanks a lot for your review and for having thought about the path that
> these patches would take.
>
> Bye,
>
>
>> Jonathan
>>>
>>> 1-3) The first 3 patches are cleaning up the patform_data used for at91_adc.
>>>
>>> 4-5) Then touchscreen support for older ADCs is added, this allows to use that
>>>        for the sam9m10g45ek.
>>>
>>> 6) Following those modifications, the mach/at91_adc.h is not used anywhere but
>>>      in the at91_adc driver so it is remove and its content (register definitions)
>>>      placed directly in the driver.
>>>
>>> 7-9) at91sam9rl support is added to at91_adc and is used for the at91sam9rl
>>>        based boards.
>>>
>>> 10-11) Prepare the atmel_tsadcc removal by switching sam9rl and sam9g45 to use
>>>          only at91_adc instead of atmel_tsadcc.
>>>
>>> 12-15) atmel_tsadcc removal
>>>
>>>
>>> Alexandre Belloni (15):
>>>     ARM: at91: sam9g45: remove unused platform_data
>>>     ARM: at91: sam9260: remove unused platform_data
>>>     iio: adc: at91: cleanup platform_data
>>>     iio: adc: at91_adc: Add support for touchscreens without TSMR
>>>     ARM: at91: sam9m10g45ek: Add touchscreen support through at91_adc
>>>     iio: adc: at91: remove unused include from include/mach
>>>     iio: adc: at91: add sam9rl support
>>>     ARM: at91: sam9rl: add at91_adc to support adc and touchscreen
>>>     ARM: at91: sam9rlek add touchscreen support through at91_adc
>>>     ARM: at91: sam9g45: switch from atmel_tsadcc to at91_adc
>>>     ARM: at91: sam9rl: switch from atmel_tsadcc to at91_adc
>>>     ARM: at91: remove atmel_tsadcc platform_data
>>>     ARM: at91: remove atmel_tsadcc from sama5_defconfig
>>>     Input: atmel_tsadcc: remove driver
>>>     ARM: at91/dt: at91-cosino_mega2560 remove useless tsadcc node
>>>
>>>    MAINTAINERS                                |   6 -
>>>    arch/arm/boot/dts/at91-cosino_mega2560.dts |   5 -
>>>    arch/arm/configs/at91sam9g45_defconfig     |   3 +-
>>>    arch/arm/configs/at91sam9rl_defconfig      |   3 +-
>>>    arch/arm/configs/sama5_defconfig           |   1 -
>>>    arch/arm/mach-at91/at91sam9260_devices.c   |  10 -
>>>    arch/arm/mach-at91/at91sam9g45.c           |   2 +-
>>>    arch/arm/mach-at91/at91sam9g45_devices.c   |  63 +----
>>>    arch/arm/mach-at91/at91sam9rl.c            |   7 +
>>>    arch/arm/mach-at91/at91sam9rl_devices.c    |  83 +++++--
>>>    arch/arm/mach-at91/board-sam9m10g45ek.c    |  16 +-
>>>    arch/arm/mach-at91/board-sam9rlek.c        |  16 +-
>>>    arch/arm/mach-at91/board.h                 |   3 -
>>>    arch/arm/mach-at91/include/mach/at91_adc.h | 107 ---------
>>>    drivers/iio/adc/at91_adc.c                 | 339 +++++++++++++++++++++++----
>>>    drivers/input/touchscreen/Kconfig          |  12 -
>>>    drivers/input/touchscreen/Makefile         |   1 -
>>>    drivers/input/touchscreen/atmel_tsadcc.c   | 358 -----------------------------
>>>    include/linux/platform_data/at91_adc.h     |  27 +--
>>>    include/linux/platform_data/atmel.h        |   7 -
>>>    20 files changed, 380 insertions(+), 689 deletions(-)
>>>    delete mode 100644 arch/arm/mach-at91/include/mach/at91_adc.h
>>>    delete mode 100644 drivers/input/touchscreen/atmel_tsadcc.c
>>>
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@...ts.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>
>
>

--
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/

Powered by blists - more mailing lists