Re: [RFC V4 0/2] Add support for pci/non-pci probing for UFS

["S, Venkatraman" <svenkatr@xxxxxx>]

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>
--
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