lkml.org 
[lkml]   [2016]   [May]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.5 70/88] ata: ahci-platform: Add ports-implemented DT bindings.
    Date
    4.5-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

    commit 17dcc37e3e847bc0e67a5b1ec52471fcc6c18682 upstream.

    On some SOCs PORTS_IMPL register value is never programmed by the
    firmware and left at zero value. Which means that no sata ports are
    available for software. AHCI driver used to cope up with this by
    fabricating the port_map if the PORTS_IMPL register is read zero,
    but recent patch broke this workaround as zero value was valid for
    NVMe disks.

    This patch adds ports-implemented DT bindings as workaround for this issue
    in a way that DT can can override the PORTS_IMPL register in cases where
    the firmware did not program it already.

    Fixes: 566d1827df2e ("libata: disable forced PORTS_IMPL for >= AHCI 1.3")
    Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
    Acked-by: Tejun Heo <tj@kernel.org>
    Reviewed-by: Andy Gross <andy.gross@linaro.org>
    Signed-off-by: Tejun Heo <tj@kernel.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    Documentation/devicetree/bindings/ata/ahci-platform.txt | 4 ++++
    drivers/ata/ahci_platform.c | 3 +++
    2 files changed, 7 insertions(+)

    --- a/Documentation/devicetree/bindings/ata/ahci-platform.txt
    +++ b/Documentation/devicetree/bindings/ata/ahci-platform.txt
    @@ -30,6 +30,10 @@ Optional properties:
    - target-supply : regulator for SATA target power
    - phys : reference to the SATA PHY node
    - phy-names : must be "sata-phy"
    +- ports-implemented : Mask that indicates which ports that the HBA supports
    + are available for software to use. Useful if PORTS_IMPL
    + is not programmed by the BIOS, which is true with
    + some embedded SOC's.

    Required properties when using sub-nodes:
    - #address-cells : number of cells to encode an address
    --- a/drivers/ata/ahci_platform.c
    +++ b/drivers/ata/ahci_platform.c
    @@ -51,6 +51,9 @@ static int ahci_probe(struct platform_de
    if (rc)
    return rc;

    + of_property_read_u32(dev->of_node,
    + "ports-implemented", &hpriv->force_port_map);
    +
    if (of_device_is_compatible(dev->of_node, "hisilicon,hisi-ahci"))
    hpriv->flags |= AHCI_HFLAG_NO_FBS | AHCI_HFLAG_NO_NCQ;


    \
     
     \ /
      Last update: 2016-05-09 10:01    [W:5.498 / U:0.192 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site