Re: [RFC PATCH 1/7] pwm: rename the PWM_POLARITY_INVERSED enum

From: Oleksandr Suvorov
Date: Tue Mar 17 2020 - 12:41:02 EST


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