mirror of https://lore.kernel.org/linux-amlogic/
 help / color / mirror / Atom feed
* [PATCH 0/6] clk: meson: Delete a meaningless spinlock from the MPLL
@ 2024-09-20  8:16 Chuan Liu via B4 Relay
  2024-09-20  8:16 ` [PATCH 1/6] clk: meson: mpll: " Chuan Liu via B4 Relay
                   ` (6 more replies)
  0 siblings, 7 replies; 9+ messages in thread
From: Chuan Liu via B4 Relay @ 2024-09-20  8:16 UTC (permalink / raw)
  To: Neil Armstrong, Jerome Brunet, Michael Turquette, Stephen Boyd,
	Kevin Hilman, Martin Blumenstingl
  Cc: linux-amlogic, linux-clk, linux-arm-kernel, linux-kernel, Chuan Liu

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@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@amlogic.com>



_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2024-09-24  8:48 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-09-20  8:16 [PATCH 0/6] clk: meson: Delete a meaningless spinlock from the MPLL Chuan Liu via B4 Relay
2024-09-20  8:16 ` [PATCH 1/6] clk: meson: mpll: " Chuan Liu via B4 Relay
2024-09-20  8:16 ` [PATCH 2/6] clk: meson: axg: Delete the " Chuan Liu via B4 Relay
2024-09-20  8:16 ` [PATCH 3/6] clk: meson: meson8b: " Chuan Liu via B4 Relay
2024-09-20  8:16 ` [PATCH 4/6] clk: meson: gxbb: " Chuan Liu via B4 Relay
2024-09-20  8:16 ` [PATCH 5/6] clk: meson: g12a: " Chuan Liu via B4 Relay
2024-09-20  8:16 ` [PATCH 6/6] clk: meson: s4: " Chuan Liu via B4 Relay
2024-09-24  8:35 ` [PATCH 0/6] clk: meson: Delete a meaningless " Jerome Brunet
2024-09-24  8:48   ` Chuan Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®