Skip to contentSkip to conversation

r8152: add missing Makefile rule

PATCH In review [email protected] 2 participants 3 messages Aug 1, 2014 – Aug 3, 2014

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 r8152: add missing Makefile rule

From: JF Le Fillatre <[email protected]>

Add missing Makefile rule for r8152 driver

In the current kernel the r8152 driver is *never* built because of a missing rule in drivers/net/Makefile, despite being selected as built-in or module in the .config file. There is no error message or warning to indicate that the driver isn't built. This change adds the rule and lets the driver build.

Tested as built-in and module for 3.15.8.

Signed-off by: JF Le Fillatre <[email protected]>

---

Is seems that this issue has existed ever since the driver was added back in 3.10. In Linus' git tree the drivers/net/Makefile was last modified before the inclusion of the driver, so I don't know if it's ever been possible to build it.

drivers/net/Makefile 2014-08-01 21:52:15.418345294 +0200 renamed from drivers/net/Makefile.orig 2014-08-01 21:51:43.058347887 +0200 +1 -0
@@ -65,6 +65,7 @@ obj-$(CONFIG_USB_CATC)          += usb/
 obj-$(CONFIG_USB_KAWETH)        += usb/
 obj-$(CONFIG_USB_PEGASUS)       += usb/
 obj-$(CONFIG_USB_RTL8150)       += usb/
+obj-$(CONFIG_USB_RTL8152)       += usb/
 obj-$(CONFIG_USB_HSO)        += usb/
 obj-$(CONFIG_USB_USBNET)        += usb/
 obj-$(CONFIG_USB_ZD1201)        += usb/
maintainer
#1 lore raw

From: [email protected]
Date: Sat, 02 Aug 2014 01:26:40 +0200

Quoted 4 lines

Is seems that this issue has existed ever since the driver was added back in 3.10. In Linus' git tree the drivers/net/Makefile was last modified before the inclusion of the driver, so I don't know if it's ever been possible to build it.

It gets built, as long as at least one other USB networking driver has been selected.

Most people (distributions, developers, maintainers) select all drivers or some large subset thereof, so would never actually run into this problem.

You'd have to specifically have a configuration that enabled r8152 and no other USB networking driver, in order to trigger this.

Anyways, I'll apply this, thanks.

#2 lore raw
Quoted 11 lines · David Miller

From: [email protected]
Date: Sat, 02 Aug 2014 01:26:40 +0200

Quoted 4 lines

Is seems that this issue has existed ever since the driver was added back in 3.10. In Linus' git tree the drivers/net/Makefile was last modified before the inclusion of the driver, so I don't know if it's ever been possible to build it.

It gets built, as long as at least one other USB networking driver has been selected.

Quoted 8 lines

Most people (distributions, developers, maintainers) select all drivers or some large subset thereof, so would never actually run into this problem.

You'd have to specifically have a configuration that enabled r8152 and no other USB networking driver, in order to trigger this.

Anyways, I'll apply this, thanks.

Very true, thank you for pointing it out. If you haven't done so yet, the patch description could be changed to something like:

When it is the only USB net driver selected, the r8152 driver doesn't get built because of a missing rule in drivers/net/Makefile. This change adds the rule and lets the driver build.

Thank you!
JF