Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1214747

Re: [PATCH v3 6/8] scsi: ufs: make the UFS variant a platform device

Show all headers | View raw

On Sun, Aug 23, 2015 at 8:09 AM, Yaniv Gardi <ygardi@codeaurora.org> wrote:
> This change turns the UFS variant (SCSI_UFS_QCOM) into a UFS
> a platform device.
> In order to do so a few additional changes are required:
> 1. The ufshcd-pltfrm is no longer serves as a platform device.
>    Now it only serves as a group of platform APIs such as PM APIs
>    (runtime suspend/resume, system suspend/resume etc), parsers of
>    clocks, regulators and pm_levels from DT.
> 2. What used to be the old platform "probe" is now "only"
>    a pltfrm_init() routine, that does exactly the same, but only
>    being called by the new probe function of the UFS variant.
>
> Signed-off-by: Yaniv Gardi <ygardi@codeaurora.org>
>
> ---
>  .../devicetree/bindings/ufs/ufshcd-pltfrm.txt      |  2 +-
>  drivers/scsi/ufs/ufs-qcom.c                        | 78 +++++++++++++++++-
>  drivers/scsi/ufs/ufshcd-pltfrm.c                   | 92 ++++++----------------
>  drivers/scsi/ufs/ufshcd-pltfrm.h                   | 41 ++++++++++
>  drivers/scsi/ufs/ufshcd.c                          | 10 +++
>  drivers/scsi/ufs/ufshcd.h                          |  1 +
>  6 files changed, 152 insertions(+), 72 deletions(-)
>  create mode 100644 drivers/scsi/ufs/ufshcd-pltfrm.h
>
> diff --git a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
> index 5357919..b39e765 100644
> --- a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
> +++ b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
> @@ -4,7 +4,7 @@ UFSHC nodes are defined to describe on-chip UFS host controllers.
>  Each UFS controller instance should have its own node.
>
>  Required properties:
> -- compatible        : compatible list, contains "jedec,ufs-1.1"
> +- compatible        : compatible list, contains "jedec,ufs-1.1" or "qcom,ufshc"

Replying again as I inadvertently dropped everyone.

This should also have a more specific compatible string with the SOC
name/number in it. It may be "the same in all SOCs", but there is
always the possibility for bugs/limitations to be found that are
specific to an SOC even if all RTL versions are identical (e.g.
different max clock speeds). It is about making the dtb future proof,
not about what exactly you need today. You can keep qcom,ufshc for
driver matching if you want.

>  - interrupts        : <interrupt mapping for UFS host controller IRQ>
>  - reg               : <registers mapping>

What about phy properties? No Unipro PHY block that requires setup?

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/8] Fix error message and present UFS variant Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
  [PATCH v3 1/8] phy: qcom-ufs: fix build error when the component is built as a module Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
  [PATCH v3 2/8] scsi: ufs-qcom: fix compilation warning if compiled as a module Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
  [PATCH v3 8/8] scsi: ufs-qcom: add QUniPro hardware support and power optimizations Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
    Re: [PATCH v3 8/8] scsi: ufs-qcom: add QUniPro hardware support and  power optimizations ygardi@codeaurora.org - 2015-08-25 14:00 +0200
  [PATCH v3 3/8] scsi: ufs-qcom: update configuration option of SCSI_UFS_QCOM component Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
  [PATCH v3 7/8] scsi: ufs-qcom: add debug prints for test bus Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
  [PATCH v3 4/8] add ufshcd_get_variant ufshcd_set_variant Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
  [PATCH v3 6/8] scsi: ufs: make the UFS variant a platform device Yaniv Gardi <ygardi@codeaurora.org> - 2015-08-23 15:20 +0200
    Re: [PATCH v3 6/8] scsi: ufs: make the UFS variant a platform device Rob Herring <robherring2@gmail.com> - 2015-08-27 20:00 +0200
      Re: [PATCH v3 6/8] scsi: ufs: make the UFS variant a platform  device ygardi@codeaurora.org - 2015-08-30 10:50 +0200
        Re: [PATCH v3 6/8] scsi: ufs: make the UFS variant a platform device Rob Herring <robherring2@gmail.com> - 2015-08-30 15:20 +0200
  Re: [PATCH v2 0/8] Fix error message and present UFS variant Akinobu Mita <akinobu.mita@gmail.com> - 2015-08-25 01:50 +0200
    Re: [PATCH v2 0/8] Fix error message and present UFS variant ygardi@codeaurora.org - 2015-08-25 14:00 +0200

csiph-web