Re: [PATCH 10/10] scsi: ufs-exynos: add UFS host support for Exynos SoCs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



HI Amit,

On 08/26/2015 12:12 PM, amit daniel kachhap wrote:
On Fri, Aug 21, 2015 at 2:58 PM, Alim Akhtar <alim.akhtar@xxxxxxxxxxx> wrote:
From: Seungwon Jeon <tgih.jun@xxxxxxxxxxx>

This patch introduces Exynos UFS host controller driver,
which mainly handles vendor-specific operations including
link startup, power mode change and hibernation/unhibernation.

Signed-off-by: Seungwon Jeon <tgih.jun@xxxxxxxxxxx>
Signed-off-by: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>
---
  .../devicetree/bindings/ufs/ufs-exynos.txt         |   92 ++
  drivers/scsi/ufs/Kconfig                           |   12 +
  drivers/scsi/ufs/Makefile                          |    1 +
  drivers/scsi/ufs/ufs-exynos-hw.c                   |  147 +++
  drivers/scsi/ufs/ufs-exynos-hw.h                   |   43 +
  drivers/scsi/ufs/ufs-exynos.c                      | 1175 ++++++++++++++++++++
  drivers/scsi/ufs/ufs-exynos.h                      |  463 ++++++++
  drivers/scsi/ufs/ufshci.h                          |   26 +-
  drivers/scsi/ufs/unipro.h                          |   47 +
  9 files changed, 2005 insertions(+), 1 deletion(-)
  create mode 100644 Documentation/devicetree/bindings/ufs/ufs-exynos.txt
  create mode 100644 drivers/scsi/ufs/ufs-exynos-hw.c
  create mode 100644 drivers/scsi/ufs/ufs-exynos-hw.h
  create mode 100644 drivers/scsi/ufs/ufs-exynos.c
  create mode 100644 drivers/scsi/ufs/ufs-exynos.h

diff --git a/Documentation/devicetree/bindings/ufs/ufs-exynos.txt b/Documentation/devicetree/bindings/ufs/ufs-exynos.txt
new file mode 100644
index 0000000..1a6184d
--- /dev/null
+++ b/Documentation/devicetree/bindings/ufs/ufs-exynos.txt
@@ -0,0 +1,92 @@
+* Exynos Universal Flash Storage (UFS) Host Controller
+
+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 "samsung,exynos7-ufs"
+- interrupts        : <interrupt mapping for UFS host controller IRQ>
+- reg               : <registers mapping>
+
+Optional properties:
+- vdd-hba-supply        : phandle to UFS host controller supply regulator node
+- vcc-supply            : phandle to VCC supply regulator node
+- vccq-supply           : phandle to VCCQ supply regulator node
+- vccq2-supply          : phandle to VCCQ2 supply regulator node
+- vcc-supply-1p8        : For embedded UFS devices, valid VCC range is 1.7-1.95V
+                          or 2.7-3.6V. This boolean property when set, specifies
+                         to use low voltage range of 1.7-1.95V. Note for external
+                         UFS cards this property is invalid and valid VCC range is
+                         always 2.7-3.6V.
+- vcc-max-microamp      : specifies max. load that can be drawn from vcc supply
+- vccq-max-microamp     : specifies max. load that can be drawn from vccq supply
+- vccq2-max-microamp    : specifies max. load that can be drawn from vccq2 supply
+- <name>-fixed-regulator : boolean property specifying that <name>-supply is a fixed regulator
+
+- clocks                : List of phandle and clock specifier pairs
+- clock-names           : List of clock input name strings sorted in the same
+                          order as the clocks property.
+- freq-table-hz                : Array of <min max> operating frequencies stored in the same
+                          order as the clocks property. If this property is not
+                         defined or a value in the array is "0" then it is assumed
+                         that the frequency is set by the parent clock or a
+                         fixed rate clock source.
+- pclk-freq-avail-range : specifies available frequency range(min/max) for APB clock
+- ufs,pwr-attr-mode : specifies mode value for power mode change
+- ufs,pwr-attr-lane : specifies lane count value for power mode change
+- ufs,pwr-attr-gear : specifies gear count value for power mode change
+- ufs,pwr-attr-hs-series : specifies HS rate series for power mode change
+- ufs,pwr-local-l2-timer : specifies array of local UNIPRO L2 timer values
+                          <FC0ProtectionTimeOutVal,TC0ReplayTimeOutVal, AFC0ReqTimeOutVal>
+- ufs,pwr-remote-l2-timer : specifies array of remote UNIPR L2 timer values
s/UNIPR/UNIPRO
ok
+                          <FC0ProtectionTimeOutVal,TC0ReplayTimeOutVal, AFC0ReqTimeOutVal>
+- ufs-rx-adv-fine-gran-sup_en : specifies support of fine granularity of MPHY
I suppose this field is bool type. This can be mentioned here.
+- ufs-rx-adv-fine-gran-step : specifies granularity steps of MPHY
+- ufs-rx-adv-min-activate-time-cap : specifies rx advanced minimum activate time of MPHY
+- ufs-pa-granularity : specifies Granularity for PA_TActivate and PA_Hibern8Time
+- ufs-pa-tacctivate : specifies time wake-up remote M-RX
+- ufs-pa-hibern8time : specifies minimum time to wait in HIBERN8 state
+
+Note: If above properties are not defined it can be assumed that the supply
+regulators or clocks are always on.
+
+Example:
+       ufshc@0x15570000 {
+               compatible = "samsung,exynos7-ufs";
+               reg = <0xfc598000 0x800>;
+               reg = <0x15570000 0x100>,
+                     <0x15570100 0x100>,
+                     <0x15571000 0x200>,
+                     <0x15572000 0x300>;
+               reg-names = "hci", "vs_hci", "unipro", "ufsp";
+               interrupts = <0 200 0>;
+
+               vdd-hba-supply = <&xxx_reg0>;
+               vdd-hba-fixed-regulator;
+               vcc-supply = <&xxx_reg1>;
+               vcc-supply-1p8;
+               vccq-supply = <&xxx_reg2>;
+               vccq2-supply = <&xxx_reg3>;
+               vcc-max-microamp = 500000;
+               vccq-max-microamp = 200000;
+               vccq2-max-microamp = 200000;
+
+               clocks = <&core 0>, <&ref 0>, <&iface 0>;
+               clock-names = "core_clk", "ref_clk", "iface_clk";
+               freq-table-hz = <100000000 200000000>, <0 0>, <0 0>;
+
+               pclk-freq-avail-range = <70000000 133000000>;
+
+               ufs,pwr-attr-mode = "FAST";
+               ufs,pwr-attr-lane = /bits/ 8 <2>;
+               ufs,pwr-attr-gear = /bits/ 8 <2>;
+               ufs,pwr-attr-hs-series = "HS_rate_b";
+               ufs,pwr-local-l2-timer = <8000 28000 20000>;
+               ufs,pwr-remote-l2-timer = <12000 32000 16000>;
+               ufs-rx-adv-fine-gran-sup_en = <1>;
+               ufs-rx-adv-fine-gran-step = <3>;
+               ufs-rx-adv-min-activate-time-cap = <10>;
+               ufs-pa-granularity = <6>;
+               ufs-pa-tacctivate = <6>;
+               ufs-pa-hibern8time = <20>;
+       };
diff --git a/drivers/scsi/ufs/Kconfig b/drivers/scsi/ufs/Kconfig
index e945383..14544d6 100644
--- a/drivers/scsi/ufs/Kconfig
+++ b/drivers/scsi/ufs/Kconfig
@@ -83,3 +83,15 @@ config SCSI_UFS_QCOM

           Select this if you have UFS controller on QCOM chipset.
           If unsure, say N.
+
+config SCSI_UFS_EXYNOS
+       bool "EXYNOS specific hooks to UFS controller platform driver"
+       depends on SCSI_UFSHCD_PLATFORM && ARCH_EXYNOS
+       select PHY_EXYNOS_UFS
+       help
+         This selects the EXYNOS specific additions to UFSHCD platform driver.
+         UFS host on EXYNOS includes HCI and UNIPRO layer, and associates with
+         UFS-PHY driver.
+
+         Select this if you have UFS host controller on EXYNOS chipset.
+         If unsure, say N.
diff --git a/drivers/scsi/ufs/Makefile b/drivers/scsi/ufs/Makefile
index 8303bcc..2accf1e 100644
--- a/drivers/scsi/ufs/Makefile
+++ b/drivers/scsi/ufs/Makefile
@@ -1,5 +1,6 @@
  # UFSHCD makefile
  obj-$(CONFIG_SCSI_UFS_QCOM) += ufs-qcom.o
+obj-$(CONFIG_SCSI_UFS_EXYNOS) += ufs-exynos.o ufs-exynos-hw.o
  obj-$(CONFIG_SCSI_UFSHCD) += ufshcd.o
  obj-$(CONFIG_SCSI_UFSHCD_PCI) += ufshcd-pci.o
  obj-$(CONFIG_SCSI_UFSHCD_PLATFORM) += ufshcd-pltfrm.o
diff --git a/drivers/scsi/ufs/ufs-exynos-hw.c b/drivers/scsi/ufs/ufs-exynos-hw.c
new file mode 100644
index 0000000..c18e45f
--- /dev/null
+++ b/drivers/scsi/ufs/ufs-exynos-hw.c
@@ -0,0 +1,147 @@
+/*
+ * UFS Host Controller driver for Exynos specific extensions
+ *
+ * Copyright (C) 2014-2015 Samsung Electronics Co., Ltd.
+ * Author: Seungwon Jeon  <tgih.jun@xxxxxxxxxxx>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#include "ufshcd.h"
+#include "unipro.h"
+#include "ufs-exynos.h"
+#include "ufs-exynos-hw.h"
The headers can be sorted here.
+
+static int ufs_clk_set_parent(struct device *dev, const char *c, const char *p)
+{
+       struct clk *_c, *_p;
+
+       _c = devm_clk_get(dev, c);
+       if (IS_ERR(_c)) {
+               dev_err(dev, "failed to get clock %s\n", c);
+               return -EINVAL;
+       }
+
+       _p = devm_clk_get(dev, p);
+       if (IS_ERR(_p)) {
+               dev_err(dev, "failed to get clock %s\n", p);
+               return -EINVAL;
+       }
+
+       return clk_set_parent(_c, _p);
+}
+
+static int exynos7_ufs_drv_init(struct device *dev, struct exynos_ufs *ufs)
+{
+       int ret;
+       const char *const clks[] = {
+               "mout_sclk_combo_phy_embedded",
+               "top_sclk_phy_fsys1_26m",
+       };
+
+       ret = ufs_clk_set_parent(dev, clks[0], clks[1]);
+       if (ret)
+               dev_err(dev, "failed to set parent %s of clock %s\n",
+                               clks[1], clks[0]);
+
+       return ret;
+}
+
+static int exynos7_ufs_pre_link(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       u32 val = ufs->drv_data->uic_attr->pa_dbg_option_suite;
+       int i;
+
+       exynos_ufs_enable_ov_tm(hba);
+       for_each_ufs_tx_lane(ufs, i)
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(0x297, i), 0x17);
+       for_each_ufs_rx_lane(ufs, i) {
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(0x362, i), 0xff);
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(0x363, i), 0x00);
+       }
+       exynos_ufs_disable_ov_tm(hba);
+
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OPTION_SUITE_DYN), 0xf);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OPTION_SUITE_DYN), 0xf);
+       for_each_ufs_tx_lane(ufs, i)
+               ufshcd_dme_set(hba,
+                       UIC_ARG_MIB_SEL(TX_HIBERN8_CONTROL, i), 0x0);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_TXPHY_CFGUPDT), 0x1);
+       udelay(1);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OPTION_SUITE), val | (1 << 12));
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_SKIP_RESET_PHY), 0x1);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_SKIP_LINE_RESET), 0x1);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_LINE_RESET_REQ), 0x1);
+       udelay(1600);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OPTION_SUITE), val);
+
+       return 0;
+}
+
+static int exynos7_ufs_post_link(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       int i;
+
+       exynos_ufs_enable_ov_tm(hba);
+       for_each_ufs_tx_lane(ufs, i) {
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(0x28b, i), 0x83);
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(0x29a, i), 0x07);
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(0x277, i),
+                       TX_LINERESET_N(exynos_ufs_calc_time_cntr(ufs, 200000)));
+       }
+       exynos_ufs_disable_ov_tm(hba);
+
+       exynos_ufs_enable_dbg_mode(hba);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_SAVECONFIGTIME), 0xbb8);
+       exynos_ufs_disable_dbg_mode(hba);
+
+       return 0;
+}
+
+static int exynos7_ufs_pre_pwr_change(struct exynos_ufs *ufs,
+                                               struct uic_pwr_mode *pwr)
+{
+       unipro_writel(ufs, 0x22, UNIPRO_DBG_FORCE_DME_CTRL_STATE);
+
+       return 0;
+}
+
+static int exynos7_ufs_post_pwr_change(struct exynos_ufs *ufs,
+                                               struct uic_pwr_mode *pwr)
+{
+       struct ufs_hba *hba = ufs->hba;
+
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_RXPHY_CFGUPDT), 0x1);
+
+       if (pwr->lane == 1) {
+               exynos_ufs_enable_dbg_mode(hba);
+               ufshcd_dme_set(hba, UIC_ARG_MIB(PA_CONNECTEDTXDATALANES), 0x1);
+               exynos_ufs_disable_dbg_mode(hba);
+       }
+
+       return 0;
+}
+
+struct exynos_ufs_drv_data exynos_ufs_drvs[] = {
+{
+       .compatible             = "samsung,exynos7-ufs",
+       .uic_attr               = &exynos7_uic_attr,
+       .quirks                 = UFSHCI_QUIRK_BROKEN_UTRD |
+                                 UFSHCI_QUIRK_BROKEN_REQ_LIST_CLR |
+                                 UFSHCI_QUIRK_BROKEN_HCE |
+                                 UFSHCI_QUIRK_BROKEN_RESET_INTR_AGGR,
+       .opts                   = EXYNOS_UFS_OPT_HAS_APB_CLK_CTRL |
+                                 EXYNOS_UFS_OPT_BROKEN_AUTO_CLK_CTRL |
+                                 EXYNOS_UFS_OPT_BROKEN_RX_SEL_IDX,
+       .drv_init               = exynos7_ufs_drv_init,
+       .pre_link               = exynos7_ufs_pre_link,
+       .post_link              = exynos7_ufs_post_link,
+       .pre_pwr_change         = exynos7_ufs_pre_pwr_change,
+       .post_pwr_change        = exynos7_ufs_post_pwr_change,
+}, {
+}, };
diff --git a/drivers/scsi/ufs/ufs-exynos-hw.h b/drivers/scsi/ufs/ufs-exynos-hw.h
new file mode 100644
index 0000000..8464ec8
--- /dev/null
+++ b/drivers/scsi/ufs/ufs-exynos-hw.h
@@ -0,0 +1,43 @@
+/*
+ * UFS Host Controller driver for Exynos specific extensions
+ *
+ * Copyright (C) 2014-2015 Samsung Electronics Co., Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#ifndef _UFS_EXYNOS_HW_H_
+#define _UFS_EXYNOS_HW_H_
+
+#include "ufs-exynos.h"
+#include "unipro.h"
+
+static struct exynos_ufs_uic_attr exynos7_uic_attr = {
+       .tx_trailingclks                = 0x10,
This can be shown in decimal as others below.
we need then in hex
+       .tx_dif_p_nsec                  = 3000000,      /* unit: ns */
+       .tx_dif_n_nsec                  = 1000000,      /* unit: ns */
+       .tx_high_z_cnt_nsec             = 20000,        /* unit: ns */
+       .tx_base_unit_nsec              = 100000,       /* unit: ns */
+       .tx_gran_unit_nsec              = 4000,         /* unit: ns */
+       .tx_sleep_cnt                   = 1000,         /* unit: ns */
+       .tx_min_activatetime            = 0xa,
same as above.
+       .rx_filler_enable               = 0x2,
+       .rx_dif_p_nsec                  = 1000000,      /* unit: ns */
+       .rx_hibern8_wait_nsec           = 4000000,      /* unit: ns */
+       .rx_base_unit_nsec              = 100000,       /* unit: ns */
+       .rx_gran_unit_nsec              = 4000,         /* unit: ns */
+       .rx_sleep_cnt                   = 1280,         /* unit: ns */
+       .rx_stall_cnt                   = 320,          /* unit: ns */
+       .rx_hs_g1_sync_len_cap          = SYNC_LEN_COARSE(0xf),
+       .rx_hs_g2_sync_len_cap          = SYNC_LEN_COARSE(0xf),
+       .rx_hs_g3_sync_len_cap          = SYNC_LEN_COARSE(0xf),
+       .rx_hs_g1_prep_sync_len_cap     = PREP_LEN(0xf),
+       .rx_hs_g2_prep_sync_len_cap     = PREP_LEN(0xf),
+       .rx_hs_g3_prep_sync_len_cap     = PREP_LEN(0xf),
+       .pa_dbg_option_suite            = 0x30103,
+};
+
+#endif /* _UFS_EXYNOS_HW_H_ */
diff --git a/drivers/scsi/ufs/ufs-exynos.c b/drivers/scsi/ufs/ufs-exynos.c
new file mode 100644
index 0000000..df2804e
--- /dev/null
+++ b/drivers/scsi/ufs/ufs-exynos.c
@@ -0,0 +1,1175 @@
+/*
+ * UFS Host Controller driver for Exynos specific extensions
+ *
+ * Copyright (C) 2014-2015 Samsung Electronics Co., Ltd.
+ * Author: Seungwon Jeon  <tgih.jun@xxxxxxxxxxx>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/of.h>
+#include <linux/of_address.h>
+#include <linux/clk.h>
+#include <linux/phy/phy-exynos-ufs.h>
+
+#include "ufshcd.h"
+#include "unipro.h"
+#include "ufs-exynos.h"
These headers can be sorted.
+
+#define IATOVAL_NSEC           20000   /* unit: ns */
+#define UNIPRO_PCLK_PERIOD(ufs) (NSEC_PER_SEC / ufs->pclk_rate)
+
+static void exynos_ufs_auto_ctrl_hcc(struct exynos_ufs *ufs, bool en);
+static void exynos_ufs_ctrl_clkstop(struct exynos_ufs *ufs, bool en);
+
+static inline struct exynos_ufs *to_exynos_ufs(struct ufs_hba *hba)
+{
+       return hba->priv;
+}
+
+static inline void exynos_ufs_enable_auto_ctrl_hcc(struct exynos_ufs *ufs)
+{
+       exynos_ufs_auto_ctrl_hcc(ufs, true);
+}
+
+static inline void exynos_ufs_disable_auto_ctrl_hcc(struct exynos_ufs *ufs)
+{
+       exynos_ufs_auto_ctrl_hcc(ufs, false);
+}
+
+static inline void exynos_ufs_disable_auto_ctrl_hcc_save(
+                                       struct exynos_ufs *ufs, u32 *val)
+{
+       *val = hci_readl(ufs, HCI_MISC);
+       exynos_ufs_auto_ctrl_hcc(ufs, false);
+}
+
+static inline void exynos_ufs_auto_ctrl_hcc_restore(
+                                       struct exynos_ufs *ufs, u32 *val)
+{
+       hci_writel(ufs, *val, HCI_MISC);
+}
+
+static inline void exynos_ufs_gate_clks(struct exynos_ufs *ufs)
+{
+       exynos_ufs_ctrl_clkstop(ufs, true);
+}
+
+static inline void exynos_ufs_ungate_clks(struct exynos_ufs *ufs)
+{
+       exynos_ufs_ctrl_clkstop(ufs, false);
+}
+
+/**
+ * exynos_ufs_auto_ctrl_hcc - HCI core clock control by h/w
+ * Control should be disabled in the below cases
+ * - Before host controller S/W reset
+ * - Access to UFS protector's register
+ */
+void exynos_ufs_auto_ctrl_hcc(struct exynos_ufs *ufs, bool en)
This function can me made static so as to make it consistent with
others. Although they are declared static above.
will do
+{
+       u32 misc = hci_readl(ufs, HCI_MISC);
+
+       if (en)
+               hci_writel(ufs, misc | HCI_CORECLK_CTRL_EN, HCI_MISC);
+       else
+               hci_writel(ufs, misc & ~HCI_CORECLK_CTRL_EN, HCI_MISC);
+}
+
+void exynos_ufs_ctrl_clkstop(struct exynos_ufs *ufs, bool en)
same as above.
will do
+{
+       u32 ctrl = hci_readl(ufs, HCI_CLKSTOP_CTRL);
+       u32 misc = hci_readl(ufs, HCI_MISC);
+
+       if (en) {
+               hci_writel(ufs, misc | CLK_CTRL_EN_MASK, HCI_MISC);
+               hci_writel(ufs, ctrl | CLK_STOP_MASK, HCI_CLKSTOP_CTRL);
+       } else {
+               hci_writel(ufs, ctrl & ~CLK_STOP_MASK, HCI_CLKSTOP_CTRL);
+               hci_writel(ufs, misc & ~CLK_CTRL_EN_MASK, HCI_MISC);
+       }
+}
+
+static int exynos_ufs_get_clk_info(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       struct list_head *head = &hba->clk_list_head;
+       struct ufs_clk_info *clki;
+       u32 pclk_rate;
+       u32 f_min, f_max;
+       u8 div = 0;
+       int ret = 0;
+
+       if (!head || list_empty(head))
+               goto out;
+
+       list_for_each_entry(clki, head, list) {
+               if (!IS_ERR_OR_NULL(clki->clk)) {
+                       if (!strcmp(clki->name, "aclk_ufs"))
+                               ufs->clk_hci_core = clki->clk;
+                       else if (!strcmp(clki->name, "sclk_unipro_apb"))
+                               ufs->clk_apb = clki->clk;
+                       else if (!strcmp(clki->name, "sclk_unipro_main"))
+                               ufs->clk_unipro_main = clki->clk;
+               }
+       }
+
+       if (!ufs->clk_hci_core || !ufs->clk_unipro_main || !ufs->clk_apb) {
+               dev_err(hba->dev, "failed to get clk info\n");
+               ret = -EINVAL;
+               goto out;
+       }
+
+       ufs->mclk_rate = clk_get_rate(ufs->clk_unipro_main);
Some if checks for the clocks can be given as ufs->clk_unipro_main is
undefined for some cases.
+       pclk_rate = clk_get_rate(ufs->clk_apb);
Same as above.
we are tracing through a list of clocks above, if some clocks are not available it will return from above.
+       f_min = ufs->pclk_avail_min;
+       f_max = ufs->pclk_avail_max;
+
+       if (ufs->opts & EXYNOS_UFS_OPT_HAS_APB_CLK_CTRL) {
Some cases can occur where pclk_rate can be 0. From the code looks
like only pclk or ufs->clk_apb is must. Check the purpose of other
clocks.
other clocks are needed for unipro, see exynos_ufs_config_unipro().
+               do {
+                       pclk_rate /= (div + 1);
+
+                       if (pclk_rate <= f_max)
+                               break;
+                       div++;
+               } while (pclk_rate >= f_min);
+       }
+
+       if (unlikely(pclk_rate < f_min || pclk_rate > f_max)) {
+               dev_err(hba->dev, "not available pclk range %d\n", pclk_rate);
+               ret = -EINVAL;
+               goto out;
+       }
+
+       ufs->pclk_rate = pclk_rate;
+       ufs->pclk_div = div;
+
+out:
+       return ret;
+}
+
+static void exynos_ufs_set_unipro_pclk_div(struct exynos_ufs *ufs)
+{
+       if (ufs->opts & EXYNOS_UFS_OPT_HAS_APB_CLK_CTRL) {
+               u32 val;
+
+               val = hci_readl(ufs, HCI_UNIPRO_APB_CLK_CTRL);
+               hci_writel(ufs, UNIPRO_APB_CLK(val, ufs->pclk_div),
+                          HCI_UNIPRO_APB_CLK_CTRL);
+       }
+}
+
+static void exynos_ufs_set_pwm_clk_div(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
+
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(CMN_PWM_CLK_CTRL), attr->cmn_pwm_clk_ctrl);
+}
+
+static void exynos_ufs_calc_pwm_clk_div(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
+       const unsigned int div = 30, mult = 20;
+       const unsigned long pwm_min = 3 * 1000 * 1000;
+       const unsigned long pwm_max = 9 * 1000 * 1000;
+       const int divs[] = {32, 16, 8, 4};
+       unsigned long clk = 0, _clk, clk_period;
+       int i = 0, clk_idx = -1;
+
+       clk_period = UNIPRO_PCLK_PERIOD(ufs);
+       for (i = 0; i < ARRAY_SIZE(divs); i++) {
+               _clk = NSEC_PER_SEC * mult / (clk_period * divs[i] * div);
+               if (_clk >= pwm_min && _clk <= pwm_max) {
+                       if (_clk > clk) {
+                               clk_idx = i;
+                               clk = _clk;
+                       }
+               }
+       }
+
+       if (clk_idx == -1) {
+               ufshcd_dme_get(hba, UIC_ARG_MIB(CMN_PWM_CLK_CTRL), &clk_idx);
+               dev_err(hba->dev,
+                       "failed to decide pwm clock divider, will not change\n");
+       }
+
+       attr->cmn_pwm_clk_ctrl = clk_idx & PWM_CLK_CTRL_MASK;
+}
+
+long exynos_ufs_calc_time_cntr(struct exynos_ufs *ufs, long period)
+{
+       const int precise = 10;
+       long pclk_rate = ufs->pclk_rate;
+       long clk_period, fraction;
+
+       clk_period = UNIPRO_PCLK_PERIOD(ufs);
+       fraction = ((NSEC_PER_SEC % pclk_rate) * precise) / pclk_rate;
+
+       return (period * precise) / ((clk_period * precise) + fraction);
+}
+
+static void exynos_ufs_specify_phy_time_attr(struct exynos_ufs *ufs)
+{
+       struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
+       struct ufs_phy_time_cfg *t_cfg = &ufs->t_cfg;
+
+       t_cfg->tx_linereset_p =
+               exynos_ufs_calc_time_cntr(ufs, attr->tx_dif_p_nsec);
+       t_cfg->tx_linereset_n =
+               exynos_ufs_calc_time_cntr(ufs, attr->tx_dif_n_nsec);
+       t_cfg->tx_high_z_cnt =
+               exynos_ufs_calc_time_cntr(ufs, attr->tx_high_z_cnt_nsec);
+       t_cfg->tx_base_n_val =
+               exynos_ufs_calc_time_cntr(ufs, attr->tx_base_unit_nsec);
+       t_cfg->tx_gran_n_val =
+               exynos_ufs_calc_time_cntr(ufs, attr->tx_gran_unit_nsec);
+       t_cfg->tx_sleep_cnt =
+               exynos_ufs_calc_time_cntr(ufs, attr->tx_sleep_cnt);
+
+       t_cfg->rx_linereset =
+               exynos_ufs_calc_time_cntr(ufs, attr->rx_dif_p_nsec);
+       t_cfg->rx_hibern8_wait =
+               exynos_ufs_calc_time_cntr(ufs, attr->rx_hibern8_wait_nsec);
+       t_cfg->rx_base_n_val =
+               exynos_ufs_calc_time_cntr(ufs, attr->rx_base_unit_nsec);
+       t_cfg->rx_gran_n_val =
+               exynos_ufs_calc_time_cntr(ufs, attr->rx_gran_unit_nsec);
+       t_cfg->rx_sleep_cnt =
+               exynos_ufs_calc_time_cntr(ufs, attr->rx_sleep_cnt);
+       t_cfg->rx_stall_cnt =
+               exynos_ufs_calc_time_cntr(ufs, attr->rx_stall_cnt);
+}
+
+static void exynos_ufs_config_phy_time_attr(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       struct ufs_phy_time_cfg *t_cfg = &ufs->t_cfg;
+       int i;
+
+       exynos_ufs_set_pwm_clk_div(ufs);
+
+       exynos_ufs_enable_ov_tm(hba);
+
+       for_each_ufs_rx_lane(ufs, i) {
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_FILLER_ENABLE, i),
+                               ufs->drv_data->uic_attr->rx_filler_enable);
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_LINERESET_VAL, i),
+                               RX_LINERESET(t_cfg->rx_linereset));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_BASE_NVAL_07_00, i),
+                               RX_BASE_NVAL_L(t_cfg->rx_base_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_BASE_NVAL_15_08, i),
+                               RX_BASE_NVAL_H(t_cfg->rx_base_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_GRAN_NVAL_07_00, i),
+                               RX_GRAN_NVAL_L(t_cfg->rx_gran_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_GRAN_NVAL_10_08, i),
+                               RX_GRAN_NVAL_H(t_cfg->rx_gran_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_OV_SLEEP_CNT_TIMER, i),
+                               RX_OV_SLEEP_CNT(t_cfg->rx_sleep_cnt));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(RX_OV_STALL_CNT_TIMER, i),
+                               RX_OV_STALL_CNT(t_cfg->rx_stall_cnt));
+       }
+
+       for_each_ufs_tx_lane(ufs, i) {
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_LINERESET_P_VAL, i),
+                               TX_LINERESET_P(t_cfg->tx_linereset_p));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_HIGH_Z_CNT_07_00, i),
+                               TX_HIGH_Z_CNT_L(t_cfg->tx_high_z_cnt));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_HIGH_Z_CNT_11_08, i),
+                               TX_HIGH_Z_CNT_H(t_cfg->tx_high_z_cnt));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_BASE_NVAL_07_00, i),
+                               TX_BASE_NVAL_L(t_cfg->tx_base_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_BASE_NVAL_15_08, i),
+                               TX_BASE_NVAL_H(t_cfg->tx_base_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_GRAN_NVAL_07_00, i),
+                               TX_GRAN_NVAL_L(t_cfg->tx_gran_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_GRAN_NVAL_10_08, i),
+                               TX_GRAN_NVAL_H(t_cfg->tx_gran_n_val));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_OV_SLEEP_CNT_TIMER, i),
+                               TX_OV_H8_ENTER_EN |
+                               TX_OV_SLEEP_CNT(t_cfg->tx_sleep_cnt));
+               ufshcd_dme_set(hba, UIC_ARG_MIB_SEL(TX_MIN_ACTIVATETIME, i),
+                               ufs->drv_data->uic_attr->tx_min_activatetime);
+       }
+
+       exynos_ufs_disable_ov_tm(hba);
+}
+
+static void exynos_ufs_config_phy_cap_attr(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
+       int i;
+
+       exynos_ufs_enable_ov_tm(hba);
+
+       for_each_ufs_rx_lane(ufs, i) {
+               ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_HS_G1_SYNC_LENGTH_CAP, i),
+                               attr->rx_hs_g1_sync_len_cap);
+               ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_HS_G2_SYNC_LENGTH_CAP, i),
+                               attr->rx_hs_g2_sync_len_cap);
+               ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_HS_G3_SYNC_LENGTH_CAP, i),
+                               attr->rx_hs_g3_sync_len_cap);
+               ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_HS_G1_PREP_LENGTH_CAP, i),
+                               attr->rx_hs_g1_prep_sync_len_cap);
+               ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_HS_G2_PREP_LENGTH_CAP, i),
+                               attr->rx_hs_g2_prep_sync_len_cap);
+               ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_HS_G3_PREP_LENGTH_CAP, i),
+                               attr->rx_hs_g3_prep_sync_len_cap);
+       }
+
+       if (attr->rx_adv_fine_gran_sup_en == 0) {
+               for_each_ufs_rx_lane(ufs, i) {
+                       ufshcd_dme_set(hba,
+                               UIC_ARG_MIB_SEL(RX_ADV_GRANULARITY_CAP, i), 0);
+
+                       if (attr->rx_min_actv_time_cap)
+                               ufshcd_dme_set(hba,
+                                       UIC_ARG_MIB_SEL(RX_MIN_ACTIVATETIME_CAP, i),
+                                       attr->rx_min_actv_time_cap);
+
+                       if (attr->rx_hibern8_time_cap)
+                               ufshcd_dme_set(hba,
+                                       UIC_ARG_MIB_SEL(RX_HIBERN8TIME_CAP, i),
+                                       attr->rx_hibern8_time_cap);
+               }
+       } else if (attr->rx_adv_fine_gran_sup_en == 1) {
+               for_each_ufs_rx_lane(ufs, i) {
+                       if (attr->rx_adv_fine_gran_step)
+                               ufshcd_dme_set(hba,
+                                       UIC_ARG_MIB_SEL(RX_ADV_GRANULARITY_CAP, i),
+                                       RX_ADV_FINE_GRAN_STEP(attr->rx_adv_fine_gran_step));
+
+                       if (attr->rx_adv_min_actv_time_cap)
+                               ufshcd_dme_set(hba,
+                                       UIC_ARG_MIB_SEL(RX_ADV_MIN_ACTIVATETIME_CAP, i),
+                                       attr->rx_adv_min_actv_time_cap);
+
+                       if (attr->rx_adv_hibern8_time_cap)
+                               ufshcd_dme_set(hba,
+                                       UIC_ARG_MIB_SEL(RX_ADV_HIBERN8TIME_CAP, i),
+                                       attr->rx_adv_hibern8_time_cap);
+               }
+       }
+
+       exynos_ufs_disable_ov_tm(hba);
+}
+
+static void exynos_ufs_establish_connt(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+       enum {
+               DEV_ID          = 0x00,
+               PEER_DEV_ID     = 0x01,
+               PEER_CPORT_ID   = 0x00,
+               TRAFFIC_CLASS   = 0x00,
+       };
+
+       /* allow cport attributes to be set */
+       ufshcd_dme_set(hba, UIC_ARG_MIB(T_CONNECTIONSTATE), CPORT_IDLE);
+
+       /* local unipro attributes */
+       ufshcd_dme_set(hba, UIC_ARG_MIB(N_DEVICEID), DEV_ID);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(N_DEVICEID_VALID), TRUE);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(T_PEERDEVICEID), PEER_DEV_ID);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(T_PEERCPORTID), PEER_CPORT_ID);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(T_CPORTFLAGS), CPORT_DEF_FLAGS);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(T_TRAFFICCLASS), TRAFFIC_CLASS);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(T_CONNECTIONSTATE), CPORT_CONNECTED);
+}
+
+static void exynos_ufs_config_smu(struct exynos_ufs *ufs)
+{
+       u32 reg, val;
+
+       exynos_ufs_disable_auto_ctrl_hcc_save(ufs, &val);
+
+       /* make encryption disabled by default */
+       reg = ufsp_readl(ufs, UFSPRSECURITY);
+       ufsp_writel(ufs, reg | NSSMU, UFSPRSECURITY);
+       ufsp_writel(ufs, 0x0, UFSPSBEGIN0);
+       ufsp_writel(ufs, 0xffffffff, UFSPSEND0);
+       ufsp_writel(ufs, 0xff, UFSPSLUN0);
+       ufsp_writel(ufs, 0xf1, UFSPSCTRL0);
+
+       exynos_ufs_auto_ctrl_hcc_restore(ufs, &val);
+}
+
+static void exynos_ufs_config_sync_pattern_mask(struct exynos_ufs *ufs,
+                                       struct uic_pwr_mode *pwr)
+{
+       struct ufs_hba *hba = ufs->hba;
+       u8 g = pwr->gear;
+       u32 mask, sync_len;
+       enum {
+               SYNC_LEN_G1 = 80 * 1000, /* 80us */
+               SYNC_LEN_G2 = 40 * 1000, /* 44us */
+               SYNC_LEN_G3 = 20 * 1000, /* 20us */
+       };
+       int i;
+
+       if (g == 1)
+               sync_len = SYNC_LEN_G1;
+       else if (g == 2)
+               sync_len = SYNC_LEN_G2;
+       else if (g == 3)
+               sync_len = SYNC_LEN_G3;
+       else
+               return;
+
+       mask = exynos_ufs_calc_time_cntr(ufs, sync_len);
+       mask = (mask >> 8) & 0xff;
+
+       exynos_ufs_enable_ov_tm(hba);
+
+       for_each_ufs_rx_lane(ufs, i)
+               ufshcd_dme_set(hba,
+                       UIC_ARG_MIB_SEL(RX_SYNC_MASK_LENGTH, i), mask);
+
+       exynos_ufs_disable_ov_tm(hba);
+}
+
+static int exynos_ufs_pre_pwr_mode(struct ufs_hba *hba,
+                               struct ufs_pa_layer_attr *pwr_max,
+                               struct ufs_pa_layer_attr *final)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       struct uic_pwr_mode *pwr_req = &ufs->pwr_req;
+       struct uic_pwr_mode *pwr_act = &ufs->pwr_act;
+
+       final->gear_rx
+               = pwr_act->gear = min_t(u8, pwr_max->gear_rx, pwr_req->gear);
+       final->gear_tx
+               = pwr_act->gear = min_t(u8, pwr_max->gear_tx, pwr_req->gear);
+       final->lane_rx
+               = pwr_act->lane = min_t(u8, pwr_max->lane_rx, pwr_req->lane);
+       final->lane_tx
+               = pwr_act->lane = min_t(u8, pwr_max->lane_tx, pwr_req->lane);
+       final->pwr_rx = pwr_act->mode = pwr_req->mode;
+       final->pwr_tx = pwr_act->mode = pwr_req->mode;
+       final->hs_rate = pwr_act->hs_series = pwr_req->hs_series;
+
+       /* save and configure l2 timer */
+       pwr_act->l_l2_timer[0] = pwr_req->l_l2_timer[0];
+       pwr_act->l_l2_timer[1] = pwr_req->l_l2_timer[1];
+       pwr_act->l_l2_timer[2] = pwr_req->l_l2_timer[2];
+       pwr_act->r_l2_timer[0] = pwr_req->r_l2_timer[0];
+       pwr_act->r_l2_timer[1] = pwr_req->r_l2_timer[1];
+       pwr_act->r_l2_timer[2] = pwr_req->r_l2_timer[2];
+
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(DL_FC0PROTTIMEOUTVAL), pwr_act->l_l2_timer[0]);
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(DL_TC0REPLAYTIMEOUTVAL), pwr_act->l_l2_timer[1]);
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(DL_AFC0REQTIMEOUTVAL), pwr_act->l_l2_timer[2]);
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(PA_PWRMODEUSERDATA0), pwr_act->r_l2_timer[0]);
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(PA_PWRMODEUSERDATA1), pwr_act->r_l2_timer[1]);
+       ufshcd_dme_set(hba,
+               UIC_ARG_MIB(PA_PWRMODEUSERDATA2), pwr_act->r_l2_timer[2]);
+
+       unipro_writel(ufs,
+               pwr_act->l_l2_timer[0], UNIPRO_DME_PWR_REQ_LOCALL2TIMER0);
+       unipro_writel(ufs,
+               pwr_act->l_l2_timer[1], UNIPRO_DME_PWR_REQ_LOCALL2TIMER1);
+       unipro_writel(ufs,
+               pwr_act->l_l2_timer[2], UNIPRO_DME_PWR_REQ_LOCALL2TIMER2);
+       unipro_writel(ufs,
+               pwr_act->r_l2_timer[0], UNIPRO_DME_PWR_REQ_REMOTEL2TIMER0);
+       unipro_writel(ufs,
+               pwr_act->r_l2_timer[1], UNIPRO_DME_PWR_REQ_REMOTEL2TIMER1);
+       unipro_writel(ufs,
+               pwr_act->r_l2_timer[2], UNIPRO_DME_PWR_REQ_REMOTEL2TIMER2);
+
+       if (ufs->drv_data->pre_pwr_change)
+               ufs->drv_data->pre_pwr_change(ufs, pwr_act);
+
+       if (IS_UFS_PWR_MODE_HS(pwr_act->mode)) {
+               exynos_ufs_config_sync_pattern_mask(ufs, pwr_act);
+
+               switch (pwr_act->hs_series) {
+               case PA_HS_MODE_A:
+               case PA_HS_MODE_B:
+                       exynos_ufs_phy_calibrate_pre_pwr_hs(ufs->phy,
+                               PWR_MODE_HS(pwr_act->gear, pwr_act->hs_series));
+                       break;
+               }
+       }
+
+       return 0;
+}
+
+#define PWR_MODE_STR_LEN       64
+static int exynos_ufs_post_pwr_mode(struct ufs_hba *hba,
+                               struct ufs_pa_layer_attr *pwr_max,
+                               struct ufs_pa_layer_attr *pwr_req)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       struct uic_pwr_mode *pwr = &ufs->pwr_act;
+       char pwr_str[PWR_MODE_STR_LEN] = "";
+       int ret = 0;
+
+       if (ufs->drv_data->post_pwr_change)
+               ufs->drv_data->post_pwr_change(ufs, pwr);
+
+       if (IS_UFS_PWR_MODE_HS(pwr->mode)) {
+               switch (pwr->hs_series) {
+               case PA_HS_MODE_A:
+               case PA_HS_MODE_B:
+                       exynos_ufs_phy_calibrate_post_pwr_hs(ufs->phy,
+                               PWR_MODE_HS(pwr->gear, pwr->hs_series));
+                       break;
+               }
+
+               ret = exynos_ufs_phy_wait_for_lock_acq(ufs->phy);
+               snprintf(pwr_str, sizeof(pwr_str), "Fast%s series_%s G_%d L_%d",
+                       pwr->mode == FASTAUTO_MODE ? "_Auto" : "",
+                       pwr->hs_series == PA_HS_MODE_A ? "A" : "B",
+                       pwr->gear, pwr->lane);
+       } else if (IS_UFS_PWR_MODE_PWM(pwr->mode)) {
+               snprintf(pwr_str, sizeof(pwr_str), "Slow%s G_%d L_%d",
+                       pwr->mode == SLOWAUTO_MODE ? "_Auto" : "",
+                       pwr->gear, pwr->lane);
+       }
+
+       dev_info(hba->dev, "Power mode change %d : %s\n", ret, pwr_str);
+       return ret;
+}
+
+static void exynos_ufs_specify_nexus_t_xfer_req(struct ufs_hba *hba,
+                                               int tag, struct scsi_cmnd *cmd)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       u32 type;
+
+       type =  hci_readl(ufs, HCI_UTRL_NEXUS_TYPE);
+
+       if (cmd)
+               hci_writel(ufs, type | (1 << tag), HCI_UTRL_NEXUS_TYPE);
+       else
+               hci_writel(ufs, type & ~(1 << tag), HCI_UTRL_NEXUS_TYPE);
+}
+
+static void exynos_ufs_specify_nexus_t_tm_req(struct ufs_hba *hba,
+                                               int tag, u8 func)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       u32 type;
+
+       type =  hci_readl(ufs, HCI_UTMRL_NEXUS_TYPE);
+
+       switch (func) {
+       case UFS_ABORT_TASK:
+       case UFS_QUERY_TASK:
+               hci_writel(ufs, type | (1 << tag), HCI_UTMRL_NEXUS_TYPE);
+               break;
+       case UFS_ABORT_TASK_SET:
+       case UFS_CLEAR_TASK_SET:
+       case UFS_LOGICAL_RESET:
+       case UFS_QUERY_TASK_SET:
+               hci_writel(ufs, type & ~(1 << tag), HCI_UTMRL_NEXUS_TYPE);
+               break;
+       }
+}
+
+static void exynos_ufs_phy_init(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+
+       if (ufs->avail_ln_rx == 0 || ufs->avail_ln_tx == 0) {
+               ufshcd_dme_get(hba, UIC_ARG_MIB(PA_AVAILRXDATALANES),
+                       &ufs->avail_ln_rx);
+               ufshcd_dme_get(hba, UIC_ARG_MIB(PA_AVAILTXDATALANES),
+                       &ufs->avail_ln_tx);
+               WARN(ufs->avail_ln_rx != ufs->avail_ln_tx,
+                       "available data lane is not equal(rx:%d, tx:%d)\n",
+                       ufs->avail_ln_rx, ufs->avail_ln_tx);
+       }
+
+       exynos_ufs_phy_set_lane_cnt(ufs->phy, ufs->avail_ln_rx);
+       exynos_ufs_phy_calibrate_pre_init(ufs->phy);
+}
+
+static void exynos_ufs_config_unipro(struct exynos_ufs *ufs)
+{
+       struct ufs_hba *hba = ufs->hba;
+
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_CLK_PERIOD),
+               DIV_ROUND_UP(NSEC_PER_SEC, ufs->mclk_rate));
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_TXTRAILINGCLOCKS),
+                       ufs->drv_data->uic_attr->tx_trailingclks);
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OPTION_SUITE),
+                       ufs->drv_data->uic_attr->pa_dbg_option_suite);
+}
+
+static void exynos_ufs_config_intr(struct exynos_ufs *ufs, u32 errs, u8 index)
+{
+       switch (index) {
+       case UNIPRO_L1_5:
+               hci_writel(ufs, DFES_ERR_EN | errs, HCI_ERR_EN_PA_LAYER);
+               break;
+       case UNIPRO_L2:
+               hci_writel(ufs, DFES_ERR_EN | errs, HCI_ERR_EN_DL_LAYER);
+               break;
+       case UNIPRO_L3:
+               hci_writel(ufs, DFES_ERR_EN | errs, HCI_ERR_EN_N_LAYER);
+               break;
+       case UNIPRO_L4:
+               hci_writel(ufs, DFES_ERR_EN | errs, HCI_ERR_EN_T_LAYER);
+               break;
+       case UNIPRO_DME:
+               hci_writel(ufs, DFES_ERR_EN | errs, HCI_ERR_EN_DME_LAYER);
+               break;
+       }
+}
+
+static int exynos_ufs_pre_link(struct ufs_hba *hba)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+
+       /* hci */
+       exynos_ufs_config_intr(ufs, DFES_DEF_L2_ERRS, UNIPRO_L2);
+       exynos_ufs_config_intr(ufs, DFES_DEF_L3_ERRS, UNIPRO_L3);
+       exynos_ufs_config_intr(ufs, DFES_DEF_L4_ERRS, UNIPRO_L4);
+       exynos_ufs_set_unipro_pclk_div(ufs);
+
+       /* unipro */
+       exynos_ufs_config_unipro(ufs);
+
+       /* m-phy */
+       exynos_ufs_phy_init(ufs);
+       exynos_ufs_config_phy_time_attr(ufs);
+       exynos_ufs_config_phy_cap_attr(ufs);
+
+       if (ufs->drv_data->pre_link)
+               ufs->drv_data->pre_link(ufs);
+
+       return 0;
+}
+
+static void exynos_ufs_fit_aggr_timeout(struct exynos_ufs *ufs)
+{
+       const u8 cntr_div = 40;
+       u32 val;
+
+       val = exynos_ufs_calc_time_cntr(ufs, IATOVAL_NSEC / cntr_div);
+       hci_writel(ufs, val & CNT_VAL_1US_MASK, HCI_1US_TO_CNT_VAL);
+}
+
+static int exynos_ufs_post_link(struct ufs_hba *hba)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
+
+       exynos_ufs_establish_connt(ufs);
+       exynos_ufs_fit_aggr_timeout(ufs);
+
+       hci_writel(ufs, 0xa, HCI_DATA_REORDER);
+       hci_writel(ufs, PRDT_PREFECT_EN | PRDT_SET_SIZE(12),
+                       HCI_TXPRDT_ENTRY_SIZE);
+       hci_writel(ufs, PRDT_SET_SIZE(12), HCI_RXPRDT_ENTRY_SIZE);
+       hci_writel(ufs, (1 << hba->nutrs) - 1, HCI_UTRL_NEXUS_TYPE);
+       hci_writel(ufs, (1 << hba->nutmrs) - 1, HCI_UTMRL_NEXUS_TYPE);
+       hci_writel(ufs, 0xf, HCI_AXIDMA_RWDATA_BURST_LEN);
+
+       if (ufs->opts & EXYNOS_UFS_OPT_SKIP_CONNECTION_ESTAB)
+               ufshcd_dme_set(hba,
+                       UIC_ARG_MIB(T_DBG_SKIP_INIT_HIBERN8_EXIT), TRUE);
+
+       if (attr->pa_granularity) {
+               exynos_ufs_enable_dbg_mode(hba);
+               ufshcd_dme_set(hba, UIC_ARG_MIB(PA_GRANULARITY),
+                               attr->pa_granularity);
+               exynos_ufs_disable_dbg_mode(hba);
+
+               if (attr->pa_tactivate)
+                       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_TACTIVATE),
+                                       attr->pa_tactivate);
+               if (attr->pa_hibern8time &&
+                   !(ufs->opts & EXYNOS_UFS_OPT_USE_SW_HIBERN8_TIMER))
+                       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_HIBERN8TIME),
+                                       attr->pa_hibern8time);
+       }
+
+       if (ufs->opts & EXYNOS_UFS_OPT_USE_SW_HIBERN8_TIMER) {
+               if (!attr->pa_granularity)
+                       ufshcd_dme_get(hba, UIC_ARG_MIB(PA_GRANULARITY),
+                                       &attr->pa_granularity);
+               if (!attr->pa_hibern8time)
+                       ufshcd_dme_get(hba, UIC_ARG_MIB(PA_HIBERN8TIME),
+                                       &attr->pa_hibern8time);
+               /*
+                * not wait for HIBERN8 time to exit hibernation
+                */
+               ufshcd_dme_set(hba, UIC_ARG_MIB(PA_HIBERN8TIME), 0);
+
+               if (attr->pa_granularity < 1 || attr->pa_granularity > 6) {
+                       /* Valid range for granularity: 1 ~ 6 */
+                       dev_warn(hba->dev,
+                               "%s: pa_granularty %d is invalid, assuming backwards compatibility\n",
+                               __func__,
+                               attr->pa_granularity);
+                       attr->pa_granularity = 6;
+               }
+       }
+
+       exynos_ufs_phy_calibrate_post_init(ufs->phy);
+
+       if (ufs->drv_data->post_link)
+               ufs->drv_data->post_link(ufs);
+
+       return 0;
+}
+
+static void exynos_ufs_specify_pwr_mode(struct device_node *np,
+                               struct exynos_ufs *ufs)
+{
+       struct uic_pwr_mode *pwr = &ufs->pwr_req;
+       const char *str = NULL;
+
+       if (!of_property_read_string(np, "ufs,pwr-attr-mode", &str)) {
+               if (!strncmp(str, "FAST", sizeof("FAST")))
+                       pwr->mode = FAST_MODE;
+               else if (!strncmp(str, "SLOW", sizeof("SLOW")))
+                       pwr->mode = SLOW_MODE;
+               else if (!strncmp(str, "FAST_auto", sizeof("FAST_auto")))
+                       pwr->mode = FASTAUTO_MODE;
+               else if (!strncmp(str, "SLOW_auto", sizeof("SLOW_auto")))
+                       pwr->mode = SLOWAUTO_MODE;
+               else
+                       pwr->mode = FAST_MODE;
+       } else {
+               pwr->mode = FAST_MODE;
+       }
+
+       if (of_property_read_u8(np, "ufs,pwr-attr-lane", &pwr->lane))
+               pwr->lane = 1;
+
+       if (of_property_read_u8(np, "ufs,pwr-attr-gear", &pwr->gear))
+               pwr->gear = 1;
+
+       if (IS_UFS_PWR_MODE_HS(pwr->mode)) {
+               if (!of_property_read_string(np,
+                                       "ufs,pwr-attr-hs-series", &str)) {
+                       if (!strncmp(str, "HS_rate_b", sizeof("HS_rate_b")))
+                               pwr->hs_series = PA_HS_MODE_B;
+                       else if (!strncmp(str, "HS_rate_a", sizeof("HS_rate_a")))
+                               pwr->hs_series = PA_HS_MODE_A;
+                       else
+                               pwr->hs_series = PA_HS_MODE_A;
+               } else {
+                       pwr->hs_series = PA_HS_MODE_A;
+               }
+       }
+
+       if (of_property_read_u32_array(
+               np, "ufs,pwr-local-l2-timer", pwr->l_l2_timer, 3)) {
+               pwr->l_l2_timer[0] = FC0PROTTIMEOUTVAL;
+               pwr->l_l2_timer[1] = TC0REPLAYTIMEOUTVAL;
+               pwr->l_l2_timer[2] = AFC0REQTIMEOUTVAL;
+       }
+
+       if (of_property_read_u32_array(
+               np, "ufs,pwr-remote-l2-timer", pwr->r_l2_timer, 3)) {
+               pwr->r_l2_timer[0] = FC0PROTTIMEOUTVAL;
+               pwr->r_l2_timer[1] = TC0REPLAYTIMEOUTVAL;
+               pwr->r_l2_timer[2] = AFC0REQTIMEOUTVAL;
+       }
+}
+
+static int exynos_ufs_parse_dt(struct device *dev, struct exynos_ufs *ufs)
+{
+       struct device_node *np = dev->of_node;
+       struct exynos_ufs_drv_data *drv_data = exynos_ufs_drvs;
+       struct exynos_ufs_uic_attr *attr;
+       u32 freq[2];
+       int ret;
+
+       while (drv_data->compatible) {
+               if (of_device_is_compatible(np, drv_data->compatible)) {
+                       ufs->drv_data = drv_data;
+                       break;
+               }
+               drv_data++;
+       }
+
+       if (ufs->drv_data && ufs->drv_data->uic_attr) {
+               attr = ufs->drv_data->uic_attr;
+       } else {
+               dev_err(dev, "failed to get uic attributes\n");
+               ret = -EINVAL;
+               goto out;
+       }
+
+       ret = of_property_read_u32_array(np,
+                       "pclk-freq-avail-range", freq, ARRAY_SIZE(freq));
+       if (!ret) {
+               ufs->pclk_avail_min = freq[0];
+               ufs->pclk_avail_max = freq[1];
+       } else {
+               dev_err(dev, "failed to get available pclk range\n");
+               goto out;
+       }
+
+       exynos_ufs_specify_pwr_mode(np, ufs);
+
+       if (!of_property_read_u32(np, "ufs-rx-adv-fine-gran-sup_en",
of_property_read_bool can be used instead.
+                               &attr->rx_adv_fine_gran_sup_en)) {
+               if (attr->rx_adv_fine_gran_sup_en == 0) {
+                       /* 100us step */
+                       if (of_property_read_u32(np,
+                                       "ufs-rx-min-activate-time-cap",
+                                       &attr->rx_min_actv_time_cap))
+                               dev_warn(dev,
+                                       "ufs-rx-min-activate-time-cap is empty\n");
+
+                       if (of_property_read_u32(np,
+                                       "ufs-rx-hibern8-time-cap",
+                                       &attr->rx_hibern8_time_cap))
+                               dev_warn(dev,
+                                       "ufs-rx-hibern8-time-cap is empty\n");
+               } else if (attr->rx_adv_fine_gran_sup_en == 1) {
+                       /* fine granularity step */
+                       if (of_property_read_u32(np,
+                                       "ufs-rx-adv-fine-gran-step",
+                                       &attr->rx_adv_fine_gran_step))
+                               dev_warn(dev,
+                                       "ufs-rx-adv-fine-gran-step is empty\n");
+
+                       if (of_property_read_u32(np,
+                                       "ufs-rx-adv-min-activate-time-cap",
+                                       &attr->rx_adv_min_actv_time_cap))
+                               dev_warn(dev,
+                                       "ufs-rx-adv-min-activate-time-cap is empty\n");
+
+                       if (of_property_read_u32(np,
+                                       "ufs-rx-adv-hibern8-time-cap",
+                                       &attr->rx_adv_hibern8_time_cap))
+                               dev_warn(dev,
+                                       "ufs-rx-adv-hibern8-time-cap is empty\n");
+               } else {
+                       dev_warn(dev,
+                               "not supported val for ufs-rx-adv-fine-gran-sup_en %d\n",
+                               attr->rx_adv_fine_gran_sup_en);
+               }
+       } else {
+               attr->rx_adv_fine_gran_sup_en = 0xf;
+       }
+
+       if (!of_property_read_u32(np,
+                               "ufs-pa-granularity", &attr->pa_granularity)) {
+               if (of_property_read_u32(np,
+                               "ufs-pa-tacctivate", &attr->pa_tactivate))
+                       dev_warn(dev, "ufs-pa-tacctivate is empty\n");
+
+               if (of_property_read_u32(np,
+                               "ufs-pa-hibern8time", &attr->pa_hibern8time))
+                       dev_warn(dev, "ufs-pa-hibern8time is empty\n");
+       }
+
+out:
+       return ret;
+}
+
+static int exynos_ufs_init(struct ufs_hba *hba)
+{
+       struct device *dev = hba->dev;
+       struct platform_device *pdev = to_platform_device(dev);
+       struct exynos_ufs *ufs;
+       struct resource *res;
+       int ret;
+
+       ufs = devm_kzalloc(dev, sizeof(*ufs), GFP_KERNEL);
+       if (!ufs)
+               return -ENOMEM;
+
+       /* exynos-specific hci */
+       res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "vs_hci");
+       ufs->reg_hci = devm_ioremap_resource(dev, res);
+       if (!ufs->reg_hci) {
+               dev_err(dev, "cannot ioremap for hci vendor register\n");
+               return -ENOMEM;
+       }
+
+       /* unipro */
+       res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "unipro");
+       ufs->reg_unipro = devm_ioremap_resource(dev, res);
+       if (!ufs->reg_unipro) {
+               dev_err(dev, "cannot ioremap for unipro register\n");
+               return -ENOMEM;
+       }
+
+       /* ufs protector */
+       res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "ufsp");
+       ufs->reg_ufsp = devm_ioremap_resource(dev, res);
+       if (!ufs->reg_ufsp) {
+               dev_err(dev, "cannot ioremap for ufs protector register\n");
+               return -ENOMEM;
+       }
+
+       ret = exynos_ufs_parse_dt(dev, ufs);
+       if (ret) {
+               dev_err(dev, "failed to get dt info.\n");
+               goto out;
+       }
+
+       ufs->phy = devm_phy_get(dev, "ufs-phy");
+       if (IS_ERR(ufs->phy)) {
+               ret = PTR_ERR(ufs->phy);
+               dev_err(dev, "failed to get ufs-phy\n");
+               goto out;
+       }
+
+       phy_init(ufs->phy);
+       ret = phy_power_on(ufs->phy);
+       if (ret)
+               goto phy_exit;
+
+       ufs->hba = hba;
+       ufs->opts = ufs->drv_data->opts |
+               EXYNOS_UFS_OPT_SKIP_CONNECTION_ESTAB |
+               EXYNOS_UFS_OPT_USE_SW_HIBERN8_TIMER;
+       ufs->rx_sel_idx = PA_MAXDATALANES;
+       if (ufs->opts & EXYNOS_UFS_OPT_BROKEN_RX_SEL_IDX)
+               ufs->rx_sel_idx = 0;
+       hba->priv = (void *)ufs;
+       hba->quirks = ufs->drv_data->quirks;
+       if (ufs->drv_data->drv_init) {
+               ret = ufs->drv_data->drv_init(dev, ufs);
+               if (ret) {
+                       dev_err(dev, "failed to init drv-data\n");
+                       goto phy_off;
+               }
+       }
+
+       ret = exynos_ufs_get_clk_info(ufs);
+       if (ret)
+               goto phy_off;
+       exynos_ufs_specify_phy_time_attr(ufs);
+       exynos_ufs_config_smu(ufs);
+       return 0;
+
+phy_off:
+       phy_power_off(ufs->phy);
+phy_exit:
+       phy_exit(ufs->phy);
+       hba->priv = NULL;
+out:
+       return ret;
+}
+
+static int exynos_ufs_host_reset(struct ufs_hba *hba)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       unsigned long timeout = jiffies + msecs_to_jiffies(1);
+       u32 val;
+       int ret = 0;
+
+       exynos_ufs_disable_auto_ctrl_hcc_save(ufs, &val);
+
+       hci_writel(ufs, UFS_SW_RST_MASK, HCI_SW_RST);
+
+       do {
+               if (!(hci_readl(ufs, HCI_SW_RST) & UFS_SW_RST_MASK))
+                       goto out;
+       } while (time_before(jiffies, timeout));
+
+       dev_err(hba->dev, "timeout host sw-reset\n");
+       ret = -ETIMEDOUT;
+
+out:
+       exynos_ufs_auto_ctrl_hcc_restore(ufs, &val);
+       return ret;
+}
+
+static void exynos_ufs_dev_hw_reset(struct ufs_hba *hba)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+
+       hci_writel(ufs, 0 << 0, HCI_GPIO_OUT);
+       udelay(5);
+       hci_writel(ufs, 1 << 0, HCI_GPIO_OUT);
+}
+
+static void exynos_ufs_pre_hibern8(struct ufs_hba *hba, u8 enter)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
+
+       if (!enter) {
+               if (ufs->opts & EXYNOS_UFS_OPT_BROKEN_AUTO_CLK_CTRL)
+                       exynos_ufs_disable_auto_ctrl_hcc(ufs);
+               exynos_ufs_ungate_clks(ufs);
+
+               if (ufs->opts & EXYNOS_UFS_OPT_USE_SW_HIBERN8_TIMER) {
+                       const unsigned int granularity_tbl[] = {
+                               1, 4, 8, 16, 32, 100
+                       };
+                       int h8_time = attr->pa_hibern8time *
+                               granularity_tbl[attr->pa_granularity - 1];
+                       unsigned long us;
+                       s64 delta;
+
+                       do {
+                               delta = h8_time - ktime_us_delta(ktime_get(),
+                                                       ufs->entry_hibern8_t);
+                               if (delta <= 0)
+                                       break;
+
+                               us = min_t(s64, delta, USEC_PER_MSEC);
+                               if (us >= 10)
+                                       usleep_range(us, us + 10);
+                               else
+                                       udelay(us);
+                       } while (1);
+               }
+       }
+}
+
+static void exynos_ufs_post_hibern8(struct ufs_hba *hba, u8 enter)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+
+       if (!enter) {
+               struct uic_pwr_mode *pwr = &ufs->pwr_act;
+               u32 mode = 0;
+
+               ufshcd_dme_get(hba, UIC_ARG_MIB(PA_PWRMODE), &mode);
+               if (mode != (pwr->mode << 4 | pwr->mode)) {
+                       dev_warn(hba->dev, "%s: power mode change\n", __func__);
+                       hba->pwr_info.pwr_rx = (mode >> 4) & 0xf;
+                       hba->pwr_info.pwr_tx = mode & 0xf;
+                       ufshcd_config_pwr_mode(hba, &hba->max_pwr_info.info);
+               }
+
+               if (!(ufs->opts & EXYNOS_UFS_OPT_SKIP_CONNECTION_ESTAB))
+                       exynos_ufs_establish_connt(ufs);
+       } else {
+               ufs->entry_hibern8_t = ktime_get();
+               exynos_ufs_gate_clks(ufs);
+               if (ufs->opts & EXYNOS_UFS_OPT_BROKEN_AUTO_CLK_CTRL)
+                       exynos_ufs_enable_auto_ctrl_hcc(ufs);
+       }
+}
+
+static int exynos_ufs_hce_enable_notify(struct ufs_hba *hba, bool notify)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+       int ret = 0;
+
+       switch ((u8)notify) {
+       case PRE_CHANGE:
+               ret = exynos_ufs_host_reset(hba);
+               if (ret)
+                       return ret;
+               exynos_ufs_dev_hw_reset(hba);
+               break;
+       case POST_CHANGE:
+               exynos_ufs_calc_pwm_clk_div(ufs);
+               if (!(ufs->opts & EXYNOS_UFS_OPT_BROKEN_AUTO_CLK_CTRL))
+                       exynos_ufs_enable_auto_ctrl_hcc(ufs);
+               break;
+       }
+
+       return ret;
+}
+
+static int exynos_ufs_link_startup_notify(struct ufs_hba *hba, bool notify)
+{
+       int ret = 0;
+
+       switch ((u8)notify) {
+       case PRE_CHANGE:
+               ret = exynos_ufs_pre_link(hba);
+               break;
+       case POST_CHANGE:
+               ret = exynos_ufs_post_link(hba);
+               break;
+       }
+
+       return ret;
+}
+
+static int exynos_ufs_pwr_change_notify(struct ufs_hba *hba, bool notify,
+                                       struct ufs_pa_layer_attr *pwr_max,
+                                       struct ufs_pa_layer_attr *pwr_req)
+{
+       int ret = 0;
+
+       switch ((u8)notify) {
+       case PRE_CHANGE:
+               ret = exynos_ufs_pre_pwr_mode(hba, pwr_max, pwr_req);
+               break;
+       case POST_CHANGE:
+               ret = exynos_ufs_post_pwr_mode(hba, NULL, pwr_req);
+               break;
+       }
+
+       return ret;
+}
+
+static int exynos_ufs_hibern8_notify(struct ufs_hba *hba,
+                                    bool enter, bool notify)
+{
+       switch ((u8)notify) {
+       case PRE_CHANGE:
+               exynos_ufs_pre_hibern8(hba, enter);
+               break;
+       case POST_CHANGE:
+               exynos_ufs_post_hibern8(hba, enter);
+               break;
+       }
+
+       return 0;
+}
+
+static int exynos_ufs_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+
+       if (!ufshcd_is_link_active(hba))
+               phy_power_off(ufs->phy);
+
+       return 0;
+}
+
+static int exynos_ufs_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op)
+{
+       struct exynos_ufs *ufs = to_exynos_ufs(hba);
+
+       if (!ufshcd_is_link_active(hba))
+               phy_power_on(ufs->phy);
+
+       exynos_ufs_config_smu(ufs);
+
+       return 0;
+}
+
+const struct ufs_hba_variant_ops ufs_hba_exynos_ops = {
+       .name                           = "exynos_ufs",
+       .init                           = exynos_ufs_init,
+       .hce_enable_notify              = exynos_ufs_hce_enable_notify,
+       .link_startup_notify            = exynos_ufs_link_startup_notify,
+       .pwr_change_notify              = exynos_ufs_pwr_change_notify,
+       .specify_nexus_t_xfer_req       = exynos_ufs_specify_nexus_t_xfer_req,
+       .specify_nexus_t_tm_req         = exynos_ufs_specify_nexus_t_tm_req,
+       .hibern8_notify                 = exynos_ufs_hibern8_notify,
+       .suspend                        = exynos_ufs_suspend,
+       .resume                         = exynos_ufs_resume,
+};
I suppose this ops should be registered as platform data inside the
ufshcd-pltfrm.c file, which i could not found so this driver will not
get invoked. Please add this in the next iteration.

Ok will add this as a seperate patch and include in this series.
After all these changes you can add for the whole series.
Reviewed by: Amit Daniel Kachhap <amit.daniel@xxxxxxxxxxx>

Thanks!!
Thanks,
Amit
+EXPORT_SYMBOL(ufs_hba_exynos_ops);
diff --git a/drivers/scsi/ufs/ufs-exynos.h b/drivers/scsi/ufs/ufs-exynos.h
new file mode 100644
index 0000000..58aa714
--- /dev/null
+++ b/drivers/scsi/ufs/ufs-exynos.h
@@ -0,0 +1,463 @@
+/*
+ * UFS Host Controller driver for Exynos specific extensions
+ *
+ * Copyright (C) 2014-2015 Samsung Electronics Co., Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#ifndef _UFS_EXYNOS_H_
+#define _UFS_EXYNOS_H_
+
+/*
+ * Exynos's Vendor specific registers for UFSHCI
+ */
+#define HCI_TXPRDT_ENTRY_SIZE  0x00
+#define  PRDT_PREFECT_EN       BIT(31)
+#define  PRDT_SET_SIZE(x)      ((x) & 0x1F)
+#define HCI_RXPRDT_ENTRY_SIZE  0x04
+#define HCI_TO_CNT_DIV_VAL     0x08
+#define HCI_1US_TO_CNT_VAL     0x0C
+#define         CNT_VAL_1US_MASK       0x3FF
+#define HCI_INVALID_UPIU_CTRL  0x10
+#define HCI_INVALID_UPIU_BADDR 0x14
+#define HCI_INVALID_UPIU_UBADDR        0x18
+#define HCI_INVALID_UTMR_OFFSET_ADDR   0x1C
+#define HCI_INVALID_UTR_OFFSET_ADDR    0x20
+#define HCI_INVALID_DIN_OFFSET_ADDR    0x24
+#define HCI_DBR_TIMER_CONFIG           0x28
+#define HCI_DBR_TIMER_STATUS   0x2C
+#define HCI_VENDOR_SPECIFIC_IS 0x38
+#define HCI_VENDOR_SPECIFIC_IE 0x3C
+#define HCI_UTRL_NEXUS_TYPE    0x40
+#define HCI_UTMRL_NEXUS_TYPE   0x44
+#define HCI_E2EFC_CTRL         0x48
+#define HCI_SW_RST             0x50
+#define  UFS_LINK_SW_RST       BIT(0)
+#define  UFS_UNIPRO_SW_RST     BIT(1)
+#define  UFS_SW_RST_MASK       (UFS_UNIPRO_SW_RST | UFS_LINK_SW_RST)
+#define HCI_LINK_VERSION       0x54
+#define HCI_IDLE_TIMER_CONFIG  0x58
+#define HCI_RX_UPIU_MATCH_ERROR_CODE   0x5C
+#define HCI_DATA_REORDER       0x60
+#define HCI_MAX_DOUT_DATA_SIZE 0x64
+#define HCI_UNIPRO_APB_CLK_CTRL        0x68
+#define  UNIPRO_APB_CLK(v, x)  (((v) & ~0xF) | ((x) & 0xF))
+#define HCI_AXIDMA_RWDATA_BURST_LEN    0x6C
+#define HCI_GPIO_OUT           0x70
+#define HCI_WRITE_DMA_CTRL     0x74
+#define HCI_ERR_EN_PA_LAYER    0x78
+#define HCI_ERR_EN_DL_LAYER    0x7C
+#define HCI_ERR_EN_N_LAYER     0x80
+#define HCI_ERR_EN_T_LAYER     0x84
+#define HCI_ERR_EN_DME_LAYER   0x88
+#define HCI_REQ_HOLD_EN                0xAC
+#define HCI_CLKSTOP_CTRL       0xB0
+#define  REFCLK_STOP           BIT(2)
+#define  UNIPRO_MCLK_STOP      BIT(1)
+#define  UNIPRO_PCLK_STOP      BIT(0)
+#define  CLK_STOP_MASK         (REFCLK_STOP |\
+                                UNIPRO_MCLK_STOP |\
+                                UNIPRO_PCLK_STOP)
+#define HCI_MISC                       0xB4
+#define  REFCLK_CTRL_EN                BIT(7)
+#define  UNIPRO_PCLK_CTRL_EN   BIT(6)
+#define  UNIPRO_MCLK_CTRL_EN   BIT(5)
+#define  HCI_CORECLK_CTRL_EN   BIT(4)
+#define  CLK_CTRL_EN_MASK      (REFCLK_CTRL_EN |\
+                                UNIPRO_PCLK_CTRL_EN |\
+                                UNIPRO_MCLK_CTRL_EN)
+#define HCI_FSM_MONITOR                0xC0
+#define HCI_PRDT_HIT_RATIO     0xC4
+#define HCI_DMA0_MONITOR_STATE 0xC8
+#define HCI_DMA0_MONITOR_CNT   0xCC
+#define HCI_DMA1_MONITOR_STATE 0xD0
+#define HCI_DMA1_MONITOR_CNT   0xD4
+
+/* Device fatal error */
+#define DFES_ERR_EN    BIT(31)
+#define DFES_DEF_L2_ERRS       (UIC_DATA_LINK_LAYER_ERROR_RX_BUF_OVERFLOW |\
+                                UIC_DATA_LINK_LAYER_ERROR_PA_INIT)
+#define DFES_DEF_L3_ERRS       (UIC_NETWORK_LAYER_ERROR_UNSUPPORTED_HEADER_TYPE |\
+                                UIC_NETWORK_LAYER_ERROR_BAD_DEVICEID_ENC |\
+                                UIC_NETWORK_LAYER_ERROR_LHDR_TRAP_PACKET_DROPPING)
+#define DFES_DEF_L4_ERRS       (UIC_TRANSPORT_LAYER_ERROR_UNSUPPORTED_HEADER_TYPE |\
+                                UIC_TRANSPORT_LAYER_ERROR_UNKNOWN_CPORTID |\
+                                UIC_TRANSPORT_LAYER_ERROR_NO_CONNECTION_RX |\
+                                UIC_TRANSPORT_LAYER_ERROR_BAD_TC)
+
+enum {
+       UNIPRO_L1_5 = 0,/* PHY Adapter */
+       UNIPRO_L2,      /* Data Link */
+       UNIPRO_L3,      /* Network */
+       UNIPRO_L4,      /* Transport */
+       UNIPRO_DME,     /* DME */
+};
+
+/*
+ * UNIPRO registers
+ */
+#define UNIPRO_COMP_VERSION            0x000
+#define UNIPRO_COMP_VERSION            0x000
+#define UNIPRO_COMP_INFO               0x004
+#define UNIPRO_COMP_RESET              0x010
+#define UNIPRO_DME_POWERON_REQ         0x040
+#define UNIPRO_DME_POWERON_CNF_RESULT  0x044
+#define UNIPRO_DME_POWEROFF_REQ                0x048
+#define UNIPRO_DME_POWEROFF_CNF_RESULT 0x04C
+#define UNIPRO_DME_RESET_REQ           0x050
+#define UNIPRO_DME_RESET_REQ_LEVEL     0x054
+#define UNIPRO_DME_ENABLE_REQ          0x058
+#define UNIPRO_DME_ENABLE_CNF_RESULT   0x05C
+#define UNIPRO_DME_ENDPOINTRESET_REQ   0x060
+#define UNIPRO_DME_ENDPOINTRESET_CNF_RESULT    0x064
+#define UNIPRO_DME_LINKSTARTUP_REQ     0x068
+#define UNIPRO_DME_LINKSTARTUP_CNF_RESULT      0x06C
+#define UNIPRO_DME_HIBERN8_ENTER_REQ   0x070
+#define UNIPRO_DME_HIBERN8_ENTER_CNF_RESULT    0x074
+#define UNIPRO_DME_HIBERN8_ENTER_IND_RESULT    0x078
+#define UNIPRO_DME_HIBERN8_EXIT_REQ    0x080
+#define UNIPRO_DME_HIBERN8_EXIT_CNF_RESULT     0x084
+#define UNIPRO_DME_HIBERN8_EXIT_IND_RESULT     0x088
+#define UNIPRO_DME_PWR_REQ             0x090
+#define UNIPRO_DME_PWR_REQ_POWERMODE   0x094
+#define UNIPRO_DME_PWR_REQ_LOCALL2TIMER0       0x098
+#define UNIPRO_DME_PWR_REQ_LOCALL2TIMER1       0x09C
+#define UNIPRO_DME_PWR_REQ_LOCALL2TIMER2       0x0A0
+#define UNIPRO_DME_PWR_REQ_REMOTEL2TIMER0      0x0A4
+#define UNIPRO_DME_PWR_REQ_REMOTEL2TIMER1      0x0A8
+#define UNIPRO_DME_PWR_REQ_REMOTEL2TIMER2      0x0AC
+#define UNIPRO_DME_PWR_CNF_RESULT      0x0B0
+#define UNIPRO_DME_PWR_IND_RESULT      0x0B4
+#define UNIPRO_DME_TEST_MODE_REQ       0x0B8
+#define UNIPRO_DME_TEST_MODE_CNF_RESULT        0x0BC
+#define UNIPRO_DME_ERROR_IND_LAYER     0x0C0
+#define UNIPRO_DME_ERROR_IND_ERRCODE   0x0C4
+#define UNIPRO_DME_PACP_CNFBIT         0x0C8
+#define UNIPRO_DME_DL_FRAME_IND                0x0D0
+#define UNIPRO_DME_INTR_STATUS         0x0E0
+#define UNIPRO_DME_INTR_ENABLE         0x0E4
+#define UNIPRO_DME_GETSET_ADDR         0x100
+#define UNIPRO_DME_GETSET_WDATA                0x104
+#define UNIPRO_DME_GETSET_RDATA                0x108
+#define UNIPRO_DME_GETSET_CONTROL      0x10C
+#define UNIPRO_DME_GETSET_RESULT       0x110
+#define UNIPRO_DME_PEER_GETSET_ADDR    0x120
+#define UNIPRO_DME_PEER_GETSET_WDATA   0x124
+#define UNIPRO_DME_PEER_GETSET_RDATA   0x128
+#define UNIPRO_DME_PEER_GETSET_CONTROL 0x12C
+#define UNIPRO_DME_PEER_GETSET_RESULT  0x130
+#define UNIPRO_DBG_FORCE_DME_CTRL_STATE        0x150
+
+/*
+ * UFS Protector registers
+ */
+#define UFSPRCTRL      0x000
+#define UFSPRSTAT      0x008
+#define UFSPRSECURITY  0x010
+#define  NSSMU         BIT(14)
+#define  DESCTYPE(type)        ((type & 0x3) << 19)
+#define  PROTBYTZPC    BIT(31)
+#define UFSPVERSION    0x01C
+#define UFSPRENCKEY0   0x020
+#define UFSPRENCKEY1   0x024
+#define UFSPRENCKEY2   0x028
+#define UFSPRENCKEY3   0x02C
+#define UFSPRENCKEY4   0x030
+#define UFSPRENCKEY5   0x034
+#define UFSPRENCKEY6   0x038
+#define UFSPRENCKEY7   0x03C
+#define UFSPRTWKEY0    0x040
+#define UFSPRTWKEY1    0x044
+#define UFSPRTWKEY2    0x048
+#define UFSPRTWKEY3    0x04C
+#define UFSPRTWKEY4    0x050
+#define UFSPRTWKEY5    0x054
+#define UFSPRTWKEY6    0x058
+#define UFSPRTWKEY7    0x05C
+#define UFSPWCTRL      0x100
+#define UFSPWSTAT      0x108
+#define UFSPWSECURITY  0x110
+#define UFSPWENCKEY0   0x120
+#define UFSPWENCKEY1   0x124
+#define UFSPWENCKEY2   0x128
+#define UFSPWENCKEY3   0x12C
+#define UFSPWENCKEY4   0x130
+#define UFSPWENCKEY5   0x134
+#define UFSPWENCKEY6   0x138
+#define UFSPWENCKEY7   0x13C
+#define UFSPWTWKEY0    0x140
+#define UFSPWTWKEY1    0x144
+#define UFSPWTWKEY2    0x148
+#define UFSPWTWKEY3    0x14C
+#define UFSPWTWKEY4    0x150
+#define UFSPWTWKEY5    0x154
+#define UFSPWTWKEY6    0x158
+#define UFSPWTWKEY7    0x15C
+#define UFSPSBEGIN0    0x200
+#define UFSPSEND0      0x204
+#define UFSPSLUN0      0x208
+#define UFSPSCTRL0     0x20C
+#define UFSPSBEGIN1    0x210
+#define UFSPSEND1      0x214
+#define UFSPSLUN1      0x218
+#define UFSPSCTRL1     0x21C
+#define UFSPSBEGIN2    0x220
+#define UFSPSEND2      0x224
+#define UFSPSLUN2      0x228
+#define UFSPSCTRL2     0x22C
+#define UFSPSBEGIN3    0x230
+#define UFSPSEND3      0x234
+#define UFSPSLUN3      0x238
+#define UFSPSCTRL3     0x23C
+#define UFSPSBEGIN4    0x240
+#define UFSPSEND4      0x244
+#define UFSPSLUN4      0x248
+#define UFSPSCTRL4     0x24C
+#define UFSPSBEGIN5    0x250
+#define UFSPSEND5      0x254
+#define UFSPSLUN5      0x258
+#define UFSPSCTRL5     0x25C
+#define UFSPSBEGIN6    0x260
+#define UFSPSEND6      0x264
+#define UFSPSLUN6      0x268
+#define UFSPSCTRL6     0x26C
+#define UFSPSBEGIN7    0x270
+#define UFSPSEND7      0x274
+#define UFSPSLUN7      0x278
+#define UFSPSCTRL7     0x27C
+
+/*
+ * MIBs for PA debug registers
+ */
+#define PA_DBG_CLK_PERIOD      0x9514
+#define PA_DBG_TXPHY_CFGUPDT   0x9518
+#define PA_DBG_RXPHY_CFGUPDT   0x9519
+#define PA_DBG_MODE            0x9529
+#define PA_DBG_TX_PHY_LATENCY  0x952C
+#define PA_DBG_AUTOMODE_THLD   0x9536
+#define PA_DBG_SKIP_RESET_PHY  0x9539
+#define PA_DBG_OV_TM           0x9540
+#define PA_DBG_SKIP_LINE_RESET 0x9541
+#define PA_DBG_LINE_RESET_REQ  0x9543
+#define PA_DBG_MIB_CAP_SEL     0x9546
+#define PA_DBG_RESUME_HIBERN8  0x9550
+#define PA_DBG_TX_PHY_LATENCY_EN       0x9552
+#define PA_DBG_IGNORE_INCOMING 0x9559
+#define PA_DBG_LINE_RESET_THLD 0x9561
+#define PA_DBG_OPTION_SUITE    0x9564
+#define PA_DBG_OPTION_SUITE_DYN        0x9565
+
+/*
+ * MIBs for Transport Layer debug registers
+ */
+#define T_DBG_SKIP_INIT_HIBERN8_EXIT   0xc001
+
+/*
+ * Exynos MPHY attributes
+ */
+#define TX_LINERESET_N_VAL     0x0277
+#define  TX_LINERESET_N(v)     (((v) >> 10) & 0xFF)
+#define TX_LINERESET_P_VAL     0x027D
+#define  TX_LINERESET_P(v)     (((v) >> 12) & 0xFF)
+#define TX_OV_SLEEP_CNT_TIMER  0x028E
+#define  TX_OV_H8_ENTER_EN     (1 << 7)
+#define  TX_OV_SLEEP_CNT(v)    (((v) >> 5) & 0x7F)
+#define TX_HIGH_Z_CNT_11_08    0x028C
+#define  TX_HIGH_Z_CNT_H(v)    (((v) >> 8) & 0xF)
+#define TX_HIGH_Z_CNT_07_00    0x028D
+#define  TX_HIGH_Z_CNT_L(v)    ((v) & 0xFF)
+#define TX_BASE_NVAL_07_00     0x0293
+#define  TX_BASE_NVAL_L(v)     ((v) & 0xFF)
+#define TX_BASE_NVAL_15_08     0x0294
+#define  TX_BASE_NVAL_H(v)     (((v) >> 8) & 0xFF)
+#define TX_GRAN_NVAL_07_00     0x0295
+#define  TX_GRAN_NVAL_L(v)     ((v) & 0xFF)
+#define TX_GRAN_NVAL_10_08     0x0296
+#define  TX_GRAN_NVAL_H(v)     (((v) >> 8) & 0x3)
+
+#define RX_FILLER_ENABLE       0x0316
+#define  RX_FILLER_EN          (1 << 1)
+#define RX_LINERESET_VAL       0x0317
+#define  RX_LINERESET(v)       (((v) >> 12) & 0xFF)
+#define RX_LCC_IGNORE          0x0318
+#define RX_SYNC_MASK_LENGTH    0x0321
+#define RX_HIBERN8_WAIT_VAL_BIT_20_16  0x0331
+#define RX_HIBERN8_WAIT_VAL_BIT_15_08  0x0332
+#define RX_HIBERN8_WAIT_VAL_BIT_07_00  0x0333
+#define RX_OV_SLEEP_CNT_TIMER  0x0340
+#define  RX_OV_SLEEP_CNT(v)    (((v) >> 6) & 0x1F)
+#define RX_OV_STALL_CNT_TIMER  0x0341
+#define  RX_OV_STALL_CNT(v)    (((v) >> 4) & 0xFF)
+#define RX_BASE_NVAL_07_00     0x0355
+#define  RX_BASE_NVAL_L(v)     ((v) & 0xFF)
+#define RX_BASE_NVAL_15_08     0x0354
+#define  RX_BASE_NVAL_H(v)     (((v) >> 8) & 0xFF)
+#define RX_GRAN_NVAL_07_00     0x0353
+#define  RX_GRAN_NVAL_L(v)     ((v) & 0xFF)
+#define RX_GRAN_NVAL_10_08     0x0352
+#define  RX_GRAN_NVAL_H(v)     (((v) >> 8) & 0x3)
+
+#define CMN_PWM_CLK_CTRL       0x0402
+#define  PWM_CLK_CTRL_MASK     0x3
+
+struct exynos_ufs;
+
+struct uic_pwr_mode {
+       u8 lane;
+       u8 gear;
+       u8 mode;
+       u8 hs_series;
+       u32 l_l2_timer[3];      /* local */
+       u32 r_l2_timer[3];      /* remote */
+};
+
+struct exynos_ufs_uic_attr {
+       /* TX Attributes */
+       unsigned int tx_trailingclks;
+       unsigned int tx_dif_p_nsec;
+       unsigned int tx_dif_n_nsec;
+       unsigned int tx_high_z_cnt_nsec;
+       unsigned int tx_base_unit_nsec;
+       unsigned int tx_gran_unit_nsec;
+       unsigned int tx_sleep_cnt;
+       unsigned int tx_min_activatetime;
+       /* RX Attributes */
+       unsigned int rx_filler_enable;
+       unsigned int rx_dif_p_nsec;
+       unsigned int rx_hibern8_wait_nsec;
+       unsigned int rx_base_unit_nsec;
+       unsigned int rx_gran_unit_nsec;
+       unsigned int rx_sleep_cnt;
+       unsigned int rx_stall_cnt;
+       unsigned int rx_hs_g1_sync_len_cap;
+       unsigned int rx_hs_g2_sync_len_cap;
+       unsigned int rx_hs_g3_sync_len_cap;
+       unsigned int rx_hs_g1_prep_sync_len_cap;
+       unsigned int rx_hs_g2_prep_sync_len_cap;
+       unsigned int rx_hs_g3_prep_sync_len_cap;
+       /* Common Attributes */
+       unsigned int cmn_pwm_clk_ctrl;
+       /* Internal Attributes */
+       unsigned int pa_dbg_option_suite;
+       /* Changeable Attributes */
+       unsigned int rx_adv_fine_gran_sup_en;
+       unsigned int rx_adv_fine_gran_step;
+       unsigned int rx_min_actv_time_cap;
+       unsigned int rx_hibern8_time_cap;
+       unsigned int rx_adv_min_actv_time_cap;
+       unsigned int rx_adv_hibern8_time_cap;
+       unsigned int pa_granularity;
+       unsigned int pa_tactivate;
+       unsigned int pa_hibern8time;
+};
+
+struct exynos_ufs_drv_data {
+       char *compatible;
+       struct exynos_ufs_uic_attr *uic_attr;
+       unsigned int quirks;
+       unsigned int opts;
+       /* SoC's specific operations */
+       int (*drv_init)(struct device *dev, struct exynos_ufs *ufs);
+       int (*pre_link)(struct exynos_ufs *ufs);
+       int (*post_link)(struct exynos_ufs *ufs);
+       int (*pre_pwr_change)(struct exynos_ufs *ufs, struct uic_pwr_mode *pwr);
+       int (*post_pwr_change)(struct exynos_ufs *ufs, struct uic_pwr_mode *pwr);
+};
+
+struct ufs_phy_time_cfg {
+       u32 tx_linereset_p;
+       u32 tx_linereset_n;
+       u32 tx_high_z_cnt;
+       u32 tx_base_n_val;
+       u32 tx_gran_n_val;
+       u32 tx_sleep_cnt;
+       u32 rx_linereset;
+       u32 rx_hibern8_wait;
+       u32 rx_base_n_val;
+       u32 rx_gran_n_val;
+       u32 rx_sleep_cnt;
+       u32 rx_stall_cnt;
+};
+
+struct exynos_ufs {
+       struct ufs_hba *hba;
+       struct phy *phy;
+       void __iomem *reg_hci;
+       void __iomem *reg_unipro;
+       void __iomem *reg_ufsp;
+       struct clk *clk_hci_core;
+       struct clk *clk_unipro_main;
+       struct clk *clk_apb;
+       u32 pclk_rate;
+       u32 pclk_div;
+       u32 pclk_avail_min;
+       u32 pclk_avail_max;
+       u32 mclk_rate;
+       int avail_ln_rx;
+       int avail_ln_tx;
+       int rx_sel_idx;
+       struct uic_pwr_mode pwr_req;    /* requested power mode */
+       struct uic_pwr_mode pwr_act;    /* actual power mode */
+       struct ufs_phy_time_cfg t_cfg;
+       ktime_t entry_hibern8_t;
+       struct exynos_ufs_drv_data *drv_data;
+
+       u32 opts;
+#define EXYNOS_UFS_OPT_HAS_APB_CLK_CTRL                BIT(0)
+#define EXYNOS_UFS_OPT_SKIP_CONNECTION_ESTAB   BIT(1)
+#define EXYNOS_UFS_OPT_BROKEN_AUTO_CLK_CTRL    BIT(2)
+#define EXYNOS_UFS_OPT_BROKEN_RX_SEL_IDX       BIT(3)
+#define EXYNOS_UFS_OPT_USE_SW_HIBERN8_TIMER    BIT(4)
+};
+
+#define for_each_ufs_rx_lane(ufs, i) \
+       for (i = (ufs)->rx_sel_idx; \
+               i < (ufs)->rx_sel_idx + (ufs)->avail_ln_rx; i++)
+#define for_each_ufs_tx_lane(ufs, i) \
+       for (i = 0; i < (ufs)->avail_ln_tx; i++)
+
+#define EXYNOS_UFS_MMIO_FUNC(name)                                       \
+static inline void name##_writel(struct exynos_ufs *ufs, u32 val, u32 reg)\
+{                                                                        \
+       writel(val, ufs->reg_##name + reg);                               \
+}                                                                        \
+                                                                         \
+static inline u32 name##_readl(struct exynos_ufs *ufs, u32 reg)                  \
+{                                                                        \
+       return readl(ufs->reg_##name + reg);                              \
+}
+
+EXYNOS_UFS_MMIO_FUNC(hci);
+EXYNOS_UFS_MMIO_FUNC(unipro);
+EXYNOS_UFS_MMIO_FUNC(ufsp);
+#undef EXYNOS_UFS_MMIO_FUNC
+
+extern long exynos_ufs_calc_time_cntr(struct exynos_ufs *, long);
+
+static inline void exynos_ufs_enable_ov_tm(struct ufs_hba *hba)
+{
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OV_TM), TRUE);
+}
+
+static inline void exynos_ufs_disable_ov_tm(struct ufs_hba *hba)
+{
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_OV_TM), FALSE);
+}
+
+static inline void exynos_ufs_enable_dbg_mode(struct ufs_hba *hba)
+{
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_MODE), TRUE);
+}
+
+static inline void exynos_ufs_disable_dbg_mode(struct ufs_hba *hba)
+{
+       ufshcd_dme_set(hba, UIC_ARG_MIB(PA_DBG_MODE), FALSE);
+}
+
+extern struct exynos_ufs_drv_data exynos_ufs_drvs[];
+
+#endif /* _UFS_EXYNOS_H_ */
diff --git a/drivers/scsi/ufs/ufshci.h b/drivers/scsi/ufs/ufshci.h
index 0ae0967..1c73400 100644
--- a/drivers/scsi/ufs/ufshci.h
+++ b/drivers/scsi/ufs/ufshci.h
@@ -169,15 +169,39 @@ enum {
  /* UECDL - Host UIC Error Code Data Link Layer 3Ch */
  #define UIC_DATA_LINK_LAYER_ERROR              UFS_BIT(31)
  #define UIC_DATA_LINK_LAYER_ERROR_CODE_MASK    0x7FFF
-#define UIC_DATA_LINK_LAYER_ERROR_PA_INIT      0x2000
+#define UIC_DATA_LINK_LAYER_ERROR_NAC_RECEIVED         UFS_BIT(0)
+#define UIC_DATA_LINK_LAYER_ERROR_TCX_REP_TIMER_EXP    UFS_BIT(1)
+#define UIC_DATA_LINK_LAYER_ERROR_AFCX_REQ_TIMER_EXP   UFS_BIT(2)
+#define UIC_DATA_LINK_LAYER_ERROR_FCX_PRO_TIMER_EXP    UFS_BIT(3)
+#define UIC_DATA_LINK_LAYER_ERROR_CRC                  UFS_BIT(4)
+#define UIC_DATA_LINK_LAYER_ERROR_RX_BUF_OVERFLOW      UFS_BIT(5)
+#define UIC_DATA_LINK_LAYER_ERROR_MASK_FRAME_LEN_EXCD  UFS_BIT(6)
+#define UIC_DATA_LINK_LAYER_ERROR_WRONG_SEQ_NUM                UFS_BIT(7)
+#define UIC_DATA_LINK_LAYER_ERROR_AFC_FRAME_SYNTAX     UFS_BIT(8)
+#define UIC_DATA_LINK_LAYER_ERROR_NAC_FRAME_SYNTAX     UFS_BIT(9)
+#define UIC_DATA_LINK_LAYER_ERROR_EOF_SYNTAX           UFS_BIT(10)
+#define UIC_DATA_LINK_LAYER_ERROR_FRAME_SYNTAX         UFS_BIT(11)
+#define UIC_DATA_LINK_LAYER_ERROR_BAD_CTRL_SYMB_TYPE   UFS_BIT(12)
+#define UIC_DATA_LINK_LAYER_ERROR_PA_INIT              UFS_BIT(13)
+#define UIC_DATA_LINK_LAYER_ERROR_PA_ERROR_IND_RECV    UFS_BIT(14)

  /* UECN - Host UIC Error Code Network Layer 40h */
  #define UIC_NETWORK_LAYER_ERROR                        UFS_BIT(31)
  #define UIC_NETWORK_LAYER_ERROR_CODE_MASK      0x7
+#define UIC_NETWORK_LAYER_ERROR_UNSUPPORTED_HEADER_TYPE        UFS_BIT(0)
+#define UIC_NETWORK_LAYER_ERROR_BAD_DEVICEID_ENC               UFS_BIT(1)
+#define UIC_NETWORK_LAYER_ERROR_LHDR_TRAP_PACKET_DROPPING      UFS_BIT(2)

  /* UECT - Host UIC Error Code Transport Layer 44h */
  #define UIC_TRANSPORT_LAYER_ERROR              UFS_BIT(31)
  #define UIC_TRANSPORT_LAYER_ERROR_CODE_MASK    0x7F
+#define UIC_TRANSPORT_LAYER_ERROR_UNSUPPORTED_HEADER_TYPE      UFS_BIT(0)
+#define UIC_TRANSPORT_LAYER_ERROR_UNKNOWN_CPORTID      UFS_BIT(1)
+#define UIC_TRANSPORT_LAYER_ERROR_NO_CONNECTION_RX     UFS_BIT(2)
+#define UIC_TRANSPORT_LAYER_ERROR_CONTROLLED_SEGMENT_DROPPING  UFS_BIT(3)
+#define UIC_TRANSPORT_LAYER_ERROR_BAD_TC               UFS_BIT(4)
+#define UIC_TRANSPORT_LAYER_ERROR_E2E_CREDIT_OVERFLOW  UFS_BIT(5)
+#define UIC_TRANSPORT_LAYER_ERROR_SAFETY_VALUE_DROPPING        UFS_BIT(6)

  /* UECDME - Host UIC Error Code DME 48h */
  #define UIC_DME_ERROR                  UFS_BIT(31)
diff --git a/drivers/scsi/ufs/unipro.h b/drivers/scsi/ufs/unipro.h
index 816a8a4..68cfece 100644
--- a/drivers/scsi/ufs/unipro.h
+++ b/drivers/scsi/ufs/unipro.h
@@ -49,7 +49,27 @@
  #define RX_BYPASS_8B10B_ENABLE                 0x00A8
  #define RX_TERMINATION_FORCE_ENABLE            0x0089

+/*
+ * M-RX Capability Attributes
+ */
+#define RX_HS_G1_SYNC_LENGTH_CAP               0x008B
+#define RX_HS_G1_PREP_LENGTH_CAP               0x008C
+#define RX_HS_G2_SYNC_LENGTH_CAP               0x0094
+#define RX_HS_G3_SYNC_LENGTH_CAP               0x0095
+#define RX_HS_G2_PREP_LENGTH_CAP               0x0096
+#define RX_HS_G3_PREP_LENGTH_CAP               0x0097
+#define RX_ADV_GRANULARITY_CAP                 0x0098
+#define RX_MIN_ACTIVATETIME_CAP                        0x008F
+#define RX_HIBERN8TIME_CAP                     0x0092
+#define RX_ADV_HIBERN8TIME_CAP                 0x0099
+#define RX_ADV_MIN_ACTIVATETIME_CAP            0x009A
+
  #define is_mphy_tx_attr(attr)                  (attr < RX_MODE)
+#define RX_ADV_FINE_GRAN_STEP(x)               ((((x) & 0x3) << 1) | 0x1)
+#define SYNC_LEN_FINE(x)                       ((x) & 0x3F)
+#define SYNC_LEN_COARSE(x)                     ((1 << 6) | ((x) & 0x3F))
+#define PREP_LEN(x)                            ((x) & 0xF)
+
  /*
   * PHY Adpater attributes
   */
@@ -87,6 +107,7 @@
  #define PA_PACPREQEOBTIMEOUT   0x1591
  #define PA_HIBERN8TIME         0x15A7
  #define PA_LOCALVERINFO                0x15A9
+#define PA_GRANULARITY         0x15AA
  #define PA_TACTIVATE           0x15A8
  #define PA_PACPFRAMECOUNT      0x15C0
  #define PA_PACPERRORCOUNT      0x15C1
@@ -110,6 +131,9 @@
  #define PA_STALLNOCONFIGTIME   0x15A3
  #define PA_SAVECONFIGTIME      0x15A4

+/* PHY Adapter protocal constants */
+#define PA_MAXDATALANES                4
+
  /* PA power modes */
  enum {
         FAST_MODE       = 1,
@@ -119,6 +143,9 @@ enum {
         UNCHANGED       = 7,
  };

+#define IS_UFS_PWR_MODE_HS(m)  (((m) == FAST_MODE) || ((m) == FASTAUTO_MODE))
+#define IS_UFS_PWR_MODE_PWM(m) (((m) == SLOW_MODE) || ((m) == SLOWAUTO_MODE))
+
  /* PA TX/RX Frequency Series */
  enum {
         PA_HS_MODE_A    = 1,
@@ -170,6 +197,11 @@ enum ufs_hs_gear_tag {
  #define DL_PEERTC1PRESENT      0x2066
  #define DL_PEERTC1RXINITCREVAL 0x2067

+/* Default value of L2 Timer */
+#define FC0PROTTIMEOUTVAL      8191
+#define TC0REPLAYTIMEOUTVAL    65535
+#define AFC0REQTIMEOUTVAL      32767
+
  /*
   * Network Layer Attributes
   */
@@ -212,4 +244,19 @@ enum {
         TRUE,
  };

+/* CPort setting */
+#define E2EFC_ON       (1 << 0)
+#define E2EFC_OFF      (0 << 0)
+#define CSD_N_ON       (0 << 1)
+#define CSD_N_OFF      (1 << 1)
+#define CSV_N_ON       (0 << 2)
+#define CSV_N_OFF      (1 << 2)
+#define CPORT_DEF_FLAGS        (CSV_N_OFF | CSD_N_OFF | E2EFC_OFF)
+
+/* CPort connection state */
+enum {
+       CPORT_IDLE = 0,
+       CPORT_CONNECTED,
+};
+
  #endif /* _UNIPRO_H_ */
--
1.7.10.4

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

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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux