From: Lu Baolu <baolu.lu@linux.intel.com>
To: felipe.balbi@linux.intel.com,
Mathias Nyman <mathias.nyman@intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Lee Jones <lee.jones@linaro.org>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
MyungJoo Ham <myungjoo.ham@samsung.com>,
Chanwoo Choi <cw00.choi@samsung.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
Lu Baolu <baolu.lu@linux.intel.com>
Subject: [PATCH v6 00/10] usb: add support for Intel dual role port mux
Date: Mon, 25 Apr 2016 16:04:46 +0800 [thread overview]
Message-ID: <1461571496-9600-1-git-send-email-baolu.lu@linux.intel.com> (raw)
Intel SOC chips are featured with USB dual role. The host role is
provided by Intel xHCI IP, and the gadget role is provided by IP
from designware. Tablet platform designs always share a single
port for both host and gadget controllers. There is a mux to
switch the port to the right controller according to the cable
type. OS needs to provide the callback to control the mux when
a plug-in event raises. The method to control the mux is platform
dependent. At least three types of implementation can be found
across current devices. 1) GPIO pins; 2) a unit which can be
controlled by memory mapped registers; 3) ACPI ASL code.
This patch series adds supports for Intel dual role port mux.
It includes:
(1) A helper layer on top of extcon for individual mux driver.
It listens to the USB-HOST extcon cable and call the switch
call-back when the cable state changes.
(2) Drivers for GPIO controlled port mux which could be found
on Baytrail devices. A mfd driver is used to split the GPIOs
into a USB gpio extcon device, a fixed regulator for gpio
controlled USB VCC, and a USB mux device. Driver for USB
gpio extcon device is already in upstream Linux. This patch
series includes a driver for GPIO USB mux.
(3) Drivers for USB port mux controlled through memory mapped
registers and the logic to create the mux device. This type
of dual role port mux could be found in Cherry Trail and
Broxton devices.
Lu Baolu (10):
extcon: usb-gpio: add device binding for platform device
extcon: usb-gpio: add support for ACPI gpio interface
regulator: fixed: add device binding for platform device
regulator: fixed: add support for ACPI interface
usb: mux: add generic code for dual role port mux
usb: mux: add driver for Intel gpio controlled port mux
usb: mux: add driver for Intel drcfg controlled port mux
mfd: intel_vuport: Add Intel virtual USB port MFD Driver
usb: pci-quirks: add Intel USB drcfg mux device
MAINTAINERS: add maintainer entry for Intel USB dual role mux drivers
Change log:
v5->v6:
Work internally with Felipe to improve the whole patch series.
Below changes have been made since last version.
- rework the common code to make it a generic interface for mux devices;
- split the vbus gpio handling to a fixed regulator device;
- removed unnecessary filtering for state change;
- removed unnecessary WARN statement;
- removed globals in mux drivers;
- removed unnecessary register polling and waiting in drcfg driver;
v4->v5:
- Change the extcon interfaces with the new ones suggested by
2a9de9c0f08d6 (extcon: Use the unique id for external connector
instead of string)
- remove patch "usb: pci-quirks: add Intel USB drcfg mux device"
from this serial due to that it's not driver staff. Will be
submitted seperately.
v3->v4:
- Check all patches with "checkpatch.pl --strict", and fix all
CHECKs;
- Change sysfs node from "intel_mux" to "port_mux";
- Refines below confusing functions:
intel_usb_mux_register() -> intel_usb_mux_bind_cable()
intel_usb_mux_unregister() -> intel_usb_mux_unbind_cable();
- Remove unnecessary struct intel_mux_dev.
v2->v3:
- uvport mfd driver got reviewed by Lee Jones, the following
changes were made accordingly.
- seperate uvport driver from the mux drivers in MAINTAINERS file
- refine the description in Kconfig
- refine the mfd_cell structure data
v1->v2:
- move mux driver from drivers/usb/misc to drivers/usb/mux;
- replace debugfs with sysfs for user level mux control;
- remove unnecessary register restore if mux registeration failed;
- Add "Acked-by: Chanwoo Choi <cw00.choi@samsung.com>" to extcon changes;
- Make the file names and exported function names more specific;
- Remove the usb_mux_get_dev() interface;
- Move "struct intel_usb_mux" from .h to .c file;
- Fix various kbuild robot warnings.
Documentation/ABI/testing/sysfs-bus-platform | 17 +++
MAINTAINERS | 10 ++
drivers/extcon/extcon-usb-gpio.c | 10 +-
drivers/mfd/Kconfig | 8 +
drivers/mfd/Makefile | 1 +
drivers/mfd/intel-vuport.c | 90 +++++++++++
drivers/regulator/fixed.c | 55 +++++++
drivers/usb/Kconfig | 2 +
drivers/usb/Makefile | 1 +
drivers/usb/host/pci-quirks.c | 45 +++++-
drivers/usb/host/xhci-ext-caps.h | 2 +
drivers/usb/mux/Kconfig | 30 ++++
drivers/usb/mux/Makefile | 6 +
drivers/usb/mux/portmux-core.c | 217 +++++++++++++++++++++++++++
drivers/usb/mux/portmux-intel-drcfg.c | 171 +++++++++++++++++++++
drivers/usb/mux/portmux-intel-gpio.c | 149 ++++++++++++++++++
include/linux/usb/portmux.h | 78 ++++++++++
17 files changed, 889 insertions(+), 3 deletions(-)
create mode 100644 drivers/mfd/intel-vuport.c
create mode 100644 drivers/usb/mux/Kconfig
create mode 100644 drivers/usb/mux/Makefile
create mode 100644 drivers/usb/mux/portmux-core.c
create mode 100644 drivers/usb/mux/portmux-intel-drcfg.c
create mode 100644 drivers/usb/mux/portmux-intel-gpio.c
create mode 100644 include/linux/usb/portmux.h
--
2.1.4
next reply other threads:[~2016-04-25 8:05 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-25 8:04 Lu Baolu [this message]
2016-04-25 8:04 ` [PATCH v6 01/10] extcon: usb-gpio: add device binding for platform device Lu Baolu
2016-04-25 23:51 ` Chanwoo Choi
2016-04-26 0:42 ` Lu Baolu
2016-04-25 8:04 ` [PATCH v6 02/10] extcon: usb-gpio: add support for ACPI gpio interface Lu Baolu
2016-04-25 23:51 ` Chanwoo Choi
2016-04-26 0:43 ` Lu Baolu
2016-04-25 8:04 ` [PATCH v6 03/10] regulator: fixed: add device binding for platform device Lu Baolu
2016-04-25 16:40 ` Mark Brown
2016-04-26 2:12 ` Lu Baolu
2016-04-25 8:04 ` [PATCH v6 04/10] regulator: fixed: add support for ACPI interface Lu Baolu
2016-04-25 17:30 ` Mark Brown
2016-04-26 2:24 ` Lu Baolu
2016-04-26 10:23 ` Mark Brown
2016-04-27 1:54 ` Lu Baolu
2016-04-27 12:33 ` Mark Brown
2016-04-28 5:55 ` Lu Baolu
2016-04-28 17:15 ` Mark Brown
2016-04-29 0:31 ` Lu Baolu
2016-04-25 8:04 ` [PATCH v6 05/10] usb: mux: add generic code for dual role port mux Lu Baolu
2016-04-25 8:04 ` [PATCH v6 06/10] usb: mux: add driver for Intel gpio controlled " Lu Baolu
2016-04-25 8:04 ` [PATCH v6 07/10] usb: mux: add driver for Intel drcfg " Lu Baolu
2016-04-25 8:04 ` [PATCH v6 08/10] mfd: intel_vuport: Add Intel virtual USB port MFD Driver Lu Baolu
2016-04-25 8:04 ` [PATCH v6 09/10] usb: pci-quirks: add Intel USB drcfg mux device Lu Baolu
2016-04-25 8:04 ` [PATCH v6 10/10] MAINTAINERS: add maintainer entry for Intel USB dual role mux drivers Lu Baolu
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=1461571496-9600-1-git-send-email-baolu.lu@linux.intel.com \
--to=baolu.lu@linux.intel.com \
--cc=broonie@kernel.org \
--cc=cw00.choi@samsung.com \
--cc=felipe.balbi@linux.intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=heikki.krogerus@linux.intel.com \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mathias.nyman@intel.com \
--cc=myungjoo.ham@samsung.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 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.