From: Yassine Oudjana <yassine.oudjana@gmail.com>
To: Robert Foss <rfoss@kernel.org>, Todor Tomov <todor.too@gmail.com>,
"Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@linaro.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Hans Verkuil <hansverk@cisco.com>
Cc: Yassine Oudjana <y.oudjana@protonmail.com>,
Yassine Oudjana <yassine.oudjana@gmail.com>,
Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>,
linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 0/3] media: camss: Link CAMSS power domain on MSM8996
Date: Wed, 3 May 2023 10:25:40 +0300 [thread overview]
Message-ID: <20230503072543.4837-1-y.oudjana@protonmail.com> (raw)
From: Yassine Oudjana <y.oudjana@protonmail.com>
CAMSS on MSM8996 has been broken since commit
46cc03175498 (media: camss: Split power domain management, 2022-07-04).
This would happen when trying to start streaming:
[ 199.097810] ------------[ cut here ]------------
[ 199.097893] camss_top_ahb_clk status stuck at 'off'
[ 199.097913] WARNING: CPU: 3 PID: 728 at drivers/clk/qcom/clk-branch.c:91 clk_branch_wait+0x140/0x160
...
[ 199.100064] clk_branch_wait+0x140/0x160
[ 199.100112] clk_branch2_enable+0x30/0x40
[ 199.100159] clk_core_enable+0x6c/0xb0
[ 199.100211] clk_enable+0x2c/0x50
[ 199.100257] camss_enable_clocks+0x94/0xe0 [qcom_camss]
[ 199.100342] csiphy_set_power+0x154/0x2a0 [qcom_camss]
...
[ 199.101594] ---[ end trace 0000000000000000 ]---
[ 199.101736] qcom-camss a34000.camss: clock enable failed: -16
[ 199.101813] qcom-camss a34000.camss: Failed to power up pipeline: -16
Turns out camss_top_ahb_clk needs the CAMSS power domain to be on. Before the change,
VFE power domains were enabled before CSIPHY enabled clocks, and since the CAMSS power
domain was their parent, it got enabled as well. With the VFE power domains now enabled
after CSIPHY is powered on, the CAMSS power domain remains off and things go south when
CSIPHY tries to enable camss_top_ahb_clk.
Link the CAMSS power domain in camss_configure_pd to make sure it gets enabled before
CSIPHY tries to enable clocks.
Yassine Oudjana (3):
media: dt-bindings: media: camss: qcom,msm8996-camss: Add CAMSS power
domain and power-domain-names
arm64: dts: qcom: msm8996: Add CAMSS power domain and
power-domain-names to CAMSS
media: camss: Link CAMSS power domain
.../bindings/media/qcom,msm8996-camss.yaml | 13 ++++++++++++-
arch/arm64/boot/dts/qcom/msm8996.dtsi | 4 +++-
drivers/media/platform/qcom/camss/camss.c | 9 ++++++++-
3 files changed, 23 insertions(+), 3 deletions(-)
--
2.40.0
next reply other threads:[~2023-05-03 7:28 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-03 7:25 Yassine Oudjana [this message]
2023-05-03 7:25 ` [PATCH 1/3] media: dt-bindings: media: camss: qcom,msm8996-camss: Add CAMSS power domain and power-domain-names Yassine Oudjana
2023-05-03 9:10 ` Bryan O'Donoghue
2023-05-05 17:50 ` Krzysztof Kozlowski
2023-05-03 7:25 ` [PATCH 2/3] arm64: dts: qcom: msm8996: Add CAMSS power domain and power-domain-names to CAMSS Yassine Oudjana
2023-05-03 9:12 ` Bryan O'Donoghue
2023-05-05 17:51 ` Krzysztof Kozlowski
2023-05-03 7:25 ` [PATCH 3/3] media: camss: Link CAMSS power domain Yassine Oudjana
2023-05-03 9:15 ` Bryan O'Donoghue
2023-05-03 8:57 ` [PATCH 0/3] media: camss: Link CAMSS power domain on MSM8996 Bryan O'Donoghue
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=20230503072543.4837-1-y.oudjana@protonmail.com \
--to=yassine.oudjana@gmail.com \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=bryan.odonoghue@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=hansverk@cisco.com \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=rfoss@kernel.org \
--cc=robh+dt@kernel.org \
--cc=todor.too@gmail.com \
--cc=vladimir.zapolskiy@linaro.org \
--cc=y.oudjana@protonmail.com \
/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
all inboxes | Powered by JetHome®