[PATCH] [MTD] bcmring: mtd nand driver
dwmw2 at infradead.org
Mon Sep 28 18:37:10 EDT 2009
On Mon, 2009-09-28 at 15:24 -0700, Leo (Hao) Chen wrote:
> I've updated the patch of mtd nand driver support for bcmring arch.
> It's now applicable to the latest kernel git tree (Linux 2.6.32-rc1).
> Please merge this patch into upstream.
> If further modification needed, let me know.
Thank you. On closer inspection, this patch is actually a whole lot
saner than it appears at first glance. The amount of superfluous
commenting made it look bad, when I first looked.
For example, a function called 'nand_dma_write' does _not_ need a
five-line comment block explaining that it 'Performs a write via DMA'.
Other comments, on a brief re-reading...
You're using a platform device, but you still hard-code the I/O address
you ioremap for bcm_imi_io_base -- is it possible for you to fix that,
and encode it in the platform device instead?
Also, do you need the USE_DMA and USE_HWECC macros? If they're always
enabled, can we lose some ifdefs?
Finally, can you convince me that this isn't an "additional restriction"
as prohibited by the GPL? :
+* Notwithstanding the above, under no circumstances may you combine this
+* software in any way with any other Broadcom software provided under a
+* license other than the GPL, without Broadcom's express prior written
David Woodhouse Open Source Technology Centre
David.Woodhouse at intel.com Intel Corporation
More information about the linux-mtd