* Re: mwifiex: PCIe8997 chip specific handling
[not found] <1469788731-5361-1-git-send-email-akarwar@marvell.com>
@ 2016-08-09 18:44 ` Brian Norris
2016-08-10 7:07 ` Amitkumar Karwar
0 siblings, 1 reply; 5+ messages in thread
From: Brian Norris @ 2016-08-09 18:44 UTC (permalink / raw)
To: Amitkumar Karwar
Cc: linux-wireless, Cathy Luo, Nishant Sarmukadam, linux-kernel
Hi,
On Fri, Jul 29, 2016 at 04:08:51PM +0530, Amitkumar Karwar wrote:
> The patch corrects the revision id register and uses it along with
> magic value and chip version registers to download appropriate firmware
> image.
>
> PCIe8997 Z chipset variant code has been removed, as it won't be used in
> production.
>
> Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> ---
> drivers/net/wireless/marvell/mwifiex/pcie.c | 35 ++++++++++-------------------
> drivers/net/wireless/marvell/mwifiex/pcie.h | 14 +++++-------
> 2 files changed, 18 insertions(+), 31 deletions(-)
[...]
> diff --git a/drivers/net/wireless/marvell/mwifiex/pcie.h b/drivers/net/wireless/marvell/mwifiex/pcie.h
> index f6992f0..46f99ca 100644
> --- a/drivers/net/wireless/marvell/mwifiex/pcie.h
> +++ b/drivers/net/wireless/marvell/mwifiex/pcie.h
> @@ -32,12 +32,9 @@
> #define PCIE8897_DEFAULT_FW_NAME "mrvl/pcie8897_uapsta.bin"
> #define PCIE8897_A0_FW_NAME "mrvl/pcie8897_uapsta_a0.bin"
> #define PCIE8897_B0_FW_NAME "mrvl/pcie8897_uapsta.bin"
> -#define PCIE8997_DEFAULT_FW_NAME "mrvl/pcieusb8997_combo_v2.bin"
> -#define PCIEUART8997_FW_NAME_Z "mrvl/pcieuart8997_combo.bin"
> -#define PCIEUART8997_FW_NAME_V2 "mrvl/pcieuart8997_combo_v2.bin"
> -#define PCIEUSB8997_FW_NAME_Z "mrvl/pcieusb8997_combo.bin"
> -#define PCIEUSB8997_FW_NAME_V2 "mrvl/pcieusb8997_combo_v2.bin"
> -#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan.bin"
> +#define PCIEUART8997_FW_NAME_V4 "mrvl/pcieuart8997_combo_v4.bin"
> +#define PCIEUSB8997_FW_NAME_V4 "mrvl/pcieusb8997_combo_v4.bin"
> +#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan_v4.bin"
Why do version bumps require firmware renames? Is this just to make sure
you don't load the new firmware on old chip revs that you don't plan to
support for production (i.e., only early revs like the _Z you're
dropping)? This doesn't seems like a good long-term solution, at least
once you start getting this silicon out in the wild. At some point, I'd
expect to see a stable file name.
Brian
>
> #define PCIE_VENDOR_ID_MARVELL (0x11ab)
> #define PCIE_VENDOR_ID_V2_MARVELL (0x1b4b)
> @@ -47,9 +44,10 @@
>
> #define PCIE8897_A0 0x1100
> #define PCIE8897_B0 0x1200
> -#define PCIE8997_Z 0x0
> -#define PCIE8997_V2 0x471
> +#define PCIE8997_A0 0x10
> +#define PCIE8997_A1 0x11
> #define CHIP_VER_PCIEUART 0x3
> +#define CHIP_MAGIC_VALUE 0x24
>
> /* Constants for Buffer Descriptor (BD) rings */
> #define MWIFIEX_MAX_TXRX_BD 0x20
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: mwifiex: PCIe8997 chip specific handling
2016-08-09 18:44 ` mwifiex: PCIe8997 chip specific handling Brian Norris
@ 2016-08-10 7:07 ` Amitkumar Karwar
2016-08-10 21:08 ` Steve deRosier
0 siblings, 1 reply; 5+ messages in thread
From: Amitkumar Karwar @ 2016-08-10 7:07 UTC (permalink / raw)
To: Brian Norris
Cc: linux-wireless@vger.kernel.org, Cathy Luo, Nishant Sarmukadam,
linux-kernel@vger.kernel.org, Wei-Ning Huang
Hi Brian,
> From: Brian Norris [mailto:briannorris@chromium.org]
> Sent: Wednesday, August 10, 2016 12:14 AM
> To: Amitkumar Karwar
> Cc: linux-wireless@vger.kernel.org; Cathy Luo; Nishant Sarmukadam;
> linux-kernel@vger.kernel.org
> Subject: Re: mwifiex: PCIe8997 chip specific handling
>
> Hi,
>
> On Fri, Jul 29, 2016 at 04:08:51PM +0530, Amitkumar Karwar wrote:
> > The patch corrects the revision id register and uses it along with
> > magic value and chip version registers to download appropriate
> > firmware image.
> >
> > PCIe8997 Z chipset variant code has been removed, as it won't be used
> > in production.
> >
> > Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> > ---
> > drivers/net/wireless/marvell/mwifiex/pcie.c | 35
> > ++++++++++-------------------
> > drivers/net/wireless/marvell/mwifiex/pcie.h | 14 +++++-------
> > 2 files changed, 18 insertions(+), 31 deletions(-)
>
> [...]
>
> > diff --git a/drivers/net/wireless/marvell/mwifiex/pcie.h
> > b/drivers/net/wireless/marvell/mwifiex/pcie.h
> > index f6992f0..46f99ca 100644
> > --- a/drivers/net/wireless/marvell/mwifiex/pcie.h
> > +++ b/drivers/net/wireless/marvell/mwifiex/pcie.h
> > @@ -32,12 +32,9 @@
> > #define PCIE8897_DEFAULT_FW_NAME "mrvl/pcie8897_uapsta.bin"
> > #define PCIE8897_A0_FW_NAME "mrvl/pcie8897_uapsta_a0.bin"
> > #define PCIE8897_B0_FW_NAME "mrvl/pcie8897_uapsta.bin"
> > -#define PCIE8997_DEFAULT_FW_NAME "mrvl/pcieusb8997_combo_v2.bin"
> > -#define PCIEUART8997_FW_NAME_Z "mrvl/pcieuart8997_combo.bin"
> > -#define PCIEUART8997_FW_NAME_V2 "mrvl/pcieuart8997_combo_v2.bin"
> > -#define PCIEUSB8997_FW_NAME_Z "mrvl/pcieusb8997_combo.bin"
> > -#define PCIEUSB8997_FW_NAME_V2 "mrvl/pcieusb8997_combo_v2.bin"
> > -#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan.bin"
> > +#define PCIEUART8997_FW_NAME_V4 "mrvl/pcieuart8997_combo_v4.bin"
> > +#define PCIEUSB8997_FW_NAME_V4 "mrvl/pcieusb8997_combo_v4.bin"
> > +#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan_v4.bin"
>
> Why do version bumps require firmware renames? Is this just to make sure
> you don't load the new firmware on old chip revs that you don't plan to
> support for production (i.e., only early revs like the _Z you're
> dropping)? This doesn't seems like a good long-term solution, at least
> once you start getting this silicon out in the wild. At some point, I'd
> expect to see a stable file name.
>
> Brian
>
We haven't yet submitted any firmware image upstream for 8997 chipset.
pcieuart8997_combo_v4.bin/pcieusb8997_combo_v4.bin would be our firmware candidate for upstream submission. The filename would remain same hereafter.
pcie*8997_combo_v2.bin had support only for A0 chipset
pcie*8997_combo_v3.bin was our internal development version which had support for A1 chipset
pcie*8997_combo_v4.bin has support for both A0 and A1 chipsets and this is the version that shall be released to customers/upstream from now on.
Regards,
Amitkumar Karwar
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: mwifiex: PCIe8997 chip specific handling
2016-08-10 7:07 ` Amitkumar Karwar
@ 2016-08-10 21:08 ` Steve deRosier
2016-08-11 10:41 ` Amitkumar Karwar
0 siblings, 1 reply; 5+ messages in thread
From: Steve deRosier @ 2016-08-10 21:08 UTC (permalink / raw)
To: Amitkumar Karwar
Cc: Brian Norris, linux-wireless@vger.kernel.org, Cathy Luo,
Nishant Sarmukadam, linux-kernel@vger.kernel.org, Wei-Ning Huang
Hi,
On Wed, Aug 10, 2016 at 12:07 AM, Amitkumar Karwar <akarwar@marvell.com> wrote:
> Hi Brian,
>
>> From: Brian Norris [mailto:briannorris@chromium.org]
>> Sent: Wednesday, August 10, 2016 12:14 AM
>> To: Amitkumar Karwar
>> Cc: linux-wireless@vger.kernel.org; Cathy Luo; Nishant Sarmukadam;
>> linux-kernel@vger.kernel.org
>> Subject: Re: mwifiex: PCIe8997 chip specific handling
>>
>> Hi,
>>
>> On Fri, Jul 29, 2016 at 04:08:51PM +0530, Amitkumar Karwar wrote:
>> > The patch corrects the revision id register and uses it along with
>> > magic value and chip version registers to download appropriate
>> > firmware image.
>> >
>> > PCIe8997 Z chipset variant code has been removed, as it won't be used
>> > in production.
>> >
>> > Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
>> > ---
>> > drivers/net/wireless/marvell/mwifiex/pcie.c | 35
>> > ++++++++++-------------------
>> > drivers/net/wireless/marvell/mwifiex/pcie.h | 14 +++++-------
>> > 2 files changed, 18 insertions(+), 31 deletions(-)
>>
>> [...]
>>
>> > diff --git a/drivers/net/wireless/marvell/mwifiex/pcie.h
>> > b/drivers/net/wireless/marvell/mwifiex/pcie.h
>> > index f6992f0..46f99ca 100644
>> > --- a/drivers/net/wireless/marvell/mwifiex/pcie.h
>> > +++ b/drivers/net/wireless/marvell/mwifiex/pcie.h
>> > @@ -32,12 +32,9 @@
>> > #define PCIE8897_DEFAULT_FW_NAME "mrvl/pcie8897_uapsta.bin"
>> > #define PCIE8897_A0_FW_NAME "mrvl/pcie8897_uapsta_a0.bin"
>> > #define PCIE8897_B0_FW_NAME "mrvl/pcie8897_uapsta.bin"
>> > -#define PCIE8997_DEFAULT_FW_NAME "mrvl/pcieusb8997_combo_v2.bin"
>> > -#define PCIEUART8997_FW_NAME_Z "mrvl/pcieuart8997_combo.bin"
>> > -#define PCIEUART8997_FW_NAME_V2 "mrvl/pcieuart8997_combo_v2.bin"
>> > -#define PCIEUSB8997_FW_NAME_Z "mrvl/pcieusb8997_combo.bin"
>> > -#define PCIEUSB8997_FW_NAME_V2 "mrvl/pcieusb8997_combo_v2.bin"
>> > -#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan.bin"
>> > +#define PCIEUART8997_FW_NAME_V4 "mrvl/pcieuart8997_combo_v4.bin"
>> > +#define PCIEUSB8997_FW_NAME_V4 "mrvl/pcieusb8997_combo_v4.bin"
>> > +#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan_v4.bin"
>>
>> Why do version bumps require firmware renames? Is this just to make sure
>> you don't load the new firmware on old chip revs that you don't plan to
>> support for production (i.e., only early revs like the _Z you're
>> dropping)? This doesn't seems like a good long-term solution, at least
>> once you start getting this silicon out in the wild. At some point, I'd
>> expect to see a stable file name.
>>
>> Brian
>>
>
> We haven't yet submitted any firmware image upstream for 8997 chipset.
> pcieuart8997_combo_v4.bin/pcieusb8997_combo_v4.bin would be our firmware candidate for upstream submission. The filename would remain same hereafter.
>
> pcie*8997_combo_v2.bin had support only for A0 chipset
> pcie*8997_combo_v3.bin was our internal development version which had support for A1 chipset
> pcie*8997_combo_v4.bin has support for both A0 and A1 chipsets and this is the version that shall be released to customers/upstream from now on.
>
Seems to me then it should just be named pcie*8997_wlan.bin. A
version number shouldn't be part of the file name in this case. Having
to update the driver for a firmware name change is silly. Most
wireless drivers have different names for different hardware/chip revs
and/or an incompatible API change. Most distributions would typically
only carry a single instance of the firmware for a particular chip.
Speaking for the ones I work with, I usually keep the original
filename intact (with a version number) and make a symlink to it with
the name the driver expects. eg:
fw-4.bin -> fw_v3.4.0.94.bin
fw_v3.2.0.144.bin
fw_v3.4.0.94.bin
That way I can keep track of the version in my filesystem, but I'm not
hacking the driver every couple of weeks. And we do issue new
firmware every few weeks. I can't imagine asking our customers to keep
updating the driver for each firmware enhancement.
IMHO changing the driver to rename the firmwares on new versions seems
both inconvenient to people using it, and extra non-useful commit
noise.
- Steve
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: mwifiex: PCIe8997 chip specific handling
2016-08-10 21:08 ` Steve deRosier
@ 2016-08-11 10:41 ` Amitkumar Karwar
2016-09-07 10:13 ` Amitkumar Karwar
0 siblings, 1 reply; 5+ messages in thread
From: Amitkumar Karwar @ 2016-08-11 10:41 UTC (permalink / raw)
To: Steve deRosier
Cc: Brian Norris, linux-wireless@vger.kernel.org, Cathy Luo,
Nishant Sarmukadam, linux-kernel@vger.kernel.org, Wei-Ning Huang
Hi Steve,
> From: Steve deRosier [mailto:derosier@gmail.com]
> Sent: Thursday, August 11, 2016 2:39 AM
> To: Amitkumar Karwar
> Cc: Brian Norris; linux-wireless@vger.kernel.org; Cathy Luo; Nishant
> Sarmukadam; linux-kernel@vger.kernel.org; Wei-Ning Huang
> Subject: Re: mwifiex: PCIe8997 chip specific handling
>
> Hi,
>
> On Wed, Aug 10, 2016 at 12:07 AM, Amitkumar Karwar <akarwar@marvell.com>
> wrote:
> > Hi Brian,
> >
> >> From: Brian Norris [mailto:briannorris@chromium.org]
> >> Sent: Wednesday, August 10, 2016 12:14 AM
> >> To: Amitkumar Karwar
> >> Cc: linux-wireless@vger.kernel.org; Cathy Luo; Nishant Sarmukadam;
> >> linux-kernel@vger.kernel.org
> >> Subject: Re: mwifiex: PCIe8997 chip specific handling
> >>
> >> Hi,
> >>
> >> On Fri, Jul 29, 2016 at 04:08:51PM +0530, Amitkumar Karwar wrote:
> >> > The patch corrects the revision id register and uses it along with
> >> > magic value and chip version registers to download appropriate
> >> > firmware image.
> >> >
> >> > PCIe8997 Z chipset variant code has been removed, as it won't be
> >> > used in production.
> >> >
> >> > Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> >> > ---
> >> > drivers/net/wireless/marvell/mwifiex/pcie.c | 35
> >> > ++++++++++-------------------
> >> > drivers/net/wireless/marvell/mwifiex/pcie.h | 14 +++++-------
> >> > 2 files changed, 18 insertions(+), 31 deletions(-)
> >>
> >> [...]
> >>
> >> > diff --git a/drivers/net/wireless/marvell/mwifiex/pcie.h
> >> > b/drivers/net/wireless/marvell/mwifiex/pcie.h
> >> > index f6992f0..46f99ca 100644
> >> > --- a/drivers/net/wireless/marvell/mwifiex/pcie.h
> >> > +++ b/drivers/net/wireless/marvell/mwifiex/pcie.h
> >> > @@ -32,12 +32,9 @@
> >> > #define PCIE8897_DEFAULT_FW_NAME "mrvl/pcie8897_uapsta.bin"
> >> > #define PCIE8897_A0_FW_NAME "mrvl/pcie8897_uapsta_a0.bin"
> >> > #define PCIE8897_B0_FW_NAME "mrvl/pcie8897_uapsta.bin"
> >> > -#define PCIE8997_DEFAULT_FW_NAME "mrvl/pcieusb8997_combo_v2.bin"
> >> > -#define PCIEUART8997_FW_NAME_Z "mrvl/pcieuart8997_combo.bin"
> >> > -#define PCIEUART8997_FW_NAME_V2 "mrvl/pcieuart8997_combo_v2.bin"
> >> > -#define PCIEUSB8997_FW_NAME_Z "mrvl/pcieusb8997_combo.bin"
> >> > -#define PCIEUSB8997_FW_NAME_V2 "mrvl/pcieusb8997_combo_v2.bin"
> >> > -#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan.bin"
> >> > +#define PCIEUART8997_FW_NAME_V4 "mrvl/pcieuart8997_combo_v4.bin"
> >> > +#define PCIEUSB8997_FW_NAME_V4 "mrvl/pcieusb8997_combo_v4.bin"
> >> > +#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan_v4.bin"
> >>
> >> Why do version bumps require firmware renames? Is this just to make
> >> sure you don't load the new firmware on old chip revs that you don't
> >> plan to support for production (i.e., only early revs like the _Z
> >> you're dropping)? This doesn't seems like a good long-term solution,
> >> at least once you start getting this silicon out in the wild. At some
> >> point, I'd expect to see a stable file name.
> >>
> >> Brian
> >>
> >
> > We haven't yet submitted any firmware image upstream for 8997 chipset.
> > pcieuart8997_combo_v4.bin/pcieusb8997_combo_v4.bin would be our
> firmware candidate for upstream submission. The filename would remain
> same hereafter.
> >
> > pcie*8997_combo_v2.bin had support only for A0 chipset
> > pcie*8997_combo_v3.bin was our internal development version which had
> > support for A1 chipset pcie*8997_combo_v4.bin has support for both A0
> and A1 chipsets and this is the version that shall be released to
> customers/upstream from now on.
> >
>
> Seems to me then it should just be named pcie*8997_wlan.bin. A version
> number shouldn't be part of the file name in this case. Having to update
> the driver for a firmware name change is silly. Most wireless drivers
> have different names for different hardware/chip revs and/or an
> incompatible API change. Most distributions would typically only carry
> a single instance of the firmware for a particular chip.
> Speaking for the ones I work with, I usually keep the original filename
> intact (with a version number) and make a symlink to it with the name
> the driver expects. eg:
>
> fw-4.bin -> fw_v3.4.0.94.bin
> fw_v3.2.0.144.bin
> fw_v3.4.0.94.bin
>
> That way I can keep track of the version in my filesystem, but I'm not
> hacking the driver every couple of weeks. And we do issue new firmware
> every few weeks. I can't imagine asking our customers to keep updating
> the driver for each firmware enhancement.
>
> IMHO changing the driver to rename the firmwares on new versions seems
> both inconvenient to people using it, and extra non-useful commit noise.
>
Thanks. I agree with you. We have also maintained single instance/name for all our chipsets for last few years. We do release new firmware periodically for these chipsets, but firmware name always remains the same.
--------
root@pe-lt949:/linux-firmware/mrvl# ls
pcie8897_uapsta.bin sd8688_helper.bin sd8797_uapsta.bin sd8897_uapsta.bin usb8797_uapsta.bin
sd8688.bin sd8787_uapsta.bin sd8887_uapsta.bin usb8766_uapsta.bin usb8897_uapsta.bin
---------
It’s just that for our new chipset 8997 for which we haven't yet submitted the firmware image upstream, we want to finalize the name as pcie*8997_combo_v4.bin
Regards,
Amitkumar Karwar
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: mwifiex: PCIe8997 chip specific handling
2016-08-11 10:41 ` Amitkumar Karwar
@ 2016-09-07 10:13 ` Amitkumar Karwar
0 siblings, 0 replies; 5+ messages in thread
From: Amitkumar Karwar @ 2016-09-07 10:13 UTC (permalink / raw)
To: Steve deRosier
Cc: Brian Norris, linux-wireless@vger.kernel.org, Cathy Luo,
Nishant Sarmukadam, linux-kernel@vger.kernel.org, Wei-Ning Huang
Hi Kalle,
> From: linux-wireless-owner@vger.kernel.org [mailto:linux-wireless-
> owner@vger.kernel.org] On Behalf Of Amitkumar Karwar
> Sent: Thursday, August 11, 2016 4:11 PM
> To: Steve deRosier
> Cc: Brian Norris; linux-wireless@vger.kernel.org; Cathy Luo; Nishant
> Sarmukadam; linux-kernel@vger.kernel.org; Wei-Ning Huang
> Subject: RE: mwifiex: PCIe8997 chip specific handling
>
> Hi Steve,
>
> > From: Steve deRosier [mailto:derosier@gmail.com]
> > Sent: Thursday, August 11, 2016 2:39 AM
> > To: Amitkumar Karwar
> > Cc: Brian Norris; linux-wireless@vger.kernel.org; Cathy Luo; Nishant
> > Sarmukadam; linux-kernel@vger.kernel.org; Wei-Ning Huang
> > Subject: Re: mwifiex: PCIe8997 chip specific handling
> >
> > Hi,
> >
> > On Wed, Aug 10, 2016 at 12:07 AM, Amitkumar Karwar
> > <akarwar@marvell.com>
> > wrote:
> > > Hi Brian,
> > >
> > >> From: Brian Norris [mailto:briannorris@chromium.org]
> > >> Sent: Wednesday, August 10, 2016 12:14 AM
> > >> To: Amitkumar Karwar
> > >> Cc: linux-wireless@vger.kernel.org; Cathy Luo; Nishant Sarmukadam;
> > >> linux-kernel@vger.kernel.org
> > >> Subject: Re: mwifiex: PCIe8997 chip specific handling
> > >>
> > >> Hi,
> > >>
> > >> On Fri, Jul 29, 2016 at 04:08:51PM +0530, Amitkumar Karwar wrote:
> > >> > The patch corrects the revision id register and uses it along
> > >> > with magic value and chip version registers to download
> > >> > appropriate firmware image.
> > >> >
> > >> > PCIe8997 Z chipset variant code has been removed, as it won't be
> > >> > used in production.
> > >> >
> > >> > Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> > >> > ---
> > >> > drivers/net/wireless/marvell/mwifiex/pcie.c | 35
> > >> > ++++++++++-------------------
> > >> > drivers/net/wireless/marvell/mwifiex/pcie.h | 14 +++++-------
> > >> > 2 files changed, 18 insertions(+), 31 deletions(-)
> > >>
> > >> [...]
> > >>
> > >> > diff --git a/drivers/net/wireless/marvell/mwifiex/pcie.h
> > >> > b/drivers/net/wireless/marvell/mwifiex/pcie.h
> > >> > index f6992f0..46f99ca 100644
> > >> > --- a/drivers/net/wireless/marvell/mwifiex/pcie.h
> > >> > +++ b/drivers/net/wireless/marvell/mwifiex/pcie.h
> > >> > @@ -32,12 +32,9 @@
> > >> > #define PCIE8897_DEFAULT_FW_NAME "mrvl/pcie8897_uapsta.bin"
> > >> > #define PCIE8897_A0_FW_NAME "mrvl/pcie8897_uapsta_a0.bin"
> > >> > #define PCIE8897_B0_FW_NAME "mrvl/pcie8897_uapsta.bin"
> > >> > -#define PCIE8997_DEFAULT_FW_NAME "mrvl/pcieusb8997_combo_v2.bin"
> > >> > -#define PCIEUART8997_FW_NAME_Z "mrvl/pcieuart8997_combo.bin"
> > >> > -#define PCIEUART8997_FW_NAME_V2 "mrvl/pcieuart8997_combo_v2.bin"
> > >> > -#define PCIEUSB8997_FW_NAME_Z "mrvl/pcieusb8997_combo.bin"
> > >> > -#define PCIEUSB8997_FW_NAME_V2 "mrvl/pcieusb8997_combo_v2.bin"
> > >> > -#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan.bin"
> > >> > +#define PCIEUART8997_FW_NAME_V4 "mrvl/pcieuart8997_combo_v4.bin"
> > >> > +#define PCIEUSB8997_FW_NAME_V4 "mrvl/pcieusb8997_combo_v4.bin"
> > >> > +#define PCIE8997_DEFAULT_WIFIFW_NAME "mrvl/pcie8997_wlan_v4.bin"
> > >>
> > >> Why do version bumps require firmware renames? Is this just to make
> > >> sure you don't load the new firmware on old chip revs that you
> > >> don't plan to support for production (i.e., only early revs like
> > >> the _Z you're dropping)? This doesn't seems like a good long-term
> > >> solution, at least once you start getting this silicon out in the
> > >> wild. At some point, I'd expect to see a stable file name.
> > >>
> > >> Brian
> > >>
> > >
> > > We haven't yet submitted any firmware image upstream for 8997
> chipset.
> > > pcieuart8997_combo_v4.bin/pcieusb8997_combo_v4.bin would be our
> > firmware candidate for upstream submission. The filename would remain
> > same hereafter.
> > >
> > > pcie*8997_combo_v2.bin had support only for A0 chipset
> > > pcie*8997_combo_v3.bin was our internal development version which
> > > had support for A1 chipset pcie*8997_combo_v4.bin has support for
> > > both A0
> > and A1 chipsets and this is the version that shall be released to
> > customers/upstream from now on.
> > >
> >
> > Seems to me then it should just be named pcie*8997_wlan.bin. A
> > version number shouldn't be part of the file name in this case. Having
> > to update the driver for a firmware name change is silly. Most
> > wireless drivers have different names for different hardware/chip revs
> > and/or an incompatible API change. Most distributions would typically
> > only carry a single instance of the firmware for a particular chip.
> > Speaking for the ones I work with, I usually keep the original
> > filename intact (with a version number) and make a symlink to it with
> > the name the driver expects. eg:
> >
> > fw-4.bin -> fw_v3.4.0.94.bin
> > fw_v3.2.0.144.bin
> > fw_v3.4.0.94.bin
> >
> > That way I can keep track of the version in my filesystem, but I'm not
> > hacking the driver every couple of weeks. And we do issue new
> > firmware every few weeks. I can't imagine asking our customers to keep
> > updating the driver for each firmware enhancement.
> >
> > IMHO changing the driver to rename the firmwares on new versions seems
> > both inconvenient to people using it, and extra non-useful commit
> noise.
> >
>
> Thanks. I agree with you. We have also maintained single instance/name
> for all our chipsets for last few years. We do release new firmware
> periodically for these chipsets, but firmware name always remains the
> same.
>
> --------
> root@pe-lt949:/linux-firmware/mrvl# ls
> pcie8897_uapsta.bin sd8688_helper.bin sd8797_uapsta.bin
> sd8897_uapsta.bin usb8797_uapsta.bin
> sd8688.bin sd8787_uapsta.bin sd8887_uapsta.bin
> usb8766_uapsta.bin usb8897_uapsta.bin
> ---------
>
> It’s just that for our new chipset 8997 for which we haven't yet
> submitted the firmware image upstream, we want to finalize the name as
> pcie*8997_combo_v4.bin
>
Could you please accept this patch?
I wanted to submit our first firmware for 8997 chipset.
Regards,
Amitkumar
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-09-07 10:13 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1469788731-5361-1-git-send-email-akarwar@marvell.com>
2016-08-09 18:44 ` mwifiex: PCIe8997 chip specific handling Brian Norris
2016-08-10 7:07 ` Amitkumar Karwar
2016-08-10 21:08 ` Steve deRosier
2016-08-11 10:41 ` Amitkumar Karwar
2016-09-07 10:13 ` Amitkumar Karwar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox