Patchwork Fix cdc-phonet build

login
register
mail settings
Submitter Chris Clayton
Date July 26, 2011, 10:20 p.m.
Message ID <201107262320.22338.chris2553@googlemail.com>
Download mbox | patch
Permalink /patch/106943/
State Accepted
Delegated to: David Miller
Headers show

Comments

Chris Clayton - July 26, 2011, 10:20 p.m.
Try to send to correct address this time!

----------  Forwarded Message  ----------

Subject: [PATCH] Fix cdc-phonet build
Date: Saturday 23 Jul 2011
From: Chris Clayton <chris2553@googlemail.com>
To: linux-net@vger.kernel.org

cdc-phonet does not presently build on linux-3.0 because there is no entry for it in 
drivers/net/Makefile. This patch adds that entry.

Signed-off-by: Chris Clayton <chris2553@googlemail.com>
---


-------------------------------------------------------
David Miller - July 28, 2011, 5:41 a.m.
From: Chris Clayton <chris2553@googlemail.com>
Date: Tue, 26 Jul 2011 23:20:22 +0100

> cdc-phonet does not presently build on linux-3.0 because there is no entry for it in 
> drivers/net/Makefile. This patch adds that entry.
> 
> Signed-off-by: Chris Clayton <chris2553@googlemail.com>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Chris Clayton - July 29, 2011, 8:49 p.m.
On Thursday 28 July 2011 06:41:17 David Miller wrote:
> From: Chris Clayton <chris2553@googlemail.com>
> Date: Tue, 26 Jul 2011 23:20:22 +0100
>
> > cdc-phonet does not presently build on linux-3.0 because there is no
> > entry for it in drivers/net/Makefile. This patch adds that entry.
> >
> > Signed-off-by: Chris Clayton <chris2553@googlemail.com>
>
> Applied, thanks.

Actually, I've just checked and the same problem exists in 2.6.39.3, so once the patch is Linus' 
tree, I guess it should go in stable too. Do you have that in hand, David or should I send to 
stable once the patch is applied to mainline?
David Miller - July 30, 2011, 12:45 a.m.
From: Chris Clayton <chris2553@googlemail.com>
Date: Fri, 29 Jul 2011 21:49:58 +0100

> On Thursday 28 July 2011 06:41:17 David Miller wrote:
>> From: Chris Clayton <chris2553@googlemail.com>
>> Date: Tue, 26 Jul 2011 23:20:22 +0100
>>
>> > cdc-phonet does not presently build on linux-3.0 because there is no
>> > entry for it in drivers/net/Makefile. This patch adds that entry.
>> >
>> > Signed-off-by: Chris Clayton <chris2553@googlemail.com>
>>
>> Applied, thanks.
> 
> Actually, I've just checked and the same problem exists in 2.6.39.3, so once the patch is Linus' 
> tree, I guess it should go in stable too. Do you have that in hand, David or should I send to 
> stable once the patch is applied to mainline?

I know it's also broken there, and I'll take care of submitting it to
-stable.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

--- linux-3.0/drivers/net/Makefile.orig 2011-07-23 22:07:45.000000000 +0100
+++ linux-3.0/drivers/net/Makefile      2011-07-23 22:10:33.000000000 +0100
@@ -283,6 +283,7 @@ 
 obj-$(CONFIG_USB_USBNET)        += usb/
 obj-$(CONFIG_USB_ZD1201)        += usb/
 obj-$(CONFIG_USB_IPHETH)        += usb/
+obj-$(CONFIG_USB_CDC_PHONET)   += usb/

 obj-$(CONFIG_WLAN) += wireless/
 obj-$(CONFIG_NET_TULIP) += tulip/