In the subject, "pcie_ep_set_bar" looks like *part* of a function name. Please include the entire function name and add "()" after it. On Mon, Feb 14, 2022 at 11:38:41PM -0600, Frank Li wrote: > ntb_transfer will set memory map windows after probe. > So the inbound map address need be updated dynamtically. I don't see "ntb_transfer" in the tree. If it's a function, please add "()" after the name. Otherwise, please say more about what "ntb_transfer" is. s/dynamtically/dynamically/ Please make the commit log say what the patch *does*, not just what needs to happen. Bjorn
