On Tue, Mar 17, 2020 at 6:27 PM <Claudiu.Beznea@xxxxxxxxxxxxx> wrote: > > > > On 17.03.2020 14:32, Oleksandr Suvorov wrote: > > @@ -187,7 +187,7 @@ static ssize_t polarity_store(struct device *child, > > if (sysfs_streq(buf, "normal")) > > polarity = PWM_POLARITY_NORMAL; > > else if (sysfs_streq(buf, "inversed")) > > You may also consider this string ^ Thanks for the feedback, Claudiu. I thought about it and decided not to change the ABI, as this change can impact lots of user-land applications. As a minimum, I can push this change as a separate patch to be able to revert the change of ABI only. What do you think? > > - polarity = PWM_POLARITY_INVERSED; > > + polarity = PWM_POLARITY_INVERTED; -- Best regards Oleksandr Suvorov Toradex AG Ebenaustrasse 10 | 6048 Horw | Switzerland | T: +41 41 500 48 00