Search Linux Wireless

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

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

 



On Friday 19 September 2008, John Daiker wrote:
> Resend: Premature send to Linville only... this version copies 
> 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 
> 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.

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

> Signed-off-by: John Daiker <daikerjohn@xxxxxxxxx>
> 
> ---
> 
> 
> 


--
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