Re: [External] : [PATCH v4 2/3] PCI: endpoint: Add API pci_epf_assign_bar_space()

[ALOK TIWARI]

On 10/7/2025 8:41 PM, Frank Li wrote:
> On Tue, Oct 07, 2025 at 12:25:59AM +0530, ALOK TIWARI wrote:
> > On 10/1/2025 2:09 AM, Frank Li wrote:
> > > +int pci_epf_assign_bar_space(struct pci_epf *epf, size_t size,
> > > +			     enum pci_barno bar,
> > > +			     const struct pci_epc_features *epc_features,
> > > +			     enum pci_epc_interface_type type,
> > > +			     dma_addr_t inbound_addr);
> > nit: Any particular reason for using inbound_addr instead of bar_addr ?
> bar_addr is easy to confuse with RC side's bar windows's address. This one
> means the ATU convert bar's inbound transfer to what EP side's address.
>
> The below API also use term 'inbound_addr'.
>
> It is not big deal. Manivannan or Niklas Cassel, do you have any perfer?
>
> I am okay for both names.

It is not a major issue, just noticed that the .c file refers to 
dma_addr_t bar_addr while the .h file uses inbound_addr.
Just curious to know the reason.

> Frank
> > > +
> > >    int pci_epf_align_inbound_addr(struct pci_epf *epf, enum pci_barno bar,
> > >    			       u64 addr, dma_addr_t *base, size_t *off);
> > >    int pci_epf_bind(struct pci_epf *epf);
> >
> > Thanks,
> > Alok



Thanks,
Alok