forked from ~ljy/RK356X_SDK_RELEASE
blame | history | patch | commit | commitdiff
hc
2024-10-09 244b2c5ca8b14627e4a17755e5922221e121c771 
 kernel/drivers/pwm/pwm-meson.c
....@@ -1,65 +1,40 @@
1
+// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause
12 /*
2
- * This file is provided under a dual BSD/GPLv2 license.  When using or
3
- * redistributing this file, you may do so under either license.
3
+ * PWM controller driver for Amlogic Meson SoCs.
44  *
5
- * GPL LICENSE SUMMARY
5
+ * This PWM is only a set of Gates, Dividers and Counters:
6
+ * PWM output is achieved by calculating a clock that permits calculating
7
+ * two periods (low and high). The counter then has to be set to switch after
8
+ * N cycles for the first half period.
9
+ * The hardware has no "polarity" setting. This driver reverses the period
10
+ * cycles (the low length is inverted with the high length) for
11
+ * PWM_POLARITY_INVERSED. This means that .get_state cannot read the polarity
12
+ * from the hardware.
13
+ * Setting the duty cycle will disable and re-enable the PWM output.
14
+ * Disabling the PWM stops the output immediately (without waiting for the
15
+ * current period to complete first).
16
+ *
17
+ * The public S912 (GXM) datasheet contains some documentation for this PWM
18
+ * controller starting on page 543:
19
+ * https://dl.khadas.com/Hardware/VIM2/Datasheet/S912_Datasheet_V0.220170314publicversion-Wesion.pdf
20
+ * An updated version of this IP block is found in S922X (G12B) SoCs. The
21
+ * datasheet contains the description for this IP block revision starting at
22
+ * page 1084:
23
+ * https://dn.odroid.com/S922X/ODROID-N2/Datasheet/S922X_Public_Datasheet_V0.2.pdf
624  *
725  * Copyright (c) 2016 BayLibre, SAS.
826  * Author: Neil Armstrong <narmstrong@baylibre.com>
927  * Copyright (C) 2014 Amlogic, Inc.
10
- *
11
- * This program is free software; you can redistribute it and/or modify
12
- * it under the terms of version 2 of the GNU General Public License as
13
- * published by the Free Software Foundation.
14
- *
15
- * This program is distributed in the hope that it will be useful, but
16
- * WITHOUT ANY WARRANTY; without even the implied warranty of
17
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
18
- * General Public License for more details.
19
- *
20
- * You should have received a copy of the GNU General Public License
21
- * along with this program; if not, see <http://www.gnu.org/licenses/>.
22
- * The full GNU General Public License is included in this distribution
23
- * in the file called COPYING.
24
- *
25
- * BSD LICENSE
26
- *
27
- * Copyright (c) 2016 BayLibre, SAS.
28
- * Author: Neil Armstrong <narmstrong@baylibre.com>
29
- * Copyright (C) 2014 Amlogic, Inc.
30
- *
31
- * Redistribution and use in source and binary forms, with or without
32
- * modification, are permitted provided that the following conditions
33
- * are met:
34
- *
35
- *   * Redistributions of source code must retain the above copyright
36
- *     notice, this list of conditions and the following disclaimer.
37
- *   * Redistributions in binary form must reproduce the above copyright
38
- *     notice, this list of conditions and the following disclaimer in
39
- *     the documentation and/or other materials provided with the
40
- *     distribution.
41
- *   * Neither the name of Intel Corporation nor the names of its
42
- *     contributors may be used to endorse or promote products derived
43
- *     from this software without specific prior written permission.
44
- *
45
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
46
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
47
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
48
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
49
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
50
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
51
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
52
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
53
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
54
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
55
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
5628  */
5729 
30
+#include <linux/bitfield.h>
31
+#include <linux/bits.h>
5832 #include <linux/clk.h>
5933 #include <linux/clk-provider.h>
6034 #include <linux/err.h>
6135 #include <linux/io.h>
6236 #include <linux/kernel.h>
37
+#include <linux/math64.h>
6338 #include <linux/module.h>
6439 #include <linux/of.h>
6540 #include <linux/of_device.h>
....@@ -70,7 +45,8 @@
7045 
7146 #define REG_PWM_A		0x0
7247 #define REG_PWM_B		0x4
73
-#define PWM_HIGH_SHIFT		16
48
+#define PWM_LOW_MASK		GENMASK(15, 0)
49
+#define PWM_HIGH_MASK		GENMASK(31, 16)
7450 
7551 #define REG_MISC_AB		0x8
7652 #define MISC_B_CLK_EN		BIT(23)
....@@ -80,21 +56,39 @@
8056 #define MISC_A_CLK_DIV_SHIFT	8
8157 #define MISC_B_CLK_SEL_SHIFT	6
8258 #define MISC_A_CLK_SEL_SHIFT	4
83
-#define MISC_CLK_SEL_WIDTH	2
59
+#define MISC_CLK_SEL_MASK	0x3
8460 #define MISC_B_EN		BIT(1)
8561 #define MISC_A_EN		BIT(0)
8662 
87
-static const unsigned int mux_reg_shifts[] = {
88
-	MISC_A_CLK_SEL_SHIFT,
89
-	MISC_B_CLK_SEL_SHIFT
63
+#define MESON_NUM_PWMS		2
64
+
65
+static struct meson_pwm_channel_data {
66
+	u8		reg_offset;
67
+	u8		clk_sel_shift;
68
+	u8		clk_div_shift;
69
+	u32		clk_en_mask;
70
+	u32		pwm_en_mask;
71
+} meson_pwm_per_channel_data[MESON_NUM_PWMS] = {
72
+	{
73
+		.reg_offset	= REG_PWM_A,
74
+		.clk_sel_shift	= MISC_A_CLK_SEL_SHIFT,
75
+		.clk_div_shift	= MISC_A_CLK_DIV_SHIFT,
76
+		.clk_en_mask	= MISC_A_CLK_EN,
77
+		.pwm_en_mask	= MISC_A_EN,
78
+	},
79
+	{
80
+		.reg_offset	= REG_PWM_B,
81
+		.clk_sel_shift	= MISC_B_CLK_SEL_SHIFT,
82
+		.clk_div_shift	= MISC_B_CLK_DIV_SHIFT,
83
+		.clk_en_mask	= MISC_B_CLK_EN,
84
+		.pwm_en_mask	= MISC_B_EN,
85
+	}
9086 };
9187 
9288 struct meson_pwm_channel {
9389 	unsigned int hi;
9490 	unsigned int lo;
9591 	u8 pre_div;
96
-
97
-	struct pwm_state state;
9892 
9993 	struct clk *clk_parent;
10094 	struct clk_mux mux;
....@@ -109,8 +103,8 @@
109103 struct meson_pwm {
110104 	struct pwm_chip chip;
111105 	const struct meson_pwm_data *data;
106
+	struct meson_pwm_channel channels[MESON_NUM_PWMS];
112107 	void __iomem *base;
113
-	u8 inverter_mask;
114108 	/*
115109 	 * Protects register (write) access to the REG_MISC_AB register
116110 	 * that is shared between the two PWMs.
....@@ -125,12 +119,16 @@
125119 
126120 static int meson_pwm_request(struct pwm_chip *chip, struct pwm_device *pwm)
127121 {
128
-	struct meson_pwm_channel *channel = pwm_get_chip_data(pwm);
122
+	struct meson_pwm *meson = to_meson_pwm(chip);
123
+	struct meson_pwm_channel *channel;
129124 	struct device *dev = chip->dev;
130125 	int err;
131126 
132
-	if (!channel)
133
-		return -ENODEV;
127
+	channel = pwm_get_chip_data(pwm);
128
+	if (channel)
129
+		return 0;
130
+
131
+	channel = &meson->channels[pwm->hwpwm];
134132 
135133 	if (channel->clk_parent) {
136134 		err = clk_set_parent(channel->clk, channel->clk_parent);
....@@ -138,7 +136,7 @@
138136 			dev_err(dev, "failed to set parent %s for %s: %d\n",
139137 				__clk_get_name(channel->clk_parent),
140138 				__clk_get_name(channel->clk), err);
141
-				return err;
139
+			return err;
142140 		}
143141 	}
144142 
....@@ -149,33 +147,37 @@
149147 		return err;
150148 	}
151149 
152
-	chip->ops->get_state(chip, pwm, &channel->state);
153
-
154150 	return 0;
155151 }
156152 
157153 static void meson_pwm_free(struct pwm_chip *chip, struct pwm_device *pwm)
158154 {
159
-	struct meson_pwm_channel *channel = pwm_get_chip_data(pwm);
155
+	struct meson_pwm *meson = to_meson_pwm(chip);
156
+	struct meson_pwm_channel *channel = &meson->channels[pwm->hwpwm];
160157 
161158 	if (channel)
162159 		clk_disable_unprepare(channel->clk);
163160 }
164161 
165
-static int meson_pwm_calc(struct meson_pwm *meson,
166
-			  struct meson_pwm_channel *channel, unsigned int id,
167
-			  unsigned int duty, unsigned int period)
162
+static int meson_pwm_calc(struct meson_pwm *meson, struct pwm_device *pwm,
163
+			  const struct pwm_state *state)
168164 {
165
+	struct meson_pwm_channel *channel = &meson->channels[pwm->hwpwm];
169166 	unsigned int pre_div, cnt, duty_cnt;
170
-	unsigned long fin_freq = -1;
171
-	u64 fin_ps;
167
+	unsigned long fin_freq;
168
+	u64 duty, period;
172169 
173
-	if (~(meson->inverter_mask >> id) & 0x1)
170
+	duty = state->duty_cycle;
171
+	period = state->period;
172
+
173
+	/*
174
+	 * Note this is wrong. The result is an output wave that isn't really
175
+	 * inverted and so is wrongly identified by .get_state as normal.
176
+	 * Fixing this needs some care however as some machines might rely on
177
+	 * this.
178
+	 */
179
+	if (state->polarity == PWM_POLARITY_INVERSED)
174180 		duty = period - duty;
175
-
176
-	if (period == channel->state.period &&
177
-	    duty == channel->state.duty_cycle)
178
-		return 0;
179181 
180182 	fin_freq = clk_get_rate(channel->clk);
181183 	if (fin_freq == 0) {
....@@ -184,25 +186,20 @@
184186 	}
185187 
186188 	dev_dbg(meson->chip.dev, "fin_freq: %lu Hz\n", fin_freq);
187
-	fin_ps = (u64)NSEC_PER_SEC * 1000;
188
-	do_div(fin_ps, fin_freq);
189189 
190
-	/* Calc pre_div with the period */
191
-	for (pre_div = 0; pre_div <= MISC_CLK_DIV_MASK; pre_div++) {
192
-		cnt = DIV_ROUND_CLOSEST_ULL((u64)period * 1000,
193
-					    fin_ps * (pre_div + 1));
194
-		dev_dbg(meson->chip.dev, "fin_ps=%llu pre_div=%u cnt=%u\n",
195
-			fin_ps, pre_div, cnt);
196
-		if (cnt <= 0xffff)
197
-			break;
198
-	}
199
-
190
+	pre_div = div64_u64(fin_freq * period, NSEC_PER_SEC * 0xffffLL);
200191 	if (pre_div > MISC_CLK_DIV_MASK) {
201192 		dev_err(meson->chip.dev, "unable to get period pre_div\n");
202193 		return -EINVAL;
203194 	}
204195 
205
-	dev_dbg(meson->chip.dev, "period=%u pre_div=%u cnt=%u\n", period,
196
+	cnt = div64_u64(fin_freq * period, NSEC_PER_SEC * (pre_div + 1));
197
+	if (cnt > 0xffff) {
198
+		dev_err(meson->chip.dev, "unable to get period cnt\n");
199
+		return -EINVAL;
200
+	}
201
+
202
+	dev_dbg(meson->chip.dev, "period=%llu pre_div=%u cnt=%u\n", period,
206203 		pre_div, cnt);
207204 
208205 	if (duty == period) {
....@@ -215,14 +212,13 @@
215212 		channel->lo = cnt;
216213 	} else {
217214 		/* Then check is we can have the duty with the same pre_div */
218
-		duty_cnt = DIV_ROUND_CLOSEST_ULL((u64)duty * 1000,
219
-						 fin_ps * (pre_div + 1));
215
+		duty_cnt = div64_u64(fin_freq * duty, NSEC_PER_SEC * (pre_div + 1));
220216 		if (duty_cnt > 0xffff) {
221217 			dev_err(meson->chip.dev, "unable to get duty cycle\n");
222218 			return -EINVAL;
223219 		}
224220 
225
-		dev_dbg(meson->chip.dev, "duty=%u pre_div=%u duty_cnt=%u\n",
221
+		dev_dbg(meson->chip.dev, "duty=%llu pre_div=%u duty_cnt=%u\n",
226222 			duty, pre_div, duty_cnt);
227223 
228224 		channel->pre_div = pre_div;
....@@ -233,147 +229,151 @@
233229 	return 0;
234230 }
235231 
236
-static void meson_pwm_enable(struct meson_pwm *meson,
237
-			     struct meson_pwm_channel *channel,
238
-			     unsigned int id)
232
+static void meson_pwm_enable(struct meson_pwm *meson, struct pwm_device *pwm)
239233 {
240
-	u32 value, clk_shift, clk_enable, enable;
241
-	unsigned int offset;
234
+	struct meson_pwm_channel *channel = &meson->channels[pwm->hwpwm];
235
+	struct meson_pwm_channel_data *channel_data;
242236 	unsigned long flags;
237
+	u32 value;
243238 
244
-	switch (id) {
245
-	case 0:
246
-		clk_shift = MISC_A_CLK_DIV_SHIFT;
247
-		clk_enable = MISC_A_CLK_EN;
248
-		enable = MISC_A_EN;
249
-		offset = REG_PWM_A;
250
-		break;
251
-
252
-	case 1:
253
-		clk_shift = MISC_B_CLK_DIV_SHIFT;
254
-		clk_enable = MISC_B_CLK_EN;
255
-		enable = MISC_B_EN;
256
-		offset = REG_PWM_B;
257
-		break;
258
-
259
-	default:
260
-		return;
261
-	}
239
+	channel_data = &meson_pwm_per_channel_data[pwm->hwpwm];
262240 
263241 	spin_lock_irqsave(&meson->lock, flags);
264242 
265243 	value = readl(meson->base + REG_MISC_AB);
266
-	value &= ~(MISC_CLK_DIV_MASK << clk_shift);
267
-	value |= channel->pre_div << clk_shift;
268
-	value |= clk_enable;
244
+	value &= ~(MISC_CLK_DIV_MASK << channel_data->clk_div_shift);
245
+	value |= channel->pre_div << channel_data->clk_div_shift;
246
+	value |= channel_data->clk_en_mask;
269247 	writel(value, meson->base + REG_MISC_AB);
270248 
271
-	value = (channel->hi << PWM_HIGH_SHIFT) | channel->lo;
272
-	writel(value, meson->base + offset);
249
+	value = FIELD_PREP(PWM_HIGH_MASK, channel->hi) |
250
+		FIELD_PREP(PWM_LOW_MASK, channel->lo);
251
+	writel(value, meson->base + channel_data->reg_offset);
273252 
274253 	value = readl(meson->base + REG_MISC_AB);
275
-	value |= enable;
254
+	value |= channel_data->pwm_en_mask;
276255 	writel(value, meson->base + REG_MISC_AB);
277256 
278257 	spin_unlock_irqrestore(&meson->lock, flags);
279258 }
280259 
281
-static void meson_pwm_disable(struct meson_pwm *meson, unsigned int id)
260
+static void meson_pwm_disable(struct meson_pwm *meson, struct pwm_device *pwm)
282261 {
283
-	u32 value, enable;
284262 	unsigned long flags;
285
-
286
-	switch (id) {
287
-	case 0:
288
-		enable = MISC_A_EN;
289
-		break;
290
-
291
-	case 1:
292
-		enable = MISC_B_EN;
293
-		break;
294
-
295
-	default:
296
-		return;
297
-	}
263
+	u32 value;
298264 
299265 	spin_lock_irqsave(&meson->lock, flags);
300266 
301267 	value = readl(meson->base + REG_MISC_AB);
302
-	value &= ~enable;
268
+	value &= ~meson_pwm_per_channel_data[pwm->hwpwm].pwm_en_mask;
303269 	writel(value, meson->base + REG_MISC_AB);
304270 
305271 	spin_unlock_irqrestore(&meson->lock, flags);
306272 }
307273 
308274 static int meson_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
309
-			   struct pwm_state *state)
275
+			   const struct pwm_state *state)
310276 {
311
-	struct meson_pwm_channel *channel = pwm_get_chip_data(pwm);
312277 	struct meson_pwm *meson = to_meson_pwm(chip);
278
+	struct meson_pwm_channel *channel = &meson->channels[pwm->hwpwm];
313279 	int err = 0;
314280 
315281 	if (!state)
316282 		return -EINVAL;
317283 
318284 	if (!state->enabled) {
319
-		meson_pwm_disable(meson, pwm->hwpwm);
320
-		channel->state.enabled = false;
285
+		if (state->polarity == PWM_POLARITY_INVERSED) {
286
+			/*
287
+			 * This IP block revision doesn't have an "always high"
288
+			 * setting which we can use for "inverted disabled".
289
+			 * Instead we achieve this using the same settings
290
+			 * that we use a pre_div of 0 (to get the shortest
291
+			 * possible duration for one "count") and
292
+			 * "period == duty_cycle". This results in a signal
293
+			 * which is LOW for one "count", while being HIGH for
294
+			 * the rest of the (so the signal is HIGH for slightly
295
+			 * less than 100% of the period, but this is the best
296
+			 * we can achieve).
297
+			 */
298
+			channel->pre_div = 0;
299
+			channel->hi = ~0;
300
+			channel->lo = 0;
321301 
322
-		return 0;
323
-	}
324
-
325
-	if (state->period != channel->state.period ||
326
-	    state->duty_cycle != channel->state.duty_cycle ||
327
-	    state->polarity != channel->state.polarity) {
328
-		if (state->polarity != channel->state.polarity) {
329
-			if (state->polarity == PWM_POLARITY_NORMAL)
330
-				meson->inverter_mask |= BIT(pwm->hwpwm);
331
-			else
332
-				meson->inverter_mask &= ~BIT(pwm->hwpwm);
302
+			meson_pwm_enable(meson, pwm);
303
+		} else {
304
+			meson_pwm_disable(meson, pwm);
333305 		}
334
-
335
-		err = meson_pwm_calc(meson, channel, pwm->hwpwm,
336
-				     state->duty_cycle, state->period);
306
+	} else {
307
+		err = meson_pwm_calc(meson, pwm, state);
337308 		if (err < 0)
338309 			return err;
339310 
340
-		channel->state.polarity = state->polarity;
341
-		channel->state.period = state->period;
342
-		channel->state.duty_cycle = state->duty_cycle;
343
-	}
344
-
345
-	if (state->enabled && !channel->state.enabled) {
346
-		meson_pwm_enable(meson, channel, pwm->hwpwm);
347
-		channel->state.enabled = true;
311
+		meson_pwm_enable(meson, pwm);
348312 	}
349313 
350314 	return 0;
315
+}
316
+
317
+static unsigned int meson_pwm_cnt_to_ns(struct pwm_chip *chip,
318
+					struct pwm_device *pwm, u32 cnt)
319
+{
320
+	struct meson_pwm *meson = to_meson_pwm(chip);
321
+	struct meson_pwm_channel *channel;
322
+	unsigned long fin_freq;
323
+	u32 fin_ns;
324
+
325
+	/* to_meson_pwm() can only be used after .get_state() is called */
326
+	channel = &meson->channels[pwm->hwpwm];
327
+
328
+	fin_freq = clk_get_rate(channel->clk);
329
+	if (fin_freq == 0)
330
+		return 0;
331
+
332
+	fin_ns = div_u64(NSEC_PER_SEC, fin_freq);
333
+
334
+	return cnt * fin_ns * (channel->pre_div + 1);
351335 }
352336 
353337 static void meson_pwm_get_state(struct pwm_chip *chip, struct pwm_device *pwm,
354338 				struct pwm_state *state)
355339 {
356340 	struct meson_pwm *meson = to_meson_pwm(chip);
357
-	u32 value, mask;
341
+	struct meson_pwm_channel_data *channel_data;
342
+	struct meson_pwm_channel *channel;
343
+	u32 value, tmp;
358344 
359345 	if (!state)
360346 		return;
361347 
362
-	switch (pwm->hwpwm) {
363
-	case 0:
364
-		mask = MISC_A_EN;
365
-		break;
366
-
367
-	case 1:
368
-		mask = MISC_B_EN;
369
-		break;
370
-
371
-	default:
372
-		return;
373
-	}
348
+	channel = &meson->channels[pwm->hwpwm];
349
+	channel_data = &meson_pwm_per_channel_data[pwm->hwpwm];
374350 
375351 	value = readl(meson->base + REG_MISC_AB);
376
-	state->enabled = (value & mask) != 0;
352
+
353
+	tmp = channel_data->pwm_en_mask | channel_data->clk_en_mask;
354
+	state->enabled = (value & tmp) == tmp;
355
+
356
+	tmp = value >> channel_data->clk_div_shift;
357
+	channel->pre_div = FIELD_GET(MISC_CLK_DIV_MASK, tmp);
358
+
359
+	value = readl(meson->base + channel_data->reg_offset);
360
+
361
+	channel->lo = FIELD_GET(PWM_LOW_MASK, value);
362
+	channel->hi = FIELD_GET(PWM_HIGH_MASK, value);
363
+
364
+	if (channel->lo == 0) {
365
+		state->period = meson_pwm_cnt_to_ns(chip, pwm, channel->hi);
366
+		state->duty_cycle = state->period;
367
+	} else if (channel->lo >= channel->hi) {
368
+		state->period = meson_pwm_cnt_to_ns(chip, pwm,
369
+						    channel->lo + channel->hi);
370
+		state->duty_cycle = meson_pwm_cnt_to_ns(chip, pwm,
371
+							channel->hi);
372
+	} else {
373
+		state->period = 0;
374
+		state->duty_cycle = 0;
375
+	}
376
+	state->polarity = PWM_POLARITY_NORMAL;
377377 }
378378 
379379 static const struct pwm_ops meson_pwm_ops = {
....@@ -425,12 +425,39 @@
425425 };
426426 
427427 static const char * const pwm_axg_ao_parent_names[] = {
428
-	"aoclk81", "xtal", "fclk_div4", "fclk_div5"
428
+	"xtal", "axg_ao_clk81", "fclk_div4", "fclk_div5"
429429 };
430430 
431431 static const struct meson_pwm_data pwm_axg_ao_data = {
432432 	.parent_names = pwm_axg_ao_parent_names,
433433 	.num_parents = ARRAY_SIZE(pwm_axg_ao_parent_names),
434
+};
435
+
436
+static const char * const pwm_g12a_ao_ab_parent_names[] = {
437
+	"xtal", "g12a_ao_clk81", "fclk_div4", "fclk_div5"
438
+};
439
+
440
+static const struct meson_pwm_data pwm_g12a_ao_ab_data = {
441
+	.parent_names = pwm_g12a_ao_ab_parent_names,
442
+	.num_parents = ARRAY_SIZE(pwm_g12a_ao_ab_parent_names),
443
+};
444
+
445
+static const char * const pwm_g12a_ao_cd_parent_names[] = {
446
+	"xtal", "g12a_ao_clk81",
447
+};
448
+
449
+static const struct meson_pwm_data pwm_g12a_ao_cd_data = {
450
+	.parent_names = pwm_g12a_ao_cd_parent_names,
451
+	.num_parents = ARRAY_SIZE(pwm_g12a_ao_cd_parent_names),
452
+};
453
+
454
+static const char * const pwm_g12a_ee_parent_names[] = {
455
+	"xtal", "hdmi_pll", "fclk_div4", "fclk_div3"
456
+};
457
+
458
+static const struct meson_pwm_data pwm_g12a_ee_data = {
459
+	.parent_names = pwm_g12a_ee_parent_names,
460
+	.num_parents = ARRAY_SIZE(pwm_g12a_ee_parent_names),
434461 };
435462 
436463 static const struct of_device_id meson_pwm_matches[] = {
....@@ -454,12 +481,23 @@
454481 		.compatible = "amlogic,meson-axg-ao-pwm",
455482 		.data = &pwm_axg_ao_data
456483 	},
484
+	{
485
+		.compatible = "amlogic,meson-g12a-ee-pwm",
486
+		.data = &pwm_g12a_ee_data
487
+	},
488
+	{
489
+		.compatible = "amlogic,meson-g12a-ao-pwm-ab",
490
+		.data = &pwm_g12a_ao_ab_data
491
+	},
492
+	{
493
+		.compatible = "amlogic,meson-g12a-ao-pwm-cd",
494
+		.data = &pwm_g12a_ao_cd_data
495
+	},
457496 	{},
458497 };
459498 MODULE_DEVICE_TABLE(of, meson_pwm_matches);
460499 
461
-static int meson_pwm_init_channels(struct meson_pwm *meson,
462
-				   struct meson_pwm_channel *channels)
500
+static int meson_pwm_init_channels(struct meson_pwm *meson)
463501 {
464502 	struct device *dev = meson->chip.dev;
465503 	struct clk_init_data init;
....@@ -468,19 +506,20 @@
468506 	int err;
469507 
470508 	for (i = 0; i < meson->chip.npwm; i++) {
471
-		struct meson_pwm_channel *channel = &channels[i];
509
+		struct meson_pwm_channel *channel = &meson->channels[i];
472510 
473511 		snprintf(name, sizeof(name), "%s#mux%u", dev_name(dev), i);
474512 
475513 		init.name = name;
476514 		init.ops = &clk_mux_ops;
477
-		init.flags = CLK_IS_BASIC;
515
+		init.flags = 0;
478516 		init.parent_names = meson->data->parent_names;
479517 		init.num_parents = meson->data->num_parents;
480518 
481519 		channel->mux.reg = meson->base + REG_MISC_AB;
482
-		channel->mux.shift = mux_reg_shifts[i];
483
-		channel->mux.mask = BIT(MISC_CLK_SEL_WIDTH) - 1;
520
+		channel->mux.shift =
521
+				meson_pwm_per_channel_data[i].clk_sel_shift;
522
+		channel->mux.mask = MISC_CLK_SEL_MASK;
484523 		channel->mux.flags = 0;
485524 		channel->mux.lock = &meson->lock;
486525 		channel->mux.table = NULL;
....@@ -495,31 +534,16 @@
495534 
496535 		snprintf(name, sizeof(name), "clkin%u", i);
497536 
498
-		channel->clk_parent = devm_clk_get(dev, name);
499
-		if (IS_ERR(channel->clk_parent)) {
500
-			err = PTR_ERR(channel->clk_parent);
501
-			if (err == -EPROBE_DEFER)
502
-				return err;
503
-
504
-			channel->clk_parent = NULL;
505
-		}
537
+		channel->clk_parent = devm_clk_get_optional(dev, name);
538
+		if (IS_ERR(channel->clk_parent))
539
+			return PTR_ERR(channel->clk_parent);
506540 	}
507541 
508542 	return 0;
509543 }
510544 
511
-static void meson_pwm_add_channels(struct meson_pwm *meson,
512
-				   struct meson_pwm_channel *channels)
513
-{
514
-	unsigned int i;
515
-
516
-	for (i = 0; i < meson->chip.npwm; i++)
517
-		pwm_set_chip_data(&meson->chip.pwms[i], &channels[i]);
518
-}
519
-
520545 static int meson_pwm_probe(struct platform_device *pdev)
521546 {
522
-	struct meson_pwm_channel *channels;
523547 	struct meson_pwm *meson;
524548 	struct resource *regs;
525549 	int err;
....@@ -537,19 +561,13 @@
537561 	meson->chip.dev = &pdev->dev;
538562 	meson->chip.ops = &meson_pwm_ops;
539563 	meson->chip.base = -1;
540
-	meson->chip.npwm = 2;
564
+	meson->chip.npwm = MESON_NUM_PWMS;
541565 	meson->chip.of_xlate = of_pwm_xlate_with_flags;
542566 	meson->chip.of_pwm_n_cells = 3;
543567 
544568 	meson->data = of_device_get_match_data(&pdev->dev);
545
-	meson->inverter_mask = BIT(meson->chip.npwm) - 1;
546569 
547
-	channels = devm_kcalloc(&pdev->dev, meson->chip.npwm,
548
-				sizeof(*channels), GFP_KERNEL);
549
-	if (!channels)
550
-		return -ENOMEM;
551
-
552
-	err = meson_pwm_init_channels(meson, channels);
570
+	err = meson_pwm_init_channels(meson);
553571 	if (err < 0)
554572 		return err;
555573 
....@@ -558,8 +576,6 @@
558576 		dev_err(&pdev->dev, "failed to register PWM chip: %d\n", err);
559577 		return err;
560578 	}
561
-
562
-	meson_pwm_add_channels(meson, channels);
563579 
564580 	platform_set_drvdata(pdev, meson);
565581