PM / AVS: rockchip-io: account for const type of of_device_id.data
authorJulia Lawall <Julia.Lawall@lip6.fr>
Tue, 2 Jan 2018 13:27:57 +0000 (14:27 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 5 Jan 2018 12:36:33 +0000 (13:36 +0100)
This driver creates a number of const structures that it stores in the
data field of an of_device_id array.

The data field of an of_device_id structure has type const void *, so
there is no need for a const-discarding cast when putting const values
into such a structure.

Furthermore, adding const to the declaration of the location that
receives a const value from such a field ensures that the compiler
will continue to check that the value is not modified.  The
const-discarding cast on the extraction from the data field is
thus no longer needed.

Done using Coccinelle.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/power/avs/rockchip-io-domain.c

index 75f63e38a8d10218842dba03f7fb15d0ba65653f..ed2b109ae8fcba5045f5bfd593bd120cbd61ae48 100644 (file)
@@ -76,7 +76,7 @@ struct rockchip_iodomain_supply {
 struct rockchip_iodomain {
        struct device *dev;
        struct regmap *grf;
-       struct rockchip_iodomain_soc_data *soc_data;
+       const struct rockchip_iodomain_soc_data *soc_data;
        struct rockchip_iodomain_supply supplies[MAX_SUPPLIES];
 };
 
@@ -382,43 +382,43 @@ static const struct rockchip_iodomain_soc_data soc_data_rv1108_pmu = {
 static const struct of_device_id rockchip_iodomain_match[] = {
        {
                .compatible = "rockchip,rk3188-io-voltage-domain",
-               .data = (void *)&soc_data_rk3188
+               .data = &soc_data_rk3188
        },
        {
                .compatible = "rockchip,rk3228-io-voltage-domain",
-               .data = (void *)&soc_data_rk3228
+               .data = &soc_data_rk3228
        },
        {
                .compatible = "rockchip,rk3288-io-voltage-domain",
-               .data = (void *)&soc_data_rk3288
+               .data = &soc_data_rk3288
        },
        {
                .compatible = "rockchip,rk3328-io-voltage-domain",
-               .data = (void *)&soc_data_rk3328
+               .data = &soc_data_rk3328
        },
        {
                .compatible = "rockchip,rk3368-io-voltage-domain",
-               .data = (void *)&soc_data_rk3368
+               .data = &soc_data_rk3368
        },
        {
                .compatible = "rockchip,rk3368-pmu-io-voltage-domain",
-               .data = (void *)&soc_data_rk3368_pmu
+               .data = &soc_data_rk3368_pmu
        },
        {
                .compatible = "rockchip,rk3399-io-voltage-domain",
-               .data = (void *)&soc_data_rk3399
+               .data = &soc_data_rk3399
        },
        {
                .compatible = "rockchip,rk3399-pmu-io-voltage-domain",
-               .data = (void *)&soc_data_rk3399_pmu
+               .data = &soc_data_rk3399_pmu
        },
        {
                .compatible = "rockchip,rv1108-io-voltage-domain",
-               .data = (void *)&soc_data_rv1108
+               .data = &soc_data_rv1108
        },
        {
                .compatible = "rockchip,rv1108-pmu-io-voltage-domain",
-               .data = (void *)&soc_data_rv1108_pmu
+               .data = &soc_data_rv1108_pmu
        },
        { /* sentinel */ },
 };
@@ -443,7 +443,7 @@ static int rockchip_iodomain_probe(struct platform_device *pdev)
        platform_set_drvdata(pdev, iod);
 
        match = of_match_node(rockchip_iodomain_match, np);
-       iod->soc_data = (struct rockchip_iodomain_soc_data *)match->data;
+       iod->soc_data = match->data;
 
        parent = pdev->dev.parent;
        if (parent && parent->of_node) {