On Wed, 30 Mar 2016 14:55:10 -0700
Stephen Boyd [off-list ref] wrote:

On 03/30, Boris Brezillon wrote:

quoted

@@ -74,6 +74,23 @@ enum pwm_polarity {
 	PWM_POLARITY_INVERSED,
 };
 
+/**
+ * struct pwm_args - PWM arguments
+ * @period: reference period
+ * @polarity: reference polarity
+ *
+ * This structure describe board-dependent arguments attached to a PWM

s/describe/describes/

quoted

+ * device. Those arguments are usually retrieved from the PWM lookup table or
+ * DT definition.
+ * This should not be confused with the PWM state: PWM args not representing

s/not representing/don't represent/ ?

Yes, I meant "are not representing", but "don't represent" is fine.

quoted

+ * the current PWM state, but the configuration the PWM user plan to use

s/plan/plans/

quoted

+ * on this PWM device.
+ */
+struct pwm_args {
+	unsigned int period;
+	enum pwm_polarity polarity;
+};
+

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com