On Thu, Feb 17, 2022 at 3:54 PM Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote: > > In the subject, "pcie_ep_set_bar" looks like *part* of a function > name. Please include the entire function name and add "()" after it. Okay > > 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. Thanks. That's ntb_mw_set_trans() at drivers/ntb/ntb_transport.c. > > s/dynamtically/dynamically/ > > Please make the commit log say what the patch *does*, not just what > needs to happen. Okay, this patch allows changing PCI BAR's address. For example, previous BAR0 map to physical address to 0xF1000000, now can change to new physical address 0xF2000000 by call function pci_epc_set_bar() The patch 3/4 is totally new method to make NTB work at RC and EP system, I hope the basic structure is acceptable. > > Bjorn
