Re: [PATCH v4 04/17] PCI: epf-test: Fix DMA transfer completion detection

[Bjorn Helgaas <helgaas@kernel.org>]

On Thu, Apr 13, 2023 at 10:50:03AM +0900, Damien Le Moal wrote:
> On 4/4/23 19:16, Damien Le Moal wrote:
> > On 4/4/23 18:47, Shunsuke Mie wrote:
> >>> @@ -120,7 +129,6 @@ static int pci_epf_test_data_transfer(struct pci_epf_test *epf_test,
> >>>   	struct dma_async_tx_descriptor *tx;
> >>>   	struct dma_slave_config sconf = {};
> >>>   	struct device *dev = &epf->dev;
> >>> -	dma_cookie_t cookie;
> >>>   	int ret;
> >>>   
> >>>   	if (IS_ERR_OR_NULL(chan)) {
> >>> @@ -152,25 +160,33 @@ static int pci_epf_test_data_transfer(struct pci_epf_test *epf_test,
> >>>   	}
> >>>   
> >>>   	reinit_completion(&epf_test->transfer_complete);
> >>> +	epf_test->transfer_chan = chan;
> >>>   	tx->callback = pci_epf_test_dma_callback;
> >>>   	tx->callback_param = epf_test;
> >>> -	cookie = tx->tx_submit(tx);
> >>> +	epf_test->transfer_cookie = tx->tx_submit(tx);
> >>
> >> How about changing the code to use dmaengine_submit() API instead of 
> >> calling a raw function pointer?
> > 
> > This is done in patch 5 of the series.
> 
> Bjorn,
> 
> My apologies for not replying to the series cover letter as I do not have it
> locally (due to WD on-going network issues).
> 
> This is a ping about this series. Can we get it queued ?
> (I do not see it in PCI next tree)

Lorenzo and Krzysztof take care of the endpoint subsystem, so I'll
wait for them.  In the meantime I'll send a couple typo fixes and a
cursory review for trivial issue.s

Bjorn