lkml.org 
[lkml]   [2019]   [Aug]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.9 203/223] usb: pci-quirks: Correct AMD PLL quirk detection
    Date
    From: Ryan Kennedy <ryan5544@gmail.com>

    commit f3dccdaade4118070a3a47bef6b18321431f9ac6 upstream.

    The AMD PLL USB quirk is incorrectly enabled on newer Ryzen
    chipsets. The logic in usb_amd_find_chipset_info currently checks
    for unaffected chipsets rather than affected ones. This broke
    once a new chipset was added in e788787ef. It makes more sense
    to reverse the logic so it won't need to be updated as new
    chipsets are added. Note that the core of the workaround in
    usb_amd_quirk_pll does correctly check the chipset.

    Signed-off-by: Ryan Kennedy <ryan5544@gmail.com>
    Fixes: e788787ef4f9 ("usb:xhci:Add quirk for Certain failing HP keyboard on reset after resume")
    Cc: stable <stable@vger.kernel.org>
    Acked-by: Alan Stern <stern@rowland.harvard.edu>
    Link: https://lore.kernel.org/r/20190704153529.9429-2-ryan5544@gmail.com
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/usb/host/pci-quirks.c | 31 +++++++++++++++++++------------
    1 file changed, 19 insertions(+), 12 deletions(-)

    --- a/drivers/usb/host/pci-quirks.c
    +++ b/drivers/usb/host/pci-quirks.c
    @@ -187,7 +187,7 @@ int usb_amd_find_chipset_info(void)
    {
    unsigned long flags;
    struct amd_chipset_info info;
    - int ret;
    + int need_pll_quirk = 0;

    spin_lock_irqsave(&amd_lock, flags);

    @@ -201,21 +201,28 @@ int usb_amd_find_chipset_info(void)
    spin_unlock_irqrestore(&amd_lock, flags);

    if (!amd_chipset_sb_type_init(&info)) {
    - ret = 0;
    goto commit;
    }

    - /* Below chipset generations needn't enable AMD PLL quirk */
    - if (info.sb_type.gen == AMD_CHIPSET_UNKNOWN ||
    - info.sb_type.gen == AMD_CHIPSET_SB600 ||
    - info.sb_type.gen == AMD_CHIPSET_YANGTZE ||
    - (info.sb_type.gen == AMD_CHIPSET_SB700 &&
    - info.sb_type.rev > 0x3b)) {
    + switch (info.sb_type.gen) {
    + case AMD_CHIPSET_SB700:
    + need_pll_quirk = info.sb_type.rev <= 0x3B;
    + break;
    + case AMD_CHIPSET_SB800:
    + case AMD_CHIPSET_HUDSON2:
    + case AMD_CHIPSET_BOLTON:
    + need_pll_quirk = 1;
    + break;
    + default:
    + need_pll_quirk = 0;
    + break;
    + }
    +
    + if (!need_pll_quirk) {
    if (info.smbus_dev) {
    pci_dev_put(info.smbus_dev);
    info.smbus_dev = NULL;
    }
    - ret = 0;
    goto commit;
    }

    @@ -234,7 +241,7 @@ int usb_amd_find_chipset_info(void)
    }
    }

    - ret = info.probe_result = 1;
    + need_pll_quirk = info.probe_result = 1;
    printk(KERN_DEBUG "QUIRK: Enable AMD PLL fix\n");

    commit:
    @@ -245,7 +252,7 @@ commit:

    /* Mark that we where here */
    amd_chipset.probe_count++;
    - ret = amd_chipset.probe_result;
    + need_pll_quirk = amd_chipset.probe_result;

    spin_unlock_irqrestore(&amd_lock, flags);

    @@ -259,7 +266,7 @@ commit:
    spin_unlock_irqrestore(&amd_lock, flags);
    }

    - return ret;
    + return need_pll_quirk;
    }
    EXPORT_SYMBOL_GPL(usb_amd_find_chipset_info);


    \
     
     \ /
      Last update: 2019-08-02 12:09    [W:4.139 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site