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;
IronPort-SDR: Ui6JidojlEaxEVB4wEVJ5O5nnfzSYSiB7JED6GD84NuRieGtnIlbfAvNyIDEJAX8uVrH/g/j06
 xF8A6aS/JgbQ==
IronPort-SDR: GXQIRMxbW+ii2kfDZ+sEivb30yP95UR7z9OHM7b228OGe6+c05HjxdSDhQlnkd6DLe1xX2VwP7
 ccXct5/T+FQA==
Date:   Wed, 1 Jul 2020 13:55:50 +0300
From:   Mika Westerberg <mika.westerberg@linux.intel.com>
To:     Mario Limonciello <mario.limonciello@dell.com>
Cc:     Andreas Noever <andreas.noever@gmail.com>,
        Michael Jamet <michael.jamet@intel.com>,
        Yehezkel Bernat <YehezkelShB@gmail.com>,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
        perry.yuan@dell.com
Subject: Re: [PATCH v3 0/2] Allow breaking up Thunderbolt/USB4 updates
Message-ID: <20200701105550.GR5180@lahna.fi.intel.com>
References: <20200623161429.24214-1-mario.limonciello@dell.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200623161429.24214-1-mario.limonciello@dell.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, Jun 23, 2020 at 11:14:27AM -0500, Mario Limonciello wrote:
> Currently updates to Thunderbolt and USB4 controllers are fully atomic
> actions. When writing into the non-active NVM nothing gets flushed to
> the hardware until authenticate is sent.
> 
> There has been some desire to improve the perceived performance of these
> updates, particularly for userland that may perform the update upon
> a performance sensitive time like logging out.
> 
> So allow userland to flush the image to hardware at runtime, and then
> allow authenticating the image at another time.
> 
> For the Dell WD19TB some specific hardware capability exists that allows
> extending this to automatically complete the update when unplugged.
> Export that functionality to userspace as well.
> 
> Changes from v2 to v3:
>  - Correct some whitespace and kernel-doc comments
>  - Add another missing 'const'
>  - For a quirk: (1<<1) -> BIT(0) 
> 
> Changes from v1 to v2:
>  - Improve documentation
>  - Drop tb-quirks.h
>  - Adjust function and parameter names to Mika's preferences
>  - Rebase onto thunderbolt.git/bleeding-edge to move on top of retimer work
> 
> Mario Limonciello (2):
>   thunderbolt: Add support for separating the flush to SPI and
>     authenticate
>   thunderbolt: Add support for authenticate on disconnect

Both applied to thunderbolt.git/next, thanks Mario!