linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/8] arm: ep93xx: CCF conversion
@ 2021-07-26 11:50 Nikita Shubin
  2021-07-26 11:50 ` [PATCH 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Nikita Shubin @ 2021-07-26 11:50 UTC (permalink / raw)
  To: Alexander Sverdlin, Geert Uytterhoeven
  Cc: Nikita Shubin,
	moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM...,
	Andrew Morton, Andrzej Pietrasiewicz, Anshuman Khandual,
	Ard Biesheuvel, Arnd Bergmann,
	open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM, Dmitry Torokhov,
	open list:FRAMEBUFFER LAYER, Geert Uytterhoeven,
	Jonathan Cameron, Krzysztof Kozlowski, Krzysztof Kozlowski,
	Kuninori Morimoto, Lars-Peter Clausen, Linus Walleij,
	moderated list:ARM PORT, open list:FRAMEBUFFER LAYER,
	open list:IIO SUBSYSTEM AND DRIVERS, open list:INPUT (KEYBOARD,
	MOUSE, JOYSTICK , TOUCHSCREEN)...,
	open list, open list:PWM SUBSYSTEM, open list:SPI SUBSYSTEM,
	Mark Brown, Mike Rapoport, Russell King, Uwe Kleine-König,
	YiFei Zhu

This series series of patches converts ep93xx to Common Clock Framework.

It consists of preparation patches to use clk_prepare_enable where it is 
needed, instead of clk_enable used in ep93xx drivers prior to CCF and
a patch converting mach-ep93xx/clock.c to CCF.

Link: https://lore.kernel.org/patchwork/cover/1445563/
Link: https://lore.kernel.org/patchwork/patch/1435884/

Alexander Sverdlin (7):
  iio: ep93xx: Prepare clock before using it
  spi: spi-ep93xx: Prepare clock before using it
  Input: ep93xx_keypad: Prepare clock before using it
  video: ep93xx: Prepare clock before using it
  dmaengine: ep93xx: Prepare clock before using it
  ASoC: cirrus: i2s: Prepare clock before using it
  pwm: ep93xx: Prepare clock before using it

Nikita Shubin (1):
  ep93xx: clock: convert in-place to COMMON_CLK

 arch/arm/Kconfig                       |   2 +-
 arch/arm/mach-ep93xx/clock.c           | 975 ++++++++++++++-----------
 arch/arm/mach-ep93xx/core.c            |   2 +-
 arch/arm/mach-ep93xx/soc.h             |  42 +-
 drivers/dma/ep93xx_dma.c               |   6 +-
 drivers/iio/adc/ep93xx_adc.c           |   6 +-
 drivers/input/keyboard/ep93xx_keypad.c |   4 +-
 drivers/pwm/pwm-ep93xx.c               |  12 +-
 drivers/spi/spi-ep93xx.c               |   4 +-
 drivers/video/fbdev/ep93xx-fb.c        |   4 +-
 sound/soc/cirrus/ep93xx-i2s.c          |  12 +-
 11 files changed, 605 insertions(+), 464 deletions(-)


base-commit: 64376a981a0e2e57c46efa63197c2ebb7dab35df
-- 
2.26.2


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

* [PATCH 4/8] video: ep93xx: Prepare clock before using it
  2021-07-26 11:50 [PATCH 0/8] arm: ep93xx: CCF conversion Nikita Shubin
@ 2021-07-26 11:50 ` Nikita Shubin
  2021-07-26 13:59 ` [PATCH v2 0/8] arm: ep93xx: CCF conversion Nikita Shubin
  2021-07-31 22:04 ` [PATCH " Linus Walleij
  2 siblings, 0 replies; 6+ messages in thread
From: Nikita Shubin @ 2021-07-26 11:50 UTC (permalink / raw)
  To: Alexander Sverdlin, Geert Uytterhoeven
  Cc: open list:FRAMEBUFFER LAYER, open list:FRAMEBUFFER LAYER, open list

From: Alexander Sverdlin <alexander.sverdlin@gmail.com>

Use clk_prepare_enable()/clk_disable_unprepare() in preparation for switch
to Common Clock Framework.

Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
---
 drivers/video/fbdev/ep93xx-fb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/video/fbdev/ep93xx-fb.c b/drivers/video/fbdev/ep93xx-fb.c
index ba33b4dce0df..2398b3d48fed 100644
--- a/drivers/video/fbdev/ep93xx-fb.c
+++ b/drivers/video/fbdev/ep93xx-fb.c
@@ -548,7 +548,7 @@ static int ep93xxfb_probe(struct platform_device *pdev)
 	}
 
 	ep93xxfb_set_par(info);
-	clk_enable(fbi->clk);
+	clk_prepare_enable(fbi->clk);
 
 	err = register_framebuffer(info);
 	if (err)
@@ -577,7 +577,7 @@ static int ep93xxfb_remove(struct platform_device *pdev)
 	struct ep93xx_fbi *fbi = info->par;
 
 	unregister_framebuffer(info);
-	clk_disable(fbi->clk);
+	clk_disable_unprepare(fbi->clk);
 	ep93xxfb_dealloc_videomem(info);
 	fb_dealloc_cmap(&info->cmap);
 
-- 
2.26.2


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

* [PATCH v2 0/8] arm: ep93xx: CCF conversion
  2021-07-26 11:50 [PATCH 0/8] arm: ep93xx: CCF conversion Nikita Shubin
  2021-07-26 11:50 ` [PATCH 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
@ 2021-07-26 13:59 ` Nikita Shubin
  2021-07-26 13:59   ` [PATCH v2 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
  2021-08-03 22:35   ` (subset) [PATCH v2 0/8] arm: ep93xx: CCF conversion Mark Brown
  2021-07-31 22:04 ` [PATCH " Linus Walleij
  2 siblings, 2 replies; 6+ messages in thread
From: Nikita Shubin @ 2021-07-26 13:59 UTC (permalink / raw)
  To: Alexander Sverdlin, Geert Uytterhoeven
  Cc: Nikita Shubin,
	moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM...,
	Andrew Morton, Andrzej Pietrasiewicz, Anshuman Khandual,
	Ard Biesheuvel, Arnd Bergmann,
	open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM, Dmitry Torokhov,
	open list:FRAMEBUFFER LAYER, Geert Uytterhoeven,
	Jonathan Cameron, Krzysztof Kozlowski, Krzysztof Kozlowski,
	Kuninori Morimoto, Lars-Peter Clausen, Linus Walleij,
	moderated list:ARM PORT, open list:FRAMEBUFFER LAYER,
	open list:IIO SUBSYSTEM AND DRIVERS, open list:INPUT (KEYBOARD,
	MOUSE, JOYSTICK , TOUCHSCREEN)...,
	open list, open list:PWM SUBSYSTEM, open list:SPI SUBSYSTEM,
	Mark Brown, Mike Rapoport, Russell King, Uwe Kleine-König,
	YiFei Zhu

This series series of patches converts ep93xx to Common Clock Framework.

It consists of preparation patches to use clk_prepare_enable where it is 
needed, instead of clk_enable used in ep93xx drivers prior to CCF and
a patch converting mach-ep93xx/clock.c to CCF.

Link: https://lore.kernel.org/patchwork/cover/1445563/
Link: https://lore.kernel.org/patchwork/patch/1435884/

v1->v2:
- added SoB

Alexander Sverdlin (7):
  iio: ep93xx: Prepare clock before using it
  spi: spi-ep93xx: Prepare clock before using it
  Input: ep93xx_keypad: Prepare clock before using it
  video: ep93xx: Prepare clock before using it
  dmaengine: ep93xx: Prepare clock before using it
  ASoC: cirrus: i2s: Prepare clock before using it
  pwm: ep93xx: Prepare clock before using it

Nikita Shubin (1):
  ep93xx: clock: convert in-place to COMMON_CLK

 arch/arm/Kconfig                       |   2 +-
 arch/arm/mach-ep93xx/clock.c           | 975 ++++++++++++++-----------
 arch/arm/mach-ep93xx/core.c            |   2 +-
 arch/arm/mach-ep93xx/soc.h             |  42 +-
 drivers/dma/ep93xx_dma.c               |   6 +-
 drivers/iio/adc/ep93xx_adc.c           |   6 +-
 drivers/input/keyboard/ep93xx_keypad.c |   4 +-
 drivers/pwm/pwm-ep93xx.c               |  12 +-
 drivers/spi/spi-ep93xx.c               |   4 +-
 drivers/video/fbdev/ep93xx-fb.c        |   4 +-
 sound/soc/cirrus/ep93xx-i2s.c          |  12 +-
 11 files changed, 605 insertions(+), 464 deletions(-)


base-commit: 64376a981a0e2e57c46efa63197c2ebb7dab35df
-- 
2.26.2


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

* [PATCH v2 4/8] video: ep93xx: Prepare clock before using it
  2021-07-26 13:59 ` [PATCH v2 0/8] arm: ep93xx: CCF conversion Nikita Shubin
@ 2021-07-26 13:59   ` Nikita Shubin
  2021-08-03 22:35   ` (subset) [PATCH v2 0/8] arm: ep93xx: CCF conversion Mark Brown
  1 sibling, 0 replies; 6+ messages in thread
From: Nikita Shubin @ 2021-07-26 13:59 UTC (permalink / raw)
  To: Alexander Sverdlin, Geert Uytterhoeven
  Cc: Nikita Shubin, open list:FRAMEBUFFER LAYER,
	open list:FRAMEBUFFER LAYER, open list

From: Alexander Sverdlin <alexander.sverdlin@gmail.com>

Use clk_prepare_enable()/clk_disable_unprepare() in preparation for switch
to Common Clock Framework.

Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me>
---
 drivers/video/fbdev/ep93xx-fb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/video/fbdev/ep93xx-fb.c b/drivers/video/fbdev/ep93xx-fb.c
index ba33b4dce0df..2398b3d48fed 100644
--- a/drivers/video/fbdev/ep93xx-fb.c
+++ b/drivers/video/fbdev/ep93xx-fb.c
@@ -548,7 +548,7 @@ static int ep93xxfb_probe(struct platform_device *pdev)
 	}
 
 	ep93xxfb_set_par(info);
-	clk_enable(fbi->clk);
+	clk_prepare_enable(fbi->clk);
 
 	err = register_framebuffer(info);
 	if (err)
@@ -577,7 +577,7 @@ static int ep93xxfb_remove(struct platform_device *pdev)
 	struct ep93xx_fbi *fbi = info->par;
 
 	unregister_framebuffer(info);
-	clk_disable(fbi->clk);
+	clk_disable_unprepare(fbi->clk);
 	ep93xxfb_dealloc_videomem(info);
 	fb_dealloc_cmap(&info->cmap);
 
-- 
2.26.2


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

* Re: [PATCH 0/8] arm: ep93xx: CCF conversion
  2021-07-26 11:50 [PATCH 0/8] arm: ep93xx: CCF conversion Nikita Shubin
  2021-07-26 11:50 ` [PATCH 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
  2021-07-26 13:59 ` [PATCH v2 0/8] arm: ep93xx: CCF conversion Nikita Shubin
@ 2021-07-31 22:04 ` Linus Walleij
  2 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2021-07-31 22:04 UTC (permalink / raw)
  To: Nikita Shubin
  Cc: Alexander Sverdlin, Geert Uytterhoeven,
	moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM...,
	Andrew Morton, Andrzej Pietrasiewicz, Anshuman Khandual,
	Ard Biesheuvel, Arnd Bergmann,
	open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM, Dmitry Torokhov,
	open list:FRAMEBUFFER LAYER, Geert Uytterhoeven,
	Jonathan Cameron, Krzysztof Kozlowski, Krzysztof Kozlowski,
	Kuninori Morimoto, Lars-Peter Clausen, moderated list:ARM PORT,
	open list:FRAMEBUFFER LAYER, open list:IIO SUBSYSTEM AND DRIVERS,
	open list:INPUT (KEYBOARD, MOUSE, JOYSTICK, TOUCHSCREEN)...,
	open list, open list:PWM SUBSYSTEM, open list:SPI SUBSYSTEM,
	Mark Brown, Mike Rapoport, Russell King, Uwe Kleine-König,
	YiFei Zhu

On Mon, Jul 26, 2021 at 1:51 PM Nikita Shubin <nikita.shubin@maquefel.me> wrote:

> This series series of patches converts ep93xx to Common Clock Framework.
>
> It consists of preparation patches to use clk_prepare_enable where it is
> needed, instead of clk_enable used in ep93xx drivers prior to CCF and
> a patch converting mach-ep93xx/clock.c to CCF.
>
> Link: https://lore.kernel.org/patchwork/cover/1445563/
> Link: https://lore.kernel.org/patchwork/patch/1435884/
>
> Alexander Sverdlin (7):
>   iio: ep93xx: Prepare clock before using it
>   spi: spi-ep93xx: Prepare clock before using it
>   Input: ep93xx_keypad: Prepare clock before using it
>   video: ep93xx: Prepare clock before using it
>   dmaengine: ep93xx: Prepare clock before using it
>   ASoC: cirrus: i2s: Prepare clock before using it
>   pwm: ep93xx: Prepare clock before using it
>
> Nikita Shubin (1):
>   ep93xx: clock: convert in-place to COMMON_CLK

This series is looking very good.
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

I suppose the per-subsystem patches can be picked up by
each subsystem maintainer and then you can send the "big patch"
to the SoC tree.

Yours,
Linus Walleij

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

* Re: (subset) [PATCH v2 0/8] arm: ep93xx: CCF conversion
  2021-07-26 13:59 ` [PATCH v2 0/8] arm: ep93xx: CCF conversion Nikita Shubin
  2021-07-26 13:59   ` [PATCH v2 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
@ 2021-08-03 22:35   ` Mark Brown
  1 sibling, 0 replies; 6+ messages in thread
From: Mark Brown @ 2021-08-03 22:35 UTC (permalink / raw)
  To: Alexander Sverdlin, Geert Uytterhoeven, Nikita Shubin
  Cc: Mark Brown, open list:PWM SUBSYSTEM, open list,
	Andrzej Pietrasiewicz, Anshuman Khandual, Russell King,
	moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM...,
	open list:INPUT KEYBOARD, MOUSE, JOYSTICK , TOUCHSCREEN...,
	YiFei Zhu, Geert Uytterhoeven, Lars-Peter Clausen, Linus Walleij,
	Krzysztof Kozlowski, moderated list:ARM PORT, Ard Biesheuvel,
	Arnd Bergmann, Dmitry Torokhov, Kuninori Morimoto,
	open list:FRAMEBUFFER LAYER, Mike Rapoport,
	Uwe Kleine-König, open list:SPI SUBSYSTEM,
	Krzysztof Kozlowski, Andrew Morton, Jonathan Cameron,
	open list:DMA GENERIC OFFLOAD ENGINE SUBSYSTEM,
	open list:FRAMEBUFFER LAYER, open list:IIO SUBSYSTEM AND DRIVERS

On Mon, 26 Jul 2021 16:59:48 +0300, Nikita Shubin wrote:
> This series series of patches converts ep93xx to Common Clock Framework.
> 
> It consists of preparation patches to use clk_prepare_enable where it is
> needed, instead of clk_enable used in ep93xx drivers prior to CCF and
> a patch converting mach-ep93xx/clock.c to CCF.
> 
> Link: https://lore.kernel.org/patchwork/cover/1445563/
> Link: https://lore.kernel.org/patchwork/patch/1435884/
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next

Thanks!

[2/8] spi: spi-ep93xx: Prepare clock before using it
      commit: 7c72dc56a631b87043e3c5838f5094db30d8c58d

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

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

end of thread, other threads:[~2021-08-03 22:36 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-26 11:50 [PATCH 0/8] arm: ep93xx: CCF conversion Nikita Shubin
2021-07-26 11:50 ` [PATCH 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
2021-07-26 13:59 ` [PATCH v2 0/8] arm: ep93xx: CCF conversion Nikita Shubin
2021-07-26 13:59   ` [PATCH v2 4/8] video: ep93xx: Prepare clock before using it Nikita Shubin
2021-08-03 22:35   ` (subset) [PATCH v2 0/8] arm: ep93xx: CCF conversion Mark Brown
2021-07-31 22:04 ` [PATCH " Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).