clk: Inline clk_get_*_optional

The optional varients of clk_get_* functions are just simple wrappers.
Reduce code size a bit by inlining them. On platforms where it is not used
(most of them), it will not be compiled in any more. On platforms where
they are used, the inlined branch should not cause any significant growth.

Signed-off-by: Sean Anderson <seanga2@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Link: https://lore.kernel.org/r/20211222171114.3091780-3-seanga2@gmail.com

diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c
index 9aa51895b3d4178abb0e74e75e6c3d2c9802dab7..fca4b8321a9d9890c2e464002c6f28904c75f691 100644 (file)
--- a/drivers/clk/clk-uclass.c
+++ b/drivers/clk/clk-uclass.c
@@ -432,18 +432,6 @@ int clk_get_by_name_nodev(ofnode node, const char *name, struct clk *clk)
        return clk_get_by_index_nodev(node, index, clk);
 }
 
-int clk_get_by_name_nodev_optional(ofnode node, const char *name,
-                                  struct clk *clk)
-{
-       int ret;
-
-       ret = clk_get_by_name_nodev(node, name, clk);
-       if (ret == -ENODATA)
-               return 0;
-
-       return ret;
-}
-
 int clk_release_all(struct clk *clk, int count)
 {
        int i, ret;
@@ -824,16 +812,6 @@ struct clk *devm_clk_get(struct udevice *dev, const char *id)
        return clk;
 }
 
-struct clk *devm_clk_get_optional(struct udevice *dev, const char *id)
-{
-       struct clk *clk = devm_clk_get(dev, id);
-
-       if (PTR_ERR(clk) == -ENODATA)
-               return NULL;
-
-       return clk;
-}
-
 void devm_clk_put(struct udevice *dev, struct clk *clk)
 {
        int rc;

diff --git a/include/clk.h b/include/clk.h
index a8225eec72af39c5edf2d8220ce09993c17ac3b8..6706afa55dd9a96d91c2f21e4db31c24761ae3c7 100644 (file)
--- a/include/clk.h
+++ b/include/clk.h
@@ -193,22 +193,6 @@ int clk_get_by_name(struct udevice *dev, const char *name, struct clk *clk);
  */
 int clk_get_by_name_nodev(ofnode node, const char *name, struct clk *clk);
 
-/**
- * clk_get_by_name_nodev_optional - Get/request an optinonal clock by name
- *             without a device.
- * @node:      The client ofnode.
- * @name:      The name of the clock to request.
- * @name:      The name of the clock to request, within the client's list of
- *             clocks.
- * @clock:     A pointer to a clock struct to initialize.
- *
- * Behaves the same as clk_get_by_name_nodev() except where there is
- * no clock producer, in this case, skip the error number -ENODATA, and
- * the function returns 0.
- */
-int clk_get_by_name_nodev_optional(ofnode node, const char *name,
-                                  struct clk *clk);
-
 /**
  * devm_clk_get - lookup and obtain a managed reference to a clock producer.
  * @dev: device for clock "consumer"
@@ -238,7 +222,16 @@ struct clk *devm_clk_get(struct udevice *dev, const char *id);
  * Behaves the same as devm_clk_get() except where there is no clock producer.
  * In this case, instead of returning -ENOENT, the function returns NULL.
  */
-struct clk *devm_clk_get_optional(struct udevice *dev, const char *id);
+static inline struct clk *devm_clk_get_optional(struct udevice *dev,
+                                               const char *id)
+{
+       struct clk *clk = devm_clk_get(dev, id);
+
+       if (PTR_ERR(clk) == -ENODATA)
+               return NULL;
+
+       return clk;
+}
 
 /**
  * clk_release_all() - Disable (turn off)/Free an array of previously
@@ -291,17 +284,36 @@ clk_get_by_name_nodev(ofnode node, const char *name, struct clk *clk)
        return -ENOSYS;
 }
 
-static inline int
-clk_get_by_name_nodev_optional(ofnode node, const char *name, struct clk *clk)
+static inline int clk_release_all(struct clk *clk, int count)
 {
        return -ENOSYS;
 }
+#endif
 
-static inline int clk_release_all(struct clk *clk, int count)
+/**
+ * clk_get_by_name_nodev_optional - Get/request an optinonal clock by name
+ *             without a device.
+ * @node:      The client ofnode.
+ * @name:      The name of the clock to request.
+ * @name:      The name of the clock to request, within the client's list of
+ *             clocks.
+ * @clock:     A pointer to a clock struct to initialize.
+ *
+ * Behaves the same as clk_get_by_name_nodev() except where there is
+ * no clock producer, in this case, skip the error number -ENODATA, and
+ * the function returns 0.
+ */
+static inline int clk_get_by_name_nodev_optional(ofnode node, const char *name,
+                                                struct clk *clk)
 {
-       return -ENOSYS;
+       int ret;
+
+       ret = clk_get_by_name_nodev(node, name, clk);
+       if (ret == -ENODATA)
+               return 0;
+
+       return ret;
 }
-#endif
 
 /**
  * enum clk_defaults_stage - What stage clk_set_defaults() is called at