pmdomain: core: constify of_phandle_args in xlate

The xlate callbacks are supposed to translate of_phandle_args to proper
provider without modifying the of_phandle_args.  Make the argument
pointer to const for code safety and readability.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20240208202822.631449-1-krzysztof.kozlowski@linaro.org
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

diff --git a/drivers/pmdomain/core.c b/drivers/pmdomain/core.c
index 46331e71108ac11765ab4730524b4a390a7dc178..ea4b29475206bfa2c9ea767983fbab5f0dcb65fd 100644 (file)
--- a/drivers/pmdomain/core.c
+++ b/drivers/pmdomain/core.c
@@ -2266,7 +2266,7 @@ static DEFINE_MUTEX(of_genpd_mutex);
  * to be a valid pointer to struct generic_pm_domain.
  */
 static struct generic_pm_domain *genpd_xlate_simple(
-                                       struct of_phandle_args *genpdspec,
+                                       const struct of_phandle_args *genpdspec,
                                        void *data)
 {
        return data;
@@ -2283,7 +2283,7 @@ static struct generic_pm_domain *genpd_xlate_simple(
  * the genpd_onecell_data struct when registering the provider.
  */
 static struct generic_pm_domain *genpd_xlate_onecell(
-                                       struct of_phandle_args *genpdspec,
+                                       const struct of_phandle_args *genpdspec,
                                        void *data)
 {
        struct genpd_onecell_data *genpd_data = data;

diff --git a/drivers/pmdomain/imx/scu-pd.c b/drivers/pmdomain/imx/scu-pd.c
index 891c1d925a9dea7f0ceecae089220e727766ec4e..05841b0bf7f30db175a8fd723d89b1b2b0306afa 100644 (file)
--- a/drivers/pmdomain/imx/scu-pd.c
+++ b/drivers/pmdomain/imx/scu-pd.c
@@ -393,7 +393,7 @@ static int imx_sc_pd_power_off(struct generic_pm_domain *domain)
        return imx_sc_pd_power(domain, false);
 }
 
-static struct generic_pm_domain *imx_scu_pd_xlate(struct of_phandle_args *spec,
+static struct generic_pm_domain *imx_scu_pd_xlate(const struct of_phandle_args *spec,
                                                  void *data)
 {
        struct generic_pm_domain *domain = ERR_PTR(-ENOENT);

diff --git a/drivers/pmdomain/tegra/powergate-bpmp.c b/drivers/pmdomain/tegra/powergate-bpmp.c
index 179ed895c2796b3516647c0024e2e2b5b681eeb3..b0138ca9f851d1d373456545b92af3db9744bdaf 100644 (file)
--- a/drivers/pmdomain/tegra/powergate-bpmp.c
+++ b/drivers/pmdomain/tegra/powergate-bpmp.c
@@ -305,7 +305,7 @@ static void tegra_bpmp_remove_powergates(struct tegra_bpmp *bpmp)
 }
 
 static struct generic_pm_domain *
-tegra_powergate_xlate(struct of_phandle_args *spec, void *data)
+tegra_powergate_xlate(const struct of_phandle_args *spec, void *data)
 {
        struct generic_pm_domain *domain = ERR_PTR(-ENOENT);
        struct genpd_onecell_data *genpd = data;

diff --git a/drivers/pmdomain/ti/ti_sci_pm_domains.c b/drivers/pmdomain/ti/ti_sci_pm_domains.c
index c091d569ecd5f3d05d84a261f1756f549ddb37e2..9dddf227a3a6b05c838761fcbfd508f8cf119f02 100644 (file)
--- a/drivers/pmdomain/ti/ti_sci_pm_domains.c
+++ b/drivers/pmdomain/ti/ti_sci_pm_domains.c
@@ -85,7 +85,7 @@ static int ti_sci_pd_power_on(struct generic_pm_domain *domain)
  * @data: genpd core data for all the powerdomains on the device
  */
 static struct generic_pm_domain *ti_sci_pd_xlate(
-                                       struct of_phandle_args *genpdspec,
+                                       const struct of_phandle_args *genpdspec,
                                        void *data)
 {
        struct genpd_onecell_data *genpd_data = data;

diff --git a/drivers/pmdomain/xilinx/zynqmp-pm-domains.c b/drivers/pmdomain/xilinx/zynqmp-pm-domains.c
index 6fd514286d82b6f36b4dd3d2a8b503247dcf49c8..0b5831e5ba1bfdac6aef88d7a3a7677dec22fbe2 100644 (file)
--- a/drivers/pmdomain/xilinx/zynqmp-pm-domains.c
+++ b/drivers/pmdomain/xilinx/zynqmp-pm-domains.c
@@ -210,7 +210,7 @@ static void zynqmp_gpd_detach_dev(struct generic_pm_domain *domain,
 }
 
 static struct generic_pm_domain *zynqmp_gpd_xlate
-                               (struct of_phandle_args *genpdspec, void *data)
+                               (const struct of_phandle_args *genpdspec, void *data)
 {
        struct genpd_onecell_data *genpd_data = data;
        unsigned int i, idx = genpdspec->args[0];

diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
index fb91770ba4bae765990da1d07341c6e04f406f90..1a391ef1b6f8b3b9399729cb17fda5752a367759 100644 (file)
--- a/include/linux/pm_domain.h
+++ b/include/linux/pm_domain.h
@@ -349,7 +349,7 @@ static inline void dev_pm_genpd_resume(struct device *dev) {}
 /* OF PM domain providers */
 struct of_device_id;
 
-typedef struct generic_pm_domain *(*genpd_xlate_t)(struct of_phandle_args *args,
+typedef struct generic_pm_domain *(*genpd_xlate_t)(const struct of_phandle_args *args,
                                                   void *data);
 
 struct genpd_onecell_data {