Re: [RFC PATCH v2 4/6] arm64: dts: pwm: replace flag 0 with PWM_NOFLAGS

From: Krzysztof Kozlowski <krzk@kernel.org>
To: Oleksandr Suvorov <oleksandr.suvorov@toradex.com>
Cc: "Heiko Stuebner" <heiko@sntech.de>,
	"Marcel Ziswiler" <marcel.ziswiler@toradex.com>,
	"Kever Yang" <kever.yang@rock-chips.com>,
	"Soeren Moch" <smoch@web.de>,
	"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
	"Miquel Raynal" <miquel.raynal@bootlin.com>,
	"Hugh Cole-Baker" <sigmaris@gmail.com>,
	linux-samsung-soc@vger.kernel.org,
	"Katsuhiro Suzuki" <katsuhiro@katsuster.net>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Daniel Lezcano" <daniel.lezcano@linaro.org>,
	"Markus Reichl" <m.reichl@fivetechno.de>,
	linux-rockchip@lists.infradead.org,
	"Chen-Yu Tsai" <wens@csie.org>, "Kukjin Kim" <kgene@kernel.org>,
	"Jagan Teki" <jagan@amarulasolutions.com>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	devicetree@vger.kernel.org, "Nick Xie" <nick@khadas.com>,
	"Maxime Ripard" <mripard@kernel.org>,
	"Philippe Schenker" <philippe.schenker@toradex.com>,
	"Paul Barker" <pbarker@konsulko.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	linux-amlogic@lists.infradead.org,
	"Johan Jonker" <jbx6244@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	"Andrius Štikonas" <andrius@stikonas.eu>,
	"Igor Opaniuk" <igor.opaniuk@toradex.com>,
	linux-kernel@vger.kernel.org,
	"Enric Balletbo i Serra" <enric.balletbo@collabora.com>,
	"Andy Yan" <andy.yan@rock-chips.com>,
	"Robin Murphy" <robin.murphy@arm.com>
Subject: Re: [RFC PATCH v2 4/6] arm64: dts: pwm: replace flag 0 with PWM_NOFLAGS
Date: Thu, 9 Apr 2020 14:14:32 +0200	[thread overview]
Message-ID: <20200409121432.GA802@kozik-lap> (raw)
In-Reply-To: <20200405192246.3741784-5-oleksandr.suvorov@toradex.com>

On Sun, Apr 05, 2020 at 10:22:44PM +0300, Oleksandr Suvorov wrote:
> The raw value '0' in the 3rd cell of the "pwms" property means there are
> no flags to be set.
> 
> This patch converts '0' flag into PWM_NOFLAGS.
> 
> Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com>
> ---
> 
> Changes in v2:
> - Use PWM_NOFLAGS instead of PWM_POLARITY_NORMAL to avoid possible
>   conflict with enum pwm_polarity in <linux/pwm.h>. Also, this name
>   reflects the sense of this value more precisely.
> 
>  arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts      | 2 +-
>  arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts       | 2 +-
>  arch/arm64/boot/dts/amlogic/meson-axg-s400.dts             | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-axg.dtsi                 | 1 +
>  arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi          | 1 +
>  arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts          | 5 +++--
>  arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts            | 2 +-
>  arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts         | 5 +++--
>  arch/arm64/boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi    | 4 ++--
>  arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts       | 4 ++--
>  arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts       | 7 ++++---
>  arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi        | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi                  | 1 +
>  arch/arm64/boot/dts/amlogic/meson-gxbb-nanopi-k2.dts       | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts     | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi           | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi       | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxbb-wetek.dtsi          | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxl-s805x-p241.dts       | 3 ++-
>  .../boot/dts/amlogic/meson-gxl-s905x-hwacom-amazetv.dts    | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts | 2 +-
>  .../arm64/boot/dts/amlogic/meson-gxl-s905x-nexbox-a95x.dts | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dtsi      | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts      | 5 +++--
>  arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts         | 3 ++-
>  arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi         | 4 +++-
>  arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dts     | 2 +-
>  arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts           | 7 ++++---
>  arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi      | 3 ++-

I understand you are going to push it through arm-soc, so:
For Exynos:
Acked-by: Krzysztof Kozlowski <krzk@kernel.org>

Otherwise if you expect individual maintainers to pick it up, please
split per architecture.

Best regards,
Krzysztof

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