diff mbox series

[-next,v2] misc: pci_endpoint_test: Remove some unused functions

Message ID 20240704023227.87039-1-jiapeng.chong@linux.alibaba.com (mailing list archive)
State Accepted
Delegated to: Krzysztof Wilczyński
Headers show
Series [-next,v2] misc: pci_endpoint_test: Remove some unused functions | expand

Commit Message

Jiapeng Chong July 4, 2024, 2:32 a.m. UTC
These functions are defined in the pci_endpoint_test.c file, but not
called elsewhere, so delete these unused functions.

drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9064
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
Changes in v2:
  -Modify the branch name in the topic. 

 drivers/misc/pci_endpoint_test.c | 12 ------------
 1 file changed, 12 deletions(-)

Comments

Krzysztof Wilczyński July 4, 2024, 7:14 a.m. UTC | #1
Hello,

> These functions are defined in the pci_endpoint_test.c file, but not
> called elsewhere, so delete these unused functions.
> 
> drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
> drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.

Have you see my question to the first version of this patch?

	Krzysztof
Greg KH July 4, 2024, 12:58 p.m. UTC | #2
On Thu, Jul 04, 2024 at 04:14:06PM +0900, Krzysztof Wilczyński wrote:
> Hello,
> 
> > These functions are defined in the pci_endpoint_test.c file, but not
> > called elsewhere, so delete these unused functions.
> > 
> > drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
> > drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.
> 
> Have you see my question to the first version of this patch?

Yeah, I don't understand this at all, how is this patch even building?

What tree/branch is it made against?

thanks,

greg k-h
Rick Wertenbroek July 4, 2024, 1:49 p.m. UTC | #3
On Thu, Jul 04, 2024 at 04:14:06PM +0900, Krzysztof Wilczyński wrote:
> Hello,
> 
> > These functions are defined in the pci_endpoint_test.c file, but not
> > called elsewhere, so delete these unused functions.
> > 
> > drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
> > drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.
> 
> Have you see my question to the first version of this patch?
> 
> 	Krzysztof

Hello,
The functions were removed in this patch : https://lore.kernel.org/linux-pci/20240322164139.678228-1-cassel@kernel.org/
So in linux-next they are indeed not used anymore.

You applied it to misc : https://lore.kernel.org/linux-pci/20240517100929.GB202520@rocinante/

Regards,
Rick
Rick Wertenbroek July 4, 2024, 2 p.m. UTC | #4
On Thu, Jul 04, 2024 at 02:58:10PM +0200, Greg KH wrote:
> On Thu, Jul 04, 2024 at 04:14:06PM +0900, Krzysztof Wilczyński wrote:
> > Hello,
> >
> > > These functions are defined in the pci_endpoint_test.c file, but not
> > > called elsewhere, so delete these unused functions.
> > >
> > > drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
> > > drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.
> >
> > Have you see my question to the first version of this patch?
>
> Yeah, I don't understand this at all, how is this patch even building?
>
> What tree/branch is it made against?
>
> thanks,
>
> greg k-h

Hi,
I can confirm in linux-next tag next-20240703 the functions are indeed removed.
See commit 0ace3d3b70ed6a474d52fc44ee9b0b1f16ca3724
misc: pci_endpoint_test: Use memcpy_toio()/memcpy_fromio() for BAR tests

Regards,
Rick
Krzysztof Wilczyński July 4, 2024, 2:22 p.m. UTC | #5
Hello,

> > > These functions are defined in the pci_endpoint_test.c file, but not
> > > called elsewhere, so delete these unused functions.
> > > 
> > > drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
> > > drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.
> > 
> > Have you see my question to the first version of this patch?
[...]
> The functions were removed in this patch : https://lore.kernel.org/linux-pci/20240322164139.678228-1-cassel@kernel.org/
> So in linux-next they are indeed not used anymore.
> 
> You applied it to misc : https://lore.kernel.org/linux-pci/20240517100929.GB202520@rocinante/

Ahh, it was sometime in May...  I assumed we already got this into the
mainline, hence my confusion.  Hard to remember what went out when.

Also, I looked only at our head branch first, as I always do, before
I noticed the updated tag.

Thank you for getting back to me.  Appreciated.

I will get this applied against our pending "misc" branch, should sort
out the warnings on next.

	Krzysztof
Krzysztof Wilczyński July 6, 2024, 3:18 a.m. UTC | #6
Hello,

> These functions are defined in the pci_endpoint_test.c file, but not
> called elsewhere, so delete these unused functions.
> 
> drivers/misc/pci_endpoint_test.c:144:19: warning: unused function 'pci_endpoint_test_bar_readl'.
> drivers/misc/pci_endpoint_test.c:150:20: warning: unused function 'pci_endpoint_test_bar_writel'.

Applied to misc, thank you!

[1/1] misc: pci_endpoint_test: Remove unused pci_endpoint_test_bar_{readl,writel} functions
      https://git.kernel.org/pci/pci/c/c22359447a19

	Krzysztof
diff mbox series

Patch

diff --git a/drivers/misc/pci_endpoint_test.c b/drivers/misc/pci_endpoint_test.c
index edced893221e..3aaaf47fa4ee 100644
--- a/drivers/misc/pci_endpoint_test.c
+++ b/drivers/misc/pci_endpoint_test.c
@@ -144,18 +144,6 @@  static inline void pci_endpoint_test_writel(struct pci_endpoint_test *test,
 	writel(value, test->base + offset);
 }
 
-static inline u32 pci_endpoint_test_bar_readl(struct pci_endpoint_test *test,
-					      int bar, int offset)
-{
-	return readl(test->bar[bar] + offset);
-}
-
-static inline void pci_endpoint_test_bar_writel(struct pci_endpoint_test *test,
-						int bar, u32 offset, u32 value)
-{
-	writel(value, test->bar[bar] + offset);
-}
-
 static irqreturn_t pci_endpoint_test_irqhandler(int irq, void *dev_id)
 {
 	struct pci_endpoint_test *test = dev_id;