From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wg0-f45.google.com ([74.125.82.45]:38020 "EHLO mail-wg0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751123AbbCMKXH (ORCPT ); Fri, 13 Mar 2015 06:23:07 -0400 Received: by wghl18 with SMTP id l18so22181676wgh.5 for ; Fri, 13 Mar 2015 03:23:06 -0700 (PDT) Message-ID: <5502BA89.8020100@linaro.org> Date: Fri, 13 Mar 2015 11:23:05 +0100 From: Daniel Lezcano MIME-Version: 1.0 To: Alexandre Belloni , Nicolas Ferre CC: Boris Brezillon , Jean-Christophe Plagniol-Villard , Wim Van Sebroeck , Guenter Roeck , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-watchdog@vger.kernel.org Subject: Re: [PATCH v5 00/10] Atmel System Timer cleanups References: <1426162054-9987-1-git-send-email-alexandre.belloni@free-electrons.com> In-Reply-To: <1426162054-9987-1-git-send-email-alexandre.belloni@free-electrons.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org On 03/12/2015 01:07 PM, Alexandre Belloni wrote: > This patch set cleans up the system timer driver. > > The main goal is to get rid of the mach/ headers dependency. At the s= ame time, > it introduces proper probing and locking (using a regmap) for the wat= chdog > driver. > > This is based on 4.0-rc1 and will have two merge conflicts with my cl= eanup #1 in > mach-at91/at91rm9200.c > > It also needs this patch to probe the watchdog properly: > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-March/3272= 74.html Hi Alexandre, through which tree do you want this patchset to be merged ? Thanks -- Daniel > Changes in v5: > - Documented the watchdog compatible > - Switched to simple-mfd to probe the system timer subnodes > - Selected ATMEL_ST from mach-at91/Kconfig instead of using def_boo= l SOC_AT91RM9200 > - Reordered the atmel-st probe cleanup and its move to driver/clock= source > - Added a patch to replace setup_irq by request_irq > - Merged the reset handling in the watchdog timer as we are using t= he watchdog > anyway to reset the board. > - Used a simple space after a #define for registers > > Changes in v4: > - Added MODULE_* metadata in at91rm9200-reset > - Proper copyright in at91rm9200-reset > - Use BIT() in atmel-st.h > - changed MODULE_ALIAS in at91rm9200_wdt.c > > Changes in v3: > - stop using an mfd and get the syscon directly from the watchdog d= river > - introduce a proper reset driver to handle reset > > Changes in v2: > - cleaned up more includes in the watchdog driver > - stop using if OF when selecting CLKSRC_OF > - stop initializing .owner > > > Alexandre Belloni (10): > ARM: at91/dt: declare atmel,at91rm9200-st as a syscon > mfd: syscon: Add atmel system timer registers definition > watchdog: at91rm9200: use the system timer syscon > watchdog: at91rm9200: implement restart handler > ARM: at91: at91rm9200: remove deprecated arm_pm_restart > ARM: at91: properly initialize timer > ARM: at91: time: move the system timer driver to drivers/clocksour= ce > clocksource: atmel-st: use syscon/regmap > clocksource: atmel-st: remove mach/hardware dependency > ARM: at91: remove useless include > > .../devicetree/bindings/arm/atmel-at91.txt | 4 +- > arch/arm/boot/dts/at91rm9200.dtsi | 6 +- > arch/arm/mach-at91/Kconfig | 1 + > arch/arm/mach-at91/Makefile | 2 +- > arch/arm/mach-at91/at91rm9200.c | 19 ---- > arch/arm/mach-at91/generic.h | 3 - > arch/arm/mach-at91/include/mach/at91_st.h | 61 ----------= - > drivers/clocksource/Kconfig | 4 + > drivers/clocksource/Makefile | 1 + > .../clocksource/timer-atmel-st.c | 115 ++++++++--= ----------- > drivers/watchdog/Kconfig | 2 +- > drivers/watchdog/at91rm9200_wdt.c | 61 +++++++++-= - > include/linux/mfd/syscon/atmel-st.h | 49 +++++++++ > 13 files changed, 164 insertions(+), 164 deletions(-) > delete mode 100644 arch/arm/mach-at91/include/mach/at91_st.h > rename arch/arm/mach-at91/at91rm9200_time.c =3D> drivers/clocksourc= e/timer-atmel-st.c (74%) > create mode 100644 include/linux/mfd/syscon/atmel-st.h > --=20 Linaro.org =E2=94=82 Open source software fo= r ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-watchdo= g" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html