On 09/09/2014 02:59 PM, Nicolin Chen wrote:
+       /*
+        * Initially mark the clock to NULL for all platforms so that later
+        * clk_prepare_enable() will ignore and return 0 for non-clock cases.
+        */
+       ssi_private->clk = NULL;

According to Mark, NULL is a valid clock, so this should be instead:

        ssi_private->clk = PTR_ERR(-EINVAL);

although that doesn't sit well with me.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to