public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Matthew Garrett <mjg@redhat.com>
Cc: Lan Tianyu <tianyu.lan@intel.com>,
	linux-acpi@vger.kernel.org, linux-usb@vger.kernel.org,
	lenb@kernel.org
Subject: Re: [PATCH V2 3/4] usb: Bind devices to ACPI devices when possible
Date: Wed, 14 Mar 2012 07:29:32 -0700	[thread overview]
Message-ID: <20120314142932.GD27770@kroah.com> (raw)
In-Reply-To: <20120314124811.GB5155@srcf.ucam.org>

On Wed, Mar 14, 2012 at 12:48:11PM +0000, Matthew Garrett wrote:
> On Wed, Mar 14, 2012 at 09:20:44AM +0800, Lan Tianyu wrote:
> > On 2012年03月13日 05:45, Matthew Garrett wrote:
> > >Built-in USB devices will typically have a representation in the system
> > >ACPI tables. Add support for binding the two together so the USB code can
> > >make use of the associated methods.
> > hi Matthew:
> > 	acpi glue framework only can cover situation that the port has been
> > connected with some device. so without device, no acpi handle can be used to
> > access "UPC" and "PLD".
> > 	Whether the usb port is user-visible or not is also useful when there
> > is no device. For example, if it is known that the usb port is not user-visible
> > and connectable. The usb hub port can be power-off.
> > 	So usb/acpi binding should consider those usb ports without plugging device.
> > Does this make sense? :)
> 
> I agree that this woud be useful, but right now we don't have port 
> objects so there's nowhere to expose this. Greg, do you have any 
> feelings about how this coud be handled?

If you wanted to do something like this, you are right, you would need
to create a "port" device and put it on every hub.  Odds are that's
probably not worth it as you will be adding more indirection on every
USB device, yet only ACPI based systems would be able to do anything
with them.

Is there really a power savings to turn off a port if nothing is present
in it?  If so, how do we know to wake it up?  What if we really wanted
it on just to "charge" something, and it never was showing up as a real
device?  Messing with that could get very tricky very quickly.

thanks,

greg k-h
--
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

  reply	other threads:[~2012-03-14 14:29 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-12 21:45 [PATCH V2 1/4] ACPI: Fix up _PLD methods Matthew Garrett
2012-03-12 21:45 ` [PATCH V2 2/4] ACPI: Add _PLD support Matthew Garrett
2012-03-13 11:07 ` [PATCH V2 1/4] ACPI: Fix up _PLD methods Sergei Shtylyov
     [not found]   ` <4F5F2A64.8030403-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
2012-03-13 11:38     ` Matthew Garrett
     [not found] ` <1331588747-1247-1-git-send-email-mjg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-03-12 21:45   ` [PATCH V2 3/4] usb: Bind devices to ACPI devices when possible Matthew Garrett
     [not found]     ` <1331588747-1247-3-git-send-email-mjg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-03-14  1:20       ` Lan Tianyu
2012-03-14 12:48         ` Matthew Garrett
2012-03-14 14:29           ` Greg KH [this message]
2012-03-14 14:44             ` Alan Stern
2012-03-12 21:45   ` [PATCH V2 4/4] usb: Set device removable state based on ACPI USB data Matthew Garrett
2012-03-13 21:32   ` [PATCH V2 1/4] ACPI: Fix up _PLD methods Greg KH
2012-03-13 21:38     ` Matthew Garrett
2012-03-13 21:49       ` Greg KH
     [not found]         ` <20120313214941.GA4534-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2012-04-10  7:56           ` Lan, Tianyu
     [not found]             ` <4CFBC02C07DA244CA19D6815A05BE6EE05C77B-0J0gbvR4kThpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2012-04-10 13:01               ` Moore, Robert

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=20120314142932.GD27770@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mjg@redhat.com \
    --cc=tianyu.lan@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox