Re: [PATCH 12/14] arm64: dts: qcom: sda660-ifc6560: describe WiFi/BT properly

From: Dmitry Baryshkov

Date: Wed Dec 31 2025 - 11:20:56 EST


On Wed, Dec 31, 2025 at 01:19:23PM +0100, Konrad Dybcio wrote:
> On 12/31/25 12:36 AM, Dmitry Baryshkov wrote:
> > The onboard WiFi / BT device, WCN3990, has a simple on-chip PMU, which
> > further spreads generated voltage. Describe the PMU in the device tree.
> >
> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> > ---
>
> [...]
>
> > +&pm660_gpios {
> > + sw_ctrl_default: sw-ctrl-default-state {
> > + pins = "gpio5";
> > + function = "normal";
> > +
> > + input-enable;
> > + bias-pull-down;
>
> Ideally
>
> func
> (no \n)
> bias-pull-down
> input-enable

I'll try remembering this for the next iteration.

>
> anyway:
>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
>
> Konrad

--
With best wishes
Dmitry