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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9CB65CD54BF for ; Mon, 25 Sep 2023 19:28:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233296AbjIYT2H (ORCPT ); Mon, 25 Sep 2023 15:28:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50084 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233308AbjIYT2E (ORCPT ); Mon, 25 Sep 2023 15:28:04 -0400 Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.24]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8FF0B10C; Mon, 25 Sep 2023 12:27:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1695670078; x=1727206078; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=G+AzBDEkkM1eADSIyN25cfwmNNwetV4UoEJVlZExwOM=; b=Pl+pSi3ZUIbM1xn4IahcvgUD+Fbz4qkphC/Z5UTzEQA5AlIr/ZKf7bxg /KehrDltwLEQUX3IyNUZRxslf4ejTB41g7c8Wr2WG4Yobe9MuQEOJDbWb V3Gb5RC/RfqqJUBCkbNAQMzYLCKeM+gKM7gFiFSgQhaTXu6TH0a9Lwzmt QtWj0QQU+FPibRpY6Eb6UVhOIwWSALEw3Ta6IIuzj1lBssAvjef4T3x4s uTR489LmPu4Q/s9VTQfoU1c8RfARWhz+iaTjQjlsZaDzW86NnN7n0Jhjm O2cJX4Iakm6QVAHCKlr2/K0yf0vjPNvNv1Ycs453q1YW5vHvQjVycdav1 g==; X-IronPort-AV: E=McAfee;i="6600,9927,10843"; a="384123235" X-IronPort-AV: E=Sophos;i="6.03,175,1694761200"; d="scan'208";a="384123235" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Sep 2023 12:27:56 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10843"; a="995506037" X-IronPort-AV: E=Sophos;i="6.03,175,1694761200"; d="scan'208";a="995506037" Received: from smile.fi.intel.com ([10.237.72.54]) by fmsmga006.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Sep 2023 12:27:53 -0700 Received: from andy by smile.fi.intel.com with local (Exim 4.97-RC0) (envelope-from ) id 1qkrFG-00000000Q7q-3Fmf; Mon, 25 Sep 2023 22:27:50 +0300 Date: Mon, 25 Sep 2023 22:27:50 +0300 From: Andy Shevchenko To: "Patel, Utkarsh H" Cc: "linux-kernel@vger.kernel.org" , "linux-usb@vger.kernel.org" , "heikki.krogerus@linux.intel.com" , "pmalani@chromium.org" , "chrome-platform@lists.linux.dev" , "bleung@chromium.org" Subject: Re: [PATCH v4 4/5] platform/chrome: cros_ec_typec: Add Displayport Alternatemode 2.1 Support Message-ID: References: <20230920023243.2494410-1-utkarsh.h.patel@intel.com> <20230920023243.2494410-5-utkarsh.h.patel@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 25, 2023 at 03:54:40PM +0000, Patel, Utkarsh H wrote: ... > > > + /** > > > > Are you sure? > > > > > + * Get cable VDO for thunderbolt cables and cables with DPSID but > > does not > > > + * support DPAM2.1. > > > + */ > > > Yes, there are TBT3 cables which advertise DPSID but does not provide any DP > capabilities in the DP discover mode VDO but does support UHBR. In that > case, need to use TBTSID and use capabilities from TBT discover mode VDO. My comment was against the style of the comment, not about content. ... > > You can also make it a bit more readable with (use better names if you think it's > > needed) > > > > u32 signalling = 0; > > u32 cable_type = 0; > > In v2 version of this patch I had them but there was feedback to remove extra > variables and use them inline. OK! -- With Best Regards, Andy Shevchenko