USB: gadget: android: Use "rndis%d" for RNDIS device name instead of "usb%d"
authorMike Lockwood <lockwood@android.com>
Fri, 12 Aug 2011 21:35:42 +0000 (14:35 -0700)
committerMike Lockwood <lockwood@android.com>
Fri, 12 Aug 2011 22:12:08 +0000 (15:12 -0700)
Signed-off-by: Mike Lockwood <lockwood@android.com>
drivers/usb/gadget/android.c
drivers/usb/gadget/u_ether.c
drivers/usb/gadget/u_ether.h

index ebe67666b20e77187bc0e65bf42d8e5681fc71f0..c82b43ba628c911660e64b91257094087b915c81 100644 (file)
@@ -373,7 +373,7 @@ static int rndis_function_bind_config(struct android_usb_function *f,
                rndis->ethaddr[0], rndis->ethaddr[1], rndis->ethaddr[2],
                rndis->ethaddr[3], rndis->ethaddr[4], rndis->ethaddr[5]);
 
-       ret = gether_setup(c->cdev->gadget, rndis->ethaddr);
+       ret = gether_setup_name(c->cdev->gadget, rndis->ethaddr, "rndis");
        if (ret) {
                pr_err("%s: gether_setup failed\n", __func__);
                return ret;
index 6590501c29e53cd70d9a3e0af565f7c6a89da780..b5a30fee014b1aeba278485b4d560e5ba81d3d58 100644 (file)
@@ -764,6 +764,26 @@ static struct device_type gadget_type = {
  * Returns negative errno, or zero on success
  */
 int gether_setup(struct usb_gadget *g, u8 ethaddr[ETH_ALEN])
+{
+       return gether_setup_name(g, ethaddr, "usb");
+}
+
+/**
+ * gether_setup_name - initialize one ethernet-over-usb link
+ * @g: gadget to associated with these links
+ * @ethaddr: NULL, or a buffer in which the ethernet address of the
+ *     host side of the link is recorded
+ * @netname: name for network device (for example, "usb")
+ * Context: may sleep
+ *
+ * This sets up the single network link that may be exported by a
+ * gadget driver using this framework.  The link layer addresses are
+ * set up using module parameters.
+ *
+ * Returns negative errno, or zero on success
+ */
+int gether_setup_name(struct usb_gadget *g, u8 ethaddr[ETH_ALEN],
+               const char *netname)
 {
        struct eth_dev          *dev;
        struct net_device       *net;
@@ -787,7 +807,7 @@ int gether_setup(struct usb_gadget *g, u8 ethaddr[ETH_ALEN])
 
        /* network device setup */
        dev->net = net;
-       strcpy(net->name, "usb%d");
+       snprintf(net->name, sizeof(net->name), "%s%%d", netname);
 
        if (get_ether_addr(dev_addr, net->dev_addr))
                dev_warn(&g->dev,
index 5bb1021f6ab7c038203d83d91878e5896136ee9e..64b65f92168ab69471603fe0b95c353ec2ed26cf 100644 (file)
@@ -86,6 +86,9 @@ struct gether {
 /* netdev setup/teardown as directed by the gadget driver */
 int gether_setup(struct usb_gadget *g, u8 ethaddr[ETH_ALEN]);
 void gether_cleanup(void);
+/* variant of gether_setup that allows customizing network device name */
+int gether_setup_name(struct usb_gadget *g, u8 ethaddr[ETH_ALEN],
+               const char *netname);
 
 /* connect/disconnect is handled by individual functions */
 struct net_device *gether_connect(struct gether *);