* [PATCH 0/2] scsi: ufs-cdns: Add M31 PHY init @ 2019-03-27 14:44 ` Jan Kotas 0 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-03-27 14:44 UTC (permalink / raw) To: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen Cc: devicetree, linux-kernel, linux-scsi, Jan Kotas This patchset adds additional initialization required by M31 PHY when used with Cadence UFS Host Controller. Jan Kotas (2): dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list scsi: ufs-cdns: Add support for UFSHCI with M31 PHY .../devicetree/bindings/ufs/cdns,ufshc.txt | 5 +- drivers/scsi/ufs/cdns-pltfrm.c | 77 +++++++++++++++++++--- 2 files changed, 70 insertions(+), 12 deletions(-) -- 2.15.0 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 0/2] scsi: ufs-cdns: Add M31 PHY init @ 2019-03-27 14:44 ` Jan Kotas 0 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-03-27 14:44 UTC (permalink / raw) To: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen Cc: devicetree, linux-kernel, linux-scsi, Jan Kotas This patchset adds additional initialization required by M31 PHY when used with Cadence UFS Host Controller. Jan Kotas (2): dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list scsi: ufs-cdns: Add support for UFSHCI with M31 PHY .../devicetree/bindings/ufs/cdns,ufshc.txt | 5 +- drivers/scsi/ufs/cdns-pltfrm.c | 77 +++++++++++++++++++--- 2 files changed, 70 insertions(+), 12 deletions(-) -- 2.15.0 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 1/2] dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list 2019-03-27 14:44 ` Jan Kotas @ 2019-03-27 14:44 ` Jan Kotas -1 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-03-27 14:44 UTC (permalink / raw) To: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen Cc: devicetree, linux-kernel, linux-scsi, Jan Kotas This patch adds a new compatible string description for CDNS UFS HCD + M31 16nm PHY. Signed-off-by: Jan Kotas <jank@cadence.com> --- Documentation/devicetree/bindings/ufs/cdns,ufshc.txt | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt index a04a4989e..02347b017 100644 --- a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt +++ b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt @@ -5,8 +5,9 @@ Each UFS controller instance should have its own node. Please see the ufshcd-pltfrm.txt for a list of all available properties. Required properties: -- compatible : Compatible list, contains the following controller: - "cdns,ufshc" +- compatible : Compatible list, contains one of the following controllers: + "cdns,ufshc" - Generic CDNS HCI, + "cdns,ufshc-m31-16nm" - CDNS UFS HC + M31 16nm PHY complemented with the JEDEC version: "jedec,ufs-2.0" -- 2.15.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 1/2] dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list @ 2019-03-27 14:44 ` Jan Kotas 0 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-03-27 14:44 UTC (permalink / raw) To: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen Cc: devicetree, linux-kernel, linux-scsi, Jan Kotas This patch adds a new compatible string description for CDNS UFS HCD + M31 16nm PHY. Signed-off-by: Jan Kotas <jank@cadence.com> --- Documentation/devicetree/bindings/ufs/cdns,ufshc.txt | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt index a04a4989e..02347b017 100644 --- a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt +++ b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt @@ -5,8 +5,9 @@ Each UFS controller instance should have its own node. Please see the ufshcd-pltfrm.txt for a list of all available properties. Required properties: -- compatible : Compatible list, contains the following controller: - "cdns,ufshc" +- compatible : Compatible list, contains one of the following controllers: + "cdns,ufshc" - Generic CDNS HCI, + "cdns,ufshc-m31-16nm" - CDNS UFS HC + M31 16nm PHY complemented with the JEDEC version: "jedec,ufs-2.0" -- 2.15.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list 2019-03-27 14:44 ` Jan Kotas (?) @ 2019-03-31 6:42 ` Rob Herring 2019-04-01 7:49 ` Jan Kotas -1 siblings, 1 reply; 11+ messages in thread From: Rob Herring @ 2019-03-31 6:42 UTC (permalink / raw) To: Jan Kotas Cc: mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen, devicetree, linux-kernel, linux-scsi On Wed, Mar 27, 2019 at 02:44:04PM +0000, Jan Kotas wrote: > This patch adds a new compatible string description for > CDNS UFS HCD + M31 16nm PHY. The phy is not a separate block? > > Signed-off-by: Jan Kotas <jank@cadence.com> > --- > Documentation/devicetree/bindings/ufs/cdns,ufshc.txt | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt > index a04a4989e..02347b017 100644 > --- a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt > +++ b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt > @@ -5,8 +5,9 @@ Each UFS controller instance should have its own node. > Please see the ufshcd-pltfrm.txt for a list of all available properties. > > Required properties: > -- compatible : Compatible list, contains the following controller: > - "cdns,ufshc" > +- compatible : Compatible list, contains one of the following controllers: > + "cdns,ufshc" - Generic CDNS HCI, > + "cdns,ufshc-m31-16nm" - CDNS UFS HC + M31 16nm PHY > complemented with the JEDEC version: > "jedec,ufs-2.0" > > -- > 2.15.0 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list 2019-03-31 6:42 ` Rob Herring @ 2019-04-01 7:49 ` Jan Kotas 0 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-04-01 7:49 UTC (permalink / raw) To: Rob Herring Cc: Jan Kotas, Mark Rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen, devicetree, linux-kernel, linux-scsi > On 31 Mar 2019, at 08:42, Rob Herring <robh@kernel.org> wrote: > > On Wed, Mar 27, 2019 at 02:44:04PM +0000, Jan Kotas wrote: >> This patch adds a new compatible string description for >> CDNS UFS HCD + M31 16nm PHY. > > The phy is not a separate block? In this case PHY is controlled by some extra registers in the controller's address space, accessible via its hardware register interface. Having an extra compatible also makes it consistent with others controllers, which are already in the kernel. ufs-qcom.txt: - compatible : compatible list, contains one of the following - "qcom,ufs-phy-qmp-20nm" for 20nm ufs phy, "qcom,ufs-phy-qmp-14nm" for legacy 14nm ufs phy, "qcom,msm8996-ufs-phy-qmp-14nm" for 14nm ufs phy present on MSM8996 chipset. tc-dwc-g210-pltfrm.txt: - compatible : compatible list must contain the PHY type & version: "snps,g210-tc-6.00-20bit" "snps,g210-tc-6.00-40bit" Regards, Jan > >> >> Signed-off-by: Jan Kotas <jank@cadence.com> >> --- >> Documentation/devicetree/bindings/ufs/cdns,ufshc.txt | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt >> index a04a4989e..02347b017 100644 >> --- a/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt >> +++ b/Documentation/devicetree/bindings/ufs/cdns,ufshc.txt >> @@ -5,8 +5,9 @@ Each UFS controller instance should have its own node. >> Please see the ufshcd-pltfrm.txt for a list of all available properties. >> >> Required properties: >> -- compatible : Compatible list, contains the following controller: >> - "cdns,ufshc" >> +- compatible : Compatible list, contains one of the following controllers: >> + "cdns,ufshc" - Generic CDNS HCI, >> + "cdns,ufshc-m31-16nm" - CDNS UFS HC + M31 16nm PHY >> complemented with the JEDEC version: >> "jedec,ufs-2.0" >> >> -- >> 2.15.0 >> > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list 2019-03-27 14:44 ` Jan Kotas (?) (?) @ 2019-04-03 4:15 ` Rob Herring -1 siblings, 0 replies; 11+ messages in thread From: Rob Herring @ 2019-04-03 4:15 UTC (permalink / raw) To: Jan Kotas Cc: Mark Rutland, Alim Akhtar, Avri Altman, pedrom.sousa, James E.J. Bottomley, Martin K. Petersen, devicetree, linux-kernel, SCSI On Wed, Mar 27, 2019 at 9:45 AM Jan Kotas <jank@cadence.com> wrote: > > This patch adds a new compatible string description for > CDNS UFS HCD + M31 16nm PHY. > > Signed-off-by: Jan Kotas <jank@cadence.com> > --- > Documentation/devicetree/bindings/ufs/cdns,ufshc.txt | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Reviewed-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 2/2] scsi: ufs-cdns: Add support for UFSHCI with M31 PHY 2019-03-27 14:44 ` Jan Kotas @ 2019-03-27 14:44 ` Jan Kotas -1 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-03-27 14:44 UTC (permalink / raw) To: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen Cc: devicetree, linux-kernel, linux-scsi, Jan Kotas This patch adds an additional PHY initialization, required for M31 PHY when used with Cadence UFS HC. A new compatible string has been added for this purpose. Signed-off-by: Jan Kotas <jank@cadence.com> --- drivers/scsi/ufs/cdns-pltfrm.c | 77 ++++++++++++++++++++++++++++++++++++------ 1 file changed, 67 insertions(+), 10 deletions(-) diff --git a/drivers/scsi/ufs/cdns-pltfrm.c b/drivers/scsi/ufs/cdns-pltfrm.c index 4a37b4f57..e0c8834df 100644 --- a/drivers/scsi/ufs/cdns-pltfrm.c +++ b/drivers/scsi/ufs/cdns-pltfrm.c @@ -17,7 +17,8 @@ #include "ufshcd-pltfrm.h" -#define CDNS_UFS_REG_HCLKDIV 0xFC +#define CDNS_UFS_REG_HCLKDIV 0xFC +#define CDNS_UFS_REG_PHY_XCFGD1 0x113C /** * Sets HCLKDIV register value based on the core_clk @@ -77,11 +78,69 @@ static int cdns_ufs_setup_clocks(struct ufs_hba *hba, bool on, return cdns_ufs_set_hclkdiv(hba); } -static struct ufs_hba_variant_ops cdns_pltfm_hba_vops = { +/** + * cdns_ufs_init - performs additional ufs initialization + * @hba: host controller instance + * + * Returns status of initialization + */ +static int cdns_ufs_init(struct ufs_hba *hba) +{ + int status = 0; + + if (hba->vops && hba->vops->phy_initialization) { + status = hba->vops->phy_initialization(hba); + if (status) + return status; + } + + return status; +} + +/** + * cdns_ufs_m31_16nm_phy_initialization - performs m31 phy initialization + * @hba: host controller instance + * + * Always returns 0 + */ +static int cdns_ufs_m31_16nm_phy_initialization(struct ufs_hba *hba) +{ + u32 data; + + /* Increase RX_Advanced_Min_ActivateTime_Capability */ + data = ufshcd_readl(hba, CDNS_UFS_REG_PHY_XCFGD1); + data |= BIT(24); + ufshcd_writel(hba, data, CDNS_UFS_REG_PHY_XCFGD1); + + return 0; +} + +static const struct ufs_hba_variant_ops cdns_ufs_pltfm_hba_vops = { + .name = "cdns-ufs-pltfm", + .setup_clocks = cdns_ufs_setup_clocks, +}; + +static const struct ufs_hba_variant_ops cdns_ufs_m31_16nm_pltfm_hba_vops = { .name = "cdns-ufs-pltfm", + .init = cdns_ufs_init, .setup_clocks = cdns_ufs_setup_clocks, + .phy_initialization = cdns_ufs_m31_16nm_phy_initialization, }; +static const struct of_device_id cdns_ufs_of_match[] = { + { + .compatible = "cdns,ufshc", + .data = &cdns_ufs_pltfm_hba_vops, + }, + { + .compatible = "cdns,ufshc-m31-16nm", + .data = &cdns_ufs_m31_16nm_pltfm_hba_vops, + }, + { }, +}; + +MODULE_DEVICE_TABLE(of, cdns_ufs_of_match); + /** * cdns_ufs_pltfrm_probe - probe routine of the driver * @pdev: pointer to platform device handle @@ -91,10 +150,15 @@ static struct ufs_hba_variant_ops cdns_pltfm_hba_vops = { static int cdns_ufs_pltfrm_probe(struct platform_device *pdev) { int err; + const struct of_device_id *of_id; + struct ufs_hba_variant_ops *vops; struct device *dev = &pdev->dev; + of_id = of_match_node(cdns_ufs_of_match, dev->of_node); + vops = (struct ufs_hba_variant_ops *)of_id->data; + /* Perform generic probe */ - err = ufshcd_pltfrm_init(pdev, &cdns_pltfm_hba_vops); + err = ufshcd_pltfrm_init(pdev, vops); if (err) dev_err(dev, "ufshcd_pltfrm_init() failed %d\n", err); @@ -115,13 +179,6 @@ static int cdns_ufs_pltfrm_remove(struct platform_device *pdev) return 0; } -static const struct of_device_id cdns_ufs_of_match[] = { - { .compatible = "cdns,ufshc" }, - {}, -}; - -MODULE_DEVICE_TABLE(of, cdns_ufs_of_match); - static const struct dev_pm_ops cdns_ufs_dev_pm_ops = { .suspend = ufshcd_pltfrm_suspend, .resume = ufshcd_pltfrm_resume, -- 2.15.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/2] scsi: ufs-cdns: Add support for UFSHCI with M31 PHY @ 2019-03-27 14:44 ` Jan Kotas 0 siblings, 0 replies; 11+ messages in thread From: Jan Kotas @ 2019-03-27 14:44 UTC (permalink / raw) To: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen Cc: devicetree, linux-kernel, linux-scsi, Jan Kotas This patch adds an additional PHY initialization, required for M31 PHY when used with Cadence UFS HC. A new compatible string has been added for this purpose. Signed-off-by: Jan Kotas <jank@cadence.com> --- drivers/scsi/ufs/cdns-pltfrm.c | 77 ++++++++++++++++++++++++++++++++++++------ 1 file changed, 67 insertions(+), 10 deletions(-) diff --git a/drivers/scsi/ufs/cdns-pltfrm.c b/drivers/scsi/ufs/cdns-pltfrm.c index 4a37b4f57..e0c8834df 100644 --- a/drivers/scsi/ufs/cdns-pltfrm.c +++ b/drivers/scsi/ufs/cdns-pltfrm.c @@ -17,7 +17,8 @@ #include "ufshcd-pltfrm.h" -#define CDNS_UFS_REG_HCLKDIV 0xFC +#define CDNS_UFS_REG_HCLKDIV 0xFC +#define CDNS_UFS_REG_PHY_XCFGD1 0x113C /** * Sets HCLKDIV register value based on the core_clk @@ -77,11 +78,69 @@ static int cdns_ufs_setup_clocks(struct ufs_hba *hba, bool on, return cdns_ufs_set_hclkdiv(hba); } -static struct ufs_hba_variant_ops cdns_pltfm_hba_vops = { +/** + * cdns_ufs_init - performs additional ufs initialization + * @hba: host controller instance + * + * Returns status of initialization + */ +static int cdns_ufs_init(struct ufs_hba *hba) +{ + int status = 0; + + if (hba->vops && hba->vops->phy_initialization) { + status = hba->vops->phy_initialization(hba); + if (status) + return status; + } + + return status; +} + +/** + * cdns_ufs_m31_16nm_phy_initialization - performs m31 phy initialization + * @hba: host controller instance + * + * Always returns 0 + */ +static int cdns_ufs_m31_16nm_phy_initialization(struct ufs_hba *hba) +{ + u32 data; + + /* Increase RX_Advanced_Min_ActivateTime_Capability */ + data = ufshcd_readl(hba, CDNS_UFS_REG_PHY_XCFGD1); + data |= BIT(24); + ufshcd_writel(hba, data, CDNS_UFS_REG_PHY_XCFGD1); + + return 0; +} + +static const struct ufs_hba_variant_ops cdns_ufs_pltfm_hba_vops = { + .name = "cdns-ufs-pltfm", + .setup_clocks = cdns_ufs_setup_clocks, +}; + +static const struct ufs_hba_variant_ops cdns_ufs_m31_16nm_pltfm_hba_vops = { .name = "cdns-ufs-pltfm", + .init = cdns_ufs_init, .setup_clocks = cdns_ufs_setup_clocks, + .phy_initialization = cdns_ufs_m31_16nm_phy_initialization, }; +static const struct of_device_id cdns_ufs_of_match[] = { + { + .compatible = "cdns,ufshc", + .data = &cdns_ufs_pltfm_hba_vops, + }, + { + .compatible = "cdns,ufshc-m31-16nm", + .data = &cdns_ufs_m31_16nm_pltfm_hba_vops, + }, + { }, +}; + +MODULE_DEVICE_TABLE(of, cdns_ufs_of_match); + /** * cdns_ufs_pltfrm_probe - probe routine of the driver * @pdev: pointer to platform device handle @@ -91,10 +150,15 @@ static struct ufs_hba_variant_ops cdns_pltfm_hba_vops = { static int cdns_ufs_pltfrm_probe(struct platform_device *pdev) { int err; + const struct of_device_id *of_id; + struct ufs_hba_variant_ops *vops; struct device *dev = &pdev->dev; + of_id = of_match_node(cdns_ufs_of_match, dev->of_node); + vops = (struct ufs_hba_variant_ops *)of_id->data; + /* Perform generic probe */ - err = ufshcd_pltfrm_init(pdev, &cdns_pltfm_hba_vops); + err = ufshcd_pltfrm_init(pdev, vops); if (err) dev_err(dev, "ufshcd_pltfrm_init() failed %d\n", err); @@ -115,13 +179,6 @@ static int cdns_ufs_pltfrm_remove(struct platform_device *pdev) return 0; } -static const struct of_device_id cdns_ufs_of_match[] = { - { .compatible = "cdns,ufshc" }, - {}, -}; - -MODULE_DEVICE_TABLE(of, cdns_ufs_of_match); - static const struct dev_pm_ops cdns_ufs_dev_pm_ops = { .suspend = ufshcd_pltfrm_suspend, .resume = ufshcd_pltfrm_resume, -- 2.15.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] scsi: ufs-cdns: Add support for UFSHCI with M31 PHY 2019-03-27 14:44 ` Jan Kotas @ 2019-04-04 3:24 ` Martin K. Petersen -1 siblings, 0 replies; 11+ messages in thread From: Martin K. Petersen @ 2019-04-04 3:24 UTC (permalink / raw) To: Jan Kotas Cc: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen, devicetree, linux-kernel, linux-scsi Jan, > + if (hba->vops && hba->vops->phy_initialization) { > + status = hba->vops->phy_initialization(hba); > + if (status) > + return status; > + } > + > + return status; I zapped the superfluous return conditional. Applied patches 1 and 2 to 5.2/scsi-queue. -- Martin K. Petersen Oracle Linux Engineering ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] scsi: ufs-cdns: Add support for UFSHCI with M31 PHY @ 2019-04-04 3:24 ` Martin K. Petersen 0 siblings, 0 replies; 11+ messages in thread From: Martin K. Petersen @ 2019-04-04 3:24 UTC (permalink / raw) To: Jan Kotas Cc: robh+dt, mark.rutland, alim.akhtar, avri.altman, pedrom.sousa, jejb, martin.petersen, devicetree, linux-kernel, linux-scsi Jan, > + if (hba->vops && hba->vops->phy_initialization) { > + status = hba->vops->phy_initialization(hba); > + if (status) > + return status; > + } > + > + return status; I zapped the superfluous return conditional. Applied patches 1 and 2 to 5.2/scsi-queue. -- Martin K. Petersen Oracle Linux Engineering ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-04-04 3:24 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-03-27 14:44 [PATCH 0/2] scsi: ufs-cdns: Add M31 PHY init Jan Kotas 2019-03-27 14:44 ` Jan Kotas 2019-03-27 14:44 ` [PATCH 1/2] dt-bindings: scsi: ufs-cdns: Update Cadence UFS compatibility list Jan Kotas 2019-03-27 14:44 ` Jan Kotas 2019-03-31 6:42 ` Rob Herring 2019-04-01 7:49 ` Jan Kotas 2019-04-03 4:15 ` Rob Herring 2019-03-27 14:44 ` [PATCH 2/2] scsi: ufs-cdns: Add support for UFSHCI with M31 PHY Jan Kotas 2019-03-27 14:44 ` Jan Kotas 2019-04-04 3:24 ` Martin K. Petersen 2019-04-04 3:24 ` Martin K. Petersen
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.