diff options
author | Tomasz Figa <t.figa@samsung.com> | 2013-12-12 11:07:17 -0500 |
---|---|---|
committer | Mike Turquette <mturquette@linaro.org> | 2014-01-08 12:46:03 -0500 |
commit | badbc542f970a40f4c15014501016584f783782d (patch) | |
tree | 3e035079ec87213154906ac573bac214b88d03ea | |
parent | b0f851777131b9fbd6f5853bea1f6144e88c1c72 (diff) |
clk: max77686: Make max77686_clk_register() return struct clk *
As a preparation for further patches, this patch modifies the clock
registration helper function to return a pointer to the newly registered
clock. No functional change is done to the driver.
Signed-off-by: Tomasz Figa <t.figa@samsung.com>
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mike Turquette <mturquette@linaro.org>
-rw-r--r-- | drivers/clk/clk-max77686.c | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/drivers/clk/clk-max77686.c b/drivers/clk/clk-max77686.c index a8e83d45ad3f..243072366aa8 100644 --- a/drivers/clk/clk-max77686.c +++ b/drivers/clk/clk-max77686.c | |||
@@ -112,27 +112,26 @@ static struct clk_init_data max77686_clks_init[MAX77686_CLKS_NUM] = { | |||
112 | }, | 112 | }, |
113 | }; | 113 | }; |
114 | 114 | ||
115 | static int max77686_clk_register(struct device *dev, | 115 | static struct clk *max77686_clk_register(struct device *dev, |
116 | struct max77686_clk *max77686) | 116 | struct max77686_clk *max77686) |
117 | { | 117 | { |
118 | struct clk *clk; | 118 | struct clk *clk; |
119 | struct clk_hw *hw = &max77686->hw; | 119 | struct clk_hw *hw = &max77686->hw; |
120 | 120 | ||
121 | clk = clk_register(dev, hw); | 121 | clk = clk_register(dev, hw); |
122 | |||
123 | if (IS_ERR(clk)) | 122 | if (IS_ERR(clk)) |
124 | return -ENOMEM; | 123 | return clk; |
125 | 124 | ||
126 | max77686->lookup = kzalloc(sizeof(struct clk_lookup), GFP_KERNEL); | 125 | max77686->lookup = kzalloc(sizeof(struct clk_lookup), GFP_KERNEL); |
127 | if (!max77686->lookup) | 126 | if (!max77686->lookup) |
128 | return -ENOMEM; | 127 | return ERR_PTR(-ENOMEM); |
129 | 128 | ||
130 | max77686->lookup->con_id = hw->init->name; | 129 | max77686->lookup->con_id = hw->init->name; |
131 | max77686->lookup->clk = clk; | 130 | max77686->lookup->clk = clk; |
132 | 131 | ||
133 | clkdev_add(max77686->lookup); | 132 | clkdev_add(max77686->lookup); |
134 | 133 | ||
135 | return 0; | 134 | return clk; |
136 | } | 135 | } |
137 | 136 | ||
138 | static int max77686_clk_probe(struct platform_device *pdev) | 137 | static int max77686_clk_probe(struct platform_device *pdev) |
@@ -154,12 +153,16 @@ static int max77686_clk_probe(struct platform_device *pdev) | |||
154 | } | 153 | } |
155 | 154 | ||
156 | for (i = 0; i < MAX77686_CLKS_NUM; i++) { | 155 | for (i = 0; i < MAX77686_CLKS_NUM; i++) { |
156 | struct clk *clk; | ||
157 | |||
157 | max77686_clks[i]->iodev = iodev; | 158 | max77686_clks[i]->iodev = iodev; |
158 | max77686_clks[i]->mask = 1 << i; | 159 | max77686_clks[i]->mask = 1 << i; |
159 | max77686_clks[i]->hw.init = &max77686_clks_init[i]; | 160 | max77686_clks[i]->hw.init = &max77686_clks_init[i]; |
160 | 161 | ||
161 | ret = max77686_clk_register(&pdev->dev, max77686_clks[i]); | 162 | clk = max77686_clk_register(&pdev->dev, max77686_clks[i]); |
162 | if (ret) { | 163 | if (IS_ERR(clk)) { |
164 | ret = PTR_ERR(clk); | ||
165 | |||
163 | switch (i) { | 166 | switch (i) { |
164 | case MAX77686_CLK_AP: | 167 | case MAX77686_CLK_AP: |
165 | dev_err(&pdev->dev, "Fail to register CLK_AP\n"); | 168 | dev_err(&pdev->dev, "Fail to register CLK_AP\n"); |