From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99B97C4743D for ; Tue, 8 Jun 2021 19:34:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 83437610A1 for ; Tue, 8 Jun 2021 19:34:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239463AbhFHTgR (ORCPT ); Tue, 8 Jun 2021 15:36:17 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:46728 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237723AbhFHTeB (ORCPT ); Tue, 8 Jun 2021 15:34:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1623180727; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=1p5fMfPKJv8flgbhVd1qMPM5Lt1Pt/3jocRKCMPKgns=; b=eBKOw3b84Czw4DPzw6se8uL8ecic/GtVvr1gmqjTiwxoN5qcIwywZV2OvSHHcbSuH/ntKX uGl8SLLXGzZGI/aJIk0eHsafvdbxKVGCcPAvy110X+r/kakEWZv4vH7cmA1B6u7hfmMo8T ARoImnYUgsLaEfxmd3LDxjs95S6iqoU= Received: from mail-ej1-f72.google.com (mail-ej1-f72.google.com [209.85.218.72]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-198-GDBiIc3zP_iwqIX7tUlO5A-1; Tue, 08 Jun 2021 15:32:05 -0400 X-MC-Unique: GDBiIc3zP_iwqIX7tUlO5A-1 Received: by mail-ej1-f72.google.com with SMTP id jy19-20020a1709077633b02903eb7acdb38cso7184779ejc.14 for ; Tue, 08 Jun 2021 12:32:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:user-agent:mime-version; bh=1p5fMfPKJv8flgbhVd1qMPM5Lt1Pt/3jocRKCMPKgns=; b=aXzmleBoUGPXnD+P568famBWxC7uTEup2s87z3gnKXDVisg1K4K5IBlJ7VxF2KhoI3 PT1TwlIq+SlzKVxKfVqz188UekBevbKEcR7HI9yY71s+Q/ehJrBD6HDIPlms23zQltWr ugNd1yOGDN27BqkWZatnYS7+UIS8ib6TxqvTBsdw51RjinpEvMONOjEaaBnNvyFrFrkJ vVN9aiyyPPB1mr6CjMS88RR2rFPfqGDOZnr8UP8Oxgm4d6MF6hr4/xGXLgJKBWclHZsM 6aa6QpGdVal55+njCAGvdINviF5HKa7//jVpY6ErCfjXaVsUcbtTtNdd0OtZZPpYjnUE Mm/g== X-Gm-Message-State: AOAM53049c+fKLPxYKfnyi6Y8BrXv9f+FmF012jQDZlTUpZCe0ZdUSJn wcQUvw6LiCoT+Qw4AO+7Lfj2tsY60kfIRhncCynH8X2wkg726gXMX7PtLaMvk+WYzMkZ1X40xAT cH52gijWyv9e4EeUYPm8IyXxV X-Received: by 2002:aa7:ca50:: with SMTP id j16mr26965479edt.158.1623180724049; Tue, 08 Jun 2021 12:32:04 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy4ZyaIjOxcVJvWEcErkpAn2qsw4vOq3MnyFnTCizlAxRHIAvplT/X46+Bw63dr7YhzwU/8aw== X-Received: by 2002:aa7:ca50:: with SMTP id j16mr26965471edt.158.1623180723815; Tue, 08 Jun 2021 12:32:03 -0700 (PDT) Received: from [10.0.4.217] (i59F7C257.versanet.de. [89.247.194.87]) by smtp.gmail.com with ESMTPSA id qh23sm224888ejb.77.2021.06.08.12.32.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Jun 2021 12:32:03 -0700 (PDT) Message-ID: Subject: Re: [RFC PATCH 0/7] usb: typec: ucsi: Polling the alt modes and PDOs From: Benjamin Berg To: Heikki Krogerus Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Date: Tue, 08 Jun 2021 21:32:01 +0200 In-Reply-To: References: <20210607131442.20121-1-heikki.krogerus@linux.intel.com> <4a76d2152f016b58298bec16aa2003a6ec55f8a8.camel@redhat.com> Content-Type: multipart/signed; micalg="pgp-sha256"; protocol="application/pgp-signature"; boundary="=-+tUZK2kr5TN2iQ/7TYMn" User-Agent: Evolution 3.40.1 (3.40.1-1.fc34) MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-+tUZK2kr5TN2iQ/7TYMn Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, 2021-06-08 at 09:54 +0300, Heikki Krogerus wrote: > On Tue, Jun 08, 2021 at 09:42:09AM +0300, Heikki Krogerus wrote: > > Please check does the partner device get removed. What do you have > > under /sys/class/typec after that happens? >=20 > Oh yes. Could you also share the trace output when that happens? >=20 > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 cd /sys/kernel/debug/tracing > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 echo 1 > events/ucsi/enable > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 # now reproduce the issue > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 cat trace > ucsi.trace So, the partner device is still there when this happens (see below). I also only see a single event in the trace for the fast plug/unplug case: kworker/u16:8-1771 [003] .... 18848.872145: ucsi_connector_change: po= rt1 status: change=3D4a04, opmode=3D5, connected=3D1, sourcing=3D0, partner= _flags=3D1, partner_type=3D1, request_data_obj=3D1304b12c, BC status=3D1 The typec port/partner states is: port1-partner/accessory_mode:none port1-partner/supports_usb_power_delivery:no port1-partner/uevent:DEVTYPE=3Dtypec_partner port1-partner/usb_power_delivery_revision:0.0 and port0/data_role:host [device] port0/power_operation_mode:default port0/power_role:source [sink] port0/supported_accessory_modes:none port0/uevent:DEVTYPE=3Dtypec_port port0/uevent:TYPEC_PORT=3Dport0 port0/usb_power_delivery_revision:2.0 port0/usb_typec_revision:1.0 port0/vconn_source:no port1/data_role:host [device] port1/power_operation_mode:3.0A port1/power_role:source [sink] port1/supported_accessory_modes:none port1/uevent:DEVTYPE=3Dtypec_port port1/uevent:TYPEC_PORT=3Dport1 port1/usb_power_delivery_revision:2.0 port1/usb_typec_revision:1.0 port1/vconn_source:no Note that for a normal plug I am usually getting a second event. This second event is occasionally missing though: kworker/u16:38-1800 [001] .... 19522.325885: ucsi_connector_change: po= rt1 status: change=3D4a04, opmode=3D5, connected=3D1, sourcing=3D0, partner= _flags=3D1, partner_type=3D1, request_data_obj=3D1304b12c, BC status=3D1 kworker/u16:38-1800 [004] .... 19522.552613: ucsi_connector_change: po= rt1 status: change=3D0044, opmode=3D3, connected=3D1, sourcing=3D0, partner= _flags=3D1, partner_type=3D1, request_data_obj=3D1304b12c, BC status=3D1 and a working unplug consistently looks like: kworker/u16:8-1771 [003] .... 19670.020085: ucsi_connector_change: po= rt1 status: change=3D4804, opmode=3D0, connected=3D0, sourcing=3D0, partner= _flags=3D1, partner_type=3D1, request_data_obj=3D1304b12c, BC status=3D0 Benjamin --=-+tUZK2kr5TN2iQ/7TYMn Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEED2NO4vMS33W8E4AFq6ZWhpmFY3AFAmC/xbEACgkQq6ZWhpmF Y3DdfA/+OcGOiFiPtEDgR+6t+HnMr7KJsPjP+LjffBEeea5bCtiPswjeH6bqlt8U pquAB/bj5qQ1VO2SyczMzwuy8eJSYNyJ6Yc/cb5CxUtFIjAE6xXhgMNguj2T8C3V 62EMl/WzFI0jfJ4Ow6bhgwlnA4CJXq0qX04MuRWnEJ2YMwEoMnnh9buxtoPVu2pm cfZcXB/QFkOGC17IEq/d1PmnZQZNtisFz6l1RnhrwL/IvJC4FoUhzC6jmJlsXl9b y0b6lFNZHM4euLvp38TUNMAEubxRa+v3wYa2bwMbBF75S+A8jvQntYrpWurNJEit c67Qrnu5OhcjZuHONCmXJIS5i8icegBUavAwfuX+5EGfd0N0rxH3vfP3AmHy+3iu /UfmEEZrA0QMsLzOUsjY80P7NaesxcFHph21EugBAeKGgvtMxZ3evLLlKUxIWi+5 tNQOCS0Xg2Ql5A4pMAFwY2QUZ3dDyeAWEqeOmcOAL/btnCne/VmRHNiIyX/Ldpqc 5Th7zHShYQNuFypo7y2DxL7lltsXAaYjKGMD6QycqoSLwpLEV1bimsI0V4EoX/vo 8aqsN87Of/jKw6FXa+UstKon5iHnxqKGz7QDkJ/8gtWG5GZHhZpR1sQ+P864V8cI KD48/6bsNBv2i7pxo5EQkqKKoCzBcTErRt4D744M2VTIltwJ2wI= =SJ8r -----END PGP SIGNATURE----- --=-+tUZK2kr5TN2iQ/7TYMn--