linux-kernel - Re: [PATCH 08/10] ARM: dts: Cygnus: Fix SP805 clocks

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: <4d9e64dd-c081-f68e-0682-a9e30e8e4793@gmail.com>
Date:   Sun, 30 Aug 2020 20:59:35 -0700
From:   Florian Fainelli <f.fainelli@...il.com>
To:     bcm-kernel-feedback-list@...adcom.com,
        Andre Przywara <andre.przywara@....com>,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org
Cc:     Guenter Roeck <linux@...ck-us.net>,
        Wim Van Sebroeck <wim@...ux-watchdog.org>,
        Viresh Kumar <vireshk@...nel.org>,
        linux-watchdog@...r.kernel.org, linux-kernel@...r.kernel.org,
        Ray Jui <rjui@...adcom.com>,
        Scott Branden <sbranden@...adcom.com>
Subject: Re: [PATCH 08/10] ARM: dts: Cygnus: Fix SP805 clocks



On 8/30/2020 8:55 PM, Florian Fainelli wrote:
> On Fri, 28 Aug 2020 14:06:00 +0100, Andre Przywara <andre.przywara@....com> wrote:
>> The SP805 DT binding requires two clocks to be specified, but the
>> Broadcom Cygnus DT currently only specifies one clock.
>>
>> In practice, Linux would pick a clock named "apb_pclk" for the bus
>> clock, and the Linux and U-Boot SP805 driver would use the first clock
>> to derive the actual watchdog counter frequency.
>>
>> Since currently both are the very same clock, we can just double the
>> clock reference, and add the correct clock-names, to match the binding.
>>
>> Signed-off-by: Andre Przywara <andre.przywara@....com>
>> ---
> 
> Applied to qspi-fixes, thanks!

Applied to devicetree/next actually, likewise for the next patch.
-- 
Florian