lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CADGdYn6M0o5YZ8Q8P0W9pK2-6EFTb9BZH8VejEQ2yvNbiHFpCQ@mail.gmail.com>
Date:	Wed, 26 Aug 2015 12:12:30 +0530
From:	amit daniel kachhap <amit.daniel@...sung.com>
To:	Alim Akhtar <alim.akhtar@...sung.com>
Cc:	linux-scsi@...r.kernel.org,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	vinholikatti@...il.com, JBottomley@...n.com,
	Seungwon Jeon <tgih.jun@...sung.com>
Subject: Re: [PATCH 10/10] scsi: ufs-exynos: add UFS host support for Exynos SoCs

On Fri, Aug 21, 2015 at 2:58 PM, Alim Akhtar <alim.akhtar@...sung.com> wrote:
> From: Seungwon Jeon <tgih.jun@...sung.com>
>
> 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@...sung.com>
> Signed-off-by: Alim Akhtar <alim.akhtar@...sung.com>
> ---
>  .../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
> +                          <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@...5570000 {
> +               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@...sung.com>
> + *
> + * 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.
> +       .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@...sung.com>
> + *
> + * 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.
> +{
> +       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.
> +{
> +       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.
> +       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.
> +               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.

After all these changes you can add for the whole series.
Reviewed by: Amit Daniel Kachhap <amit.daniel@...sung.com>

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@...r.kernel.org
> 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-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ