On Mon, 24 Feb 2014, Dan Williams wrote:

> @@ -163,12 +227,18 @@ int usb_hub_create_port_device(struct usb_hub *hub, int 
> port1)
>       port_dev->dev.parent = hub->intfdev;
>       port_dev->dev.groups = port_dev_group;
>       port_dev->dev.type = &usb_port_device_type;
> -     dev_set_name(&port_dev->dev, "port%d", port1);
> -
> +     dev_set_name(&port_dev->dev, "%s-port%d", dev_name(&hub->hdev->dev),
> +                     port1);

By the way, check out the usage of sscanf in usb_acpi_find_companion().  
It seems like a bad idea; better just to store the port number in the 
port structure.

Alan Stern

PS: Why are the usb_get_hub_port_connect_type() call and following 
material in that routine?  Why isn't they in usb_new_device() instead?

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to