linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: reinette chatre <reinette.chatre@linux.intel.com>
To: Marcel Holtmann <holtmann@linux.intel.com>
Cc: Johannes Berg <johannes@sipsolutions.net>,
	Tomas Winkler <tomasw@gmail.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: New iwlwifi 3945 uCode available
Date: Wed, 22 Oct 2008 16:14:21 -0700	[thread overview]
Message-ID: <1224717261.10863.219.camel@rc-desk> (raw)
In-Reply-To: <1224716347.9386.151.camel@californication>

On Wed, 2008-10-22 at 15:59 -0700, Marcel Holtmann wrote:

> in some cases it might be possible with a set of quirks or special
> functions that can be switched at runtime. Check the whole bunch of USB
> drivers that have special handling depending on what hardware it finds.
> We could do the same. In some cases this effort might be too much, but
> if possible we should at least try.

I don't think we can easily apply the hardware quirk solution here. This
is because hardware can be identified by id that does not change and the
hardware self does not really change (unless it has firmware that
changes ...). In this firmware case we do not have "an id" (we have one
file that is loaded by firmware loader so we cannot know what we get
just by "looking at it" unless we enforce a version number in the
filename as we currently do in our driver) and the firmware can change,
potentially frequently, with the consequence of many "quirks" that need
to be maintained.

> It all depends on how much the ucode API changes and sometimes we really
> wanna cleanup the driver and remove the old code, but in that case we
> have to tell the users via modules_install that this kernel will break
> or we have to keep the old driver around. Breaking it from one kernel
> version to the next one without the user noticing only after reboot is
> just not good enough.

I agree with you and Johannes in this regard and look forward to the
modules_install solution. I do hope this will be acceptable to
everybody.

Reinette



  reply	other threads:[~2008-10-22 23:14 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-21 18:27 New iwlwifi 3945 uCode available reinette chatre
2008-10-21 21:25 ` Johannes Berg
2008-10-21 21:33   ` Tomas Winkler
2008-10-21 21:34     ` Johannes Berg
2008-10-21 21:38       ` John W. Linville
2008-10-21 21:53         ` Tomas Winkler
2008-10-21 22:13           ` Marcel Holtmann
2008-10-21 22:23             ` Tomas Winkler
2008-10-21 23:01               ` Marcel Holtmann
2008-10-21 23:09                 ` Luis R. Rodriguez
2008-10-21 23:11                   ` Marcel Holtmann
2008-10-21 23:27                     ` Luis R. Rodriguez
2008-10-21 23:31                       ` Marcel Holtmann
2008-10-21 23:34                         ` Luis R. Rodriguez
2008-10-21 22:28             ` reinette chatre
2008-10-21 23:06               ` Marcel Holtmann
2008-10-21 23:34                 ` Tomas Winkler
2008-10-22  1:03                   ` Marcel Holtmann
2008-10-22  7:27               ` Johannes Berg
2008-10-22  7:56                 ` Marcel Holtmann
2008-10-22  7:57                   ` Johannes Berg
2008-10-22 20:30                   ` reinette chatre
2008-10-22 22:59                     ` Marcel Holtmann
2008-10-22 23:14                       ` reinette chatre [this message]
2008-10-22 23:39                         ` John W. Linville
2008-10-23  1:12                         ` Marcel Holtmann
2008-10-23  7:22                         ` Johannes Berg
2008-10-22  9:10                 ` Tomas Winkler
2008-10-22  9:14                   ` Johannes Berg
2008-10-22  9:23                     ` Tomas Winkler
2008-10-22 10:07               ` Christoph Hellwig
2008-10-22 12:28                 ` Tomas Winkler
2008-10-22 18:25                   ` Marcel Holtmann
2008-10-24 12:45                   ` Florian Mickler
2008-10-21 21:59       ` reinette chatre
  -- strict thread matches above, loose matches on Subject: below --
2009-04-28 20:52 new " reinette chatre

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=1224717261.10863.219.camel@rc-desk \
    --to=reinette.chatre@linux.intel.com \
    --cc=holtmann@linux.intel.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=tomasw@gmail.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).