From: Darren Hart <dvhart@infradead.org>
To: Azael Avalos <coproscefalo@gmail.com>
Cc: "platform-driver-x86@vger.kernel.org"
<platform-driver-x86@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Len Brown <len.brown@intel.com>,
mjg59@srcf.ucam.org
Subject: Re: [PATCH resend 5/6] toshiba_acpi: Add support to enable/disable USB 3
Date: Mon, 9 Feb 2015 20:55:52 -0800 [thread overview]
Message-ID: <20150210045552.GF37927@fury.dvhart.com> (raw)
In-Reply-To: <CAGdLNWGvqcTX09_kqZWScJFj96+L3=kjKE1d0aQKnQ_wrawazQ@mail.gmail.com>
On Mon, Feb 09, 2015 at 09:46:07PM -0700, Azael Avalos wrote:
> Hi Darren,
>
> 2015-02-09 21:11 GMT-07:00 Darren Hart <dvhart@infradead.org>:
> > On Mon, Feb 09, 2015 at 08:34:53PM -0700, Azael Avalos wrote:
> >> Toshiba laptops that come with USB 3 ports have a feature that lets
> >> them disable USB 3 functionality and act as a regular USB 2 port, and
> >> thus, saving power.
> >>
> >> This patch adds support to that feature, by creating a sysfs entry
> >> named "usb_three", acceptig only two parameters, 0 to disable the
> >> USB 3 (acting as a USB 2) and 1 to enable it.
> >>
> >
> > Similar question, does the system BIOS allow you to configure this setting? If
> > so, what would be the value for enabling it in the OS directly?
>
> Yes.
Yes it's available in the BIOS?
> A Toshiba provided app lets you configure all these parameters from Windows,
> and so, I'm developing a similar app for this, a few clicks and your
> settings are
> changed on the next reboot :-)
>
And I understand now that independent of the BIOS question, Windows has a
program to make these changes. Blech. Going to think on these overnight and get
back to you.
I'd welcome the thoughts of the previous maintainers as to whether we want to
include these kinds of OS-parity changes in the kernel. I'm probably leaning
toward taking them.
Matthew, Len, is there some kind of precedent I need to be aware of here?
--
Darren Hart
Intel Open Source Technology Center
next prev parent reply other threads:[~2015-02-10 4:55 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-10 3:34 [PATCH resend 0/6] toshiba_acpi: Add new features plus some procfs ports Azael Avalos
2015-02-10 3:34 ` [PATCH resend 1/6] toshiba_acpi: Add version entry to sysfs Azael Avalos
2015-02-10 3:34 ` [PATCH resend 2/6] toshiba_acpi: Add fan " Azael Avalos
2015-02-10 4:02 ` Darren Hart
2015-02-10 4:25 ` Azael Avalos
2015-02-10 3:34 ` [PATCH resend 3/6] toshiba_acpi: Add support for Keyboard functions mode Azael Avalos
2015-02-10 4:08 ` Darren Hart
2015-02-10 4:38 ` Azael Avalos
2015-02-10 3:34 ` [PATCH resend 4/6] toshiba_acpi: Add support for Panel Power ON Azael Avalos
2015-02-10 3:34 ` [PATCH resend 5/6] toshiba_acpi: Add support to enable/disable USB 3 Azael Avalos
2015-02-10 4:11 ` Darren Hart
2015-02-10 4:46 ` Azael Avalos
2015-02-10 4:55 ` Darren Hart [this message]
2015-02-10 5:02 ` Azael Avalos
2015-02-10 19:20 ` Darren Hart
2015-02-10 5:24 ` Matthew Garrett
2015-02-10 19:20 ` Darren Hart
2015-02-10 3:34 ` [PATCH resend 6/6] toshiba_acpi: Bump version number to 0.21 Azael Avalos
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=20150210045552.GF37927@fury.dvhart.com \
--to=dvhart@infradead.org \
--cc=coproscefalo@gmail.com \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=platform-driver-x86@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox