RE: [EXT] Re: [PATCH v6 3/3] Bluetooth: NXP: Add protocol support for NXP Bluetooth chipsets

From: Neeraj sanjay kale <neeraj.sanjaykale@nxp.com>
To: Francesco Dolcini <francesco@dolcini.it>
Cc: "davem@davemloft.net" <davem@davemloft.net>,
	"edumazet@google.com" <edumazet@google.com>,
	"kuba@kernel.org" <kuba@kernel.org>,
	"pabeni@redhat.com" <pabeni@redhat.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org" 
	<krzysztof.kozlowski+dt@linaro.org>,
	"marcel@holtmann.org" <marcel@holtmann.org>,
	"johan.hedberg@gmail.com" <johan.hedberg@gmail.com>,
	"luiz.dentz@gmail.com" <luiz.dentz@gmail.com>,
	"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
	"jirislaby@kernel.org" <jirislaby@kernel.org>,
	"alok.a.tiwari@oracle.com" <alok.a.tiwari@oracle.com>,
	"hdanton@sina.com" <hdanton@sina.com>,
	"ilpo.jarvinen@linux.intel.com" <ilpo.jarvinen@linux.intel.com>,
	"leon@kernel.org" <leon@kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-bluetooth@vger.kernel.org"
	<linux-bluetooth@vger.kernel.org>,
	"linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>,
	Amitkumar Karwar <amitkumar.karwar@nxp.com>,
	Rohit Fule <rohit.fule@nxp.com>, Sherry Sun <sherry.sun@nxp.com>
Subject: RE: [EXT] Re: [PATCH v6 3/3] Bluetooth: NXP: Add protocol support for NXP Bluetooth chipsets
Date: Fri, 10 Mar 2023 13:51:48 +0000	[thread overview]
Message-ID: <AM9PR04MB8603DA275FB41A5A22491D3AE7BA9@AM9PR04MB8603.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <ZAsp9fm779DR0Vuz@francesco-nb.int.toradex.com>

Hi Francesco

> 
> On Fri, Mar 10, 2023 at 12:09:09PM +0000, Neeraj sanjay kale wrote:
> > > > +#define FIRMWARE_W8987       "nxp/uartuart8987_bt.bin"
> > > > +#define FIRMWARE_W8997       "nxp/uartuart8997_bt_v4.bin"
> > > > +#define FIRMWARE_W9098       "nxp/uartuart9098_bt_v1.bin"
> > > > +#define FIRMWARE_IW416       "nxp/uartiw416_bt_v0.bin"
> > > > +#define FIRMWARE_IW612       "nxp/uartspi_n61x_v1.bin.se"
> > >
> > > Where are this files coming from? Where can I download those?
> > > Is loading a combo firmware from the mwifiex driver supported?
> > We are working on submitting these files to linux-firmware. They will
> > be available under nxp/ directory once merged.
> 
> What about the combo firmware that would be downloaded by mwifiex
> driver? How is this supposed to interact with it?
If combo firmware is loaded by the mwifiex, then this driver would not detect any bootloader signatures, and would skip FW downloading and move on. Please check the nxp_setup() function.

> 
> > > > +#define HCI_NXP_PRI_BAUDRATE 115200 #define
> HCI_NXP_SEC_BAUDRATE
> > > > +3000000
> > > What if the UART device does not support 3000000 baudrate (think at
> > > limitation on the clock source/divider of the UART)? Shouldn't this
> > > be configurable?
> > We have noted this requirement and decided to design and implement on
> > this in upcoming patches along with other new features.  We have a
> > number of customers out there who have been using these chips as well
> > as the legacy Marvell chips, which need FW download at 3000000
> > baudrate, and so far there were no issues reported.  Using a lower
> > standard baudrate affects the time it takes to download the FW, which
> > we are trying to keep strictly under 5 seconds.
> 
> ok, just for you to know our hardware, using NXP SoC, will not work with this
> baudrate (no way to have it given the clock tree we have).
Got it! We will work on this requirement for sure and come up with something soon in the next patch for this driver.
FYI, firmware download takes around 20 seconds at 115200 baudrate, which is highly unacceptable by many customers. 😊

Thanks,
Neeraj