Re: [PATCH v6 04/13] clk: qcom: gdsc: Manage clocks with !CONFIG_PM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/23/2015 01:35 AM, Rajendra Nayak wrote:
[]..

+
+#ifndef CONFIG_PM
+static void enable_clock(struct device *dev, const char *con_id)
+{
+    struct clk *clk;
+
+    clk = clk_get(dev, con_id);
+    if (!IS_ERR(clk)) {
+        clk_prepare_enable(clk);
+        clk_put(clk);
+    }
+}
+
+static void disable_clock(struct device *dev, const char *con_id)
+{
+    struct clk *clk;
+
+    clk = clk_get(dev, con_id);
+    if (!IS_ERR(clk)) {
+        clk_disable_unprepare(clk);
+        clk_put(clk);
+    }
+}

Is there a reason why this whole patch isn't generic code? I recall some
discussion but I forgot now and there isn't any mention of why this
isn't generic code in the commit text.

If by generic code, you mean using PM clocks, then this thread should
give some context..
http://www.spinics.net/lists/arm-kernel/msg414072.html

Sorry, I read the thread and I tried to understand what was going on but I'm still lost. Can you clarify further in the commit text somehow?

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux