public inbox for linux-staging@lists.linux.dev
 help / color / mirror / Atom feed
* [PATCH] staging: rtl8192u: remove entry from Makefile
@ 2023-10-17  8:13 Greg Kroah-Hartman
  2023-10-17 10:35 ` Thorsten Leemhuis
  0 siblings, 1 reply; 5+ messages in thread
From: Greg Kroah-Hartman @ 2023-10-17  8:13 UTC (permalink / raw)
  To: linux-staging
  Cc: linux-kernel, Greg Kroah-Hartman, Thorsten Leemhuis,
	Philipp Hortmann, Arnd Bergmann

In commit 697455ce4110 ("staging: rtl8192u: Remove broken driver"), the
driver was removed, along with the Kconfig entry, but the Makefile line
in drivers/staging/Makefile was not updated, so things like 'make clean'
fail to work properly as they will decend into all subdirectories to try
to clean things up.

Resolve this by removing the entry in the main staging Makefile.

Reported-by: Thorsten Leemhuis <linux@leemhuis.info>
Cc: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Fixes: 697455ce4110 ("staging: rtl8192u: Remove broken driver")
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/staging/Makefile | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/staging/Makefile b/drivers/staging/Makefile
index ffa70dda481d..0f6379436443 100644
--- a/drivers/staging/Makefile
+++ b/drivers/staging/Makefile
@@ -4,7 +4,6 @@
 obj-y				+= media/
 obj-$(CONFIG_PRISM2_USB)	+= wlan-ng/
 obj-$(CONFIG_FB_OLPC_DCON)	+= olpc_dcon/
-obj-$(CONFIG_RTL8192U)		+= rtl8192u/
 obj-$(CONFIG_RTL8192E)		+= rtl8192e/
 obj-$(CONFIG_RTL8723BS)		+= rtl8723bs/
 obj-$(CONFIG_R8712U)		+= rtl8712/
-- 
2.42.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] staging: rtl8192u: remove entry from Makefile
  2023-10-17  8:13 [PATCH] staging: rtl8192u: remove entry from Makefile Greg Kroah-Hartman
@ 2023-10-17 10:35 ` Thorsten Leemhuis
  2023-10-17 13:35   ` Greg Kroah-Hartman
  0 siblings, 1 reply; 5+ messages in thread
From: Thorsten Leemhuis @ 2023-10-17 10:35 UTC (permalink / raw)
  To: Greg Kroah-Hartman, linux-staging
  Cc: linux-kernel, Philipp Hortmann, Arnd Bergmann

On 17.10.23 10:13, Greg Kroah-Hartman wrote:
> In commit 697455ce4110 ("staging: rtl8192u: Remove broken driver"), the
> driver was removed, along with the Kconfig entry, but the Makefile line
> in drivers/staging/Makefile was not updated, so things like 'make clean'
> fail to work properly as they will decend into all subdirectories to try
> to clean things up.
> 
> Resolve this by removing the entry in the main staging Makefile.
> 
> Reported-by: Thorsten Leemhuis <linux@leemhuis.info>
> Cc: Philipp Hortmann <philipp.g.hortmann@gmail.com>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Fixes: 697455ce4110 ("staging: rtl8192u: Remove broken driver")
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Thx Greg, that fixed things for my case. In case anyone cares:

Tested-by: Thorsten Leemhuis <linux@leemhuis.info>

Ciao, Thorsten

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] staging: rtl8192u: remove entry from Makefile
  2023-10-17 10:35 ` Thorsten Leemhuis
@ 2023-10-17 13:35   ` Greg Kroah-Hartman
  2023-10-17 18:07     ` Philipp Hortmann
  0 siblings, 1 reply; 5+ messages in thread
From: Greg Kroah-Hartman @ 2023-10-17 13:35 UTC (permalink / raw)
  To: Thorsten Leemhuis
  Cc: linux-staging, linux-kernel, Philipp Hortmann, Arnd Bergmann

On Tue, Oct 17, 2023 at 12:35:57PM +0200, Thorsten Leemhuis wrote:
> On 17.10.23 10:13, Greg Kroah-Hartman wrote:
> > In commit 697455ce4110 ("staging: rtl8192u: Remove broken driver"), the
> > driver was removed, along with the Kconfig entry, but the Makefile line
> > in drivers/staging/Makefile was not updated, so things like 'make clean'
> > fail to work properly as they will decend into all subdirectories to try
> > to clean things up.
> > 
> > Resolve this by removing the entry in the main staging Makefile.
> > 
> > Reported-by: Thorsten Leemhuis <linux@leemhuis.info>
> > Cc: Philipp Hortmann <philipp.g.hortmann@gmail.com>
> > Cc: Arnd Bergmann <arnd@arndb.de>
> > Fixes: 697455ce4110 ("staging: rtl8192u: Remove broken driver")
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> 
> Thx Greg, that fixed things for my case. In case anyone cares:

I care, thanks!

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] staging: rtl8192u: remove entry from Makefile
  2023-10-17 13:35   ` Greg Kroah-Hartman
@ 2023-10-17 18:07     ` Philipp Hortmann
  2023-10-17 18:29       ` Greg Kroah-Hartman
  0 siblings, 1 reply; 5+ messages in thread
From: Philipp Hortmann @ 2023-10-17 18:07 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Thorsten Leemhuis
  Cc: linux-staging, linux-kernel, Arnd Bergmann

On 10/17/23 15:35, Greg Kroah-Hartman wrote:
> On Tue, Oct 17, 2023 at 12:35:57PM +0200, Thorsten Leemhuis wrote:
>> On 17.10.23 10:13, Greg Kroah-Hartman wrote:
>>> In commit 697455ce4110 ("staging: rtl8192u: Remove broken driver"), the
>>> driver was removed, along with the Kconfig entry, but the Makefile line
>>> in drivers/staging/Makefile was not updated, so things like 'make clean'
>>> fail to work properly as they will decend into all subdirectories to try
>>> to clean things up.
>>>
>>> Resolve this by removing the entry in the main staging Makefile.
>>>
>>> Reported-by: Thorsten Leemhuis <linux@leemhuis.info>
>>> Cc: Philipp Hortmann <philipp.g.hortmann@gmail.com>
>>> Cc: Arnd Bergmann <arnd@arndb.de>
>>> Fixes: 697455ce4110 ("staging: rtl8192u: Remove broken driver")
>>> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>>
>> Thx Greg, that fixed things for my case. In case anyone cares:
> 
> I care, thanks!


Sorry for messing up. I did a compile test of the entire kernel but I 
was not aware that I also need to test 'make clean'

Bye Philipp

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] staging: rtl8192u: remove entry from Makefile
  2023-10-17 18:07     ` Philipp Hortmann
@ 2023-10-17 18:29       ` Greg Kroah-Hartman
  0 siblings, 0 replies; 5+ messages in thread
From: Greg Kroah-Hartman @ 2023-10-17 18:29 UTC (permalink / raw)
  To: Philipp Hortmann
  Cc: Thorsten Leemhuis, linux-staging, linux-kernel, Arnd Bergmann

On Tue, Oct 17, 2023 at 08:07:33PM +0200, Philipp Hortmann wrote:
> On 10/17/23 15:35, Greg Kroah-Hartman wrote:
> > On Tue, Oct 17, 2023 at 12:35:57PM +0200, Thorsten Leemhuis wrote:
> > > On 17.10.23 10:13, Greg Kroah-Hartman wrote:
> > > > In commit 697455ce4110 ("staging: rtl8192u: Remove broken driver"), the
> > > > driver was removed, along with the Kconfig entry, but the Makefile line
> > > > in drivers/staging/Makefile was not updated, so things like 'make clean'
> > > > fail to work properly as they will decend into all subdirectories to try
> > > > to clean things up.
> > > > 
> > > > Resolve this by removing the entry in the main staging Makefile.
> > > > 
> > > > Reported-by: Thorsten Leemhuis <linux@leemhuis.info>
> > > > Cc: Philipp Hortmann <philipp.g.hortmann@gmail.com>
> > > > Cc: Arnd Bergmann <arnd@arndb.de>
> > > > Fixes: 697455ce4110 ("staging: rtl8192u: Remove broken driver")
> > > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > 
> > > Thx Greg, that fixed things for my case. In case anyone cares:
> > 
> > I care, thanks!
> 
> 
> Sorry for messing up. I did a compile test of the entire kernel but I was
> not aware that I also need to test 'make clean'

Not a problem, I didn't either, and neither did the 0-day bot.

greg k-h

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-10-17 18:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-17  8:13 [PATCH] staging: rtl8192u: remove entry from Makefile Greg Kroah-Hartman
2023-10-17 10:35 ` Thorsten Leemhuis
2023-10-17 13:35   ` Greg Kroah-Hartman
2023-10-17 18:07     ` Philipp Hortmann
2023-10-17 18:29       ` Greg Kroah-Hartman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox