netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Drake <dsd@gentoo.org>
To: Rami Rosen <rosenrami@gmail.com>
Cc: Oliver Neukum <oliver@neukum.org>, John Que <qwejohn@gmail.com>,
	linux-usb-devel@lists.sourceforge.net,
	"John W. Linville" <linville@tuxdriver.com>,
	netdev@vger.kernel.org, Ulrich Kunitz <kune@deine-taler.de>
Subject: Re: [linux-usb-devel] [PATCH RFC] ZyDAS ZD1211 USB-WLAN driver
Date: Sun, 04 Jun 2006 22:51:08 +0100	[thread overview]
Message-ID: <448355CC.3030401@gentoo.org> (raw)
In-Reply-To: <ae5426fb0606041103od6bf095sd4403ab2966a1e1f@mail.gmail.com>

Rami Rosen wrote:
> It is mentioned there that these definitions are taken from the ZYDAS
> driver and  not all of them are relevant for the rewrite driver;
> however, it seems to me that
> removing them may be thought of.

They are mostly relevant, not all come from the vendor driver directly, 
some will be used later, many are useful for documenting the bits and 
bobs that we *do* know about the CR registers (not a lot).

I don't think there are any readability issues while all defines are 
kept out of the way from the code.

Daniel

  reply	other threads:[~2006-06-04 21:50 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-03 11:20 [PATCH RFC] ZyDAS ZD1211 USB-WLAN driver Daniel Drake
2006-06-03 17:51 ` [linux-usb-devel] " Oliver Neukum
2006-06-03 19:35   ` Daniel Drake
2006-06-03 22:25     ` Oliver Neukum
2006-06-04 16:29       ` John Que
2006-06-04 17:17         ` Oliver Neukum
2006-06-04 18:03           ` Rami Rosen
2006-06-04 21:51             ` Daniel Drake [this message]
2006-06-04 18:22           ` John Que
2006-06-04 19:06             ` Oliver Neukum
2006-06-04 21:45         ` Daniel Drake
2006-06-06  5:41       ` David Brownell
2006-06-10 11:23       ` Ulrich Kunitz
2006-06-10 11:49         ` Oliver Neukum
2006-06-10 12:40         ` [linux-usb-devel] " Daniel Drake
2006-06-10 19:37   ` Daniel Drake

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=448355CC.3030401@gentoo.org \
    --to=dsd@gentoo.org \
    --cc=kune@deine-taler.de \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=linville@tuxdriver.com \
    --cc=netdev@vger.kernel.org \
    --cc=oliver@neukum.org \
    --cc=qwejohn@gmail.com \
    --cc=rosenrami@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).