All of lore.kernel.org
 help / color / mirror / Atom feed
From: Abel Vesa <abel.vesa@linaro.org>
To: Adam Ford <aford173@gmail.com>
Cc: linux-clk@vger.kernel.org, aford@beaconembedded.com,
	Abel Vesa <abelvesa@kernel.org>, Peng Fan <peng.fan@nxp.com>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH V4 0/4] clk: imx: Improve imx8mm/imx8mn LCDIF clocks
Date: Fri, 31 Mar 2023 15:35:38 +0300	[thread overview]
Message-ID: <ZCbTmitvBTQu26PX@linaro.org> (raw)
In-Reply-To: <20230323230127.120883-1-aford173@gmail.com>

On 23-03-23 18:01:23, Adam Ford wrote:
> Both the i.MX8M Mini and Nano have a video_pll which can be used 
> to source a clock which feeds the lcdif interface.  This interface
> currently fixes video_pll and divides down the clock feeding LCDIF.
> However, when connected to an HDMI bridge chip that supports a
> variety of video resolutions and refresh rates, the only settings
> that properly sync are ones that evenly divide from the video_pll_out
> clock.
> 
> This series adds the ability for the clk-compolsite-8m to
> request a better parent clock rate if the proper clock flag is
> enable and sets that flag in the corresponding imx8mm and 
> imx8mn video_pll clocks to increase the number of resolutions
> and refresh rates timings that the LCDIF can produce.
> 
> This also has a side benefit of allowing the video-pll to run
> at a lower clock speed which can potentially save some power
> depending on the requested resolution and refresh rate.
> 

Applied, thanks!

[1/4] clk: imx: composite-8m: Add support to determine_rate
      commit: 156e96ff2172518b6f83e97d8f11f677bc668e22
[2/4] clk: imx: Add imx8m_clk_hw_composite_flags macro
      commit: 784a9b3916e949c00666588fd167c4ab245ec9d6
[3/4] clk: imx8mm: Let IMX8MM_CLK_LCDIF_PIXEL set parent rate
      commit: 5fe6ec93f10b0765d59e0efb6ecba419a6a49d48
[4/4] clk: imx: Let IMX8MN_CLK_DISP_PIXEL set parent rate
      commit: 46a974433ea7fa468b45db70536f7cea81feb87c


Best regards,
--
Abel Vesa <abel.vesa@linaro.org>



> v4:  Sorry for the noise.  I forgot to 'git ammend' so V4
>      is what V3 should have been.
> V3:  Change the name of clk_divider_determine_rate to
>       imx8m_clk_divider_determine_rate
> V2:  Split off the new imx8m_clk_hw_composite_flags definition
>      into its own patch and re-order to fix build error.
>       
> Adam Ford (3):
>   clk: imx: composite-8m: Add support to determine_rate
>   clk: imx8mm: Let IMX8MM_CLK_LCDIF_PIXEL set parent rate
>   clk: imx: Let IMX8MN_CLK_DISP_PIXEL set parent rate
> 
>  drivers/clk/imx/clk-composite-8m.c | 7 +++++++
>  drivers/clk/imx/clk-imx8mm.c       | 2 +-
>  drivers/clk/imx/clk-imx8mn.c       | 2 +-
>  drivers/clk/imx/clk.h              | 4 ++++
>  4 files changed, 13 insertions(+), 2 deletions(-)
> 
> -- 
> 2.34.1
> 

WARNING: multiple messages have this Message-ID (diff)
From: Abel Vesa <abel.vesa@linaro.org>
To: Adam Ford <aford173@gmail.com>
Cc: linux-clk@vger.kernel.org, aford@beaconembedded.com,
	Abel Vesa <abelvesa@kernel.org>, Peng Fan <peng.fan@nxp.com>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH V4 0/4] clk: imx: Improve imx8mm/imx8mn LCDIF clocks
Date: Fri, 31 Mar 2023 15:35:38 +0300	[thread overview]
Message-ID: <ZCbTmitvBTQu26PX@linaro.org> (raw)
In-Reply-To: <20230323230127.120883-1-aford173@gmail.com>

On 23-03-23 18:01:23, Adam Ford wrote:
> Both the i.MX8M Mini and Nano have a video_pll which can be used 
> to source a clock which feeds the lcdif interface.  This interface
> currently fixes video_pll and divides down the clock feeding LCDIF.
> However, when connected to an HDMI bridge chip that supports a
> variety of video resolutions and refresh rates, the only settings
> that properly sync are ones that evenly divide from the video_pll_out
> clock.
> 
> This series adds the ability for the clk-compolsite-8m to
> request a better parent clock rate if the proper clock flag is
> enable and sets that flag in the corresponding imx8mm and 
> imx8mn video_pll clocks to increase the number of resolutions
> and refresh rates timings that the LCDIF can produce.
> 
> This also has a side benefit of allowing the video-pll to run
> at a lower clock speed which can potentially save some power
> depending on the requested resolution and refresh rate.
> 

Applied, thanks!

[1/4] clk: imx: composite-8m: Add support to determine_rate
      commit: 156e96ff2172518b6f83e97d8f11f677bc668e22
[2/4] clk: imx: Add imx8m_clk_hw_composite_flags macro
      commit: 784a9b3916e949c00666588fd167c4ab245ec9d6
[3/4] clk: imx8mm: Let IMX8MM_CLK_LCDIF_PIXEL set parent rate
      commit: 5fe6ec93f10b0765d59e0efb6ecba419a6a49d48
[4/4] clk: imx: Let IMX8MN_CLK_DISP_PIXEL set parent rate
      commit: 46a974433ea7fa468b45db70536f7cea81feb87c


Best regards,
--
Abel Vesa <abel.vesa@linaro.org>



> v4:  Sorry for the noise.  I forgot to 'git ammend' so V4
>      is what V3 should have been.
> V3:  Change the name of clk_divider_determine_rate to
>       imx8m_clk_divider_determine_rate
> V2:  Split off the new imx8m_clk_hw_composite_flags definition
>      into its own patch and re-order to fix build error.
>       
> Adam Ford (3):
>   clk: imx: composite-8m: Add support to determine_rate
>   clk: imx8mm: Let IMX8MM_CLK_LCDIF_PIXEL set parent rate
>   clk: imx: Let IMX8MN_CLK_DISP_PIXEL set parent rate
> 
>  drivers/clk/imx/clk-composite-8m.c | 7 +++++++
>  drivers/clk/imx/clk-imx8mm.c       | 2 +-
>  drivers/clk/imx/clk-imx8mn.c       | 2 +-
>  drivers/clk/imx/clk.h              | 4 ++++
>  4 files changed, 13 insertions(+), 2 deletions(-)
> 
> -- 
> 2.34.1
> 

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

  parent reply	other threads:[~2023-03-31 12:36 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-23 23:01 [PATCH V4 0/4] clk: imx: Improve imx8mm/imx8mn LCDIF clocks Adam Ford
2023-03-23 23:01 ` Adam Ford
2023-03-23 23:01 ` [PATCH V4 1/4] clk: imx: composite-8m: Add support to determine_rate Adam Ford
2023-03-23 23:01   ` Adam Ford
2023-03-23 23:01 ` [PATCH V4 2/4] clk: imx: Add imx8m_clk_hw_composite_flags macro Adam Ford
2023-03-23 23:01   ` Adam Ford
2023-03-23 23:01 ` [PATCH V4 3/4] clk: imx8mm: Let IMX8MM_CLK_LCDIF_PIXEL set parent rate Adam Ford
2023-03-23 23:01   ` Adam Ford
2023-03-23 23:01 ` [PATCH V4 4/4] clk: imx: Let IMX8MN_CLK_DISP_PIXEL " Adam Ford
2023-03-23 23:01   ` Adam Ford
2023-03-31 12:35 ` Abel Vesa [this message]
2023-03-31 12:35   ` [PATCH V4 0/4] clk: imx: Improve imx8mm/imx8mn LCDIF clocks Abel Vesa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZCbTmitvBTQu26PX@linaro.org \
    --to=abel.vesa@linaro.org \
    --cc=abelvesa@kernel.org \
    --cc=aford173@gmail.com \
    --cc=aford@beaconembedded.com \
    --cc=festevam@gmail.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=peng.fan@nxp.com \
    --cc=s.hauer@pengutronix.de \
    --cc=sboyd@kernel.org \
    --cc=shawnguo@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.