From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C35274569C for ; Wed, 24 Jul 2024 13:34:49 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id AEB6842D89; Wed, 24 Jul 2024 13:34:49 +0200 (CEST) Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) by mails.dpdk.org (Postfix) with ESMTP id B0224433DE for ; Wed, 24 Jul 2024 13:34:48 +0200 (CEST) Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-4257d5fc9b7so58329815e9.2 for ; Wed, 24 Jul 2024 04:34:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1721820888; x=1722425688; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=Y4BALn/vWM3vmi8bbTftveGEy8BBXtz1bPmsKdEi1Rc=; b=Nur159dWZS3qehrL2bHRYQ7lh3nIeOVFukNf04LgD++Mll79dE4E9cfH9IdfiW7Ujt kwnU2m4hY7ui0CHdedHD7cx3EKiAJoyCn8uDNbYsKJgUiZmf/Rtd6alJH+weBZ7W98b0 iBpPahEIbv8tC/cxhAxbrBPxSn72P4QqAbJGyqx6he8MaIcRLxwxC/QzKldISK/8YEqG Qe/rsPnxtY+2QcAt6fSvu8zwNwMjrNfKNbJhFkVL4rx4NdT+Q84VChLXK09aAUXUTL9h Gbod50jG9cqZdCsEgwbjIP+0GBjnd5NpVDKNcWTDdn1tReaCosLPwiGwKGjXjwifzaNC RvRw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1721820888; x=1722425688; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Y4BALn/vWM3vmi8bbTftveGEy8BBXtz1bPmsKdEi1Rc=; b=iY549uCaE0+sFodnHZlDymXg64Q81xOOSb0mUYWW1NL8ivuQbx1mnCr7VCaoeYKEd0 4Y373j93gOXMDfkSEZYTGn0O6MUyc0jOdYewlFKyvyeFvH5e4w2ZsprfSWhjbI+zKeWM RibeD47x/9Nmf8YHeqyeWuGyMC5FiYrdDtyYHFis6fF5RCsSwzyUMDuSiy/79iImjjXX B//UhxZgO5K9S6Yn7W8ttqc7I+Ovl3G8kgPq4dehehVNhNaSeQEJZafYgCIpbqnd/76l IYJ0eBRpRTEvuu+2bL/Xg2ApRxPGYSHkny3TTINNI8AZrDrYCsXsTOAeQSAXAvw5yH8U myQg== X-Gm-Message-State: AOJu0YyqiWxDkxNlY9MOUmx0iroSTEAOqmH7cMzqJ5qw8qj4gNs9S5fU Oar2byfG6hWvY6AOSVSBLrJvqBlngoVbMb9CytNYkspMdmWgJJpXmMJRg5Yh X-Google-Smtp-Source: AGHT+IGRHiwMxLgHedkSSOb017G+JYaKhNmGri1VORJd8mpXu8TqCVQM/KzjhgodHj+MezU0zVRdBQ== X-Received: by 2002:a05:600c:3b88:b0:425:64c5:5780 with SMTP id 5b1f17b1804b1-427f7ab64dfmr22948545e9.1.1721820888099; Wed, 24 Jul 2024 04:34:48 -0700 (PDT) Received: from localhost ([2a01:4b00:d036:ae00:7aef:1aaa:3dff:d546]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3687868b29fsm14121474f8f.32.2024.07.24.04.34.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 24 Jul 2024 04:34:47 -0700 (PDT) From: luca.boccassi@gmail.com To: Chengwen Feng Cc: dpdk stable Subject: patch 'dma/hisilicon: remove support for HIP09 platform' has been queued to stable release 22.11.6 Date: Wed, 24 Jul 2024 12:33:06 +0100 Message-Id: <20240724113318.616754-28-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20240724113318.616754-1-luca.boccassi@gmail.com> References: <20240715152704.2229503-86-luca.boccassi@gmail.com> <20240724113318.616754-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Hi, FYI, your patch has been queued to stable release 22.11.6 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 07/26/24. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/1b1337cc9316675d9cd4982d78d4abf5891c965f Thanks. Luca Boccassi --- >From 1b1337cc9316675d9cd4982d78d4abf5891c965f Mon Sep 17 00:00:00 2001 From: Chengwen Feng Date: Thu, 4 Jul 2024 02:53:17 +0000 Subject: [PATCH] dma/hisilicon: remove support for HIP09 platform [ upstream commit 2a3f42942a5f4df62108679ef44a34d21b4a2553 ] The DMA for HIP09 is no longer available, so delete it. Signed-off-by: Chengwen Feng --- doc/guides/dmadevs/hisilicon.rst | 1 - drivers/dma/hisilicon/hisi_dmadev.c | 40 +---------------------------- drivers/dma/hisilicon/hisi_dmadev.h | 35 +------------------------ 3 files changed, 2 insertions(+), 74 deletions(-) diff --git a/doc/guides/dmadevs/hisilicon.rst b/doc/guides/dmadevs/hisilicon.rst index 8c1f0f8886..974bc49376 100644 --- a/doc/guides/dmadevs/hisilicon.rst +++ b/doc/guides/dmadevs/hisilicon.rst @@ -13,7 +13,6 @@ Supported Kunpeng SoCs ---------------------- * Kunpeng 920 -* Kunpeng 930 Device Setup diff --git a/drivers/dma/hisilicon/hisi_dmadev.c b/drivers/dma/hisilicon/hisi_dmadev.c index 0e11ca14cc..4db3b0554c 100644 --- a/drivers/dma/hisilicon/hisi_dmadev.c +++ b/drivers/dma/hisilicon/hisi_dmadev.c @@ -39,8 +39,6 @@ hisi_dma_queue_base(struct hisi_dma_dev *hw) { if (hw->reg_layout == HISI_DMA_REG_LAYOUT_HIP08) return HISI_DMA_HIP08_QUEUE_BASE; - else if (hw->reg_layout == HISI_DMA_REG_LAYOUT_HIP09) - return HISI_DMA_HIP09_QUEUE_BASE; else return 0; } @@ -216,25 +214,6 @@ hisi_dma_init_hw(struct hisi_dma_dev *hw) HISI_DMA_HIP08_QUEUE_INT_MASK_M, true); hisi_dma_update_queue_mbit(hw, HISI_DMA_QUEUE_INT_MASK_REG, HISI_DMA_HIP08_QUEUE_INT_MASK_M, true); - } else if (hw->reg_layout == HISI_DMA_REG_LAYOUT_HIP09) { - hisi_dma_update_queue_mbit(hw, HISI_DMA_QUEUE_CTRL0_REG, - HISI_DMA_HIP09_QUEUE_CTRL0_ERR_ABORT_M, false); - hisi_dma_update_queue_mbit(hw, HISI_DMA_QUEUE_INT_STATUS_REG, - HISI_DMA_HIP09_QUEUE_INT_MASK_M, true); - hisi_dma_update_queue_mbit(hw, HISI_DMA_QUEUE_INT_MASK_REG, - HISI_DMA_HIP09_QUEUE_INT_MASK_M, true); - hisi_dma_update_queue_mbit(hw, - HISI_DMA_HIP09_QUEUE_ERR_INT_STATUS_REG, - HISI_DMA_HIP09_QUEUE_ERR_INT_MASK_M, true); - hisi_dma_update_queue_mbit(hw, - HISI_DMA_HIP09_QUEUE_ERR_INT_MASK_REG, - HISI_DMA_HIP09_QUEUE_ERR_INT_MASK_M, true); - hisi_dma_update_queue_bit(hw, HISI_DMA_QUEUE_CTRL1_REG, - HISI_DMA_HIP09_QUEUE_CTRL1_VA_ENABLE_B, true); - hisi_dma_update_bit(hw, - HISI_DMA_HIP09_QUEUE_CFG_REG(hw->queue_id), - HISI_DMA_HIP09_QUEUE_CFG_LINK_DOWN_MASK_B, - true); } } @@ -256,8 +235,6 @@ hisi_dma_reg_layout(uint8_t revision) { if (revision == HISI_DMA_REVISION_HIP08B) return HISI_DMA_REG_LAYOUT_HIP08; - else if (revision >= HISI_DMA_REVISION_HIP09A) - return HISI_DMA_REG_LAYOUT_HIP09; else return HISI_DMA_REG_LAYOUT_INVALID; } @@ -328,14 +305,11 @@ hisi_dma_info_get(const struct rte_dma_dev *dev, struct rte_dma_info *dev_info, uint32_t info_sz) { - struct hisi_dma_dev *hw = dev->data->dev_private; + RTE_SET_USED(dev); RTE_SET_USED(info_sz); dev_info->dev_capa = RTE_DMA_CAPA_MEM_TO_MEM | RTE_DMA_CAPA_OPS_COPY; - if (hw->reg_layout == HISI_DMA_REG_LAYOUT_HIP09) - dev_info->dev_capa |= RTE_DMA_CAPA_HANDLES_ERRORS; - dev_info->max_vchans = 1; dev_info->max_desc = HISI_DMA_MAX_DESC_NUM; dev_info->min_desc = HISI_DMA_MIN_DESC_NUM; @@ -514,18 +488,6 @@ hisi_dma_dump_common(struct hisi_dma_dev *hw, FILE *f) { HISI_DMA_REG_LAYOUT_HIP08, HISI_DMA_HIP08_DUMP_START_REG, HISI_DMA_HIP08_DUMP_END_REG }, - { HISI_DMA_REG_LAYOUT_HIP09, - HISI_DMA_HIP09_DUMP_REGION_A_START_REG, - HISI_DMA_HIP09_DUMP_REGION_A_END_REG }, - { HISI_DMA_REG_LAYOUT_HIP09, - HISI_DMA_HIP09_DUMP_REGION_B_START_REG, - HISI_DMA_HIP09_DUMP_REGION_B_END_REG }, - { HISI_DMA_REG_LAYOUT_HIP09, - HISI_DMA_HIP09_DUMP_REGION_C_START_REG, - HISI_DMA_HIP09_DUMP_REGION_C_END_REG }, - { HISI_DMA_REG_LAYOUT_HIP09, - HISI_DMA_HIP09_DUMP_REGION_D_START_REG, - HISI_DMA_HIP09_DUMP_REGION_D_END_REG }, }; uint32_t i; diff --git a/drivers/dma/hisilicon/hisi_dmadev.h b/drivers/dma/hisilicon/hisi_dmadev.h index 5a17f9f69e..a57b5c759a 100644 --- a/drivers/dma/hisilicon/hisi_dmadev.h +++ b/drivers/dma/hisilicon/hisi_dmadev.h @@ -25,22 +25,14 @@ #define HISI_DMA_DEVICE_ID 0xA122 #define HISI_DMA_PCI_REVISION_ID_REG 0x08 #define HISI_DMA_REVISION_HIP08B 0x21 -#define HISI_DMA_REVISION_HIP09A 0x30 #define HISI_DMA_MAX_HW_QUEUES 4 #define HISI_DMA_MAX_DESC_NUM 8192 #define HISI_DMA_MIN_DESC_NUM 32 -/** - * The HIP08B(HiSilicon IP08) and HIP09B(HiSilicon IP09) are DMA iEPs, they - * have the same pci device id but different pci revision. - * Unfortunately, they have different register layouts, so two layout - * enumerations are defined. - */ enum { HISI_DMA_REG_LAYOUT_INVALID = 0, - HISI_DMA_REG_LAYOUT_HIP08, - HISI_DMA_REG_LAYOUT_HIP09 + HISI_DMA_REG_LAYOUT_HIP08 }; /** @@ -69,9 +61,6 @@ enum { * length of queue-region. The global offset for a single queue register is * calculated by: * offset = queue-base + (queue-id * queue-region) + reg-offset-in-region. - * - * The first part of queue region is basically the same for HIP08 and HIP09 - * register layouts, therefore, HISI_QUEUE_* registers are defined for it. */ #define HISI_DMA_QUEUE_SQ_BASE_L_REG 0x0 #define HISI_DMA_QUEUE_SQ_BASE_H_REG 0x4 @@ -110,28 +99,6 @@ enum { #define HISI_DMA_HIP08_DUMP_START_REG 0x2000 #define HISI_DMA_HIP08_DUMP_END_REG 0x2280 -/** - * HiSilicon IP09 DMA register and field define: - */ -#define HISI_DMA_HIP09_QUEUE_BASE 0x2000 -#define HISI_DMA_HIP09_QUEUE_CTRL0_ERR_ABORT_M GENMASK(31, 28) -#define HISI_DMA_HIP09_QUEUE_CTRL1_VA_ENABLE_B 2 -#define HISI_DMA_HIP09_QUEUE_INT_MASK_M 0x1 -#define HISI_DMA_HIP09_QUEUE_ERR_INT_STATUS_REG 0x48 -#define HISI_DMA_HIP09_QUEUE_ERR_INT_MASK_REG 0x4C -#define HISI_DMA_HIP09_QUEUE_ERR_INT_MASK_M GENMASK(18, 1) -#define HISI_DMA_HIP09_QUEUE_CFG_REG(queue_id) (0x800 + \ - (queue_id) * 0x20) -#define HISI_DMA_HIP09_QUEUE_CFG_LINK_DOWN_MASK_B 16 -#define HISI_DMA_HIP09_DUMP_REGION_A_START_REG 0x0 -#define HISI_DMA_HIP09_DUMP_REGION_A_END_REG 0x368 -#define HISI_DMA_HIP09_DUMP_REGION_B_START_REG 0x800 -#define HISI_DMA_HIP09_DUMP_REGION_B_END_REG 0xA08 -#define HISI_DMA_HIP09_DUMP_REGION_C_START_REG 0x1800 -#define HISI_DMA_HIP09_DUMP_REGION_C_END_REG 0x1A4C -#define HISI_DMA_HIP09_DUMP_REGION_D_START_REG 0x1C00 -#define HISI_DMA_HIP09_DUMP_REGION_D_END_REG 0x1CC4 - /** * In fact, there are multiple states, but it need to pay attention to * the following three states for the driver: -- 2.39.2 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2024-07-24 12:29:22.884110361 +0100 +++ 0028-dma-hisilicon-remove-support-for-HIP09-platform.patch 2024-07-24 12:29:20.799026695 +0100 @@ -1 +1 @@ -From 2a3f42942a5f4df62108679ef44a34d21b4a2553 Mon Sep 17 00:00:00 2001 +From 1b1337cc9316675d9cd4982d78d4abf5891c965f Mon Sep 17 00:00:00 2001 @@ -6 +6 @@ -The DMA for HIP09 is no longer available, so delete it. +[ upstream commit 2a3f42942a5f4df62108679ef44a34d21b4a2553 ] @@ -8 +8 @@ -Cc: stable@dpdk.org +The DMA for HIP09 is no longer available, so delete it. @@ -12,5 +12,4 @@ - doc/guides/dmadevs/hisilicon.rst | 1 - - doc/guides/rel_notes/release_24_07.rst | 5 ++++ - drivers/dma/hisilicon/hisi_dmadev.c | 40 +------------------------- - drivers/dma/hisilicon/hisi_dmadev.h | 35 +--------------------- - 4 files changed, 7 insertions(+), 74 deletions(-) + doc/guides/dmadevs/hisilicon.rst | 1 - + drivers/dma/hisilicon/hisi_dmadev.c | 40 +---------------------------- + drivers/dma/hisilicon/hisi_dmadev.h | 35 +------------------------ + 3 files changed, 2 insertions(+), 74 deletions(-) @@ -30,16 +28,0 @@ -diff --git a/doc/guides/rel_notes/release_24_07.rst b/doc/guides/rel_notes/release_24_07.rst -index 058609b0f3..eb2ed1a55f 100644 ---- a/doc/guides/rel_notes/release_24_07.rst -+++ b/doc/guides/rel_notes/release_24_07.rst -@@ -204,6 +204,11 @@ Removed Items - BPF is not supported and the librte-bpf test fails on 32-bit x86 kernels. - So disable the library and the pmd. - -+* **Removed hisilicon DMA support for HIP09 platform.** -+ -+ The DMA for HIP09 is no longer available, -+ so the support is removed from hisilicon driver for HIP09 platform. -+ - - API Changes - ----------- @@ -47 +30 @@ -index e96bc1d654..7cd6ebc1e0 100644 +index 0e11ca14cc..4db3b0554c 100644 @@ -94 +77 @@ -@@ -327,14 +304,11 @@ hisi_dma_info_get(const struct rte_dma_dev *dev, +@@ -328,14 +305,11 @@ hisi_dma_info_get(const struct rte_dma_dev *dev, @@ -110 +93 @@ -@@ -513,18 +487,6 @@ hisi_dma_dump_common(struct hisi_dma_dev *hw, FILE *f) +@@ -514,18 +488,6 @@ hisi_dma_dump_common(struct hisi_dma_dev *hw, FILE *f)