On Sun, 9 Mar 2025 at 23:39, David Heidelberg <david@xxxxxxx> wrote: > > > > On 09/03/2025 10:36, Dmitry Baryshkov wrote: > > On Sun, Mar 09, 2025 at 10:17:29AM +0100, David Heidelberg wrote: > >> Hello Dmitry! > >> > >> Thank you for looking into it. See replies. > >> > >> On 09/03/2025 09:33, Dmitry Baryshkov wrote: > >>> On Sun, Mar 09, 2025 at 01:45:51AM +0100, David Heidelberg via B4 Relay wrote: > >>>> + > >>>> +&riva { > >>>> + status = "okay"; > >>>> + pinctrl-names = "default"; > >>>> + pinctrl-0 = <&riva_wlan_pin_a>; > >>> > >>> Where is it defined? Also pinctrl-names should come after pinctrl-N. > >> > >> definition is kinda aside in qcom-apq8064-pins.dtsi . > > > > Ack, missed it. > > > >> > >> All other suggestions incorporated, if it's OK otherwise, let me send v2 > > > > I think this also needs several supplies in the riva device itself and > > in the iris subdevice. See qcom-apq8064-sony-xperia-lagan-yuga.dts. > > > > I don't have the device in my hands, so if I figure out the regulators, > I'll not be able to test new WiFi setup. I would drop the WiFi node for > now, so at least the base regulators and eMMC gets and possible someone > with devices in his hand can push more functionality forward, if you > don't object. > > There is slowly forming some APQ8064/MSM8960 community, so I think there > will be updates, now I would just settle with minimum, so our downstream > changes will contain only small chunks to get upstreamed. > > Would you be ok with this approach? Yes, that sounds perfectly fine. > > >> > >> Thank you > >> David > >> > >>> > >>>> +}; > >>>> > >>>> --- > >>>> base-commit: 0a2f889128969dab41861b6e40111aa03dc57014 > >>>> change-id: 20250309-lg-nexus4-mako-da0833885b26 > >>>> > >>>> Best regards, > >>>> -- > >>>> David Heidelberg <david@xxxxxxx> > >>>> > >>>> > >>> > >> > >> -- > >> David Heidelberg > >> > > > > -- > David Heidelberg > -- With best wishes Dmitry