linux-kernel - Re: [PATCH v5 00/10] Atmel System Timer cleanups

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]

Message-ID: <5502BA89.8020100@linaro.org>
Date:	Fri, 13 Mar 2015 11:23:05 +0100
From:	Daniel Lezcano <daniel.lezcano@...aro.org>
To:	Alexandre Belloni <alexandre.belloni@...e-electrons.com>,
	Nicolas Ferre <nicolas.ferre@...el.com>
CC:	Boris Brezillon <boris.brezillon@...e-electrons.com>,
	Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
	Wim Van Sebroeck <wim@...ana.be>,
	Guenter Roeck <linux@...ck-us.net>,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	linux-watchdog@...r.kernel.org
Subject: Re: [PATCH v5 00/10] Atmel System Timer cleanups

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 same time,
> it introduces proper probing and locking (using a regmap) for the watchdog
> driver.
>
> This is based on 4.0-rc1 and will have two merge conflicts with my cleanup #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/327274.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_bool SOC_AT91RM9200
>   - Reordered the atmel-st probe cleanup and its move to driver/clocksource
>   - Added a patch to replace setup_irq by request_irq
>   - Merged the reset handling in the watchdog timer as we are using the 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 driver
>   - 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/clocksource
>    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 => drivers/clocksource/timer-atmel-st.c (74%)
>   create mode 100644 include/linux/mfd/syscon/atmel-st.h
>


-- 
  <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

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