On 23 May 2012 14:36, S, Venkatraman <svenkatr@xxxxxx> wrote: > On Tue, May 22, 2012 at 5:03 PM, Girish K S > <girish.shivananjappa@xxxxxxxxxx> wrote: >> The existing UFS driver is tightly bound to only pci specific devices. >> It cannot work for non-pci devices. >> >> "SCSI: UFS: Create separate UFS core and pci modules" patch splits the >> ufs driver into core driver and pci probe driver. >> >> "SCSI: UFS: Add platform support for ufs host controller" patch adds >> support for the platform probing for non-pci devices >> >> changes in v4: >> -Rebased to the latest linux-next/master branch. >> -Removed the Copyright string from the module probe function and >> deleted redundant code from ufshcd-pltfm.c suspend/resume functions. >> >> changes in v3: >> -Modified the ufshcd.c ufshcd-pci.c ufshcd0pltfm.c to follow the >> driver module initialization standard. suggested by venkatraman. >> >> changes in v2: >> -Reduced the number of patches from 4 to 2. merged the core and >> pci probing modules as single patch. suggested by Christoph Hellwig. >> >> changes in v1: >> -Changes suggested by Namjae to split the driver as 3 independent >> modules and retain the author's name in the files created by me. >> >> Girish K S (2): >> SCSI: UFS: Create separate UFS core and pci modules >> SCSI: UFS: Add platform support for ufs host controller >> >> drivers/scsi/ufs/Kconfig | 32 +++- >> drivers/scsi/ufs/Makefile | 2 + >> drivers/scsi/ufs/ufs.h | 103 +++++++++++ >> drivers/scsi/ufs/ufshcd-pci.c | 206 ++++++++++++++++++++++ >> drivers/scsi/ufs/ufshcd-pltfm.c | 176 +++++++++++++++++++ >> drivers/scsi/ufs/ufshcd.c | 367 ++++++++++----------------------------- >> 6 files changed, 608 insertions(+), 278 deletions(-) >> create mode 100644 drivers/scsi/ufs/ufshcd-pci.c >> create mode 100644 drivers/scsi/ufs/ufshcd-pltfm.c >> >> -- >> 1.7.4.1 >> > Thanks Girish. > V4 looks ok to me. If it helps, you can add > Reviewed-by: Venkatraman S <svenkatr@xxxxxx> Will generate a patch including your Ack. If Namjae and Christoph Hellwi agree will include their ack in the patch -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html