linux-kernel - Re: [PATCH v2 0/2] clk: meson: g12a: Add CPU Clock support

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] [day] [month] [year] [list]

Message-ID: <cd33907c-9b58-3e62-c490-35c184746367@baylibre.com>
Date:   Tue, 19 Mar 2019 21:23:09 +0100
From:   Neil Armstrong <narmstrong@...libre.com>
To:     Jerome Brunet <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 v2 0/2] clk: meson: g12a: Add CPU Clock support

On 19/03/2019 10:39, Jerome Brunet wrote:
> On Mon, 2019-03-04 at 14:11 +0100, Neil Armstrong wrote:
>> This patchset adds support for the clock tree feeding the 4xCortex A53
>> cpu cluster.
>>
>> This patchet does not handle clock changing, this will be added in a
>> secondary patchset.
>>
>> The CPU clock can either use the SYS_PLL for > 1GHz frequencies or
>> use a couple of div+mux from 1GHz/667MHz/24MHz source with 2 non-glitch
>> muxes.
>>
>> The CPU clock must be switched to a safe clock while changing the clocks
>> before the non-glitch muxes. Proper support will be added later.
>>
>> In this patchset, clocks are set read-only.
>>
>> Changes since v1:
>> - moved to clk_regmap_gate_ro_ops for R/O gates
>> - added comments with datasheet field names
>> - moved pribate bindings changes to driver patch
>> - removed invalid PCIE IDs
>>
>> Neil Armstrong (2):
>>   clk: meson-g12a: add cpu clock bindings
>>   clk: meson: g12a: add cpu clocks
>>
>>  drivers/clk/meson/g12a.c              | 350 ++++++++++++++++++++++++++
>>  drivers/clk/meson/g12a.h              |  22 +-
>>  include/dt-bindings/clock/g12a-clkc.h |   1 +
>>  3 files changed, 372 insertions(+), 1 deletion(-)
>>
> 
> With the small naming comment on patch 2
> 
> Acked-by: Jerome Brunet <jbrunet@...libre.com>
> 

I fixed the naming while applying on next/headers for patch 1, next/drivers for patch 2

Neil