linux-kernel - Re: [PATCH v3 0/7] clocksource: sh_cmt: Update driver for DT binding rework

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: <2100093.DY5UYTmyGT@avalon>
Date:   Tue, 26 Sep 2017 11:26:43 +0300
From:   Laurent Pinchart <laurent.pinchart@...asonboard.com>
To:     Geert Uytterhoeven <geert+renesas@...der.be>
Cc:     Daniel Lezcano <daniel.lezcano@...aro.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Magnus Damm <damm+renesas@...nsource.se>,
        devicetree@...r.kernel.org, linux-renesas-soc@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/7] clocksource: sh_cmt: Update driver for DT binding rework

Hi Geert,

Thank you for the patches.

On Monday, 18 September 2017 16:46:40 EEST Geert Uytterhoeven wrote:
> 	Hi Daniel, Thomas, Rob, Mark, Magnus,
> 
> This patch series updates the sh_cmt clocksource driver for the recent DT
> binding rework that was merged in v4.14-rc1.
> 
> This series is an evolutionary improvement of Magnus Damm's series "[PATCH
> v2 00/11] clocksource: sh_cmt: DT binding rework V2".
> It complements Magnus' series "[PATCH v3 00/06] clocksource: sh_cmt: DT
> binding rework V3" and "[PATCH v4 00/06] clocksource: sh_cmt: DT binding
> rework V4", which included DT binding updates only.
> 
>   - Patch 1 fixes wrong vendor/soc ordering in the SoC-specific compatible
>     values  (This issue was raised during review before, but was not
>     addressed in later iterations of the DT binding rework.  Oh well,
>     things like that happen when there's more than 2 years between initial
>     submission and acceptance...),
>   - Patches 2 to 6 implement the updated DT bindings, as already accepted
>     upstream, and amended in patch 1.
>   - Patch 7 is a small cleanup.
> 
> Changes compared to v2:
>   - Take over from Magnus,
>   - Drop DT binding updates that were revised and accepted upstream,
>   - Fix SoC-specific compatible values,
>   - Change channels_mask from "unsigned long" to "unsigned int,
>   - Remove support for "renesas,cmt-32*",
>   - Mark "renesas,cmt-48-gen2" deprecated,
>   - Remove unused "renesas,channels-mask" handling,
>   - Use of_device_get_match_data() helper.
> 
> Thanks!
> 
> Geert Uytterhoeven (5):
>   dt-bindings: timer: renesas,cmt: Fix SoC-specific compatible values
>   clocksource: sh_cmt: Remove support for "renesas,cmt-32*"
>   clocksource: sh_cmt: Mark "renesas,cmt-48-gen2" deprecated
>   clocksource: sh_cmt: Remove unused "renesas,channels-mask" handling
>   clocksource: sh_cmt: Use of_device_get_match_data() helper
> 
> Magnus Damm (2):
>   clocksource: sh_cmt: Use 0x3f mask for SH_CMT_48BIT case
>   clocksource: sh_cmt: Support separate R-Car Gen2 CMT0/1

For the whole series,

Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>

>  .../devicetree/bindings/timer/renesas,cmt.txt      | 24 +++----
>  drivers/clocksource/sh_cmt.c                       | 76 +++++++++----------
>  2 files changed, 45 insertions(+), 55 deletions(-)

-- 
Regards,

Laurent Pinchart