* [PATCH 5/6] staging: usb: tcpm: usb: typec: tcpm: Follow Try.SRC exit requirements
@ 2017-08-27 5:25 Badhri Jagan Sridharan
2017-08-27 17:22 ` Guenter Roeck
0 siblings, 1 reply; 2+ messages in thread
From: Badhri Jagan Sridharan @ 2017-08-27 5:25 UTC (permalink / raw)
To: Guenter Roeck, Greg Kroah-Hartman
Cc: devel, linux-kernel, Badhri Jagan Sridharan
According to spec:
" 4.5.2.2.9.2 Exiting from Try.SRC State:
The port shall transition to Attached.SRC when the SRC.Rd
state is detected on exactly one of the CC1 or CC2 pins for
at least tPDDebounce. The port shall transition to
TryWait.SNK after tDRPTry and the SRC.Rd state has not been
detected."
Signed-off-by: Badhri Jagan Sridharan <Badhri@google.com>
---
drivers/staging/typec/tcpm.c | 33 ++++++++++++++++++++++++++++-----
1 file changed, 28 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/typec/tcpm.c b/drivers/staging/typec/tcpm.c
index 7eed04698ebe..fc179bdea7e4 100644
--- a/drivers/staging/typec/tcpm.c
+++ b/drivers/staging/typec/tcpm.c
@@ -112,6 +112,7 @@
S(SRC_TRYWAIT_UNATTACHED), \
\
S(SRC_TRY), \
+ S(SRC_TRY_WAIT), \
S(SRC_TRY_DEBOUNCE), \
S(SNK_TRYWAIT), \
S(SNK_TRYWAIT_DEBOUNCE), \
@@ -2158,6 +2159,7 @@ static void run_state_machine(struct tcpm_port *port)
{
int ret;
enum typec_pwr_opmode opmode;
+ unsigned int msecs;
port->enter_state = port->state;
switch (port->state) {
@@ -2379,7 +2381,22 @@ static void run_state_machine(struct tcpm_port *port)
case SRC_TRY:
port->try_src_count++;
tcpm_set_cc(port, tcpm_rp_cc(port));
- tcpm_set_state(port, SNK_TRYWAIT, PD_T_DRP_TRY);
+ port->max_wait = 0;
+ tcpm_set_state(port, SRC_TRY_WAIT, 0);
+ break;
+ case SRC_TRY_WAIT:
+ if (port->max_wait == 0) {
+ port->max_wait = jiffies +
+ msecs_to_jiffies(PD_T_DRP_TRY);
+ msecs = PD_T_DRP_TRY;
+ } else {
+ if (time_is_after_jiffies(port->max_wait))
+ msecs = jiffies_to_msecs(port->max_wait -
+ jiffies);
+ else
+ msecs = 0;
+ }
+ tcpm_set_state(port, SNK_TRYWAIT, msecs);
break;
case SRC_TRY_DEBOUNCE:
tcpm_set_state(port, SRC_ATTACHED, PD_T_PD_DEBOUNCE);
@@ -2935,12 +2952,12 @@ static void _tcpm_cc_change(struct tcpm_port *port, enum typec_cc_status cc1,
tcpm_set_state(port, SRC_TRYWAIT, 0);
}
break;
- case SRC_TRY:
+ case SRC_TRY_WAIT:
if (tcpm_port_is_source(port))
tcpm_set_state(port, SRC_TRY_DEBOUNCE, 0);
break;
case SRC_TRY_DEBOUNCE:
- tcpm_set_state(port, SRC_TRY, 0);
+ tcpm_set_state(port, SRC_TRY_WAIT, 0);
break;
case SNK_TRYWAIT_DEBOUNCE:
if (port->vbus_present) {
@@ -3015,7 +3032,10 @@ static void _tcpm_pd_vbus_on(struct tcpm_port *port)
case SNK_TRYWAIT:
tcpm_set_state(port, SNK_TRYWAIT_VBUS, 0);
break;
-
+ case SRC_TRY_WAIT:
+ case SRC_TRY_DEBOUNCE:
+ /* Do nothing, waiting for sink detection */
+ break;
default:
break;
}
@@ -3069,7 +3089,10 @@ static void _tcpm_pd_vbus_off(struct tcpm_port *port)
case PORT_RESET_WAIT_OFF:
tcpm_set_state(port, tcpm_default_state(port), 0);
break;
-
+ case SRC_TRY_WAIT:
+ case SRC_TRY_DEBOUNCE:
+ /* Do nothing, waiting for sink detection */
+ break;
default:
if (port->pwr_role == TYPEC_SINK &&
port->attached)
--
2.14.1.342.g6490525c54-goog
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 5/6] staging: usb: tcpm: usb: typec: tcpm: Follow Try.SRC exit requirements
2017-08-27 5:25 [PATCH 5/6] staging: usb: tcpm: usb: typec: tcpm: Follow Try.SRC exit requirements Badhri Jagan Sridharan
@ 2017-08-27 17:22 ` Guenter Roeck
0 siblings, 0 replies; 2+ messages in thread
From: Guenter Roeck @ 2017-08-27 17:22 UTC (permalink / raw)
To: Badhri Jagan Sridharan; +Cc: Greg Kroah-Hartman, devel, linux-kernel
On Sat, Aug 26, 2017 at 10:25:07PM -0700, Badhri Jagan Sridharan wrote:
> According to spec:
> " 4.5.2.2.9.2 Exiting from Try.SRC State:
> The port shall transition to Attached.SRC when the SRC.Rd
> state is detected on exactly one of the CC1 or CC2 pins for
> at least tPDDebounce. The port shall transition to
> TryWait.SNK after tDRPTry and the SRC.Rd state has not been
> detected."
>
> Signed-off-by: Badhri Jagan Sridharan <Badhri@google.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> ---
> drivers/staging/typec/tcpm.c | 33 ++++++++++++++++++++++++++++-----
> 1 file changed, 28 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/typec/tcpm.c b/drivers/staging/typec/tcpm.c
> index 7eed04698ebe..fc179bdea7e4 100644
> --- a/drivers/staging/typec/tcpm.c
> +++ b/drivers/staging/typec/tcpm.c
> @@ -112,6 +112,7 @@
> S(SRC_TRYWAIT_UNATTACHED), \
> \
> S(SRC_TRY), \
> + S(SRC_TRY_WAIT), \
> S(SRC_TRY_DEBOUNCE), \
> S(SNK_TRYWAIT), \
> S(SNK_TRYWAIT_DEBOUNCE), \
> @@ -2158,6 +2159,7 @@ static void run_state_machine(struct tcpm_port *port)
> {
> int ret;
> enum typec_pwr_opmode opmode;
> + unsigned int msecs;
>
> port->enter_state = port->state;
> switch (port->state) {
> @@ -2379,7 +2381,22 @@ static void run_state_machine(struct tcpm_port *port)
> case SRC_TRY:
> port->try_src_count++;
> tcpm_set_cc(port, tcpm_rp_cc(port));
> - tcpm_set_state(port, SNK_TRYWAIT, PD_T_DRP_TRY);
> + port->max_wait = 0;
> + tcpm_set_state(port, SRC_TRY_WAIT, 0);
> + break;
> + case SRC_TRY_WAIT:
> + if (port->max_wait == 0) {
> + port->max_wait = jiffies +
> + msecs_to_jiffies(PD_T_DRP_TRY);
> + msecs = PD_T_DRP_TRY;
> + } else {
> + if (time_is_after_jiffies(port->max_wait))
> + msecs = jiffies_to_msecs(port->max_wait -
> + jiffies);
> + else
> + msecs = 0;
> + }
> + tcpm_set_state(port, SNK_TRYWAIT, msecs);
> break;
> case SRC_TRY_DEBOUNCE:
> tcpm_set_state(port, SRC_ATTACHED, PD_T_PD_DEBOUNCE);
> @@ -2935,12 +2952,12 @@ static void _tcpm_cc_change(struct tcpm_port *port, enum typec_cc_status cc1,
> tcpm_set_state(port, SRC_TRYWAIT, 0);
> }
> break;
> - case SRC_TRY:
> + case SRC_TRY_WAIT:
> if (tcpm_port_is_source(port))
> tcpm_set_state(port, SRC_TRY_DEBOUNCE, 0);
> break;
> case SRC_TRY_DEBOUNCE:
> - tcpm_set_state(port, SRC_TRY, 0);
> + tcpm_set_state(port, SRC_TRY_WAIT, 0);
> break;
> case SNK_TRYWAIT_DEBOUNCE:
> if (port->vbus_present) {
> @@ -3015,7 +3032,10 @@ static void _tcpm_pd_vbus_on(struct tcpm_port *port)
> case SNK_TRYWAIT:
> tcpm_set_state(port, SNK_TRYWAIT_VBUS, 0);
> break;
> -
> + case SRC_TRY_WAIT:
> + case SRC_TRY_DEBOUNCE:
> + /* Do nothing, waiting for sink detection */
> + break;
> default:
> break;
> }
> @@ -3069,7 +3089,10 @@ static void _tcpm_pd_vbus_off(struct tcpm_port *port)
> case PORT_RESET_WAIT_OFF:
> tcpm_set_state(port, tcpm_default_state(port), 0);
> break;
> -
> + case SRC_TRY_WAIT:
> + case SRC_TRY_DEBOUNCE:
> + /* Do nothing, waiting for sink detection */
> + break;
> default:
> if (port->pwr_role == TYPEC_SINK &&
> port->attached)
> --
> 2.14.1.342.g6490525c54-goog
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-08-27 17:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-27 5:25 [PATCH 5/6] staging: usb: tcpm: usb: typec: tcpm: Follow Try.SRC exit requirements Badhri Jagan Sridharan
2017-08-27 17:22 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®