| Message ID | 20210528081613.730661-1-kyletso@google.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 6490fa565534fa83593278267785a694fd378a2b |
| Headers | show |
| Series | usb: pd: Set PD_T_SINK_WAIT_CAP to 310ms | expand |
On 5/28/21 1:16 AM, Kyle Tso wrote: > Current timer PD_T_SINK_WAIT_CAP is set to 240ms which will violate the > SinkWaitCapTimer (tTypeCSinkWaitCap 310 - 620 ms) defined in the PD > Spec if the port is faster enough when running the state machine. Set it > to the lower bound 310ms to ensure the timeout is in Spec. > > Fixes: f0690a25a140 ("staging: typec: USB Type-C Port Manager (tcpm)") > Signed-off-by: Kyle Tso <kyletso@google.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> > --- > include/linux/usb/pd.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/usb/pd.h b/include/linux/usb/pd.h > index bf00259493e0..96b7ff66f074 100644 > --- a/include/linux/usb/pd.h > +++ b/include/linux/usb/pd.h > @@ -460,7 +460,7 @@ static inline unsigned int rdo_max_power(u32 rdo) > #define PD_T_RECEIVER_RESPONSE 15 /* 15ms max */ > #define PD_T_SOURCE_ACTIVITY 45 > #define PD_T_SINK_ACTIVITY 135 > -#define PD_T_SINK_WAIT_CAP 240 > +#define PD_T_SINK_WAIT_CAP 310 /* 310 - 620 ms */ > #define PD_T_PS_TRANSITION 500 > #define PD_T_SRC_TRANSITION 35 > #define PD_T_DRP_SNK 40 >
diff --git a/include/linux/usb/pd.h b/include/linux/usb/pd.h index bf00259493e0..96b7ff66f074 100644 --- a/include/linux/usb/pd.h +++ b/include/linux/usb/pd.h @@ -460,7 +460,7 @@ static inline unsigned int rdo_max_power(u32 rdo) #define PD_T_RECEIVER_RESPONSE 15 /* 15ms max */ #define PD_T_SOURCE_ACTIVITY 45 #define PD_T_SINK_ACTIVITY 135 -#define PD_T_SINK_WAIT_CAP 240 +#define PD_T_SINK_WAIT_CAP 310 /* 310 - 620 ms */ #define PD_T_PS_TRANSITION 500 #define PD_T_SRC_TRANSITION 35 #define PD_T_DRP_SNK 40
Current timer PD_T_SINK_WAIT_CAP is set to 240ms which will violate the SinkWaitCapTimer (tTypeCSinkWaitCap 310 - 620 ms) defined in the PD Spec if the port is faster enough when running the state machine. Set it to the lower bound 310ms to ensure the timeout is in Spec. Fixes: f0690a25a140 ("staging: typec: USB Type-C Port Manager (tcpm)") Signed-off-by: Kyle Tso <kyletso@google.com> --- include/linux/usb/pd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)