Hi, Sandeep Singh <[email protected]> writes: >>>> Sandeep Singh <[email protected]> writes: >>>>> @@ -260,11 +266,12 @@ int usb_hcd_amd_remote_wakeup_quirk(struct pci_dev >>>>> *pdev) >>>>> { >>>>> /* Make sure amd chipset type has already been initialized */ >>>>> usb_amd_find_chipset_info(); >>>>> - if (amd_chipset.sb_type.gen != AMD_CHIPSET_YANGTZE) >>>>> - return 0; >>>>> - >>>>> - dev_dbg(&pdev->dev, "QUIRK: Enable AMD remote wakeup fix\n"); >>>>> - return 1; >>>>> + if (amd_chipset.sb_type.gen == AMD_CHIPSET_YANGTZE || >>>>> + AMD_CHIPSET_TAISHAN) { >>>> >>>> Sure this is what you wanna do? >>>> >>> >>> Yes. Please.. >> >> Are you sure this works the way you think it will? >> > Yes. Can you please help me to understand your concern.
Try evaluating that expression in your head. When will it be true and when will it false? -- balbi
signature.asc
Description: PGP signature
