* [PATCH V5 0/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-11 10:10 ` LI Qingwu 0 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-01-11 10:10 UTC (permalink / raw) To: abelvesa, mturquette, sboyd, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, abel.vesa Cc: Qing-wu.Li, 18701859600, bsp-development.geo Changes in V5: Add Reviewed-by: Abel Vesa. LI Qingwu (1): clk: imx8mp: Alias M7 SRC/DIV to M7 CORE drivers/clk/imx/clk-imx8mp.c | 2 ++ 1 file changed, 2 insertions(+) -- 2.25.1 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V5 0/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-11 10:10 ` LI Qingwu 0 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-01-11 10:10 UTC (permalink / raw) To: abelvesa, mturquette, sboyd, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, abel.vesa Cc: Qing-wu.Li, 18701859600, bsp-development.geo Changes in V5: Add Reviewed-by: Abel Vesa. LI Qingwu (1): clk: imx8mp: Alias M7 SRC/DIV to M7 CORE drivers/clk/imx/clk-imx8mp.c | 2 ++ 1 file changed, 2 insertions(+) -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE 2023-01-11 10:10 ` LI Qingwu @ 2023-01-11 10:10 ` LI Qingwu -1 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-01-11 10:10 UTC (permalink / raw) To: abelvesa, mturquette, sboyd, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, abel.vesa Cc: Qing-wu.Li, 18701859600, bsp-development.geo, Marco Felsch Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h but never assigned. It will cause the system to hang if using them. Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE for backward compatibility. Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> Reviewed-by: Abel Vesa <abel.vesa@linaro.org> Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> --- drivers/clk/imx/clk-imx8mp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c index 652ae58c2735..601eea7ab99d 100644 --- a/drivers/clk/imx/clk-imx8mp.c +++ b/drivers/clk/imx/clk-imx8mp.c @@ -522,6 +522,8 @@ static int imx8mp_clocks_probe(struct platform_device *pdev) hws[IMX8MP_CLK_A53_SRC] = hws[IMX8MP_CLK_A53_DIV]; hws[IMX8MP_CLK_A53_CG] = hws[IMX8MP_CLK_A53_DIV]; hws[IMX8MP_CLK_M7_CORE] = imx8m_clk_hw_composite_core("m7_core", imx8mp_m7_sels, ccm_base + 0x8080); + hws[IMX8MP_CLK_M7_DIV] = hws[IMX8MP_CLK_M7_CORE]; + hws[IMX8MP_CLK_M7_SRC] = hws[IMX8MP_CLK_M7_CORE]; hws[IMX8MP_CLK_ML_CORE] = imx8m_clk_hw_composite_core("ml_core", imx8mp_ml_sels, ccm_base + 0x8100); hws[IMX8MP_CLK_GPU3D_CORE] = imx8m_clk_hw_composite_core("gpu3d_core", imx8mp_gpu3d_core_sels, ccm_base + 0x8180); hws[IMX8MP_CLK_GPU3D_SHADER_CORE] = imx8m_clk_hw_composite("gpu3d_shader_core", imx8mp_gpu3d_shader_sels, ccm_base + 0x8200); -- 2.25.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-11 10:10 ` LI Qingwu 0 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-01-11 10:10 UTC (permalink / raw) To: abelvesa, mturquette, sboyd, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, abel.vesa Cc: Qing-wu.Li, 18701859600, bsp-development.geo, Marco Felsch Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h but never assigned. It will cause the system to hang if using them. Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE for backward compatibility. Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> Reviewed-by: Abel Vesa <abel.vesa@linaro.org> Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> --- drivers/clk/imx/clk-imx8mp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c index 652ae58c2735..601eea7ab99d 100644 --- a/drivers/clk/imx/clk-imx8mp.c +++ b/drivers/clk/imx/clk-imx8mp.c @@ -522,6 +522,8 @@ static int imx8mp_clocks_probe(struct platform_device *pdev) hws[IMX8MP_CLK_A53_SRC] = hws[IMX8MP_CLK_A53_DIV]; hws[IMX8MP_CLK_A53_CG] = hws[IMX8MP_CLK_A53_DIV]; hws[IMX8MP_CLK_M7_CORE] = imx8m_clk_hw_composite_core("m7_core", imx8mp_m7_sels, ccm_base + 0x8080); + hws[IMX8MP_CLK_M7_DIV] = hws[IMX8MP_CLK_M7_CORE]; + hws[IMX8MP_CLK_M7_SRC] = hws[IMX8MP_CLK_M7_CORE]; hws[IMX8MP_CLK_ML_CORE] = imx8m_clk_hw_composite_core("ml_core", imx8mp_ml_sels, ccm_base + 0x8100); hws[IMX8MP_CLK_GPU3D_CORE] = imx8m_clk_hw_composite_core("gpu3d_core", imx8mp_gpu3d_core_sels, ccm_base + 0x8180); hws[IMX8MP_CLK_GPU3D_SHADER_CORE] = imx8m_clk_hw_composite("gpu3d_shader_core", imx8mp_gpu3d_shader_sels, ccm_base + 0x8200); -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE 2023-01-11 10:10 ` LI Qingwu @ 2023-01-16 14:28 ` Abel Vesa -1 siblings, 0 replies; 16+ messages in thread From: Abel Vesa @ 2023-01-16 14:28 UTC (permalink / raw) To: Stephen Boyd Cc: LI Qingwu, abelvesa, mturquette, sboyd, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, bsp-development.geo, Marco Felsch On 23-01-11 11:10:30, LI Qingwu wrote: > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h > but never assigned. It will cause the system to hang if using them. > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE > for backward compatibility. > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> Stephen, can you please pick this up yourself? Thanks! > --- > drivers/clk/imx/clk-imx8mp.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c > index 652ae58c2735..601eea7ab99d 100644 > --- a/drivers/clk/imx/clk-imx8mp.c > +++ b/drivers/clk/imx/clk-imx8mp.c > @@ -522,6 +522,8 @@ static int imx8mp_clocks_probe(struct platform_device *pdev) > hws[IMX8MP_CLK_A53_SRC] = hws[IMX8MP_CLK_A53_DIV]; > hws[IMX8MP_CLK_A53_CG] = hws[IMX8MP_CLK_A53_DIV]; > hws[IMX8MP_CLK_M7_CORE] = imx8m_clk_hw_composite_core("m7_core", imx8mp_m7_sels, ccm_base + 0x8080); > + hws[IMX8MP_CLK_M7_DIV] = hws[IMX8MP_CLK_M7_CORE]; > + hws[IMX8MP_CLK_M7_SRC] = hws[IMX8MP_CLK_M7_CORE]; > hws[IMX8MP_CLK_ML_CORE] = imx8m_clk_hw_composite_core("ml_core", imx8mp_ml_sels, ccm_base + 0x8100); > hws[IMX8MP_CLK_GPU3D_CORE] = imx8m_clk_hw_composite_core("gpu3d_core", imx8mp_gpu3d_core_sels, ccm_base + 0x8180); > hws[IMX8MP_CLK_GPU3D_SHADER_CORE] = imx8m_clk_hw_composite("gpu3d_shader_core", imx8mp_gpu3d_shader_sels, ccm_base + 0x8200); > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-16 14:28 ` Abel Vesa 0 siblings, 0 replies; 16+ messages in thread From: Abel Vesa @ 2023-01-16 14:28 UTC (permalink / raw) To: Stephen Boyd Cc: LI Qingwu, abelvesa, mturquette, sboyd, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, bsp-development.geo, Marco Felsch On 23-01-11 11:10:30, LI Qingwu wrote: > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h > but never assigned. It will cause the system to hang if using them. > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE > for backward compatibility. > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> Stephen, can you please pick this up yourself? Thanks! > --- > drivers/clk/imx/clk-imx8mp.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c > index 652ae58c2735..601eea7ab99d 100644 > --- a/drivers/clk/imx/clk-imx8mp.c > +++ b/drivers/clk/imx/clk-imx8mp.c > @@ -522,6 +522,8 @@ static int imx8mp_clocks_probe(struct platform_device *pdev) > hws[IMX8MP_CLK_A53_SRC] = hws[IMX8MP_CLK_A53_DIV]; > hws[IMX8MP_CLK_A53_CG] = hws[IMX8MP_CLK_A53_DIV]; > hws[IMX8MP_CLK_M7_CORE] = imx8m_clk_hw_composite_core("m7_core", imx8mp_m7_sels, ccm_base + 0x8080); > + hws[IMX8MP_CLK_M7_DIV] = hws[IMX8MP_CLK_M7_CORE]; > + hws[IMX8MP_CLK_M7_SRC] = hws[IMX8MP_CLK_M7_CORE]; > hws[IMX8MP_CLK_ML_CORE] = imx8m_clk_hw_composite_core("ml_core", imx8mp_ml_sels, ccm_base + 0x8100); > hws[IMX8MP_CLK_GPU3D_CORE] = imx8m_clk_hw_composite_core("gpu3d_core", imx8mp_gpu3d_core_sels, ccm_base + 0x8180); > hws[IMX8MP_CLK_GPU3D_SHADER_CORE] = imx8m_clk_hw_composite("gpu3d_shader_core", imx8mp_gpu3d_shader_sels, ccm_base + 0x8200); > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE 2023-01-16 14:28 ` Abel Vesa @ 2023-01-18 19:00 ` Stephen Boyd -1 siblings, 0 replies; 16+ messages in thread From: Stephen Boyd @ 2023-01-18 19:00 UTC (permalink / raw) To: Abel Vesa Cc: LI Qingwu, abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, bsp-development.geo, Marco Felsch Quoting Abel Vesa (2023-01-16 06:28:33) > On 23-01-11 11:10:30, LI Qingwu wrote: > > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h > > but never assigned. It will cause the system to hang if using them. > > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE > > for backward compatibility. > > > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) > > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> > > Stephen, can you please pick this up yourself? > Is it needed for clk-fixes? When did the system start hanging? ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-18 19:00 ` Stephen Boyd 0 siblings, 0 replies; 16+ messages in thread From: Stephen Boyd @ 2023-01-18 19:00 UTC (permalink / raw) To: Abel Vesa Cc: LI Qingwu, abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, bsp-development.geo, Marco Felsch Quoting Abel Vesa (2023-01-16 06:28:33) > On 23-01-11 11:10:30, LI Qingwu wrote: > > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h > > but never assigned. It will cause the system to hang if using them. > > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE > > for backward compatibility. > > > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) > > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> > > Stephen, can you please pick this up yourself? > Is it needed for clk-fixes? When did the system start hanging? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE 2023-01-18 19:00 ` Stephen Boyd @ 2023-01-28 18:38 ` Abel Vesa -1 siblings, 0 replies; 16+ messages in thread From: Abel Vesa @ 2023-01-28 18:38 UTC (permalink / raw) To: Stephen Boyd, LI Qingwu, Peng Fan Cc: LI Qingwu, abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, bsp-development.geo, Marco Felsch On 23-01-18 11:00:36, Stephen Boyd wrote: > Quoting Abel Vesa (2023-01-16 06:28:33) > > On 23-01-11 11:10:30, LI Qingwu wrote: > > > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h > > > but never assigned. It will cause the system to hang if using them. > > > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE > > > for backward compatibility. > > > > > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) > > > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > > > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > > > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> > > > > Stephen, can you please pick this up yourself? > > > > Is it needed for clk-fixes? When did the system start hanging? Li, Peng, can you provide some details here? ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-28 18:38 ` Abel Vesa 0 siblings, 0 replies; 16+ messages in thread From: Abel Vesa @ 2023-01-28 18:38 UTC (permalink / raw) To: Stephen Boyd, LI Qingwu, Peng Fan Cc: LI Qingwu, abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, bsp-development.geo, Marco Felsch On 23-01-18 11:00:36, Stephen Boyd wrote: > Quoting Abel Vesa (2023-01-16 06:28:33) > > On 23-01-11 11:10:30, LI Qingwu wrote: > > > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in imx8mp-clock.h > > > but never assigned. It will cause the system to hang if using them. > > > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to IMX8MP_CLK_M7_CORE > > > for backward compatibility. > > > > > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core to simplify code) > > > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > > > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > > > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> > > > > Stephen, can you please pick this up yourself? > > > > Is it needed for clk-fixes? When did the system start hanging? Li, Peng, can you provide some details here? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE 2023-01-28 18:38 ` Abel Vesa @ 2023-01-30 7:44 ` LI Qingwu -1 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-01-30 7:44 UTC (permalink / raw) To: Abel Vesa, Stephen Boyd, Peng Fan Cc: abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, GEO-CHHER-bsp-development, Marco Felsch > -----Original Message----- > From: Abel Vesa <abel.vesa@linaro.org> > Sent: Sunday, January 29, 2023 2:39 AM > To: Stephen Boyd <sboyd@kernel.org>; LI Qingwu > <Qing-wu.Li@leica-geosystems.com.cn>; Peng Fan <peng.fan@nxp.com> > Cc: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>; abelvesa@kernel.org; > mturquette@baylibre.com; shawnguo@kernel.org; s.hauer@pengutronix.de; > kernel@pengutronix.de; festevam@gmail.com; linux-imx@nxp.com; > linux-clk@vger.kernel.org; linux-arm-kernel@lists.infradead.org; > linux-kernel@vger.kernel.org; 18701859600@163.com; > GEO-CHHER-bsp-development <bsp-development.geo@leica-geosystems.com>; > Marco Felsch <m.felsch@pengutronix.de> > Subject: Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE > > [Some people who received this message don't often get email from > abel.vesa@linaro.org. Learn why this is important at > https://aka.ms/LearnAboutSenderIdentification ] > > This email is not from Hexagon's Office 365 instance. Please be careful while > clicking links, opening attachments, or replying to this email. > > > On 23-01-18 11:00:36, Stephen Boyd wrote: > > Quoting Abel Vesa (2023-01-16 06:28:33) > > > On 23-01-11 11:10:30, LI Qingwu wrote: > > > > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in > imx8mp-clock.h > > > > but never assigned. It will cause the system to hang if using them. > > > > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to > > > > IMX8MP_CLK_M7_CORE for backward compatibility. > > > > > > > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core > > > > to simplify code) > > > > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > > > > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > > > > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> > > > > > > Stephen, can you please pick this up yourself? > > > > > > > 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. ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-01-30 7:44 ` LI Qingwu 0 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-01-30 7:44 UTC (permalink / raw) To: Abel Vesa, Stephen Boyd, Peng Fan Cc: abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, GEO-CHHER-bsp-development, Marco Felsch > -----Original Message----- > From: Abel Vesa <abel.vesa@linaro.org> > Sent: Sunday, January 29, 2023 2:39 AM > To: Stephen Boyd <sboyd@kernel.org>; LI Qingwu > <Qing-wu.Li@leica-geosystems.com.cn>; Peng Fan <peng.fan@nxp.com> > Cc: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>; abelvesa@kernel.org; > mturquette@baylibre.com; shawnguo@kernel.org; s.hauer@pengutronix.de; > kernel@pengutronix.de; festevam@gmail.com; linux-imx@nxp.com; > linux-clk@vger.kernel.org; linux-arm-kernel@lists.infradead.org; > linux-kernel@vger.kernel.org; 18701859600@163.com; > GEO-CHHER-bsp-development <bsp-development.geo@leica-geosystems.com>; > Marco Felsch <m.felsch@pengutronix.de> > Subject: Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE > > [Some people who received this message don't often get email from > abel.vesa@linaro.org. Learn why this is important at > https://aka.ms/LearnAboutSenderIdentification ] > > This email is not from Hexagon's Office 365 instance. Please be careful while > clicking links, opening attachments, or replying to this email. > > > On 23-01-18 11:00:36, Stephen Boyd wrote: > > Quoting Abel Vesa (2023-01-16 06:28:33) > > > On 23-01-11 11:10:30, LI Qingwu wrote: > > > > Defined IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV in > imx8mp-clock.h > > > > but never assigned. It will cause the system to hang if using them. > > > > Alias IMX8MP_CLK_M7_SRC and IMX8MP_CLK_M7_DIV to > > > > IMX8MP_CLK_M7_CORE for backward compatibility. > > > > > > > > Fixes: 8c83a8ff4dd9 (clk: imx8mp: use imx8m_clk_hw_composite_core > > > > to simplify code) > > > > Reviewed-by: Marco Felsch <m.felsch@pengutronix.de> > > > > Reviewed-by: Abel Vesa <abel.vesa@linaro.org> > > > > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn> > > > > > > Stephen, can you please pick this up yourself? > > > > > > > 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. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <911ff831eaa686c87682d8e95e90326c.sboyd@kernel.org>]
[parent not found: <AS8PR06MB7432D9E3D31FAF8087338548D7DD9@AS8PR06MB7432.eurprd06.prod.outlook.com>]
* RE: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE [not found] ` <AS8PR06MB7432D9E3D31FAF8087338548D7DD9@AS8PR06MB7432.eurprd06.prod.outlook.com> @ 2023-02-13 5:44 ` LI Qingwu 0 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-02-13 5:44 UTC (permalink / raw) To: Stephen Boyd, Abel Vesa, Peng Fan Cc: abelvesa <kernel.org, mturquette <baylibre.com, shawnguo <kernel.org, s.hauer <pengutronix.de, kernel <pengutronix.de, festevam <gmail.com, linux-kernel <vger.kernel.org, linux-imx <nxp.com, linux-clk <vger.kernel.org, linux-arm-kernel <lists.infradead.org, 18701859600 <163.com, GEO-CHHER-bsp-development, Marco Felsch > -----Original Message----- > From: Stephen Boyd <sboyd@kernel.org> > Sent: Tuesday, January 31, 2023 6:35 AM > To: Abel Vesa <abel.vesa@linaro.org>; LI Qingwu > <qing-wu.li@leica-geosystems.com.cn>; Peng Fan <peng.fan@nxp.com> > Cc: abelvesa@ <kernel.org abelvesa@kernel.org>; mturquette@ > <baylibre.com mturquette@baylibre.com>; shawnguo@ <kernel.org > shawnguo@kernel.org>; s.hauer@ <pengutronix.de > s.hauer@pengutronix.de>; kernel@ <pengutronix.de > kernel@pengutronix.de>; festevam@ <gmail.com festevam@gmail.com>; > linux-imx@ <nxp.com linux-imx@nxp.com>; linux-clk@ <vger.kernel.org > linux-clk@vger.kernel.org>; linux-arm-kernel@ <lists.infradead.org > linux-arm-kernel@lists.infradead.org>; linux-kernel@ <vger.kernel.org > linux-kernel@vger.kernel.org>; 18701859600@ <163.com > 18701859600@163.com>; GEO-CHHER-bsp-development > <bsp-development.geo@leica-geosystems.com>; Marco Felsch > <m.felsch@pengutronix.de> > Subject: RE: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE > > This email is not from Hexagon’s Office 365 instance. Please be > careful while clicking links, opening attachments, or replying to this email. > > > 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? Just check it's OK to apply or not? Why ? ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-02-13 5:44 ` LI Qingwu 0 siblings, 0 replies; 16+ messages in thread From: LI Qingwu @ 2023-02-13 5:44 UTC (permalink / raw) To: Stephen Boyd, Abel Vesa, Peng Fan Cc: abelvesa <kernel.org, mturquette <baylibre.com, shawnguo <kernel.org, s.hauer <pengutronix.de, kernel <pengutronix.de, festevam <gmail.com, linux-kernel <vger.kernel.org, linux-imx <nxp.com, linux-clk <vger.kernel.org, linux-arm-kernel <lists.infradead.org, 18701859600 <163.com, GEO-CHHER-bsp-development, Marco Felsch > -----Original Message----- > From: Stephen Boyd <sboyd@kernel.org> > Sent: Tuesday, January 31, 2023 6:35 AM > To: Abel Vesa <abel.vesa@linaro.org>; LI Qingwu > <qing-wu.li@leica-geosystems.com.cn>; Peng Fan <peng.fan@nxp.com> > Cc: abelvesa@ <kernel.org abelvesa@kernel.org>; mturquette@ > <baylibre.com mturquette@baylibre.com>; shawnguo@ <kernel.org > shawnguo@kernel.org>; s.hauer@ <pengutronix.de > s.hauer@pengutronix.de>; kernel@ <pengutronix.de > kernel@pengutronix.de>; festevam@ <gmail.com festevam@gmail.com>; > linux-imx@ <nxp.com linux-imx@nxp.com>; linux-clk@ <vger.kernel.org > linux-clk@vger.kernel.org>; linux-arm-kernel@ <lists.infradead.org > linux-arm-kernel@lists.infradead.org>; linux-kernel@ <vger.kernel.org > linux-kernel@vger.kernel.org>; 18701859600@ <163.com > 18701859600@163.com>; GEO-CHHER-bsp-development > <bsp-development.geo@leica-geosystems.com>; Marco Felsch > <m.felsch@pengutronix.de> > Subject: RE: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE > > This email is not from Hexagon’s Office 365 instance. Please be > careful while clicking links, opening attachments, or replying to this email. > > > 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? Just check it's OK to apply or not? Why ? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE [not found] ` <911ff831eaa686c87682d8e95e90326c.sboyd@kernel.org> @ 2023-02-13 7:41 ` Abel Vesa 2023-02-13 7:41 ` Abel Vesa 1 sibling, 0 replies; 16+ messages in thread From: Abel Vesa @ 2023-02-13 7:41 UTC (permalink / raw) To: Stephen Boyd, LI Qingwu Cc: LI Qingwu, Peng Fan, abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, GEO-CHHER-bsp-development, Marco Felsch 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. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V5 1/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE @ 2023-02-13 7:41 ` Abel Vesa 0 siblings, 0 replies; 16+ messages in thread From: Abel Vesa @ 2023-02-13 7:41 UTC (permalink / raw) To: Stephen Boyd, LI Qingwu Cc: LI Qingwu, Peng Fan, abelvesa, mturquette, shawnguo, s.hauer, kernel, festevam, linux-imx, linux-clk, linux-arm-kernel, linux-kernel, 18701859600, GEO-CHHER-bsp-development, Marco Felsch 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. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2023-02-13 7:43 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-01-11 10:10 [PATCH V5 0/1] clk: imx8mp: Alias M7 SRC/DIV to M7 CORE LI Qingwu 2023-01-11 10:10 ` LI Qingwu 2023-01-11 10:10 ` [PATCH V5 1/1] " LI Qingwu 2023-01-11 10:10 ` LI Qingwu 2023-01-16 14:28 ` Abel Vesa 2023-01-16 14:28 ` Abel Vesa 2023-01-18 19:00 ` Stephen Boyd 2023-01-18 19:00 ` Stephen Boyd 2023-01-28 18:38 ` Abel Vesa 2023-01-28 18:38 ` Abel Vesa 2023-01-30 7:44 ` LI Qingwu 2023-01-30 7:44 ` LI Qingwu [not found] ` <911ff831eaa686c87682d8e95e90326c.sboyd@kernel.org> [not found] ` <AS8PR06MB7432D9E3D31FAF8087338548D7DD9@AS8PR06MB7432.eurprd06.prod.outlook.com> 2023-02-13 5:44 ` LI Qingwu 2023-02-13 5:44 ` LI Qingwu 2023-02-13 7:41 ` Abel Vesa 2023-02-13 7:41 ` Abel Vesa
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.