<snip>

quoted

+
+int gdsc_register(struct device *dev, struct gdsc **scs, size_t num,
+          struct regmap *regmap)
+{
+    int i, ret;
+    struct genpd_onecell_data *data;
+
+    data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
+    if (!data)
+        return -ENOMEM;
+
+    data->domains = devm_kcalloc(dev, num, sizeof(*data->domains),
+                     GFP_KERNEL);
+    if (!data->domains)
+        return -ENOMEM;
+
+    data->num_domains = num;
+    for (i = 0; i < num; i++) {
+        if (!scs[i])
+            continue;
+        scs[i]->regmap = regmap;
+        ret = gdsc_init(scs[i]);
+        if (ret)
+            return ret;
+        data->domains[i] = &scs[i]->pd;
+    }
+
+    return of_genpd_add_provider_onecell(dev->of_node, data);
+}

EXPORT_SYMBOL?

quoted

+
+void gdsc_unregister(struct device *dev)
+{
+    of_genpd_del_provider(dev->of_node);
+}

EXPORT_SYMBOL?

Stephen, gdsc_register/unregister are internal to the qcom clk driver
why to export them?

-- 
regards,
Stan