linux-kernel - Re: [PATCH v3.1 0/4] clk: meson: gxbb-aoclk: Add CEC 32k clock

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: <d7c27631-e50a-501a-a94f-28a124f9aa74@baylibre.com>
Date:   Fri, 4 Aug 2017 12:14:45 +0200
From:   Neil Armstrong <narmstrong@...libre.com>
To:     jbrunet@...libre.com
Cc:     linux-clk@...r.kernel.org, linux-amlogic@...ts.infradead.org,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3.1 0/4] clk: meson: gxbb-aoclk: Add CEC 32k clock

On 08/01/2017 01:56 PM, Neil Armstrong wrote:
> In order to support the standalone CEC Controller on the Amlogic SoCs,
> a specific CEC 32K clock must be handled in the AO domain.
> 
> The CEC 32K AO Clock is a dual divider with dual counter to provide a more
> precise 32768Hz clock for the CEC subsystem from the external xtal.
> 
> The AO clocks management registers are spread among the AO register space,
> so this patch also adds management of these registers mappings then uses them
> for the CEC 32K AO clock management.
> 
> This patchset :
>  - updates the bindings accordingly
>  - switch driver to new bindings
>  - adds the CEC 32k clock
>  - adds the clock binding entry
> 
> The DT Update will be sent in another patchset.
> 
> Changes since v3 at [3] :
>  - fix typo in cec_32k set_rate
> 
> Changes since v2 at [2] :
>  - fix round_rate invalid handling
>  - enhance set_rate code thanks to Chris Moore
>  - add more explicit comments thanks to Martin
> 
> Changes since v1 at [1] :
>  - move bindings to parent syscon node and move clkc to subnode
>  - switch aoclkc to use regmap only register access
>  - introduce aoclk-regmap-gate for this purpose until regmap clocks are generic
> 
> [1] https://lkml.kernel.org/r/1499336663-23875-1-git-send-email-narmstrong@baylibre.com
> [2] https://lkml.kernel.org/r/1501235589-318-1-git-send-email-narmstrong@baylibre.com
> [3] https://lkml.kernel.org/r/1501504957-19476-1-git-send-email-narmstrong@baylibre.com
> 
> Neil Armstrong (4):
>   dt-bindings: clock: amlogic,gxbb-aoclkc: Update bindings
>   clk: meson: gxbb-aoclk: Switch to regmap for register access
>   dt-bindings: clock: gxbb-aoclk: Add CEC 32k clock
>   clk: meson: gxbb-aoclk: Add CEC 32k clock
> 
>  .../bindings/clock/amlogic,gxbb-aoclkc.txt         |  22 ++-
>  drivers/clk/meson/Makefile                         |   2 +-
>  drivers/clk/meson/gxbb-aoclk-32k.c                 | 194 +++++++++++++++++++++
>  drivers/clk/meson/gxbb-aoclk-regmap.c              |  46 +++++
>  drivers/clk/meson/gxbb-aoclk.c                     |  65 ++++---
>  drivers/clk/meson/gxbb-aoclk.h                     |  42 +++++
>  include/dt-bindings/clock/gxbb-aoclkc.h            |   1 +
>  7 files changed, 342 insertions(+), 30 deletions(-)
>  create mode 100644 drivers/clk/meson/gxbb-aoclk-32k.c
>  create mode 100644 drivers/clk/meson/gxbb-aoclk-regmap.c
>  create mode 100644 drivers/clk/meson/gxbb-aoclk.h
> 

Applied to clk-meson's next/drivers except for patch 3 to next/headers