soc: qcom: ice: Remove the device_link field in qcom_ice
authorJoe Hattori <joe@pf.is.s.u-tokyo.ac.jp>
Wed, 30 Oct 2024 02:50:46 +0000 (11:50 +0900)
committerBjorn Andersson <andersson@kernel.org>
Tue, 12 Nov 2024 04:03:30 +0000 (22:03 -0600)
The struct device_link *link field in struct qcom_ice is only used to
store the result of a device_link_add call with the
DL_FLAG_AUTOREMOVE_SUPPLIER flag. With this flag, the resulting value
can only be used to check whether the link is present or not, as per the
device_link_add description, hence this commit removes the field.

Signed-off-by: Joe Hattori <joe@pf.is.s.u-tokyo.ac.jp>
Link: https://lore.kernel.org/r/20241030025046.303342-1-joe@pf.is.s.u-tokyo.ac.jp
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/soc/qcom/ice.c

index 50be7a9..393d2d1 100644 (file)
@@ -44,7 +44,6 @@
 struct qcom_ice {
        struct device *dev;
        void __iomem *base;
-       struct device_link *link;
 
        struct clk *core_clk;
 };
@@ -268,6 +267,7 @@ struct qcom_ice *of_qcom_ice_get(struct device *dev)
        struct qcom_ice *ice;
        struct resource *res;
        void __iomem *base;
+       struct device_link *link;
 
        if (!dev || !dev->of_node)
                return ERR_PTR(-ENODEV);
@@ -311,8 +311,8 @@ struct qcom_ice *of_qcom_ice_get(struct device *dev)
                return ERR_PTR(-EPROBE_DEFER);
        }
 
-       ice->link = device_link_add(dev, &pdev->dev, DL_FLAG_AUTOREMOVE_SUPPLIER);
-       if (!ice->link) {
+       link = device_link_add(dev, &pdev->dev, DL_FLAG_AUTOREMOVE_SUPPLIER);
+       if (!link) {
                dev_err(&pdev->dev,
                        "Failed to create device link to consumer %s\n",
                        dev_name(dev));