Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Szuying Chen <chensiying21@gmail.com>
To:     mika.westerberg@linux.intel.com, YehezkelShB@gmail.com
Cc:     gregkh@linuxfoundation.org, mario.limonciello@amd.com,
        andreas.noever@gmail.com, michael.jamet@intel.com,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
        Yd_Tseng@asmedia.com.tw, Chloe_Chen@asmedia.com.tw,
        Richard_Hsu@asmedia.com.tw
Subject: RE: [PATCH v9 0/6] thunderbolt: Add support for ASMedia NVM firmware image format
Date:   Tue,  6 Sep 2022 18:12:45 +0800
Message-Id: <20220906101245.6559-1-chensiying21@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

From: Szuying Chen <Chloe_Chen@asmedia.com.tw>

Hi,

> Hi all,
> 
> This series adds support for ASMedia NVM images. It is based on the
> work of Szuying Chen with a couple of changes from me. The first
> patches move Intel specific NVM validation into nvm.c and >adds a
> vendor table for both routers and retimers that can be extended to
> support different >vendor NVM image formats. Once that is done the
> ASMedia NVM image format support is added.
> 
> The previous version of the patch series can be found here:
> 
>   https://lore.kernel.org/linux-usb/20220902094010.2170-1-chensiying21@gmail.com/
> 
> Changes from the previous version:
> 
>   * Shuffled the ordering of the patches a bit
>   * Add tb_retimer_nvm_read() and tb_switch_nvm_read()
>   * Add ->write_headers() callback that is used with Thunderbolt 2
>     routers
>   * Cover retimers too
>   * Minor cleanups and tweaks
> 
> I have tested this on Intel Thunderbolt 3 and Thunderbolt 4 routers.
> ASMedia folks, can you try this also on your side and let me know if it works or not?

I send a comments in separate email please help to modify it.
I have tested this on ASMedia USB4 router. It work well.

> 
> Mika Westerberg (1):
>   thunderbolt: Provide tb_retimer_nvm_read() analogous to tb_switch_nvm_read()
> 
> Szuying Chen (5):
>   thunderbolt: Allow NVM upgrade of USB4 host routers
>   thunderbolt: Extend NVM version fields to 32-bits
>   thunderbolt: Rename and make nvm_read() available for other files
>   thunderbolt: Move vendor specific NVM handling into nvm.c
>   thunderbolt: Add support for ASMedia NVM image format
> 
>  drivers/thunderbolt/nvm.c     | 385 +++++++++++++++++++++++++++++++++-
>  drivers/thunderbolt/retimer.c | 105 ++++------  drivers/thunderbolt/switch.c  | 162 +++++---------
>  drivers/thunderbolt/tb.c      |   5 +-
>  drivers/thunderbolt/tb.h      |  30 ++-
>  5 files changed, 498 insertions(+), 189 deletions(-)

Thanks!

<p></p>