summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDiana Z <dzigterman@chromium.org>2020-03-26 22:04:37 -0600
committerCommit Bot <commit-bot@chromium.org>2020-03-27 21:14:41 +0000
commit90d47e5e3b7a957095b87330359592e19b174297 (patch)
tree4248b095e3cd70b559b2a41f04bbc7605af452f0
parente3c9d2ab77571008734279986aa9a12b1450e5ab (diff)
downloadchrome-ec-90d47e5e3b7a957095b87330359592e19b174297.tar.gz
Docs: Adding TCPMv2 overview document
General overview of TCPMv2, converted to md format. Updated new stack portion of the USB-C overview to link to this document, so only one location needs to be updated in the future with config changes. BUG=None BRANCH=None TEST=rendered in chrome with markdown preview plus extension Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: Iab74dbee97eeb8c39fe210787bb142c6a1ec6af8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2124010 Reviewed-by: Jett Rink <jettrink@chromium.org>
-rw-r--r--docs/images/TCPMv2-ExampleStates.pngbin0 -> 13779 bytes
-rw-r--r--docs/images/TCPMv2-ExampleTransitions.pngbin0 -> 58779 bytes
-rw-r--r--docs/images/TCPMv2-TaskLayers.pngbin0 -> 47867 bytes
-rw-r--r--docs/images/TCPMv2-TaskLoop.pngbin0 -> 16282 bytes
-rw-r--r--docs/sitemap.md1
-rw-r--r--docs/usb-c.md10
-rw-r--r--docs/usb-tcpmv2.md184
7 files changed, 187 insertions, 8 deletions
diff --git a/docs/images/TCPMv2-ExampleStates.png b/docs/images/TCPMv2-ExampleStates.png
new file mode 100644
index 0000000000..7068829e31
--- /dev/null
+++ b/docs/images/TCPMv2-ExampleStates.png
Binary files differ
diff --git a/docs/images/TCPMv2-ExampleTransitions.png b/docs/images/TCPMv2-ExampleTransitions.png
new file mode 100644
index 0000000000..da88a143a8
--- /dev/null
+++ b/docs/images/TCPMv2-ExampleTransitions.png
Binary files differ
diff --git a/docs/images/TCPMv2-TaskLayers.png b/docs/images/TCPMv2-TaskLayers.png
new file mode 100644
index 0000000000..7429e8f9b1
--- /dev/null
+++ b/docs/images/TCPMv2-TaskLayers.png
Binary files differ
diff --git a/docs/images/TCPMv2-TaskLoop.png b/docs/images/TCPMv2-TaskLoop.png
new file mode 100644
index 0000000000..1c0bb218fe
--- /dev/null
+++ b/docs/images/TCPMv2-TaskLoop.png
Binary files differ
diff --git a/docs/sitemap.md b/docs/sitemap.md
index b430c69e2e..12b6e034dd 100644
--- a/docs/sitemap.md
+++ b/docs/sitemap.md
@@ -36,6 +36,7 @@
* [USB-C Power Delivery and Alternate Modes](./usb-c.md)
* [USB-A and USB-C Policies for Sourcing Power](./usb_power.md)
+* [USB-C Power Delivery TCPMv2](./usb-tcpmv2.md)
## EC-3PO
diff --git a/docs/usb-c.md b/docs/usb-c.md
index 3b17e29fd0..c47d62b4f0 100644
--- a/docs/usb-c.md
+++ b/docs/usb-c.md
@@ -105,14 +105,8 @@ Chromebooks. For example, the older stack supports the Zinger, which is the
USB-C charging device that shipped with Samus, the Google Chromebook Pixel 2.
The Zinger implements the charger only side of the USB PD protocol.
-To use the newer USB-C PD stack implementation, see `CONFIG_USB_SM_FRAMEWORK`
-description in [config.h][config header link].
-
-The newer implementation only supports Chromebooks at the moment. There are
-multiple policy engine definitions to choose from depending on the different
-USB-C features the Chromebook should support. As of now, you must choose one and
-only one policy engine state machine implementation. The policy engine mostly
-defines what PD features (e.g. Try.SRC) are implemented on the Chromebook.
+To use the newer USB-C PD stack implementation, see
+[TCPMv2 Overview](usb-tcpmv2.md).
## Implementation Considerations
diff --git a/docs/usb-tcpmv2.md b/docs/usb-tcpmv2.md
new file mode 100644
index 0000000000..d80eb677a7
--- /dev/null
+++ b/docs/usb-tcpmv2.md
@@ -0,0 +1,184 @@
+# EC USB-C Power Delivery TCPMv2 Overview
+
+As the original USB-C Power Delivery (PD) solution for the ChromeOS Embedded Controller has aged, it has grown to the point where it is difficult to add new features and address bugs. A new PD stack (generally referred to as TCPMv2) has been introduced to the codebase for use moving forward. It implements a layered, state-based design which tracks more closely with the USB Type-C and USB PD specifications.
+
+[TOC]
+
+## Enabling TCPMv2
+
+Boards may enable TCPMv2 by adding the following defines:
+
+
+
+* `CONFIG_USB_PD_TCPMV2`Configures the board to use the new stack.
+* `CONFIG_USB_DRP_ACC_TRYSRC`: Configures the type of state machine to run (in this case, a DRP which performs Try.SRC behavior). Currently available are DRP and charge-through Vconn-powered device options
+* `CONFIG_USB_PD_DECODE_SOP`: Sets up messaging for SOP’ and SOP’’, which is strongly encouraged in the TCPMv2 code moving forward
+* `CONFIG_USB_PID 0x1234`: Sets the USB Product Identifier. This will be shared for all boards within one reference design, and new PIDs may be requested by sending an email to the ChromeOS FW Team.
+
+Other configurations to specify behaviors within the task still apply (ex. `CONFIG_USB_PD_ALT_MODE_DFP` and `CONFIG_USB_PD_TCPC_LOW_POWER`).
+
+
+## State Machine Framework
+
+The basis of the TCPMv2 state machines is a generic state machine framework. This framework can be found in common/usbc/usb\_sm.c. For each state, there may be defined:
+
+
+
+* Entry: Called upon entering a state
+* Run: Called while steady in a state
+* Exit: Called upon leaving a state
+* Parent: Superstate. Enters, exits, and runs alongside the child state. Only enters and exits when transitioning between states which do not share the parent.
+
+All fields are optional and may be set to NULL. A new state is transitioned to with a call into set\_state(), which exits the old state and parents and enters the new parents and state. States may be changed with set\_state() in any entry or run function, but any call from an exit function is ignored since exit is only called when a change is already in progress. While in a state, run\_state() executes the run function for the current state and parents. If set\_state() is called from either an entry function or a run function, the remaining run or entry functions are stopped.
+
+Below is a graphical example of what some states may look like. States 1 and 2 share Parent State 1, while State 3 has Parent State of 2.
+
+
+![Example States](images/TCPMv2-ExampleStates.png "Example States")
+
+
+Translated into code, this would be something like below (note it is not necessary that the states be a part of an array, but the TCPMv2 code generally organizes the states in this way):
+
+
+```
+static const struct usb_state test_states[] = {
+ [PARENT_1] = {
+ .entry = parent_1_entry,
+ .run = parent_1_run,
+ .exit = parent_1_exit,
+ },
+ [PARENT_2] = {
+ .entry = parent_2_entry,
+ .run = parent_2_run,
+ .exit = parent_2_exit,
+ },
+ [STATE_1] = {
+ .entry = state_1_entry,
+ .run = state_1_run,
+ .exit = state_1_exit,
+ .parent = &test_states[PARENT_1],
+ },
+ [STATE_2] = {
+ .entry = state_2_entry,
+ .run = state_2_run,
+ .exit = state_2_exit,
+ .parent = &test_states[PARENT_1],
+ },
+ [STATE_3] = {
+ .entry = state_3_entry,
+ .run = state_3_run,
+ .exit = state_3_exit,
+ .parent = &test_states[PARENT_2],
+ },
+};
+```
+
+
+For this example, each test state is written simply to produce a print of its function name. The two exceptions are:
+
+
+
+* parent\_1\_run() calls set\_state() into STATE\_2 when called a second time
+* state\_2\_entry() calls set\_state() into STATE\_3
+
+Graphically, this is represented below:
+
+
+![Example Transitions](images/TCPMv2-ExampleTransitions.png "Example state transitions and resulting called functions")
+
+
+And the following code output is produced:
+
+
+```
+Calling run_state()
+state_1_run
+parent_1_run
+
+Calling run_state()
+state_1_run
+state_1_run calling set_state() to state 2
+state_1_exit
+state_2_entry
+state_2_entry calling set_state() to state 3
+state_2_exit
+parent_1_exit
+parent_2_entry
+state_3_entry
+
+Calling run_state()
+state_3_run
+parent_2_run
+```
+
+
+
+## TCPMv2 PD Task
+
+The TCPMv2 PD task is built upon state machines using the above framework and is located in common/usbc/usbc\_task.c. It is separated into three layers which track with the USB Type-C and USB PD specification states and run in a loop with 5 ms periods between executions. A graphical representation of these layers is below.
+
+
+
+![PD Task Loop](images/TCPMv2-TaskLoop.png "PD task loop state machine calls")
+
+
+The task is designed such that the Type-C (TC) layer could run independently for any application which doesn’t wish to enable PD messaging. Boards define their appropriate Policy Engine (PE) and TC state machines through their selection of a CONFIG\_USB\_\* define, with current options supporting both Dual-Role Ports (DRPs) and Charge-Through Vconn-Powered Device (CTVPD). All boards use the same Protocol Layer (PRL) code.
+
+
+## TCPMv2 Layers
+
+
+### Overview
+
+The three state machines mentioned above interact with each other and the EC drivers in order to orchestrate all Type-C connection behavior. Graphically, they are represented below.
+
+
+
+![PD Task Layers](images/TCPMv2-TaskLayers.png "PD task layer interactions")
+
+
+Layers communicate with each other using atomic operations on flags and shared buffers. Functions calling into each layer are clearly named to indicate the layer they are acting on, and anything calling into the PD task should be doing so through pd\_\* named functions.
+
+Some specific examples of how this communication works between layers is below. If a port partner sends in a Vconn\_swap request, then:
+
+
+
+* PRL will communicate that a message was received to the PE layer through pe\_message\_received(), which sets PE\_FLAGS\_MSG\_RECEIVED and indicates the receive buffer has a message
+* PE asks with the TC whether the board is sourcing Vconn with tc\_is\_vconn\_src() which checks TC\_FLAGS\_VCONN\_ON
+* PE tells the PRL to send an ACCEPT message to the port partner through prl\_send\_ctrl\_msg() which fills in shared message information and sets PRL\_FLAGS\_MSG\_XMIT
+* PRL lets the PE know that the message transmit was successful through pe\_message\_sent() which sets PE\_FLAGS\_TX\_COMPLETE
+* TC tells the PE layer that the Vconn swap completed with pe\_vconn\_swap\_complete() which sets PE\_FLAGS\_VCONN\_SWAP\_COMPLETE
+
+
+### Type-C Layer
+
+Defined in the USB Type-C specification, this layer is responsible for basic connection. It takes care of setting the CC lines, detecting and debouncing the partner CC lines, and performs most of the interactions needed with the PPC and USB mux. Once the TC layer has gotten the connection to the point of being Attached.SNK or Attached.SRC, it will enable the PRL and PE layers accordingly.
+
+
+### Protocol Layer
+
+A part of the USB PD specification, the protocol layer is responsible for the actual sending and receiving of PD messages with the TCPCs. The layer is actually composed of 4 separate state machines running one after the other. These state machines are:
+
+
+
+* Chunked receive (RCH): passes messages up to the PE and requests chunks when chunking
+* Chunked transmit (TCH): receives messages from the PE and waits for chunk requests when chunking
+* Protocol transmit (PRL\_TX): passes messages to the TCPCs and handles PD 3.0 collision avoidance
+* Protocol hard reset (PRL\_HR): responds to or transmits hard resets, resets PRL layer variables, notifies PE of hard reset receipt or sent
+
+
+### Policy Engine Layer
+
+The PE layer states are defined as a part of the USB PD specification. State names are meant to track very closely with the specification so they can be easily searchable and understood. The PE’s primary responsibility is to send and process PD messages in order to implement the port’s policy.
+
+
+## Best Practices
+
+
+
+* Always call return after set\_state(). Once the state has been changed, nothing further should be done in the current state.
+* Never call set\_state() from an exit function. The call will be ignored as there is already a state transition taking place.
+* Never call set\_state() from outside the PD task. The task may be waiting in any number of locations and the context should not change around it while it does so.
+* Always use flags to communicate between layers, and to communicate with the PD task from other tasks. Flags should be accessed through atomic operations.
+* Always use pd\_\* functions to access the PD task from other tasks.
+* Always write unit tests as code is added, to verify new code works and protect against regressions as development continues.