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]
Date:	Thu, 17 Sep 2015 11:04:58 +0530
From:	Alim Akhtar <alim.akhtar@...sung.com>
To:	amit daniel kachhap <amit.daniel@...sung.com>
Cc:	linux-scsi@...r.kernel.org,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	vinholikatti@...il.com, JBottomley@...n.com, essuuj@...il.com
Subject: Re: [PATCH 10/10] scsi: ufs-exynos: add UFS host support for Exynos
 SoCs

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@...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
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@...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.
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@...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.
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@...sung.com>
>
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@...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