All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lan Tianyu <tianyu.lan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
To: Alan Stern <stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org>
Cc: gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org,
	lenb-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	sarah.a.sharp-VuQAYsv1563Yd54FQh9/CA@public.gmane.org
Subject: Re: [RFC PATCH V3 3/3] usb : Add sysfs files to control usb port's power
Date: Tue, 05 Jun 2012 16:30:51 +0800	[thread overview]
Message-ID: <4FCDC3BB.50809@intel.com> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1206041040510.1555-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>

On 2012年06月04日 23:02, Alan Stern wrote:
> On Mon, 4 Jun 2012, Lan Tianyu wrote:
> s/attribue/attribute/  -- here and below
>
>> +		allows the user space to control the power policy on the usb port
>> +
>> +		All ports have one of the following three values for portX/control
>> +		"auto" - For ports with device, if the device was not in the suspend
>> +			at that pointer, the power would remain on but it would power
>> +			off when it was suspended. If the the device was in the suspend,
>> +			"auto" means the device could be put into much lower state so
>> +			the device would be resumed and suspended again.
>
> The patch does not implement anything at all for "auto".  It should do
> _something_.
>
Yeah, if we don't power off port without device when echo "auto" > portX/control,
this patch does nothing about "auto". Original plan is to deal with port without
devices firstly. Do you mean I should add the process of port with device in this
patch?


-- 
Best Regards
Tianyu Lan
linux kernel enabling team
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2012-06-05  8:30 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1338799665-13997-1-git-send-email-tianyu.lan@intel.com>
2012-06-04  8:47 ` [RFC PATCH V3 2/3] USB/ACPI: Add usb port's acpi power control in the xhci PORT_POWER feature request process Lan Tianyu
2012-06-04  8:47 ` [RFC PATCH V3 3/3] usb : Add sysfs files to control usb port's power Lan Tianyu
     [not found]   ` <1338799665-13997-3-git-send-email-tianyu.lan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2012-06-04 15:02     ` Alan Stern
     [not found]       ` <Pine.LNX.4.44L0.1206041040510.1555-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2012-06-05  8:30         ` Lan Tianyu [this message]
2012-06-05 14:18           ` Alan Stern
     [not found]             ` <Pine.LNX.4.44L0.1206051017040.1542-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2012-06-05 15:58               ` Sarah Sharp
2012-06-06  1:53             ` Lan Tianyu
2012-06-06 14:21               ` Alan Stern
2012-06-07  6:45                 ` Lan Tianyu
2012-06-07 14:24                   ` Alan Stern

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=4FCDC3BB.50809@intel.com \
    --to=tianyu.lan-ral2jqcrhueavxtiumwx3w@public.gmane.org \
    --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
    --cc=lenb-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=sarah.a.sharp-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org \
    /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.