linux-kernel - Re: [PATCH] arm64: dts: mt8183: Add pwm and backlight node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8484ec3b-14aa-f0de-28d6-f360c491515b@collabora.com>
Date:   Wed, 25 Nov 2020 11:24:49 +0100
From:   Enric Balletbo i Serra <enric.balletbo@...labora.com>
To:     Hsin-Yi Wang <hsinyi@...omium.org>, linux-kernel@...r.kernel.org
Cc:     Rob Herring <robh+dt@...nel.org>,
        Matthias Brugger <matthias.bgg@...il.com>,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH] arm64: dts: mt8183: Add pwm and backlight node

Hi Hsin-Yi,

Thank you for your patch.

On 24/11/20 5:12, Hsin-Yi Wang wrote:
> Add pwm to mt8183 and backlight to mt8183-kukui.
> 
> Signed-off-by: Hsin-Yi Wang <hsinyi@...omium.org>
> ---

Picked the patch and checked that pwm for the backlight is working as expected
on my Lenovo Ideapad Duet.

Tested-by: Enric Balletbo i Serra <enric.balletbo@...labora.com>

>  .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 28 +++++++++++++++++++
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi      | 10 +++++++
>  2 files changed, 38 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi
> index 85f7c33ba4461..bf2ad1294dd30 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi
> @@ -19,6 +19,17 @@ chosen {
>  		stdout-path = "serial0:115200n8";
>  	};
>  
> +	backlight_lcd0: backlight_lcd0 {
> +		compatible = "pwm-backlight";
> +		pwms = <&pwm0 0 500000>;
> +		power-supply = <&bl_pp5000>;
> +		enable-gpios = <&pio 176 0>;
> +		brightness-levels = <0 1023>;
> +		num-interpolated-steps = <1023>;
> +		default-brightness-level = <576>;
> +		status = "okay";
> +	};
> +
>  	memory@...00000 {
>  		device_type = "memory";
>  		reg = <0 0x40000000 0 0x80000000>;
> @@ -536,6 +547,17 @@ pins_clk {
>  		};
>  	};
>  
> +	pwm0_pin_default: pwm0_pin_default {
> +		pins1 {
> +			pinmux = <PINMUX_GPIO176__FUNC_GPIO176>;
> +			output-high;
> +			bias-pull-up;
> +		};
> +		pins2 {
> +			pinmux = <PINMUX_GPIO43__FUNC_DISP_PWM>;
> +		};
> +	};
> +
>  	scp_pins: scp {
>  		pins_scp_uart {
>  			pinmux = <PINMUX_GPIO110__FUNC_TP_URXD1_AO>,
> @@ -670,6 +692,12 @@ pins_wifi_wakeup {
>  	};
>  };
>  
> +&pwm0 {
> +	status = "okay";
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&pwm0_pin_default>;
> +};
> +
>  &scp {
>  	status = "okay";
>  	pinctrl-names = "default";
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 08a914d3a6435..a974bad899365 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -479,6 +479,16 @@ spi0: spi@...0a000 {
>  			status = "disabled";
>  		};
>  
> +		pwm0: pwm@...0e000 {
> +			compatible = "mediatek,mt8183-disp-pwm";
> +			reg = <0 0x1100e000 0 0x1000>;
> +			interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_LOW>;
> +			#pwm-cells = <2>;
> +			clocks = <&topckgen CLK_TOP_MUX_DISP_PWM>,
> +					<&infracfg CLK_INFRA_DISP_PWM>;
> +			clock-names = "main", "mm";
> +		};
> +
>  		i2c3: i2c@...0f000 {
>  			compatible = "mediatek,mt8183-i2c";
>  			reg = <0 0x1100f000 0 0x1000>,
>