Search Linux Wireless

Re: [PATCH] rt2x00: Consolidate definition of CSR_OFFSET

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 19, 2008 at 2:16 AM, John Daiker <daikerjohn@xxxxxxxxx> wrote:
> 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 <daikerjohn@xxxxxxxxx>

NACK
This patch does more then consolidating  the CRS_OFFSET definition since 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

> ---
>
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux