Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp958896pxb; Thu, 19 Nov 2020 19:17:55 -0800 (PST) X-Google-Smtp-Source: ABdhPJwMuHSKuGWCg4CAFfJMdSyl9XvKp78qFKU8Lq30nsyOTy3/r2r/WsfEZAE+v1EHUiGfD5EH X-Received: by 2002:a17:906:c41:: with SMTP id t1mr31185885ejf.19.1605842275099; Thu, 19 Nov 2020 19:17:55 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1605842275; cv=none; d=google.com; s=arc-20160816; b=Of2owTpARMgV5W/7zHk3inN20Say0KT3f0O7kuupcdMjqeeBXPV2sia86gQ6cWg8Jz 1AbmgpdbqNc4kb/H8Zp3i8OAfErmk3j1Bc1nK2uCyldFlN+jMbDsChuTRWSMb4Ko3kd9 YNRpXXpxBC3DjNcyDEtwqOkTTNiREpIbpBRmIDxjneLV2XKKMwGa31sz1fdMn490wQ52 UZDWmQ6UONBRcCqkUW6wzDA39Yj0hb8zG0NGXjZ3tdgb/hMZrIvW2ZWVXDbPGQZyTk1W mDk4wiuMRjDs+89/y8Fbt8jLzwJEy8QjohiAIBvKncMTkmIekLRfYMS3pObqaJky4L+b LMdg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=/zGvD23D7UzPmkeJaICeLLr5BBiyIFomvZLFOZzlxm4=; b=z0/8BPw2AR4japkQCRHD5plscX3al4MokIVKvU2FYnmJXa++dS+bBUgzv/5nvy29SN CzBHYy3qF5g97JQTdwa+VC56lc8GhWEN7x+o22RbTqFVuWbyaOcj7hJB5iJE+OJbMPBn 1DfirLmZFLrCQW+7IlIHfulcuBkEiRQeuojk5xk/aaKlwh2SH9iaJncYNZ0UM6JkWtqM YSSugBuU2zaOJMYfkeief+XGfkHOgqaAJkmu2kCBQ/9FImHU2VbABSNWg9VYOYGdfCMK WyL1KyKyPRSZjaSqyLERir08VlOwrJUzSAL20cZ2XlqTATR3tw8W91kiHmJb6HSaKtAX 0xgg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b="K/ex82SL"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w21si977473edx.274.2020.11.19.19.17.31; Thu, 19 Nov 2020 19:17:55 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b="K/ex82SL"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726342AbgKTDOG (ORCPT + 99 others); Thu, 19 Nov 2020 22:14:06 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40712 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726189AbgKTDOG (ORCPT ); Thu, 19 Nov 2020 22:14:06 -0500 Received: from mail-qk1-x742.google.com (mail-qk1-x742.google.com [IPv6:2607:f8b0:4864:20::742]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 912D6C0617A7 for ; Thu, 19 Nov 2020 19:14:04 -0800 (PST) Received: by mail-qk1-x742.google.com with SMTP id q5so7708230qkc.12 for ; Thu, 19 Nov 2020 19:14:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=/zGvD23D7UzPmkeJaICeLLr5BBiyIFomvZLFOZzlxm4=; b=K/ex82SL594RfazFznOBdSPZAoIgw2gtJlLr4AhlET3fTfMnmidSmJ5v48KHjGQUiz Bz4RUYB5LACvSG4sr8W5+BjRLaw4qYxFR6fHQzN3i2jG3AE7QyKRvbYJ5MQHQay0iWAe JoX589dbEyXYyLjASuftwf6P/H1Xa/9J8MAzc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=/zGvD23D7UzPmkeJaICeLLr5BBiyIFomvZLFOZzlxm4=; b=sWZbptmGyS0HPorNQPHEBCpGT7PWD1PZ2YKUlV704TIbMKf0omwrJYNl9x1MggZZGS 3GC0IeLVmzgVRQ+swjXDkGwHoKyHVYFZBeKA8hH+6dX+jfKRDHYKXCa9fYE261x8sWZd BwGl5kY150JkX7hYFtM+6K0u0WzEg2JKt4Xu5xu8WJ4+Iuj6eBtbuIZjLYny9WQosJn+ QMep6Nb1xb26cR5rFo4uIYqSfb7FG1ipIAnIfikSpMTHBkMiBwrmRNHi6vub9c2Sj4Ro K8Bre5HXAv/H9dG4wxPiHOBGBhZl3ziMEbpHGzJa8kZ5z3v3eLuNjk0RGk7twMcfGh9T 34Ag== X-Gm-Message-State: AOAM531lFX9Lrzq20BmqlJzZ2zYNAIt7FeRjLy52Ltl1uQMqbvkbDTjC iEdcsZPb+IcUBMe7HZiDdezU7lhjsfF/EL+qpH7ENg== X-Received: by 2002:a37:6195:: with SMTP id v143mr13545153qkb.71.1605842043665; Thu, 19 Nov 2020 19:14:03 -0800 (PST) MIME-Version: 1.0 References: <20201119063211.2264-1-utkarsh.h.patel@intel.com> <20201119063211.2264-3-utkarsh.h.patel@intel.com> <20201119080906.GE3652649@google.com> In-Reply-To: From: Prashant Malani Date: Thu, 19 Nov 2020 19:13:51 -0800 Message-ID: Subject: Re: [PATCH v3 2/4] platform/chrome: cros_ec_typec: Use Thunderbolt 3 cable discover mode VDO in USB4 mode To: "Patel, Utkarsh H" Cc: "linux-kernel@vger.kernel.org" , "linux-usb@vger.kernel.org" , "heikki.krogerus@linux.intel.com" , "enric.balletbo@collabora.com" , "Mani, Rajmohan" , "Shaikh, Azhar" Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Utkarsh, On Thu, Nov 19, 2020 at 6:32 PM Patel, Utkarsh H wrote: > > Hi Prashant, > > > -----Original Message----- > > From: Prashant Malani > > Sent: Thursday, November 19, 2020 12:09 AM > > To: Patel, Utkarsh H > > Cc: linux-kernel@vger.kernel.org; linux-usb@vger.kernel.org; > > heikki.krogerus@linux.intel.com; enric.balletbo@collabora.com; Mani, > > Rajmohan ; Shaikh, Azhar > > > > Subject: Re: [PATCH v3 2/4] platform/chrome: cros_ec_typec: Use Thunderbolt > > 3 cable discover mode VDO in USB4 mode > > > > Hi Utkarsh, > > > > On Wed, Nov 18, 2020 at 10:32:09PM -0800, Utkarsh Patel wrote: > > > Configure Thunderbolt 3 cable generation value by filling Thunderbolt > > > 3 cable discover mode VDO to support rounded Thunderbolt 3 cables. > > > While we are here use Thunderbolt 3 cable discover mode VDO to fill > > > active cable plug link training value. > > > > > > Suggested-by: Heikki Krogerus > > > Signed-off-by: Utkarsh Patel > > > > > > -- > > > Changes in v3: > > > - Added a check for cable's TBT support before filling TBT3 discover mode > > > VDO. > > > > > > Changes in v2: > > > - No change. > > > -- > > > --- > > > drivers/platform/chrome/cros_ec_typec.c | 14 ++++++++++++-- > > > 1 file changed, 12 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/platform/chrome/cros_ec_typec.c > > > b/drivers/platform/chrome/cros_ec_typec.c > > > index 8111ed1fc574..68b17ee1d1ae 100644 > > > --- a/drivers/platform/chrome/cros_ec_typec.c > > > +++ b/drivers/platform/chrome/cros_ec_typec.c > > > @@ -514,8 +514,18 @@ static int cros_typec_enable_usb4(struct > > cros_typec_data *typec, > > > else if (pd_ctrl->control_flags & USB_PD_CTRL_ACTIVE_CABLE) > > > data.eudo |= EUDO_CABLE_TYPE_RE_TIMER << > > EUDO_CABLE_TYPE_SHIFT; > > > > > > - data.active_link_training = !!(pd_ctrl->control_flags & > > > - USB_PD_CTRL_ACTIVE_LINK_UNIDIR); > > > + /* > > > + * Filling TBT3 Cable VDO when TBT3 cable is being used to establish > > > + * USB4 connection. > > > + */ > > > + if (pd_ctrl->cable_gen) { > > > + data.tbt_cable_vdo = TBT_MODE; > > > + > > > + if (pd_ctrl->control_flags & > > USB_PD_CTRL_ACTIVE_LINK_UNIDIR) > > > + data.tbt_cable_vdo |= TBT_CABLE_LINK_TRAINING; > > > + > > > + data.tbt_cable_vdo |= TBT_SET_CABLE_ROUNDED(pd_ctrl- > > >cable_gen); > > > + } > > > > I think the following would decouple Rounded Support and Active Cable Link > > Training?: > > Any reason you would want to decouple them? Is there anything in the spec that says Active Cable Link Training needs Rounded Cable support (or vice versa)? If yes, could you kindly point me to the relevant portion in the spec that states this? If no, then the two should be set independently based on the response from the Chrome EC. FWIW, Table F-11 ( TBT3 Cable Discover Mode VDO Responses) from the USB Type-C Cable & Connector Spec (Rel 2.0) suggests the two are independent bits although I don't have access to the TBT3 spec to confirm. BR, -Prashant