Hi, On 7/5/2017 3:42 AM, Ethan Zhao wrote: > Sinan， > > About the patch attached, why clear the word of > PCI_EXP_DEVCTL_EXT_TAG ? does the device will be set by default after > POST it is not supported ? > > dev_info(&dev->dev, "clearing extended tags capability\n"); > > + pcie_capability_clear_word(dev, PCI_EXP_DEVCTL, > + PCI_EXP_DEVCTL_EXT_TAG); We don't usually trust the FW in Linux. Some outdated firmware might have done the same mistake and end users usually don't upgrade their BIOS in general. We end up working around issues in Linux with quirks etc. The other issue is this. Default value of the register is not zero. "Extended Tag Field Enable: Default value of this bit is implementation specific" https://pcisig.com/sites/default/files/specification_documents/ECN_Extended_Tag_Enable_Default_05Sept2008_final.pdf Hope this helps, Sinan -- Sinan Kaya Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.
