On Wed, Jan 12, 2011 at 12:56:44PM -0800, Sutharsan Ramamoorthy wrote: > This patch implements gpmc configuration functions. Why? > These gpmc functions are used by westbridge device controller driver > in staging tree to configure gpmc timings and bus width. How does the driver live today without these functions? And as this is a staging driver, any way to just leave the functions in the staging driver for now, as there are lots of other things that need to be done to this driver before it can be moved out of the staging tree. Or is this one of those things? > This patch has been compiled in linux-2.6.37-rc4 This line isn't needed. You also forgot the line: This patch is corrupted by my email client and can not be applied or built properly. :( > New files: > 1. ../arch/arm/mach-omap2/usb-cywb-pnand.c > 2. ../include/linux/usb/cywb.h > > Modified files: > 1. ../arch/arm/mach-omap2/board-zoom-peripherals.c > 2. ../arch/arm/mach-omap2/Makefile > 3. > ../drivers/staging/westbridge/astoria/arch/arm/mach-omap2/cyashalomap_kernel.c > 4. > ../drivers/staging/westbridge/astoria/arch/arm/plat-omap/include/mach/westbridge/westbridge-omap3-pnand-hal/cyashalomap_kernel.h > 5. ../drivers/staging/westbridge/astoria/device/cyasdevice.c What is this? We know what has been added and modified by virtue of the patch itself. Please use 'git format-patch' to create a patch that includes the diffstat in it automatically, if you want to show that (which you should.) > Signed-off by: Sutharsan Ramamoorthy <srmt@xxxxxxxxxxx> This differs from your "From:" line in your email. If you want to properly attribute the patch's author, please read Documentation/SubmittingPatches to do that. Care to try it again? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
