RE: [PATCH 1/2] dt-bindings: net: bluetooth: nxp: add support for supply and reset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Marco Felsch <m.felsch@xxxxxxxxxxxxxx>
> Sent: Monday, October 28, 2024 5:00 PM
> To: Sherry Sun <sherry.sun@xxxxxxx>
> Cc: POPESCU Catalin <catalin.popescu@xxxxxxxxxxxxxxxxxxxx>; Amitkumar
> Karwar <amitkumar.karwar@xxxxxxx>; Neeraj Sanjay Kale
> <neeraj.sanjaykale@xxxxxxx>; marcel@xxxxxxxxxxxx;
> luiz.dentz@xxxxxxxxx; robh@xxxxxxxxxx; krzk+dt@xxxxxxxxxx;
> conor+dt@xxxxxxxxxx; p.zabel@xxxxxxxxxxxxxx; linux-
> bluetooth@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; GEO-CHHER-bsp-development <bsp-
> development.geo@xxxxxxxxxxxxxxxxxxxx>; Krzysztof Kozlowski
> <krzk@xxxxxxxxxx>
> Subject: Re: [PATCH 1/2] dt-bindings: net: bluetooth: nxp: add support for
> supply and reset
> 
> Hi,
> 
> On 24-10-28, Sherry Sun wrote:
> >
> > > From: POPESCU Catalin <catalin.popescu@xxxxxxxxxxxxxxxxxxxx>
> > >
> > > We use the NXP downstream driver mwifiex which doesn't have support
> > > for regulator or PDn.
> > >
> > > However, regulator is already supported by the MMC core (vmmc-supply).
> > >
> > > For PDn, we use mmc pwrseq simple driver that has been patched to
> > > add support for reset-control.
> >
> > Ok, thanks, the mmc change looks good for me, so there is no problem
> > with the NXP SDIO wifi.
> >
> > But how do you plan to handle the NXP PCIe wifi? We also need to make
> > sure the BT patch won't break the PCIe wifi function.
> 
> Can you please elaborate how this could break the PCIe use-case?

Similar to the SDIO wifi, if no corresponding reset control for the PDn pin in PCIe wifi driver, the wifi part will be unexpectedly powered off when removing the BT driver.

Best Regards
Sherry





[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux