Re: [PATCH v6 3/4] arm64: dts: qcom: sdm845-oneplus: Implement panel sleep pinctrl

[David Heidelberg]

On 03/11/2025 12:45, David Heidelberg via B4 Relay wrote:
> From: David Heidelberg <david@xxxxxxx>
>
> We can DSI pin from 8mA to 2mA while suspend, do it.
>
> In theory, should give us extra 2 hours of idle battery life.
>
> cosmetic: sort pinctrl properties.
>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> Signed-off-by: David Heidelberg <david@xxxxxxx>
> ---
>   .../arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi | 26 +++++++++++++++++++++-
>   1 file changed, 25 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> index d59a5e8cdeb2c..f00dbc0fff3d4 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> @@ -449,8 +449,9 @@ display_panel: panel@0 {
>   
>   		reset-gpios = <&tlmm 6 GPIO_ACTIVE_LOW>;
>   
> -		pinctrl-names = "default";
>   		pinctrl-0 = <&panel_default>;
> +		pinctrl-1 = <&panel_sleep>;
> +		pinctrl-names = "default", "sleep";
>   

Recent Matrix discussion brought me to look into how sleep state is handled.

Should the panel driver add to pinctrl_pm_select_sleep_state to enter the state?

Thanks
David