| Message ID | 20220426233508.1762345-3-bjorn.andersson@linaro.org |
|---|---|
| State | Accepted |
| Commit | af77132e9d2bc8ca477003fef15e2c677ae958b1 |
| Headers | show |
| Series | soc: qcom: rpmhpd: Add sc8280xp and sa8540p | expand |
diff --git a/drivers/soc/qcom/rpmhpd.c b/drivers/soc/qcom/rpmhpd.c index 11102ac47769..f8d28e902942 100644 --- a/drivers/soc/qcom/rpmhpd.c +++ b/drivers/soc/qcom/rpmhpd.c @@ -613,10 +613,8 @@ static int rpmhpd_probe(struct platform_device *pdev) data->num_domains = num_pds; for (i = 0; i < num_pds; i++) { - if (!rpmhpds[i]) { - dev_warn(dev, "rpmhpds[%d] is empty\n", i); + if (!rpmhpds[i]) continue; - } rpmhpds[i]->dev = dev; rpmhpds[i]->addr = cmd_db_read_addr(rpmhpds[i]->res_name);
In some cases the DT binding will fully describe the set of available RPMh power-domains, but there is no reason for exposing them all in the implementation. Omitting individual data->domains is handle gracefully by of_genpd_add_provider_onecell(), so there's no reason for printing a warning when this occurs. Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- Changes since v1: - New patch drivers/soc/qcom/rpmhpd.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)