clk: Clean up suspend/resume coding style
authorStephen Boyd <sboyd@kernel.org>
Thu, 11 Oct 2018 16:28:13 +0000 (09:28 -0700)
committerStephen Boyd <sboyd@kernel.org>
Thu, 11 Oct 2018 16:28:13 +0000 (09:28 -0700)
The normal style is to use 'core' for struct clk_core pointers and to
directly access the core pointer from the clk_hw pointer when we're
within the common clk framework. Update the patches to make it a bit
easier to handle.

Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk.c

index dd775771a7cc079f93e070c8816025d1b8dd189a..af011974d4ecbf1226fa0d4ecb3db66bf2514afc 100644 (file)
@@ -935,39 +935,41 @@ static int clk_core_enable_lock(struct clk_core *core)
  */
 void clk_gate_restore_context(struct clk_hw *hw)
 {
-       if (hw->clk->core->enable_count)
-               hw->clk->core->ops->enable(hw);
+       struct clk_core *core = hw->core;
+
+       if (core->enable_count)
+               core->ops->enable(hw);
        else
-               hw->clk->core->ops->disable(hw);
+               core->ops->disable(hw);
 }
 EXPORT_SYMBOL_GPL(clk_gate_restore_context);
 
-static int _clk_save_context(struct clk_core *clk)
+static int clk_core_save_context(struct clk_core *core)
 {
        struct clk_core *child;
        int ret = 0;
 
-       hlist_for_each_entry(child, &clk->children, child_node) {
-               ret = _clk_save_context(child);
+       hlist_for_each_entry(child, &core->children, child_node) {
+               ret = clk_core_save_context(child);
                if (ret < 0)
                        return ret;
        }
 
-       if (clk->ops && clk->ops->save_context)
-               ret = clk->ops->save_context(clk->hw);
+       if (core->ops && core->ops->save_context)
+               ret = core->ops->save_context(core->hw);
 
        return ret;
 }
 
-static void _clk_restore_context(struct clk_core *clk)
+static void clk_core_restore_context(struct clk_core *core)
 {
        struct clk_core *child;
 
-       if (clk->ops && clk->ops->restore_context)
-               clk->ops->restore_context(clk->hw);
+       if (core->ops && core->ops->restore_context)
+               core->ops->restore_context(core->hw);
 
-       hlist_for_each_entry(child, &clk->children, child_node)
-               _clk_restore_context(child);
+       hlist_for_each_entry(child, &core->children, child_node)
+               clk_core_restore_context(child);
 }
 
 /**
@@ -983,13 +985,13 @@ int clk_save_context(void)
        int ret;
 
        hlist_for_each_entry(clk, &clk_root_list, child_node) {
-               ret = _clk_save_context(clk);
+               ret = clk_core_save_context(clk);
                if (ret < 0)
                        return ret;
        }
 
        hlist_for_each_entry(clk, &clk_orphan_list, child_node) {
-               ret = _clk_save_context(clk);
+               ret = clk_core_save_context(clk);
                if (ret < 0)
                        return ret;
        }
@@ -1006,13 +1008,13 @@ EXPORT_SYMBOL_GPL(clk_save_context);
  */
 void clk_restore_context(void)
 {
-       struct clk_core *clk;
+       struct clk_core *core;
 
-       hlist_for_each_entry(clk, &clk_root_list, child_node)
-               _clk_restore_context(clk);
+       hlist_for_each_entry(core, &clk_root_list, child_node)
+               clk_core_restore_context(core);
 
-       hlist_for_each_entry(clk, &clk_orphan_list, child_node)
-               _clk_restore_context(clk);
+       hlist_for_each_entry(core, &clk_orphan_list, child_node)
+               clk_core_restore_context(core);
 }
 EXPORT_SYMBOL_GPL(clk_restore_context);