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

From: Thierry Reding <thierry.reding@gmail.com>
To: Oleksandr Suvorov <oleksandr.suvorov@toradex.com>
Cc: "Alexandre Belloni" <alexandre.belloni@bootlin.com>,
	"Heiko Stuebner" <heiko@sntech.de>,
	linux-pwm@vger.kernel.org,
	"Marcel Ziswiler" <marcel.ziswiler@toradex.com>,
	linux-kernel@vger.kernel.org,
	"Paul Cercueil" <paul@crapouillou.net>,
	"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
	linux-riscv@lists.infradead.org,
	"Fabio Estevam" <festevam@gmail.com>,
	"Florian Fainelli" <f.fainelli@gmail.com>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Chen-Yu Tsai" <wens@csie.org>,
	linux-rockchip@lists.infradead.org,
	"Ludovic Desroches" <ludovic.desroches@microchip.com>,
	bcm-kernel-feedback-list@broadcom.com,
	"NXP Linux Team" <linux-imx@nxp.com>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	devicetree@vger.kernel.org, "Ray Jui" <rjui@broadcom.com>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Maxime Ripard" <mripard@kernel.org>,
	"Philippe Schenker" <philippe.schenker@toradex.com>,
	"Paul Barker" <pbarker@konsulko.com>,
	"Paul Walmsley" <paul.walmsley@sifive.com>,
	linux-amlogic@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	"Igor Opaniuk" <igor.opaniuk@toradex.com>,
	"Scott Branden" <sbranden@broadcom.com>,
	"Tony Prisk" <linux@prisktech.co.nz>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Pengutronix Kernel Team" <kernel@pengutronix.de>,
	"Shawn Guo" <shawnguo@kernel.org>,
	"Claudiu Beznea" <claudiu.beznea@microchip.com>
Subject: Re: [RFC PATCH 1/7] pwm: rename the PWM_POLARITY_INVERSED enum
Date: Tue, 17 Mar 2020 18:40:43 +0100	[thread overview]
Message-ID: <20200317174043.GA1464607@ulmo> (raw)
In-Reply-To: <20200317123231.2843297-2-oleksandr.suvorov@toradex.com>

[-- Attachment #1.1: Type: text/plain, Size: 877 bytes --]

On Tue, Mar 17, 2020 at 02:32:25PM +0200, Oleksandr Suvorov wrote:
> The polarity enum definition PWM_POLARITY_INVERSED is misspelled.
> Rename it to PWM_POLARITY_INVERTED.

It isn't misspelled. "inversed" is a synonym for "inverted". Both
spellings are correct.

And as you noted in the cover letter, there's a conflict between the
macro defined in dt-bindings/pwm/pwm.txt. If they end up being included
in the wrong order you'll get a compile error.

The enum was named this way on purpose to make it separate from the
definition for the DT bindings. Note that DT bindings are an ABI and can
never change, whereas the enum pwm_polarity is part of a Linux internal
API and doesn't have the same restrictions as an ABI.

As far as I'm concerned this is completely unnecessary churn that's
potentially going to come back and bite us, so I see no reason to accept
this.

Thierry

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel