linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: Sven-Haegar Koch <haegar@sdinet.de>
Cc: "Luis R. Rodriguez" <lrodriguez@atheros.com>,
	Suraj Sumangala <Suraj.Sumangala@Atheros.com>,
	Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
	David Woodhouse <dwmw2@infradead.org>,
	linux-bluetooth <linux-bluetooth@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: Firmware versioning best practices: ath3k-2.fw rename or replace ath3k-1.fw ?
Date: Sat, 09 Oct 2010 10:06:39 +0200	[thread overview]
Message-ID: <1286611599.6145.435.camel@aeonflux> (raw)
In-Reply-To: <alpine.DEB.2.02.1010090040240.19472@aurora.sdinet.de>

Hi Sven-Haeger,

> > > > I last tried to document a thread we had over this here:
> > > >
> > > > http://wireless.kernel.org/en/developers/Documentation/firmware-versioning
> > > >
> > 
> > Thanks, I've updated that link above to document bug fixing does not require
> > a filename change.
> 
> I would summarize it as:
> 
> If a new firmware version also works with an old driver, keep the filename.

correct.

> If a new firmware version also requires a new driver, change the name.
> 
> If a new driver requires a new firmware, change the name.

These two depend. The exposed API stays the same. The firmware file
itself is the same. Just the loading procedure is different. So no need
to change the firmware name.

Let me repeat this. If the API of the firmware exposed after loading it,
breaks or is incompatible, then you need a new name.

If you have generic commands to detect features in the firmware, then
you should never be needed to change your firmware name. So you could
extend the API as much as you like with keeping the same name.

The different firmware names are for the driver to be able to detect the
API of the firmware. And only if that is only possible via the filename
you should use different filenames. Otherwise don't bother and use the
generic feature detection of the firmware itself.

And for Bluetooth in specific that is HCI. So any company needed
different firmware filenames for Bluetooth have done something really
really wrong in their development cycles.

Regards

Marcel



  reply	other threads:[~2010-10-09  8:06 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-08 17:02 Firmware versioning best practices: ath3k-2.fw rename or replace ath3k-1.fw ? Luis R. Rodriguez
2010-10-08 17:27 ` Suraj Sumangala
2010-10-08 18:15   ` Luis R. Rodriguez
2010-10-08 22:47     ` Sven-Haegar Koch
2010-10-09  8:06       ` Marcel Holtmann [this message]
2010-10-12 21:17     ` Henry Ptasinski
2010-10-13 10:06       ` Marcel Holtmann
2010-10-13 17:42         ` Luis R. Rodriguez
2010-10-13 17:54           ` Kevin Hayes
2010-10-13 18:09             ` Luis R. Rodriguez
2010-10-13 18:41               ` Kevin Hayes
2010-10-14  4:23               ` Suraj Sumangala
2010-10-09  8:01 ` Marcel Holtmann
2010-10-11  7:14   ` Senthil Balasubramanian

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1286611599.6145.435.camel@aeonflux \
    --to=marcel@holtmann.org \
    --cc=Luis.Rodriguez@Atheros.com \
    --cc=Suraj.Sumangala@Atheros.com \
    --cc=dwmw2@infradead.org \
    --cc=haegar@sdinet.de \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=lrodriguez@atheros.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).