linux-kernel - Re: [PATCH RESEND v12 3/8] clk: baikal-t1: Add shared xGMAC ref/ptp clocks internal parent

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: <20220930212530.7BA97C433C1@smtp.kernel.org>
Date:   Fri, 30 Sep 2022 14:25:27 -0700
From:   Stephen Boyd <sboyd@...nel.org>
To:     Michael Turquette <mturquette@...libre.com>,
        Serge Semin <Sergey.Semin@...kalelectronics.ru>
Cc:     Serge Semin <Sergey.Semin@...kalelectronics.ru>,
        Serge Semin <fancer.lancer@...il.com>,
        Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>,
        Pavel Parkhomenko <Pavel.Parkhomenko@...kalelectronics.ru>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
        linux-clk@...r.kernel.org, linux-mips@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH RESEND v12 3/8] clk: baikal-t1: Add shared xGMAC ref/ptp clocks internal parent

Quoting Serge Semin (2022-09-29 15:53:57)
> Baikal-T1 CCU reference manual says that both xGMAC reference and xGMAC
> PTP clocks are generated by two different wrappers with the same constant
> divider thus each producing a 156.25 MHz signal. But for some reason both
> of these clock sources are gated by a single switch-flag in the CCU
> registers space - CCU_SYS_XGMAC_BASE.BIT(0). In order to make the clocks
> handled independently we need to define a shared parental gate so the base
> clock signal would be switched off only if both of the child-clocks are
> disabled.
> 
> Note the ID is intentionally set to -2 since we are going to add a one
> more internal clock identifier in the next commit.
> 
> Fixes: 353afa3a8d2e ("clk: Add Baikal-T1 CCU Dividers driver")
> Signed-off-by: Serge Semin <Sergey.Semin@...kalelectronics.ru>
> 
> ---

Applied to clk-next