Re: [PATCH v3] arm64: dts: qcom: qcs6490-rb3gen2: enable Bluetooth

From: Krzysztof Kozlowski
Date: Mon Dec 02 2024 - 04:36:22 EST


On 02/12/2024 10:32, Janaki Ramaiah Thota wrote:
>>
>>> + pinctrl-names = "default", "sleep";
>>> + pinctrl-1 = <&qup_uart7_sleep_cts>, <&qup_uart7_sleep_rts>,
>>> + <&qup_uart7_sleep_tx>, <&qup_uart7_sleep_rx>;
>>
>> This could be just one phandle to state node with multiple pins subnode.
>>
>
> Since we have different configuration for 3 GPIOs, it’s not good to
> combine all 4 GPIOs into one common sleep configuration. Each GPIO
> configuration is having separate requirements based on the Bluetooth SOC.

I did not propose that. Please read bindings and other existing DTS sources.

> This is as per hardware recommendations.



Best regards,
Krzysztof