Re: [PATCH][WAS:bcmai,axi] bcma: add Broadcom specific AMBA bus driver

[RafaÅ MiÅecki]

2011/5/7 Hauke Mehrtens <hauke@xxxxxxxxxx>:
> On 05/07/2011 06:23 PM, RafaÅ MiÅecki wrote:
>> 2011/5/7 Hauke Mehrtens <hauke@xxxxxxxxxx>:
>>> On 05/05/2011 11:59 PM, RafaÅ MiÅecki wrote:
>>>> Cc: Greg KH <greg@xxxxxxxxx>
>>>> Cc: Michael BÃsch <mb@xxxxxxxxx>
>>>> Cc: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
>>>> Cc: George Kashperko <george@xxxxxxxxxxx>
>>>> Cc: Arend van Spriel <arend@xxxxxxxxxxxx>
>>>> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
>>>> Cc: Russell King <rmk@xxxxxxxxxxxxxxxx>
>>>> Cc: Arnd Bergmann <arnd@xxxxxxxx>
>>>> Cc: Andy Botting <andy@xxxxxxxxxxxxxxx>
>>>> Cc: linuxdriverproject <devel@xxxxxxxxxxxxxxxxxxxxxx>
>>>> Cc: linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>
>>>> Signed-off-by: RafaÅ MiÅecki <zajec5@xxxxxxxxx>
>>>> ---
>>>> 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@xxxxxxxxxxxxxxx? linux-wireless@xxxxxxxxxxxxxxx? 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@xxxxxxxxxxxxxxxx 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Å
--
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/