On Thu May 12, 2022 at 8:07 PM IST, Krzysztof Kozlowski wrote: > On 12/05/2022 11:19, Sireesh Kodali wrote: > > On Wed May 11, 2022 at 11:25 PM IST, Krzysztof Kozlowski wrote: > >> On 11/05/2022 18:16, Sireesh Kodali wrote: > >>> This commit adds the modem (q6v5_mss), WiFi (wcnss-pil) and audio DSP > >>> (q6v5_pas) remote processor nodes for the MSM8953 platform. It also adds > >>> the coresponding SMP2P, SMSM and pinctrl nodes that are needed by these > >>> remote processors. > >>> > >>> Signed-off-by: Sireesh Kodali <sireeshkodali1@xxxxxxxxx> > >>> + }; > >>> + > >>> + wcnss_sleep: wcnss-sleep-pins { > >>> + wcss_wlan2 { > >> > >> No underscores in node names, unless something needs it? > >> > > > > wcnss_sleep is used by the pronto node defined below > > wcnss_sleep is not a node name and it is not in the line where I commented. > Sorry, I'll fix it in v2 > Best regards, > Krzysztof Thanks, Sireesh