From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski 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 Message-ID: <20200409121432.GA802@kozik-lap> References: <20200405192246.3741784-1-oleksandr.suvorov@toradex.com> <20200405192246.3741784-5-oleksandr.suvorov@toradex.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20200405192246.3741784-5-oleksandr.suvorov@toradex.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Oleksandr Suvorov Cc: devicetree@vger.kernel.org, Paul Barker , Marcel Ziswiler , Igor Opaniuk , Philippe Schenker , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , Laurent Pinchart , Andrius =?utf-8?B?xaB0aWtvbmFz?= , Andy Yan , Chen-Yu Tsai , Daniel Lezcano , Enric Balletbo i Serra , Heiko Stuebner , Hugh Cole-Baker , Jagan Teki , Johan Jonker , Katsuhiro Suzuki , Kever Yang List-Id: linux-rockchip.vger.kernel.org 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 > --- > > Changes in v2: > - Use PWM_NOFLAGS instead of PWM_POLARITY_NORMAL to avoid possible > conflict with enum pwm_polarity in . 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 Otherwise if you expect individual maintainers to pick it up, please split per architecture. Best regards, Krzysztof