Linux clock framework development
 help / color / mirror / Atom feed
From: Val Packett <val@packett.cool>
To: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>,
	Hans de Goede <johannes.goede@oss.qualcomm.com>
Cc: Bjorn Andersson <andersson@kernel.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>,
	linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org
Subject: Re: [PATCH] clk: qcom: x1e80100-dispcc: Stop disp_cc_mdss_mdp_clk_src from getting parked
Date: Sat, 25 Apr 2026 13:10:04 -0300	[thread overview]
Message-ID: <39679013-dacd-4804-a52f-0c36adf8e698@packett.cool> (raw)
In-Reply-To: <CAO9ioeWUp_SBnkuNejRLb9E1UvrbCJXYcZr0KgWJgCVZKRLL0w@mail.gmail.com>


On 4/25/26 9:44 AM, Dmitry Baryshkov wrote:
> On Sat, 25 Apr 2026 at 15:33, Hans de Goede
> <johannes.goede@oss.qualcomm.com> wrote:
>> Parking disp_cc_mdss_mdp_clk_src at 19.2MHz causing the EFI GOP framebuffer
>> to stop functioning. The EFI GOP framebuffer should keep working until
>> the msm display driver loads, to help with boot debugging and to ensure
>> display output when the msm module is not in the initramfs.
>>
>> Switch disp_cc_mdss_mdp_clk_src over to clk_rcg2_shared_no_init_park_ops
>> to keep the EFI GOP working after binding the x1e80100-dispcc driver.
>>
>> Suggested-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
>> Signed-off-by: Hans de Goede <johannes.goede@oss.qualcomm.com>
Thanks for finding this!!
> Most likely we need this to be performed for all dispcc drivers. At
> least for all laptop usecases.

This is relevant for phones (ex-Android) as well actually, though there 
have been attempts to fix that by adding stuff like

             clocks = <&gcc GCC_DISP_HF_AXI_CLK>,
                  <&dispcc DISP_CC_MDSS_MDP_CLK>,
                  <&dispcc DISP_CC_MDSS_BYTE0_CLK>,
                  <&dispcc DISP_CC_MDSS_BYTE0_INTF_CLK>,
                  <&dispcc DISP_CC_MDSS_PCLK0_CLK>,
                  <&dispcc DISP_CC_MDSS_VSYNC_CLK>;
             power-domains = <&dispcc DISP_CC_MDSS_CORE_GDSC>;

to the simple-framebuffer node. However there have also been some 
"random" issues with the handover to mdss drivers (even with the mdss 
reset) so some postmarketOS kernel builds completely disable the 
simplefb driver.

~val


  reply	other threads:[~2026-04-25 16:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-25 12:33 [PATCH] clk: qcom: x1e80100-dispcc: Stop disp_cc_mdss_mdp_clk_src from getting parked Hans de Goede
2026-04-25 12:44 ` Dmitry Baryshkov
2026-04-25 16:10   ` Val Packett [this message]
2026-04-26 18:20     ` Hans de Goede
2026-04-27 16:00       ` Hans de Goede
2026-04-27 17:12 ` Bjorn Andersson
2026-04-28  8:39   ` Hans de Goede
2026-04-28 11:37     ` Dmitry Baryshkov

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=39679013-dacd-4804-a52f-0c36adf8e698@packett.cool \
    --to=val@packett.cool \
    --cc=andersson@kernel.org \
    --cc=dmitry.baryshkov@oss.qualcomm.com \
    --cc=johannes.goede@oss.qualcomm.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=sboyd@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox