Thread (1 message) 1 message, 1 author, 2012-02-28

[PATCH v3 06/10] pwm: Add NVIDIA Tegra SoC support

From: Stephen Warren <hidden>
Date: 2012-02-28 21:14:35
Also in: linux-devicetree, linux-tegra

Possibly related (same subject, not in this thread)

Thierry Reding wrote at Wednesday, February 22, 2012 8:17 AM:
This commit adds a generic PWM framework driver for the PWFM controller
found on NVIDIA Tegra SoCs. The driver is based on code from the
Chromium kernel tree and was originally written by Gary King (NVIDIA)
and later modified by Simon Que (Chromium).
quoted hunk
diff --git a/drivers/pwm/pwm-tegra.c b/drivers/pwm/pwm-tegra.c
+static int tegra_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm,
+	/*
+	 * Convert from duty_ns / period_ns to a fixed number of duty
+	 * ticks per (1 << PWM_DUTY_WIDTH) cycles.
+	 */
+	c = duty_ns * ((1 << PWM_DUTY_WIDTH) - 1);
The driver in ChromeOS has the following extra line here:

    c += (period_ns / 2);

and the following extra line in the comment above:

    Also, make sure to round to the nearest integer during division.

I don't know the HW well enough to know if that line should be present?

Otherwise, this looks similar enough to the ChromeOS driver, so:

Acked-by: Stephen Warren <redacted>

-- 
nvpublic
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help