lkml.org 
[lkml]   [2016]   [Jul]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.12 56/88] powerpc/pseries: Fix PCI config address for DDW
    Date
    From: Gavin Shan <gwshan@linux.vnet.ibm.com>

    3.12-stable review patch. If anyone has any objections, please let me know.

    ===============

    commit 8a934efe94347eee843aeea65bdec8077a79e259 upstream.

    In commit 8445a87f7092 "powerpc/iommu: Remove the dependency on EEH
    struct in DDW mechanism", the PE address was replaced with the PCI
    config address in order to remove dependency on EEH. According to PAPR
    spec, firmware (pHyp or QEMU) should accept "xxBBSSxx" format PCI config
    address, not "xxxxBBSS" provided by the patch. Note that "BB" is PCI bus
    number and "SS" is the combination of slot and function number.

    This fixes the PCI address passed to DDW RTAS calls.

    Fixes: 8445a87f7092 ("powerpc/iommu: Remove the dependency on EEH struct in DDW mechanism")
    Reported-by: Guilherme G. Piccoli <gpiccoli@linux.vnet.ibm.com>
    Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>
    Tested-by: Guilherme G. Piccoli <gpiccoli@linux.vnet.ibm.com>
    Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
    Signed-off-by: Jiri Slaby <jslaby@suse.cz>
    ---
    arch/powerpc/platforms/pseries/iommu.c | 4 ++--
    1 file changed, 2 insertions(+), 2 deletions(-)

    diff --git a/arch/powerpc/platforms/pseries/iommu.c b/arch/powerpc/platforms/pseries/iommu.c
    index 88cbecbdbd6f..c06f9e75f8b1 100644
    --- a/arch/powerpc/platforms/pseries/iommu.c
    +++ b/arch/powerpc/platforms/pseries/iommu.c
    @@ -876,7 +876,7 @@ static int query_ddw(struct pci_dev *dev, const u32 *ddw_avail,
    dn = pci_device_to_OF_node(dev);
    pdn = PCI_DN(dn);
    buid = pdn->phb->buid;
    - cfg_addr = (pdn->busno << 8) | pdn->devfn;
    + cfg_addr = ((pdn->busno << 16) | (pdn->devfn << 8));

    ret = rtas_call(ddw_avail[0], 3, 5, (u32 *)query,
    cfg_addr, BUID_HI(buid), BUID_LO(buid));
    @@ -905,7 +905,7 @@ static int create_ddw(struct pci_dev *dev, const u32 *ddw_avail,
    dn = pci_device_to_OF_node(dev);
    pdn = PCI_DN(dn);
    buid = pdn->phb->buid;
    - cfg_addr = (pdn->busno << 8) | pdn->devfn;
    + cfg_addr = ((pdn->busno << 16) | (pdn->devfn << 8));

    do {
    /* extra outputs are LIOBN and dma-addr (hi, lo) */
    --
    2.9.1
    \
     
     \ /
      Last update: 2016-07-14 11:01    [W:3.798 / U:0.316 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site