From: "Guy, Wey-Yi" <wey-yi.w.guy@intel.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: "linville@tuxdriver.com" <linville@tuxdriver.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"ipw3945-devel@lists.sourceforge.net"
<ipw3945-devel@lists.sourceforge.net>
Subject: Re: [PATCH 16/16] iwlagn: downgrade warning on unknown TLV
Date: Sun, 10 Apr 2011 08:34:08 -0700 [thread overview]
Message-ID: <1302449648.14995.132.camel@wwguy-huron> (raw)
In-Reply-To: <1302423346.3519.4.camel@jlt3.sipsolutions.net>
On Sun, 2011-04-10 at 01:15 -0700, Johannes Berg wrote:
> On Fri, 2011-04-08 at 08:30 -0700, Wey-Yi Guy wrote:
> > From: Johannes Berg <johannes.berg@intel.com>
> >
> > If we maintain API properly, then there isn't
> > really a reason to warn about this since we'll
> > just be adding things that are safe to ignore,
> > so downgrade the warning to debug info level.
>
> This might be worthwhile for .39 since we'll be releasing firmware with
> some TLVs it doesn't understand eventually.
>
> johannes
>
I am not sure it is that important since .39 will not support support
the new TLV and this give a good indication to upgrade even driver
should still works.
Just like if there are newer version of firmware available for download,
driver do give a warning for it.
Wey
prev parent reply other threads:[~2011-04-10 15:54 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-08 15:30 [PATCH 00/16] update for 2.6.40 Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 01/16] iwlwifi: remove extranious macro from firmware define Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 02/16] iwlagn: remove un-necessary function pointer Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 03/16] iwlagn: fix radar frame rejection Wey-Yi Guy
2011-04-10 8:15 ` Johannes Berg
2011-04-10 15:29 ` Guy, Wey-Yi
2011-04-08 15:30 ` [PATCH 04/16] iwlwifi: fix bugs in change_interface Wey-Yi Guy
2011-04-10 8:15 ` Johannes Berg
2011-04-10 15:37 ` Guy, Wey-Yi
2011-04-08 15:30 ` [PATCH 05/16] iwlagn: PAPD read for 2000 series devices Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 06/16] iwlagn: no 3945 define needed Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 07/16] iwlagn: remove unused 3945 define Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 08/16] iwlagn: cleanup to remove the reference for 3945 Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 09/16] iwlagn: remove more reference to legacy devices Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 10/16] iwlagn: remove un-needed configuration Wey-Yi Guy
2011-04-12 10:39 ` Stanislaw Gruszka
2011-04-12 10:41 ` Stanislaw Gruszka
2011-04-12 14:39 ` wwguy
2011-04-08 15:30 ` [PATCH 11/16] iwlagn: more cleanup to remove unused reference Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 12/16] iwlagn: all _agn devices support power save mode Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 13/16] iwlagn: tx power calib always done in firmware Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 14/16] iwlagn: sensitivity and chain noise done by driver Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 15/16] iwlagn: clean up & autodetect statistics Wey-Yi Guy
2011-04-08 15:30 ` [PATCH 16/16] iwlagn: downgrade warning on unknown TLV Wey-Yi Guy
2011-04-10 8:15 ` Johannes Berg
2011-04-10 15:34 ` Guy, Wey-Yi [this message]
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=1302449648.14995.132.camel@wwguy-huron \
--to=wey-yi.w.guy@intel.com \
--cc=ipw3945-devel@lists.sourceforge.net \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.