* [PATCH v3] arm64: dts: exynos: add pwm node for exynosautov9-sadk [not found] <CGME20230717063837epcas2p3780fbf0f8c12ca070aa296766d0eb5e8@epcas2p3.samsung.com> @ 2023-07-17 6:33 ` Jaewon Kim 2023-07-17 6:42 ` Krzysztof Kozlowski 0 siblings, 1 reply; 2+ messages in thread From: Jaewon Kim @ 2023-07-17 6:33 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Alim Akhtar Cc: devicetree, linux-arm-kernel, linux-samsung-soc, linux-kernel, Jaewon Kim Add pwm node to support fan on exynosautov9-sadk board. PWM channel 3 of ExynosAutov9 is connected to fan for SoC cooling in SADK board. Signed-off-by: Jaewon Kim <jaewon02.kim@samsung.com> --- Changes in v3: - removed adding compatible to driver. Changes in v2: - add compatible string to driver. --- arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts | 6 ++++++ arch/arm64/boot/dts/exynos/exynosautov9.dtsi | 10 ++++++++++ 2 files changed, 16 insertions(+) diff --git a/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts b/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts index 898c2fc345ed..e717bb1cad81 100644 --- a/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts +++ b/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts @@ -50,6 +50,12 @@ }; }; +&pwm { + pinctrl-names = "default"; + pinctrl-0 = <&pwm_tout3>; + status = "okay"; +}; + &serial_0 { pinctrl-0 = <&uart0_bus_dual>; status = "okay"; diff --git a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi index d3c5cdeff47f..3b906f4db907 100644 --- a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi +++ b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi @@ -1560,6 +1560,16 @@ samsung,syscon-phandle = <&pmu_system_controller>; samsung,cluster-index = <1>; }; + + pwm: pwm@103f0000 { + compatible = "samsung,exynosautov9-pwm", + "samsung,exynos4210-pwm"; + reg = <0x103f0000 0x100>; + samsung,pwm-outputs = <0>, <1>, <2>, <3>; + #pwm-cells = <3>; + clocks = <&xtcxo>; + clock-names = "timers"; + }; }; }; -- 2.17.1 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v3] arm64: dts: exynos: add pwm node for exynosautov9-sadk 2023-07-17 6:33 ` [PATCH v3] arm64: dts: exynos: add pwm node for exynosautov9-sadk Jaewon Kim @ 2023-07-17 6:42 ` Krzysztof Kozlowski 0 siblings, 0 replies; 2+ messages in thread From: Krzysztof Kozlowski @ 2023-07-17 6:42 UTC (permalink / raw) To: Jaewon Kim, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Alim Akhtar Cc: devicetree, linux-arm-kernel, linux-samsung-soc, linux-kernel On 17/07/2023 08:33, Jaewon Kim wrote: > Add pwm node to support fan on exynosautov9-sadk board. > PWM channel 3 of ExynosAutov9 is connected to fan for SoC cooling > in SADK board. > > Signed-off-by: Jaewon Kim <jaewon02.kim@samsung.com> > --- > Changes in v3: > - removed adding compatible to driver. > > Changes in v2: > - add compatible string to driver. > --- > arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts | 6 ++++++ > arch/arm64/boot/dts/exynos/exynosautov9.dtsi | 10 ++++++++++ > 2 files changed, 16 insertions(+) > > diff --git a/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts b/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts > index 898c2fc345ed..e717bb1cad81 100644 > --- a/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts > +++ b/arch/arm64/boot/dts/exynos/exynosautov9-sadk.dts > @@ -50,6 +50,12 @@ > }; > }; > > +&pwm { > + pinctrl-names = "default"; > + pinctrl-0 = <&pwm_tout3>; > + status = "okay"; > +}; > + > &serial_0 { > pinctrl-0 = <&uart0_bus_dual>; > status = "okay"; > diff --git a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi > index d3c5cdeff47f..3b906f4db907 100644 > --- a/arch/arm64/boot/dts/exynos/exynosautov9.dtsi > +++ b/arch/arm64/boot/dts/exynos/exynosautov9.dtsi > @@ -1560,6 +1560,16 @@ > samsung,syscon-phandle = <&pmu_system_controller>; > samsung,cluster-index = <1>; > }; > + > + pwm: pwm@103f0000 { > + compatible = "samsung,exynosautov9-pwm", > + "samsung,exynos4210-pwm"; It does not look like you tested the DTS against bindings. Please run `make dtbs_check` (see Documentation/devicetree/bindings/writing-schema.rst or https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/ for instructions). Full DT schema compliance is a requirement for Samsung (since some time). Best regards, Krzysztof ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-07-17 6:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20230717063837epcas2p3780fbf0f8c12ca070aa296766d0eb5e8@epcas2p3.samsung.com>
2023-07-17 6:33 ` [PATCH v3] arm64: dts: exynos: add pwm node for exynosautov9-sadk Jaewon Kim
2023-07-17 6:42 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).