From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 146DB441044 for ; Tue, 5 May 2026 10:17:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777976265; cv=none; b=E2jSC9e1Old5oNAgHrksBjMk/dkqyseQWpWkAB2cc/Ch7QUI4oUuOGmYeNilSsBIQiYZRPFItWBgUmkxYczslxYS5DGHxNxBD7OzhIfbPBcqGUdBBx8fcuwwcEhVds9A6llgWHVtg0t3e52agyEeqxpxJei7hIhDoh1EX8xhZOk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777976265; c=relaxed/simple; bh=LN0w4doIU962zs+vEibFmYqoKNVztwjLgnNOjtYEzKg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=np43/Yveop2lK8zJ3/ZjJlnj2TJNQ3TfY0TrBmUbeAbYIVyAZFlZQSicEIVmM6sOAPCwkRHCdtuWLAJOFNYCwWrPpFiz6OT4asqujCE5by5ZPeoL3tQrqixaR2xOP62tbt+8N1jDl+oq6a9WMx8FH72JPGEKLtPdPHRlgh3RUpc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=QHd7t60h; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="QHd7t60h" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 26A18C2BCB4; Tue, 5 May 2026 10:17:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1777976264; bh=LN0w4doIU962zs+vEibFmYqoKNVztwjLgnNOjtYEzKg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=QHd7t60hqTaduf3A2SKmpdEiEjsyFkjifDPFD5vFuMUw+RcNFqwQBYGtAo7jOja3f FQwllo4i9FOYyAYNbJlPE/rbTWukaXFbqj8d6FOAjbJMEBXX1aO8D0CoVFC3iRVEli fvQtmo2pW129mYKDPdzUb4zZ0gkYMV6H4MZWGdB1aZYJC/D2ayVUiZIodjbpI3Qm3I sIfbwi5NBDJpnnDlcGMnVgK7i49UTgkSPQb8zpgFVTFGYXpX2ZkPO1MoSkZ++0sy0T dCqflWQiHiWqH714xm5DhncdaqGl15XK5O8yt9AXvJeHkNKOdsFlOOmMi9zWlySnZf H8TwaDH+0NwHw== From: Sasha Levin To: stable@vger.kernel.org Cc: Suman Kumar Chakraborty , Giovanni Cabiddu , Andy Shevchenko , Herbert Xu , Sasha Levin Subject: [PATCH 7.0.y 2/2] crypto: qat - fix firmware loading failure for GEN6 devices Date: Tue, 5 May 2026 06:17:38 -0400 Message-ID: <20260505101738.582879-2-sashal@kernel.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260505101738.582879-1-sashal@kernel.org> References: <2026050339-tumble-sponge-5d2f@gregkh> <20260505101738.582879-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: Suman Kumar Chakraborty [ Upstream commit e7dcb722bb75bb3f3992f580a8728a794732fd7a ] QAT GEN6 hardware requires a minimum 3 us delay during the acceleration engine reset sequence to ensure the hardware fully settles. Without this delay, the firmware load may fail intermittently. Add a delay after placing the AE into reset and before clearing the reset, matching the hardware requirements and ensuring stable firmware loading. Earlier generations remain unaffected. Fixes: 17fd7514ae68 ("crypto: qat - add qat_6xxx driver") Signed-off-by: Suman Kumar Chakraborty Cc: stable@vger.kernel.org Reviewed-by: Giovanni Cabiddu Reviewed-by: Andy Shevchenko Signed-off-by: Herbert Xu Signed-off-by: Sasha Levin --- drivers/crypto/intel/qat/qat_common/adf_accel_engine.c | 7 +++++++ .../crypto/intel/qat/qat_common/icp_qat_fw_loader_handle.h | 1 + drivers/crypto/intel/qat/qat_common/qat_hal.c | 5 ++++- 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/crypto/intel/qat/qat_common/adf_accel_engine.c b/drivers/crypto/intel/qat/qat_common/adf_accel_engine.c index f9f1018a28236..09d4f547e082c 100644 --- a/drivers/crypto/intel/qat/qat_common/adf_accel_engine.c +++ b/drivers/crypto/intel/qat/qat_common/adf_accel_engine.c @@ -1,5 +1,6 @@ // SPDX-License-Identifier: (BSD-3-Clause OR GPL-2.0-only) /* Copyright(c) 2014 - 2020 Intel Corporation */ +#include #include #include #include "adf_cfg.h" @@ -162,8 +163,14 @@ int adf_ae_stop(struct adf_accel_dev *accel_dev) static int adf_ae_reset(struct adf_accel_dev *accel_dev, int ae) { struct adf_fw_loader_data *loader_data = accel_dev->fw_loader; + unsigned long reset_delay; qat_hal_reset(loader_data->fw_loader); + + reset_delay = loader_data->fw_loader->chip_info->reset_delay_us; + if (reset_delay) + fsleep(reset_delay); + if (qat_hal_clr_reset(loader_data->fw_loader)) return -EFAULT; diff --git a/drivers/crypto/intel/qat/qat_common/icp_qat_fw_loader_handle.h b/drivers/crypto/intel/qat/qat_common/icp_qat_fw_loader_handle.h index 6887930c7995e..e74cafa95f1cc 100644 --- a/drivers/crypto/intel/qat/qat_common/icp_qat_fw_loader_handle.h +++ b/drivers/crypto/intel/qat/qat_common/icp_qat_fw_loader_handle.h @@ -27,6 +27,7 @@ struct icp_qat_fw_loader_chip_info { int mmp_sram_size; bool nn; bool lm2lm3; + u16 reset_delay_us; u32 lm_size; u32 icp_rst_csr; u32 icp_rst_mask; diff --git a/drivers/crypto/intel/qat/qat_common/qat_hal.c b/drivers/crypto/intel/qat/qat_common/qat_hal.c index 0f5a2690690a1..1c3d1311f1c7a 100644 --- a/drivers/crypto/intel/qat/qat_common/qat_hal.c +++ b/drivers/crypto/intel/qat/qat_common/qat_hal.c @@ -20,6 +20,7 @@ #define RST_CSR_QAT_LSB 20 #define RST_CSR_AE_LSB 0 #define MC_TIMESTAMP_ENABLE (0x1 << 7) +#define MIN_RESET_DELAY_US 3 #define IGNORE_W1C_MASK ((~(1 << CE_BREAKPOINT_BITPOS)) & \ (~(1 << CE_CNTL_STORE_PARITY_ERROR_BITPOS)) & \ @@ -713,8 +714,10 @@ static int qat_hal_chip_init(struct icp_qat_fw_loader_handle *handle, handle->chip_info->wakeup_event_val = 0x80000000; handle->chip_info->fw_auth = true; handle->chip_info->css_3k = true; - if (handle->pci_dev->device == PCI_DEVICE_ID_INTEL_QAT_6XXX) + if (handle->pci_dev->device == PCI_DEVICE_ID_INTEL_QAT_6XXX) { handle->chip_info->dual_sign = true; + handle->chip_info->reset_delay_us = MIN_RESET_DELAY_US; + } handle->chip_info->tgroup_share_ustore = true; handle->chip_info->fcu_ctl_csr = FCU_CONTROL_4XXX; handle->chip_info->fcu_sts_csr = FCU_STATUS_4XXX; -- 2.53.0