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

From: Janaki Ramaiah Thota
Date: Mon Dec 02 2024 - 09:09:45 EST




On 12/2/2024 4:56 PM, Konrad Dybcio wrote:
On 2.12.2024 10:36 AM, Krzysztof Kozlowski wrote:
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.

Janaki,

you can refer to my older patch (attached)

Konrad

Thank you very much for the information Konrad.

Thanks,
Janakiram