[PATCH 0/6] clk: meson: Delete a meaningless spinlock from the MPLL
Chuan Liu via B4 Relay
devnull+chuan.liu.amlogic.com at kernel.org
Fri Sep 20 01:16:10 PDT 2024
The existing locking mechanism of CCF can effectively avoid concurrent
register access. struct meson_clk_mpll_data has no meaning in defining
a spinlock repeatedly.
In addition, the register corresponding to MPLL does not share the same
register with other module drivers, so there is no concurrent access to
the register with other modules drivers.
Every driver file with mpll defines a spinlock with the same name (even
if defined as "static"), giving the illusion of repeated definitions?
Signed-off-by: Chuan Liu <chuan.liu at amlogic.com>
---
Chuan Liu (6):
clk: meson: mpll: Delete a meaningless spinlock from the MPLL
clk: meson: axg: Delete the spinlock from the MPLL
clk: meson: meson8b: Delete the spinlock from the MPLL
clk: meson: gxbb: Delete the spinlock from the MPLL
clk: meson: g12a: Delete the spinlock from the MPLL
clk: meson: s4: Delete the spinlock from the MPLL
drivers/clk/meson/axg.c | 6 ------
drivers/clk/meson/clk-mpll.c | 11 -----------
drivers/clk/meson/clk-mpll.h | 1 -
drivers/clk/meson/g12a.c | 6 ------
drivers/clk/meson/gxbb.c | 6 ------
drivers/clk/meson/meson8b.c | 3 ---
drivers/clk/meson/s4-pll.c | 6 ------
7 files changed, 39 deletions(-)
---
base-commit: 0ef513560b53d499c824b77220c537eafe1df90d
change-id: 20240918-mpll_spinlock-4b9b55c44fd5
Best regards,
--
Chuan Liu <chuan.liu at amlogic.com>
More information about the linux-amlogic
mailing list