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