Commit d5575424 authored by Lan Tianyu's avatar Lan Tianyu Committed by Greg Kroah-Hartman

usb/acpi: Bind ACPI node to USB port, not usb_device.

In the ACPI DSDT table, only usb root hub and usb ports are ACPI device
nodes.  Originally, we bound the usb port's ACPI node to the usb device
attached to the port.  However, we want to access those ACPI port
methods when the port is empty, and there's no usb_device associated
with that port.

Now that the usb port is a real device, we can bind the port's ACPI node
to struct usb_port instead.
Signed-off-by: default avatarLan Tianyu <tianyu.lan@intel.com>
Signed-off-by: default avatarSarah Sharp <sarah.a.sharp@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent ff823c79
...@@ -5077,3 +5077,21 @@ struct usb_device *usb_hub_find_child(struct usb_device *hdev, ...@@ -5077,3 +5077,21 @@ struct usb_device *usb_hub_find_child(struct usb_device *hdev,
return hub->ports[port1 - 1]->child; return hub->ports[port1 - 1]->child;
} }
EXPORT_SYMBOL_GPL(usb_hub_find_child); EXPORT_SYMBOL_GPL(usb_hub_find_child);
#ifdef CONFIG_ACPI
/**
* usb_get_hub_port_acpi_handle - Get the usb port's acpi handle
* @hdev: USB device belonging to the usb hub
* @port1: port num of the port
*
* Return port's acpi handle if successful, NULL if params are
* invaild.
*/
acpi_handle usb_get_hub_port_acpi_handle(struct usb_device *hdev,
int port1)
{
struct usb_hub *hub = hdev_to_hub(hdev);
return DEVICE_ACPI_HANDLE(&hub->ports[port1 - 1]->dev);
}
#endif
...@@ -70,22 +70,59 @@ static int usb_acpi_check_pld(struct usb_device *udev, acpi_handle handle) ...@@ -70,22 +70,59 @@ static int usb_acpi_check_pld(struct usb_device *udev, acpi_handle handle)
static int usb_acpi_find_device(struct device *dev, acpi_handle *handle) static int usb_acpi_find_device(struct device *dev, acpi_handle *handle)
{ {
struct usb_device *udev; struct usb_device *udev;
struct device *parent;
acpi_handle *parent_handle; acpi_handle *parent_handle;
int port_num;
if (!is_usb_device(dev)) /*
return -ENODEV; * In the ACPI DSDT table, only usb root hub and usb ports are
* acpi device nodes. The hierarchy like following.
udev = to_usb_device(dev); * Device (EHC1)
parent = dev->parent; * Device (HUBN)
parent_handle = DEVICE_ACPI_HANDLE(parent); * Device (PR01)
* Device (PR11)
if (!parent_handle) * Device (PR12)
return -ENODEV; * Device (PR13)
* ...
*handle = acpi_get_child(parent_handle, udev->portnum); * So all binding process is divided into two parts. binding
* root hub and usb ports.
if (!*handle) */
if (is_usb_device(dev)) {
udev = to_usb_device(dev);
if (udev->parent)
return -ENODEV;
/* root hub's parent is the usb hcd. */
parent_handle = DEVICE_ACPI_HANDLE(dev->parent);
*handle = acpi_get_child(parent_handle, udev->portnum);
if (!*handle)
return -ENODEV;
return 0;
} else if (is_usb_port(dev)) {
sscanf(dev_name(dev), "port%d", &port_num);
/* Get the struct usb_device point of port's hub */
udev = to_usb_device(dev->parent->parent);
/*
* The root hub ports' parent is the root hub. The non-root-hub
* ports' parent is the parent hub port which the hub is
* connected to.
*/
if (!udev->parent) {
*handle = acpi_get_child(DEVICE_ACPI_HANDLE(&udev->dev),
port_num);
if (!*handle)
return -ENODEV;
} else {
parent_handle =
usb_get_hub_port_acpi_handle(udev->parent,
udev->portnum);
if (!parent_handle)
return -ENODEV;
*handle = acpi_get_child(parent_handle, port_num);
if (!*handle)
return -ENODEV;
}
} else
return -ENODEV; return -ENODEV;
/* /*
...@@ -102,7 +139,7 @@ static int usb_acpi_find_device(struct device *dev, acpi_handle *handle) ...@@ -102,7 +139,7 @@ static int usb_acpi_find_device(struct device *dev, acpi_handle *handle)
static struct acpi_bus_type usb_acpi_bus = { static struct acpi_bus_type usb_acpi_bus = {
.bus = &usb_bus_type, .bus = &usb_bus_type,
.find_bridge = NULL, .find_bridge = usb_acpi_find_device,
.find_device = usb_acpi_find_device, .find_device = usb_acpi_find_device,
}; };
......
#include <linux/pm.h> #include <linux/pm.h>
#include <linux/acpi.h>
struct dev_state; struct dev_state;
...@@ -115,6 +116,7 @@ extern struct bus_type usb_bus_type; ...@@ -115,6 +116,7 @@ extern struct bus_type usb_bus_type;
extern struct device_type usb_device_type; extern struct device_type usb_device_type;
extern struct device_type usb_if_device_type; extern struct device_type usb_if_device_type;
extern struct device_type usb_ep_device_type; extern struct device_type usb_ep_device_type;
extern struct device_type usb_port_device_type;
extern struct usb_device_driver usb_generic_driver; extern struct usb_device_driver usb_generic_driver;
static inline int is_usb_device(const struct device *dev) static inline int is_usb_device(const struct device *dev)
...@@ -132,6 +134,11 @@ static inline int is_usb_endpoint(const struct device *dev) ...@@ -132,6 +134,11 @@ static inline int is_usb_endpoint(const struct device *dev)
return dev->type == &usb_ep_device_type; return dev->type == &usb_ep_device_type;
} }
static inline int is_usb_port(const struct device *dev)
{
return dev->type == &usb_port_device_type;
}
/* Do the same for device drivers and interface drivers. */ /* Do the same for device drivers and interface drivers. */
static inline int is_usb_device_driver(struct device_driver *drv) static inline int is_usb_device_driver(struct device_driver *drv)
...@@ -166,6 +173,8 @@ extern void usb_notify_remove_bus(struct usb_bus *ubus); ...@@ -166,6 +173,8 @@ extern void usb_notify_remove_bus(struct usb_bus *ubus);
#ifdef CONFIG_ACPI #ifdef CONFIG_ACPI
extern int usb_acpi_register(void); extern int usb_acpi_register(void);
extern void usb_acpi_unregister(void); extern void usb_acpi_unregister(void);
extern acpi_handle usb_get_hub_port_acpi_handle(struct usb_device *hdev,
int port1);
#else #else
static inline int usb_acpi_register(void) { return 0; }; static inline int usb_acpi_register(void) { return 0; };
static inline void usb_acpi_unregister(void) { }; static inline void usb_acpi_unregister(void) { };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment