To:     Kelvin Cao <kelvin.cao@microchip.com>,
        Kurt Schwemmer <kurt.schwemmer@microsemi.com>,
        Bjorn Helgaas <bhelgaas@google.com>,
        =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= <kw@linux.com>,
        linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     kelvincao@outlook.com
References: <20211119003803.2333-1-kelvin.cao@microchip.com>
From:   Logan Gunthorpe <logang@deltatee.com>
Message-ID: <cf1bc79c-718c-ce23-fae8-178d0f545901@deltatee.com>
Date:   Fri, 19 Nov 2021 09:49:39 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
MIME-Version: 1.0
In-Reply-To: <20211119003803.2333-1-kelvin.cao@microchip.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-CA
Content-Transfer-Encoding: 7bit
Subject: Re: [PATCH 0/2] Add Switchtec Gen4 automotive device IDs and a tweak
Precedence: bulk


On 2021-11-18 5:38 p.m., Kelvin Cao wrote:
> Hi,
> 
> This patchset introduces device IDs for the Switchtec Gen4 automotive
> variants and a minor tweak for the MRPC execution.
> 
> The first patch adds the device IDs. Patch 2 makes the tweak to improve
> the MRPC execution efficiency [1].
> 
> This patchset is based on v5.16-rc1.
> 
> [1] https://lore.kernel.org/r/20211014141859.11444-1-kelvin.cao@microchip.com/
> 
> Thanks,
> Kelvin
> 
> Kelvin Cao (2):
>   Add device IDs for the Gen4 automotive variants
>   Declare local array state_names as static
> 
>  drivers/pci/quirks.c           |  9 +++++++++
>  drivers/pci/switch/switchtec.c | 11 ++++++++++-
>  2 files changed, 19 insertions(+), 1 deletion(-)

Looks fine to me.

Reviewed-by: Logan Gunthorpe <logang@deltatee.com>

Logan