linux-kernel - Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE

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

Message-ID: <Y+npxisKxJaaSrul@linaro.org>
Date:   Mon, 13 Feb 2023 09:41:58 +0200
From:   Abel Vesa <abel.vesa@...aro.org>
To:     Stephen Boyd <sboyd@...nel.org>,
        LI Qingwu <qing-wu.li@...ca-geosystems.com.cn>
Cc:     LI Qingwu <qing-wu.li@...ca-geosystems.com.cn>,
        Peng Fan <peng.fan@....com>,
        "abelvesa@...nel.org" <abelvesa@...nel.org>,
        "mturquette@...libre.com" <mturquette@...libre.com>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "festevam@...il.com" <festevam@...il.com>,
        "linux-imx@....com" <linux-imx@....com>,
        "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "18701859600@....com" <18701859600@....com>,
        GEO-CHHER-bsp-development 
        <bsp-development.geo@...ca-geosystems.com>,
        Marco Felsch <m.felsch@...gutronix.de>
Subject: Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE

On 23-01-30 14:34:30, Stephen Boyd wrote:
> Quoting LI Qingwu (2023-01-29 23:44:22)
> > 
> > > > Is it needed for clk-fixes? When did the system start hanging?
> > > 
> > > Li, Peng, can you provide some details here?
> > 
> > Since IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV were not assigned,
> > if using any one of them in device tree, Linux system hanging after start M7 core.
> 
>  $ git grep -e IMX8MP_CLK_M7_SRC -o -e IMX8MP_CLK_M7_DIV
>  include/dt-bindings/clock/imx8mp-clock.h:IMX8MP_CLK_M7_SRC
>  include/dt-bindings/clock/imx8mp-clock.h:IMX8MP_CLK_M7_DIV
> 
> so we're OK to let Abel apply this?

Replying here as the CC emails in Li's next-in-thread email are all messed up and
the reply I sent there bounced back.

As Stephen pointed out, in upstream, there is no current user for those
two clock IDs. So I suppose we can keep the Fixes tag, but currently,
on upstream, there is no chance a consumer could hang as there is no
consumer yet for these clocks.

Stephen, I'll add this to my 6.4 material, again, since there is no
actual comsumer yet, if that's OK with you.