Re: [PATCH] pci:endpoint Remove redundant returns

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

> > [+Cc linux-pci mailing list]
> 
> Where did this patch come from?  I don't see it on the mailing list,
> so there's no signed-off-by chain for it, and the Link in the commit
> doesn't go to the patch posting.

Patch for reference:

  From: Wang Jiang <jiangwang@xxxxxxxxxx>
  
  We get 1 warnings when building kernel withW=1:
  296: FILE: ./drivers/pci/endpoint/functions/pci-epf-test.c:296:
  return;
  
  In fact, void function return statements are not generally useful.
  
  Signed-off-by: Wang Jiang <jiangwang@xxxxxxxxxx>
  ---
   drivers/pci/endpoint/functions/pci-epf-test.c | 2 --
   1 file changed, 2 deletions(-)
  
  diff --git a/drivers/pci/endpoint/functions/pci-epf-test.c b/drivers/pci/endpoint/functions/pci-epf-test.c
  index 7c2ed6eae53a..cfdb38cd8cd7 100644
  --- a/drivers/pci/endpoint/functions/pci-epf-test.c
  +++ b/drivers/pci/endpoint/functions/pci-epf-test.c
  @@ -291,8 +291,6 @@ static void pci_epf_test_clean_dma_chan(struct pci_epf_test *epf_test)
   
   	dma_release_channel(epf_test->dma_chan_rx);
   	epf_test->dma_chan_rx = NULL;
  -
  -	return;
   }
   
   static void pci_epf_test_print_rate(struct pci_epf_test *epf_test,
  -- 
  2.25.1

> Please post all PCI-related patches to linux-pci@xxxxxxxxxxxxxxx.

Tentatively dropped.  I will wait for the author to post the patch again,
hopefully including the mailing list now.

Jiang, please resend your patch again adding the mailing list, so we can
pick it up.

The process to follow is outlined here:

  - https://www.kernel.org/doc/html/latest/process/submitting-patches.html

Thank you!

	Krzysztof




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux