From: Lan Tianyu <tianyu.lan@intel.com>
To: Sarah Sharp <sarah.a.sharp@linux.intel.com>
Cc: lenb@kernel.org, gregkh@linuxfoundation.org,
linux-usb@vger.kernel.org, linux-acpi@vger.kernel.org,
stern@rowland.harvard.edu, mjg59@srcf.ucam.org
Subject: Re: [PATCH 5/7] usb: move struct usb_device->children to struct usb_hub_port->child
Date: Wed, 18 Apr 2012 10:19:59 +0800 [thread overview]
Message-ID: <4F8E24CF.8050300@intel.com> (raw)
In-Reply-To: <20120417170225.GA4670@xanatos>
hi sarah:
Oh. Thanks for review. Good catching. This is a missing
caused
by carelessness during updating patch. I will update patch soon.
于2012年04月18日 星期三 01时02分25秒,Sarah Sharp写到:
> On Tue, Apr 17, 2012 at 02:02:00PM +0800, Lan Tianyu wrote:
>> Move child's pointer to the struct usb_hub_port since the child device
>> is directly associated with the port. Provide usb_get_hub_child_device()
>> to get child's pointer.
>
> It looks like you deleted the documentation for the children pointer
> from usb_hub_port, but didn't actually remove the pointer from the
> structure. Did you mean to do that?
>
>> diff --git a/include/linux/usb.h b/include/linux/usb.h
>> index 73b68d1..1ea01a5 100644
>> --- a/include/linux/usb.h
>> +++ b/include/linux/usb.h
>> @@ -428,7 +428,6 @@ enum usb_device_removable {
>> * access from userspace
>> * @usbfs_dentry: usbfs dentry entry for the device
>> * @maxchild: number of ports if hub
>> - * @children: child devices - USB devices that are attached to this hub
>> * @quirks: quirks of the whole device
>> * @urbnum: number of URBs submitted for the whole device
>> * @active_duration: total time device is not suspended
>> @@ -527,6 +526,8 @@ static inline struct usb_device *interface_to_usbdev(struct usb_interface *intf)
>>
>> extern struct usb_device *usb_get_dev(struct usb_device *dev);
>> extern void usb_put_dev(struct usb_device *dev);
>> +extern struct usb_device *usb_get_hub_child_device(struct usb_device *hdev,
>> + int port1);
>>
>> /* USB device locking */
>> #define usb_lock_device(udev) device_lock(&(udev)->dev)
>> --
>> 1.7.6.rc2.8.g28eb
>>
>
--
Best Regards
Tianyu Lan
linux kernel enabling team
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2012-04-18 2:24 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-17 6:01 [PATCH 0/7] usb/acpi: Add binding usb device with acpi Lan Tianyu
2012-04-17 6:01 ` [PATCH 1/7] ACPI: Add _PLD support Lan Tianyu
2012-04-17 6:01 ` [PATCH 2/7] usb: Bind devices to ACPI devices when possible Lan Tianyu
[not found] ` <1334642522-22021-1-git-send-email-tianyu.lan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2012-04-17 6:01 ` [PATCH 3/7] usb: Set device removable state based on ACPI USB data Lan Tianyu
2012-04-17 6:01 ` [PATCH 4/7] usb: add struct usb_hub_port to store port related members Lan Tianyu
2012-04-17 6:02 ` [PATCH 5/7] usb: move struct usb_device->children to struct usb_hub_port->child Lan Tianyu
[not found] ` <1334642522-22021-6-git-send-email-tianyu.lan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2012-04-17 17:02 ` Sarah Sharp
2012-04-18 2:19 ` Lan Tianyu [this message]
2012-04-17 6:02 ` [PATCH 6/7] usb/acpi: add the support of usb hub ports' acpi binding without attached devices Lan Tianyu
2012-04-17 6:02 ` [PATCH 7/7] usb/acpi: add usb check for the connect type of usb port Lan Tianyu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F8E24CF.8050300@intel.com \
--to=tianyu.lan@intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=sarah.a.sharp@linux.intel.com \
--cc=stern@rowland.harvard.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.