linux-kernel - Re: [PATCH v2 0/5] qcom-ufs: phy/hcd: Refactor phy initialization code

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] [day] [month] [year] [list]

Message-ID: <238f7e71-c878-2851-c5b8-9ef4a835cf23@ti.com>
Date:   Wed, 18 Oct 2017 18:01:49 +0530
From:   Kishon Vijay Abraham I <kishon@...com>
To:     Vivek Gautam <vivek.gautam@...eaurora.org>,
        <martin.petersen@...cle.com>, <linux-scsi@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
CC:     <subhashj@...eaurora.org>, <robh+dt@...nel.org>,
        <bjorn.andersson@...aro.org>, <devicetree@...r.kernel.org>
Subject: Re: [PATCH v2 0/5] qcom-ufs: phy/hcd: Refactor phy initialization
 code



On Thursday 12 October 2017 11:49 AM, Vivek Gautam wrote:
> Refactoring the qcom-ufs phy and host controller code to move
> further towards the generic phy usage. Right now the qcom-ufs exports
> a bunch of APIs that are used by the host controller to initialize
> the phy.
> With this patch series, we populate the phy_init() which was a no-op
> earlier. The host controller then calls the phy_init() at the designated
> place rather than doing it invariably in ufs_hcd_init().
> 
> As part of this series, we introduce phy modes for ufs phy.
> The M-PHY has two data rates defined for each generations (Gears) -
> Rate A and Rate B. These can serve as the two modes of ufs HS phy.
> Host controller can direct the phy to set the respective configurations
> based on the phy modes.
> 
> The patch-series has been tested with necessary dt patches on db820c.
> 
> 
> Hi Kishon,
> 
> I shall be on vacation for the next week. Can you kindly consider
> picking this series for 4.15. Martin was good with last version
> of the patches[1], and Subhash has reviewed the patches of which
> he has given 'Reviewed-by' for [PATCH 4/5] , and I have addressed
> his comments for the last patch. I hope it's not too late for 4.15.
> Subhash, can you please consider giving your 'Reviewed-by' for
> the last patch if you find it good.

merged to phy -next, thanks!

-Kishon
> 
> [1] https://lkml.org/lkml/2017/8/10/958
> 
> Vivek Gautam (5):
>   dt-bindings: phy: Add PHY_TYPE_UFS definition
>   phy: Add UFS PHY modes
>   phy: qcom-ufs: Add support to set phy mode
>   scsi/ufs: qcom: Set phy mode based on the controllers HS MODE
>   ufs/phy: qcom: Refactor to use phy_init call
> 
>  drivers/phy/qualcomm/phy-qcom-ufs-i.h        |  5 ++--
>  drivers/phy/qualcomm/phy-qcom-ufs-qmp-14nm.c | 29 +++++++++++++++++--
>  drivers/phy/qualcomm/phy-qcom-ufs-qmp-20nm.c | 29 +++++++++++++++++--
>  drivers/phy/qualcomm/phy-qcom-ufs.c          | 42 ++++++++++------------------
>  drivers/scsi/ufs/ufs-qcom.c                  | 39 +++++++++++---------------
>  include/dt-bindings/phy/phy.h                |  1 +
>  include/linux/phy/phy-qcom-ufs.h             |  3 --
>  include/linux/phy/phy.h                      |  2 ++
>  8 files changed, 91 insertions(+), 59 deletions(-)
>