All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Gil Fine <gil.fine@intel.com>
Cc: andreas.noever@gmail.com, michael.jamet@intel.com,
	YehezkelShB@gmail.com, linux-usb@vger.kernel.org,
	lukas@wunner.de
Subject: Re: [PATCH v2 0/7] thunderbolt: CLx support for USB4 and Titan Ridge
Date: Tue, 28 Dec 2021 09:52:07 +0200	[thread overview]
Message-ID: <YcrCJ+WlUsqKFtsW@lahna> (raw)
In-Reply-To: <20211227161344.GJ24036@ccdjLinux26>

Hi Gil,

On Mon, Dec 27, 2021 at 06:13:44PM +0200, Gil Fine wrote:
> Hi Mika,
> About latest patches series adding CL0s in thunderbolt/next.git
> 1.
> Attached fix for CL0s enabled check - w/o it, the CL0s enabling fails

Thanks, I folded this to patch "thunderbolt: Enable CL0s for Intel
Titan Ridge".

> 2.
> Maybe it is better not to mention "Intel limitation..." about the first-hop
> router since I am not sure this is correct. Attached patch w/ my proposal.

Well it is targeted only for Intel Titan Ridge so I think we can keep
it for documentation purposes :)

> 3.
> Don't we prefer to stay with the name: "tb_switch_titan_ridge_pcie_l1_enable" ?
> so that it is clear that the function is Titan Ridge specific?
> If so, sending patch for this

It works with any router (as it specifically checks for Titan Ridge
before going further), I think we can keep the name more generic too.

      reply	other threads:[~2021-12-28  7:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-17  1:16 [PATCH v2 0/7] thunderbolt: CLx support for USB4 and Titan Ridge Gil Fine
2021-12-17  1:16 ` [PATCH v2 1/7] thunderbolt: Add TMU unidirectional mode Gil Fine
2021-12-17  1:16 ` [PATCH v2 2/7] thunderbolt: Add CL0s support for USB4 Gil Fine
2021-12-17  1:16 ` [PATCH v2 3/7] thunderbolt: Move usb4_switch_wait_for_bit() to switch.c Gil Fine
2021-12-17  1:16 ` [PATCH v2 4/7] thunderbolt: Enable TMU for Titan Ridge device Gil Fine
2021-12-17  1:16 ` [PATCH v2 5/7] thunderbolt: Rename Intel VSC capability Gil Fine
2021-12-17  1:16 ` [PATCH v2 6/7] thunderbolt: Enable CL0s for Titan Ridge device Gil Fine
2021-12-17  1:16 ` [PATCH v2 7/7] thunderbolt: Add kernel param for CLx disabling Gil Fine
2021-12-22 14:18 ` [PATCH v2 0/7] thunderbolt: CLx support for USB4 and Titan Ridge Mika Westerberg
2021-12-27 16:13   ` Gil Fine
2021-12-28  7:52     ` Mika Westerberg [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YcrCJ+WlUsqKFtsW@lahna \
    --to=mika.westerberg@linux.intel.com \
    --cc=YehezkelShB@gmail.com \
    --cc=andreas.noever@gmail.com \
    --cc=gil.fine@intel.com \
    --cc=linux-usb@vger.kernel.org \
    --cc=lukas@wunner.de \
    --cc=michael.jamet@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.