2008-09-19 00:22:33

by John Daiker

[permalink] [raw]
Subject: Resend: [PATCH] rt2x00: Consolidate definition of CSR_OFFSET

Resend: Premature send to Linville only... this version copies
linux-wireless and rt2400-devel.

This consolidates the definition of CSR_OFFSET when
CONFIG_RT2X00_LIB_DEBUG is also set.
The original definition appeared in 5 files, but was moved to a common
.h file instead.

Compile-tested only, as I don't have any rt2x00 HW in a working machine.

Signed-off-by: John Daiker <[email protected]>

---



Attachments:
rt2x00.patch (4.17 kB)

2008-09-19 18:09:52

by Ivo Van Doorn

[permalink] [raw]
Subject: Re: [Rt2400-devel] Resend: [PATCH] rt2x00: Consolidate definition of CSR_OFFSET

On Friday 19 September 2008, John Daiker wrote:
> Resend: Premature send to Linville only... this version copies=20
> linux-wireless and rt2400-devel.

I was in the BCC list, but here goes my official reply. :)

> This consolidates the definition of CSR_OFFSET when=20
> CONFIG_RT2X00_LIB_DEBUG is also set.
> The original definition appeared in 5 files, but was moved to a commo=
n
> .h file instead.
>=20
> Compile-tested only, as I don't have any rt2x00 HW in a working machi=
ne.

NACK
This patch does more then consolidating =A0the CRS_OFFSET definition si=
nce it
changes several lines for adding/removing whitespaces as well.
It completely breaks rt2500usb, and I really dislike using define
names in rt2x00.h
while the define itself is declared in the register headers only (and
is thus not really
visible in rt2x00.h, even though the compiler would accept it and it
would work as
expected).

Ivo

> Signed-off-by: John Daiker <[email protected]>
>=20
> ---
>=20
>=20
>=20