mirror of
https://github.com/torvalds/linux.git
synced 2026-07-28 01:55:51 +02:00
clk: qcom: clk-rpmh: Make all VRMs optional
Some VRMs aren't present on all boards, so mark them as optional. This prevents probe failures on boards where not all VRMs are present. This resolves an issue seen on the Nothing Phone (4a) Pro (Eliza) where probe fails due to RPMH_RF_CLK5 not being present on the board, this is due to this device having a slightly different PMIC configuration from the Eliza MTP. This matches the downstream approach of marking all VRMs as optional and makes the previous clka_optional handling redundant. Signed-off-by: Alexander Koskovich <akoskovich@pm.me> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Link: https://lore.kernel.org/r/20260414-clk-rpmh-vrm-opt-v3-1-8ca21469ffbc@pm.me Signed-off-by: Bjorn Andersson <andersson@kernel.org>
This commit is contained in:
parent
67121dad6c
commit
25b8f50b06
|
|
@ -66,8 +66,6 @@ struct clk_rpmh {
|
|||
struct clk_rpmh_desc {
|
||||
struct clk_hw **clks;
|
||||
size_t num_clks;
|
||||
/* RPMh clock clkaN are optional for this platform */
|
||||
bool clka_optional;
|
||||
};
|
||||
|
||||
static DEFINE_MUTEX(rpmh_clk_lock);
|
||||
|
|
@ -699,7 +697,6 @@ static struct clk_hw *sm8550_rpmh_clocks[] = {
|
|||
static const struct clk_rpmh_desc clk_rpmh_sm8550 = {
|
||||
.clks = sm8550_rpmh_clocks,
|
||||
.num_clks = ARRAY_SIZE(sm8550_rpmh_clocks),
|
||||
.clka_optional = true,
|
||||
};
|
||||
|
||||
static struct clk_hw *sm8650_rpmh_clocks[] = {
|
||||
|
|
@ -731,7 +728,6 @@ static struct clk_hw *sm8650_rpmh_clocks[] = {
|
|||
static const struct clk_rpmh_desc clk_rpmh_sm8650 = {
|
||||
.clks = sm8650_rpmh_clocks,
|
||||
.num_clks = ARRAY_SIZE(sm8650_rpmh_clocks),
|
||||
.clka_optional = true,
|
||||
};
|
||||
|
||||
static struct clk_hw *sc7280_rpmh_clocks[] = {
|
||||
|
|
@ -901,7 +897,6 @@ static struct clk_hw *sm8750_rpmh_clocks[] = {
|
|||
static const struct clk_rpmh_desc clk_rpmh_sm8750 = {
|
||||
.clks = sm8750_rpmh_clocks,
|
||||
.num_clks = ARRAY_SIZE(sm8750_rpmh_clocks),
|
||||
.clka_optional = true,
|
||||
};
|
||||
|
||||
static struct clk_hw *glymur_rpmh_clocks[] = {
|
||||
|
|
@ -1059,8 +1054,7 @@ static int clk_rpmh_probe(struct platform_device *pdev)
|
|||
if (!res_addr) {
|
||||
hw_clks[i] = NULL;
|
||||
|
||||
if (desc->clka_optional &&
|
||||
!strncmp(rpmh_clk->res_name, "clka", sizeof("clka") - 1))
|
||||
if (rpmh_clk->res_addr == CLK_RPMH_VRM_EN_OFFSET)
|
||||
continue;
|
||||
|
||||
dev_err(&pdev->dev, "missing RPMh resource address for %s\n",
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user