Re: [PATCH 00/41] rewritten rt2800 drivers

[Luis R. Rodriguez]

On Thu, Nov 5, 2009 at 12:59 PM, Gertjan van Wingerde
<gwingerde@xxxxxxxxx> wrote:
> On 11/04/09 21:19, Gertjan van Wingerde wrote:
>> On Wed, Nov 4, 2009 at 6:31 PM, Bartlomiej Zolnierkiewicz
>> <bzolnier@xxxxxxxxx> wrote:
>>>
>>> Hi,
>>>
>>> Here is the rt2800 rewrite in the form of patches (I've trimmed cc:
>>> list considerably since I'm not sure whether most people really want
>>> to see 40+ patches in their mailboxes).
>>>
>>>
>>> There were some minor changes since yesterday:
>>>
>>> - two new patches were added at the top of tree fixing Kconfig help
>>> Âentries of rt2800[pci,usb] drivers to be more helpful and to prevent
>>> Âwasting people's time (I think that patch #1 should go to Linus'
>>> Âtree as soon as possible, ditto for patch #2 and net-next tree)
>>>
>>> - patch descriptions were improved for many patches
>>> Â(suggestion from Ingo)
>>>
>>> - rt2x00_intf_is_[pci,usb]() helpers for commonly used checks were
>>> Âadded to "rt2x00: add support for different chipset interfaces" patch
>>> Â(suggestion from Ivo)
>>>
>>> - addition of separate rt2800 MAINTAINERS entry was dropped for now
>>> Âbecause it stirred needless controversies distracting people from
>>> Âtechnical issues (thanks to Julian Calaby for pointing this to me),
>>> Âthe goal of change was to make sure that people won't bother busy
>>> Ârt2x00 maintainers about rt2800 tree so it is not a big deal anyway
>>>
>>>
>>> The rt2800 tree has also been updated to reflect those changes
>>> (old branch is still available as rt2800-v1 for reference):
>>>
>>> The following changes since commit fa867e7355a1bdcd9bf7d55ebe9296f5b9c4028a:
>>> ÂJuuso Oikarinen (1):
>>> Â Â Â Âwl1271: Generalize command response reading
>>>
>>> are available in the git repository at:
>>>
>>> Âgit://git.kernel.org/pub/scm/linux/kernel/git/bart/misc.git rt2800
>>>
>>> Bartlomiej Zolnierkiewicz (41):
>>> Â Â Ârt2800usb: make Kconfig help entry more helpful
>>> Â Â Ârt2800pci: make Kconfig help entry more helpful
>>> Â Â Ârt2800usb: fix rt2800usb_rfcsr_read()
>>> Â Â Ârt2800pci: fix crypto in TX frame
>>> Â Â Ârt2800pci: fix comment about register access
>>> Â Â Ârt2800pci: fix comment about IV/EIV fields
>>> Â Â Ârt2x00: fix rt2x00usb_register_read() comment
>>> Â Â Ârt2800usb: use rt2x00usb_register_multiwrite() to set key entries
>>> Â Â Ârt2800usb: add rt2800_register_[read,write]() wrappers
>>> Â Â Ârt2800pci: add rt2800_register_[read,write]() wrappers
>>> Â Â Ârt2800usb: add rt2800_register_multi[read,write]() wrappers
>>> Â Â Ârt2800pci: add rt2800_register_multi[read,write]() wrappers
>>> Â Â Ârt2800usb: add rt2800_regbusy_read() wrapper
>>> Â Â Ârt2800pci: add rt2800_regbusy_read() wrapper
>>> Â Â Ârt2800usb: add rt2800_bbp_[read,write]() wrappers
>>> Â Â Ârt2800pci: add rt2800_bbp_[read,write]() wrappers
>>> Â Â Ârt2800usb: add rt2800_rfcsr_[read,write]() wrappers
>>> Â Â Ârt2800pci: add rt2800_rfcsr_[read,write]() wrappers
>>> Â Â Ârt2800usb: add rt2800_rf_[read,write]() wrappers
>>> Â Â Ârt2800pci: add rt2800_rf_[read,write]() wrappers
>>> Â Â Ârt2800usb: add rt2800_mcu_request() wrapper
>>> Â Â Ârt2800pci: add rt2800_mcu_request() wrapper
>>> Â Â Ârt2x00: add driver private field to struct rt2x00_dev
>>> Â Â Ârt2800usb: convert to use struct rt2800_ops methods
>>> Â Â Ârt2800pci: convert to use struct rt2800_ops methods
>>> Â Â Ârt2x00: fix rt2x00usb_register_multiwrite() arguments
>>> Â Â Ârt2x00: fix rt2x00usb_regbusy_read() arguments
>>> Â Â Ârt2x00: fix rt2x00pci_register_multi[read,write]() arguments
>>> Â Â Ârt2800: add rt2800lib.h
>>> Â Â Ârt2800usb: fix comments in rt2800usb.h
>>> Â Â Ârt2800usb: add RXINFO_DESC_SIZE definition
>>> Â Â Ârt2800: fix duplication in header files
>>> Â Â Ârt2800: fix comments in rt2800.h
>>> Â Â Ârt2x00: add support for different chipset interfaces
>>> Â Â Ârt2800: prepare for rt2800lib addition
>>> Â Â Ârt2800: add rt2800lib (part one)
>>> Â Â Ârt2x00: remove needless ifdefs from rt2x00leds.h
>>> Â Â Ârt2800: add rt2800lib (part two)
>>> Â Â Ârt2x00: move REGISTER_BUSY_* definitions to rt2x00.h
>>> Â Â Ârt2800: add rt2800lib (part three)
>>> Â Â Ârt2800: add rt2800lib (part four)
>>>
>>> Âdrivers/net/wireless/rt2x00/Kconfig   Â|  16 +-
>>> Âdrivers/net/wireless/rt2x00/Makefile   |  Â1 +
>>> Âdrivers/net/wireless/rt2x00/rt2800.h   | 1816 ++++++++++++++++++++++++++++
>>> Âdrivers/net/wireless/rt2x00/rt2800lib.c Â| 1817 ++++++++++++++++++++++++++++
>>> Âdrivers/net/wireless/rt2x00/rt2800lib.h Â| Â134 +++
>>> Âdrivers/net/wireless/rt2x00/rt2800pci.c Â| 1908 +++---------------------------
>>> Âdrivers/net/wireless/rt2x00/rt2800pci.h Â| 1780 ----------------------------
>>> Âdrivers/net/wireless/rt2x00/rt2800usb.c Â| 1828 ++---------------------------
>>> Âdrivers/net/wireless/rt2x00/rt2800usb.h Â| 1818 +----------------------------
>>> Âdrivers/net/wireless/rt2x00/rt2x00.h   |  43 +
>>> Âdrivers/net/wireless/rt2x00/rt2x00leds.h | Â Â4 -
>>> Âdrivers/net/wireless/rt2x00/rt2x00pci.h Â| Â 24 +-
>>> Âdrivers/net/wireless/rt2x00/rt2x00usb.c Â| Â Â2 +-
>>> Âdrivers/net/wireless/rt2x00/rt2x00usb.h Â| Â 17 +-
>>> Â14 files changed, 4048 insertions(+), 7160 deletions(-)
>>> Âcreate mode 100644 drivers/net/wireless/rt2x00/rt2800.h
>>> Âcreate mode 100644 drivers/net/wireless/rt2x00/rt2800lib.c
>>> Âcreate mode 100644 drivers/net/wireless/rt2x00/rt2800lib.h
>>>
>>
>> Hi Bart,
>>
>> Many thanks for these patches.
>>
>> So far I have been able to go through the first 30 patches of the
>> series. I'll finish the other 11 tomorrow (when I'm back home and have
>> better facilities to review these more elaborate patches of the
>> series).
>> They all look fine to me, I only had a comment for patch 10.
>>
>> I've sent my ACKs for the remaining 29 I reviewed.
>>
>> I guess Ivo still has to review them as well, as, so far, he has been
>> the one of the rt2x00 project to give the final ACK for inclusion,
>> unless he indicates that he is fine with my assessments.
>>
>> ---
>> Gertjan
>> rt2x00 developer
>>
>
> OK. I have now completed my review of the remaining patches and have sent my ACKs for those.

A little summary would be nice for those who don't want to read single
line acks/nacks.

  Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/