linux-kernel - Re: [PATCH v4] scsi: ufs: add support of generic PHY framework in Qualcomm MSM chips

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Wed, 17 Sep 2014 15:27:43 -0700
From:	Kumar Gala <galak@...eaurora.org>
To:	Yaniv Gardi <ygardi@...eaurora.org>
Cc:	James.Bottomley@...senPartnership.com, hch@...radead.org,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
	linux-arm-msm@...r.kernel.org, santoshsy@...il.com,
	linux-scsi-owner@...r.kernel.org, subhashj@...eaurora.org,
	noag@...eaurora.org, draviv@...eaurora.org,
	Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kishon Vijay Abraham I <kishon@...com>,
	Vinayak Holikatti <vinholikatti@...il.com>,
	"James E.J. Bottomley" <JBottomley@...allels.com>,
	Grant Likely <grant.likely@...aro.org>,
	Raviv Shvili <rshvili@...eaurora.org>,
	Sujit Reddy Thumma <sthumma@...eaurora.org>,
	Christoph Hellwig <hch@....de>,
	Sahitya Tummala <stummala@...eaurora.org>,
	"open list:OPEN FIRMWARE AND..." <devicetree@...r.kernel.org>
Subject: Re: [PATCH v4] scsi: ufs: add support of generic PHY framework in Qualcomm MSM chips


On Sep 17, 2014, at 9:26 AM, Yaniv Gardi <ygardi@...eaurora.org> wrote:

> This change depends on draviv [PATCH v3] series of 16 changes that was
> already uploaded.
> Soon draviv will uploaded again as [PATCH v4] that contain 17 patches, and
> this change will be dependent on this serie,
> 
> Thanks,
> Yaniv
> 
> -----Original Message-----
> From: Yaniv Gardi [mailto:ygardi@...eaurora.org] 
> Sent: Wednesday, September 17, 2014 7:22 PM
> To: James.Bottomley@...senPartnership.com; hch@...radead.org
> Cc: linux-kernel@...r.kernel.org; linux-scsi@...r.kernel.org;
> linux-arm-msm@...r.kernel.org; santoshsy@...il.com;
> linux-scsi-owner@...r.kernel.org; subhashj@...eaurora.org;
> ygardi@...eaurora.org; noag@...eaurora.org; draviv@...eaurora.org; Rob
> Herring; Pawel Moll; Mark Rutland; Ian Campbell; Kumar Gala; Kishon Vijay
> Abraham I; Vinayak Holikatti; James E.J. Bottomley; Grant Likely; Raviv
> Shvili; Sujit Reddy Thumma; Christoph Hellwig; Sahitya Tummala; open
> list:OPEN FIRMWARE AND...
> Subject: [PATCH v4] scsi: ufs: add support of generic PHY framework in
> Qualcomm MSM chips
> 
> In this change we add support to the generic PHY framework.
> Two UFS phys are implemented:
> qmp-20nm and qmp-28nm.
> 
> Also, the files in this change implement the UFS HW (controller & PHY)
> specific behavior in Qualcomm chips.
> Relocation of a few header files is needed in order to expose routines
> and data structures between PHY driver and UFS driver
> 
> Signed-off-by: Yaniv Gardi <ygardi@...eaurora.org>
> ---
> .../devicetree/bindings/phy/qcom-ufs-phy.txt       |  37 +
> drivers/phy/Makefile                               |   3 +
> drivers/phy/phy-qcom-ufs-qmp-20nm.c                | 259 ++++++
> drivers/phy/phy-qcom-ufs-qmp-20nm.h                | 216 +++++
> drivers/phy/phy-qcom-ufs-qmp-28nm.c                | 342 ++++++++
> drivers/phy/phy-qcom-ufs-qmp-28nm.h                | 738 ++++++++++++++++++
> drivers/phy/phy-qcom-ufs.c                         | 661 ++++++++++++++++
> drivers/scsi/ufs/Kconfig                           |  12 +
> drivers/scsi/ufs/Makefile                          |   1 +
> drivers/scsi/ufs/ufs-qcom.c                        | 866
> +++++++++++++++++++++
> drivers/scsi/ufs/ufs.h                             | 491 ------------
> drivers/scsi/ufs/ufshcd-pci.c                      |   2 +-
> drivers/scsi/ufs/ufshcd-pltfrm.c                   |   2 +-
> drivers/scsi/ufs/ufshcd.c                          |  14 +-
> drivers/scsi/ufs/ufshcd.h                          | 595 --------------
> drivers/scsi/ufs/unipro.h                          | 207 -----
> include/linux/phy/phy-qcom-ufs.h                   | 202 +++++
> include/linux/scsi/ufs/ufs-qcom.h                  | 156 ++++
> include/linux/scsi/ufs/ufs.h                       | 491 ++++++++++++
> include/linux/scsi/ufs/ufshcd.h                    | 635 +++++++++++++++
> include/linux/scsi/ufs/unipro.h                    | 207 +++++
> 21 files changed, 4840 insertions(+), 1297 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/phy/qcom-ufs-phy.txt
> create mode 100644 drivers/phy/phy-qcom-ufs-qmp-20nm.c
> create mode 100644 drivers/phy/phy-qcom-ufs-qmp-20nm.h
> create mode 100644 drivers/phy/phy-qcom-ufs-qmp-28nm.c
> create mode 100644 drivers/phy/phy-qcom-ufs-qmp-28nm.h
> create mode 100644 drivers/phy/phy-qcom-ufs.c
> create mode 100644 drivers/scsi/ufs/ufs-qcom.c
> delete mode 100644 drivers/scsi/ufs/ufs.h
> delete mode 100644 drivers/scsi/ufs/ufshcd.h
> delete mode 100644 drivers/scsi/ufs/unipro.h
> create mode 100644 include/linux/phy/phy-qcom-ufs.h
> create mode 100644 include/linux/scsi/ufs/ufs-qcom.h
> create mode 100644 include/linux/scsi/ufs/ufs.h
> create mode 100644 include/linux/scsi/ufs/ufshcd.h
> create mode 100644 include/linux/scsi/ufs/unipro.h

Can you split this into multiple patches.  At least one that just moves the headers around without any other changes.  Than another that introduces the PHY code.  Will make it far easier to review.

- k

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/