Return-path: Received: from mail-qy0-f181.google.com ([209.85.216.181]:36254 "EHLO mail-qy0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752352Ab1EGQvc convert rfc822-to-8bit (ORCPT ); Sat, 7 May 2011 12:51:32 -0400 MIME-Version: 1.0 In-Reply-To: <4DC57425.30206@hauke-m.de> References: <1304632783-8781-1-git-send-email-zajec5@gmail.com> <4DC56FB9.5000208@hauke-m.de> <4DC57425.30206@hauke-m.de> Date: Sat, 7 May 2011 18:51:30 +0200 Message-ID: (sfid-20110507_185150_462526_5F341042) Subject: Re: [PATCH][WAS:bcmai,axi] bcma: add Broadcom specific AMBA bus driver From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= To: Hauke Mehrtens Cc: linux-wireless@vger.kernel.org, "John W. Linville" , b43-dev@lists.infradead.org, Greg KH , =?UTF-8?Q?Michael_B=C3=BCsch?= , Larry Finger , George Kashperko , Arend van Spriel , linux-arm-kernel@lists.infradead.org, Russell King , Arnd Bergmann , Andy Botting , linuxdriverproject , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: 2011/5/7 Hauke Mehrtens : > On 05/07/2011 06:23 PM, Rafał Miłecki wrote: >> 2011/5/7 Hauke Mehrtens : >>> On 05/05/2011 11:59 PM, Rafał Miłecki wrote: >>>> Cc: Greg KH >>>> Cc: Michael Büsch >>>> Cc: Larry Finger >>>> Cc: George Kashperko >>>> Cc: Arend van Spriel >>>> Cc: linux-arm-kernel@lists.infradead.org >>>> Cc: Russell King >>>> Cc: Arnd Bergmann >>>> Cc: Andy Botting >>>> Cc: linuxdriverproject >>>> Cc: linux-kernel@vger.kernel.org >>>> Signed-off-by: Rafał Miłecki >>>> --- >>>> V2: Rename to axi >>>>     Use DEFINE_PCI_DEVICE_TABLE in bridge >>>>     Make use of pr_fmt and pr_* >>>>     Store core class >>>>     Rename bridge to not b43 specific >>>>     Replace magic 0x1000 with BCMAI_CORE_SIZE >>>>     Remove some old "ssb" names and defines >>>>     Move BCMAI_ADDR_BASE def >>>>     Add drvdata field >>>> V3: Fix reloading (kfree issue) >>>>     Add 14e4:0x4331 >>>>     Fix non-initialized struct issue >>>>     Drop useless inline functions wrappers for pci core drv >>>>     Proper pr_* usage >>>> V3.1: Include forgotten changes (pr_* and include related) >>>>     Explain why we dare to implement empty release function >>>> V4: Add ABI documentation >>>>     Move struct device to wrapper and alloc it dynamically >>>>     checkpatch.pl pointed fixes >>>> V5: Rename to bcma, AXI was really bad name >>>>     Use EXPORT_SYMBOL_GPL >>>>     Set pci driver fields in one place >>>>     Drop unlikely >>>>     Use BCMA_CORE_SIZE for calc in awrite32 >>>>     Add README >>>>     Fix compilation (delay.h) >>>> --- >>>>  Documentation/ABI/testing/sysfs-bus-bcma    |   31 ++ >>>>  drivers/Kconfig                             |    2 + >>>>  drivers/Makefile                            |    1 + >>>>  drivers/bcma/Kconfig                        |   33 +++ >>>>  drivers/bcma/Makefile                       |    7 + >>>>  drivers/bcma/README                         |   18 ++ >>>>  drivers/bcma/TODO                           |    3 + >>>>  drivers/bcma/bcma_private.h                 |   31 ++ >>>>  drivers/bcma/core.c                         |   51 ++++ >>>>  drivers/bcma/driver_chipcommon.c            |   87 ++++++ >>>>  drivers/bcma/driver_chipcommon_pmu.c        |  134 +++++++++ >>>>  drivers/bcma/driver_pci.c                   |  163 +++++++++++ >>>>  drivers/bcma/host_pci.c                     |  196 +++++++++++++ >>>>  drivers/bcma/main.c                         |  271 ++++++++++++++++++ >>>>  drivers/bcma/scan.c                         |  392 +++++++++++++++++++++++++++ >>>>  drivers/bcma/scan.h                         |   56 ++++ >>>>  include/linux/bcma/bcma.h                   |  232 ++++++++++++++++ >>>>  include/linux/bcma/bcma_driver_chipcommon.h |  297 ++++++++++++++++++++ >>>>  include/linux/bcma/bcma_driver_pci.h        |   89 ++++++ >>>>  include/linux/bcma/bcma_regs.h              |   34 +++ >>>>  include/linux/mod_devicetable.h             |   17 ++ >>>>  scripts/mod/file2alias.c                    |   22 ++ >>>>  22 files changed, 2167 insertions(+), 0 deletions(-) >>>>  create mode 100644 Documentation/ABI/testing/sysfs-bus-bcma >>>>  create mode 100644 drivers/bcma/Kconfig >>>>  create mode 100644 drivers/bcma/Makefile >>>>  create mode 100644 drivers/bcma/README >>>>  create mode 100644 drivers/bcma/TODO >>>>  create mode 100644 drivers/bcma/bcma_private.h >>>>  create mode 100644 drivers/bcma/core.c >>>>  create mode 100644 drivers/bcma/driver_chipcommon.c >>>>  create mode 100644 drivers/bcma/driver_chipcommon_pmu.c >>>>  create mode 100644 drivers/bcma/driver_pci.c >>>>  create mode 100644 drivers/bcma/host_pci.c >>>>  create mode 100644 drivers/bcma/main.c >>>>  create mode 100644 drivers/bcma/scan.c >>>>  create mode 100644 drivers/bcma/scan.h >>>>  create mode 100644 include/linux/bcma/bcma.h >>>>  create mode 100644 include/linux/bcma/bcma_driver_chipcommon.h >>>>  create mode 100644 include/linux/bcma/bcma_driver_pci.h >>>>  create mode 100644 include/linux/bcma/bcma_regs.h >>>> >>> Hi, >>> >>> An entry in the MAINTAINERS file would be nice to find the right >>> addresses to send patches to. >> >> Will do, thanks. Should I put some ML for this driver? >> netdev@vger.kernel.org? linux-wireless@vger.kernel.org? So far there >> are no net drivers for BCMA. Probably b43 will be the first (or >> brcm80211). >> > There is a Ethernet core used on embedded devices with this bus and > George wrote a driver for the first version of this bus implementation, > he send to the ML. Most user will use this with some wireless devices so > I would add linux-wireless@vger.kernel.org. Why does ssb uses netdev ML? Thanks, I didn't know about this ethernet driver. Have to check it! Where was this published? Do you have a thread name? I'm not sure if we want linux-wireless to be ML, if we already have some Ethernet driver. Some network-general ML makes more sense then. Maybe netdev would be better. -- Rafał