Re: [PATCH v3] ARM: dts: nexus4: Initial dts

[David Heidelberg]

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?

> > Thank you
> > David
> >
> > > > +};
> > > >
> > > > ---
> > > > base-commit: 0a2f889128969dab41861b6e40111aa03dc57014
> > > > change-id: 20250309-lg-nexus4-mako-da0833885b26
> > > >
> > > > Best regards,
> > > > --
> > > > David Heidelberg <david@xxxxxxx>
> >
> > --
> > David Heidelberg

--
David Heidelberg