Received-SPF: pass (google.com: domain of linux-crypto+bounces-2924-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: JiaJie Ho <jiajie.ho@starfivetech.com>
To: 'Herbert Xu' <herbert@gondor.apana.org.au>, "'David S . Miller'"
	<davem@davemloft.net>, 'Rob Herring' <robh+dt@kernel.org>, 'Krzysztof
 Kozlowski' <krzysztof.kozlowski+dt@linaro.org>, 'Conor Dooley'
	<conor+dt@kernel.org>, 'Eugeniy Paltsev' <Eugeniy.Paltsev@synopsys.com>,
	'Vinod Koul' <vkoul@kernel.org>, "'linux-crypto@vger.kernel.org'"
	<linux-crypto@vger.kernel.org>, "'devicetree@vger.kernel.org'"
	<devicetree@vger.kernel.org>, "'linux-kernel@vger.kernel.org'"
	<linux-kernel@vger.kernel.org>, "'dmaengine@vger.kernel.org'"
	<dmaengine@vger.kernel.org>
Subject: RE: [PATCH v4 0/7] crypto: starfive: Add support for JH8100
Thread-Topic: [PATCH v4 0/7] crypto: starfive: Add support for JH8100
Thread-Index: AQHabswuRP6vmqtmzUiwGSiP4uQyCbFK9DVg
Date: Wed, 27 Mar 2024 01:55:04 +0000
Message-ID:
 <NT0PR01MB1182C3CD53AA7CB251BDD99D8A34A@NT0PR01MB1182.CHNPR01.prod.partner.outlook.cn>
References: <20240305071006.2181158-1-jiajie.ho@starfivetech.com>
In-Reply-To: <20240305071006.2181158-1-jiajie.ho@starfivetech.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?0qffVL/8J822Ra1K1k0+YVG8qTDkWO0Nx2xGdQanPSsVTL3TgN46tpWGnAD0?=
 =?us-ascii?Q?p6Kt+n4Y5Ou4SwMwdvQ3Z5RNcDUzgN0OaykSiOVdIPLL8n8TNbvg5vV+pEBz?=
 =?us-ascii?Q?mC46CThCr3XmFH+/Zh4Q8vdgsueO6hFVxvKnRrrjvj7wJhskhkH5EBRUwESc?=
 =?us-ascii?Q?gxCCFnigREePZW6MtEkYOn0fSG6vYdOVYFz7azy66ANFZfLX6TEwpTK7jBkR?=
 =?us-ascii?Q?ombdvLNDwyyWbmHajbCpJIpQWw2nbSCByUv+5vJXk3j/wl1wf8ed5/J0BLDO?=
 =?us-ascii?Q?3TFGSMpUf+bKbtsSrrwKuUHpuQmHf+Nx5gTXL2BFdAoN+l4RxOkzbqsqXCpN?=
 =?us-ascii?Q?kTxL0AUlA2GcKxD7N4VPEGauxGd1a01WbyiMHSy3+nv6YvuaiLusyLr083WM?=
 =?us-ascii?Q?Xej7RuyBgIkCHNt+3unAy2am5ZqcJO1SyxEoMkgiE1K6p9cilDR9a7mvsXOg?=
 =?us-ascii?Q?LhoK8sjXPLQr/LruH5Puf3RBu6AihKH6Qlhs71pIH05dlNoPjgZlKHN9LmZH?=
 =?us-ascii?Q?1y2ywn+YULz9SZUF8xC43D2StkJjdxEhsouQUGE3L2NKqkZuxJRfRf7tGXHc?=
 =?us-ascii?Q?97MQZ74uEloU9a8ko5TuMwg8OVnES4XrLGitXK5olVqmLK4lCyIAY+wxmqNM?=
 =?us-ascii?Q?eGrAat5NWaw4VTK3UzoajFQvgtaihUABVzaj3laPHx8KaBrnGiwU50Ace8gO?=
 =?us-ascii?Q?I2Rp+CDepmctahLm/lTV0irjJ2Npf6OcCsZApjG+U/BTomCuS1roGjgzj9jp?=
 =?us-ascii?Q?rdkdB34oK69r0LobGzxTCb3ol0a79GHpfdquo+cz3Zbk1TZRLOtYCRQJdA6d?=
 =?us-ascii?Q?DBNvuu0TOvzJplUJQrSGhRLlcOQhIU96Mzpynn2oqniBU5VNotIwrT8xROxa?=
 =?us-ascii?Q?A0z50/OgHF1nj4yUjL3ybe1VHf496FcBJc5RbTMxLyZho4xi4eP5o3iH2a9L?=
 =?us-ascii?Q?+7ht62Lq3eDTWx3ORlLdkm9NvxPizyJSbRTY5vttf09Y5bUav5MDyOlGI4E4?=
 =?us-ascii?Q?pAcIZC6p71s7D2VzQIXJP5Uv0RjnTkw2+unLPB8shJbAx2xj9/aU6CQAlKCL?=
 =?us-ascii?Q?1tOLKXr1LNUA0/kbGz7pcWHG6n1Jtye8JWBg8SCZ5cYoDVu+psR6tnuBU2bo?=
 =?us-ascii?Q?DyPobTDcgBTQYfirzftuO2y8MOTQuxZfXUBPIHrbsI8Weix+nH6quQFaOO+h?=
 =?us-ascii?Q?8dDF02qlDQPQ3zG7K0LWA5BvAC2AmhGOeGPfWFergFmOCyHHHwLY1cXgFffG?=
 =?us-ascii?Q?SQFBDEpjZH2qwWUW5ntx9eVf9bWPMISdyA7+/87yCq0T3nTpfUwROqWD2AW4?=
 =?us-ascii?Q?/NMYGg/cEg7kaj3GopAc8G+yTzGpb6eJvqic2fv/NErtJYr9fEHiSjv3tyFv?=
 =?us-ascii?Q?XbBRgktGXeF7frmAh9q9e6UvbbvD6uaQ4EX+soNErO8FMrfyaRMWV1NEDec9?=
 =?us-ascii?Q?tn7dYwpPj60TWPU5axo94r2d1AzVISA/2KHmf+F3cIjVUbrV8eTpCxZO5qUg?=
 =?us-ascii?Q?qhTDF81FCwiX+J2SXxZlCLIAnPOVqGL1g27upDwbrl12HQ2nbqxuMR/arwGG?=
 =?us-ascii?Q?8MhOFkJ4fqjcSoEmaneJHCIr+cBxZ/oe9dsQvHr7?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-crypto@vger.kernel.org
List-Id: <linux-crypto.vger.kernel.org>
List-Subscribe: <mailto:linux-crypto+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-crypto+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: starfivetech.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: NT0PR01MB1182.CHNPR01.prod.partner.outlook.cn
X-MS-Exchange-CrossTenant-Network-Message-Id: 6a58d795-5fa5-4a36-b113-08dc4e00ebf1
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Mar 2024 01:55:04.4909
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 06fe3fa3-1221-43d3-861b-5a4ee687a85c
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Dq5uJF/FrrP6BPyhFIA0I7tE6GQgaxqQnhJErKpXE/mwm64px77vsxj4kqjM0dlkf0O9EM4YvwGHb08yPeyVjtzZw0nEaB3GBJ/oMO/C2eY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: NT0PR01MB1264

> This patch series add driver support for StarFive JH8100 SoC crypto engin=
e.
> Patch 1 adds compatible string and update irq descriptions for
> JH8100 device. Subsequent patches update current driver implementations t=
o
> support both 7110 and 8100 variants.
>=20
> v3->v4:
> - Updated interrupts descriptions for jh8100-crypto compatible. (Rob)
> - Added patch 3 to skip unneeded key freeing for RSA module.
>=20
> v2->v3:
> - Use of device data instead of #ifdef CONFIG_ for different device
>   variants.
> - Updated dt bindings compatible and interrupts descriptions.
> - Added patch 4 to support hardware quirks for dw-axi-dmac driver.
>=20
> v1->v2:
> - Resolved build warnings reported by kernel test robot
>   https://lore.kernel.org/oe-kbuild-all/202312170614.24rtwf9x-
> lkp@intel.com/
>=20
> Jia Jie Ho (7):
>   dt-bindings: crypto: starfive: Add jh8100 support
>   crypto: starfive: Update hash dma usage
>   crypto: starfive: Skip unneeded key free
>   crypto: starfive: Use dma for aes requests
>   dmaengine: dw-axi-dmac: Support hardware quirks
>   crypto: starfive: Add sm3 support for JH8100
>   crypto: starfive: Add sm4 support for JH8100
>=20
>  .../crypto/starfive,jh7110-crypto.yaml        |   30 +-
>  drivers/crypto/starfive/Kconfig               |   30 +-
>  drivers/crypto/starfive/Makefile              |    5 +-
>  drivers/crypto/starfive/jh7110-aes.c          |  592 ++++++---
>  drivers/crypto/starfive/jh7110-cryp.c         |   77 +-
>  drivers/crypto/starfive/jh7110-cryp.h         |  114 +-
>  drivers/crypto/starfive/jh7110-hash.c         |  316 +++--
>  drivers/crypto/starfive/jh7110-rsa.c          |    3 +
>  drivers/crypto/starfive/jh8100-sm3.c          |  544 ++++++++
>  drivers/crypto/starfive/jh8100-sm4.c          | 1119 +++++++++++++++++
>  .../dma/dw-axi-dmac/dw-axi-dmac-platform.c    |   32 +-
>  drivers/dma/dw-axi-dmac/dw-axi-dmac.h         |    2 +
>  include/linux/dma/dw_axi.h                    |   11 +
>  13 files changed, 2437 insertions(+), 438 deletions(-)  create mode 1006=
44
> drivers/crypto/starfive/jh8100-sm3.c
>  create mode 100644 drivers/crypto/starfive/jh8100-sm4.c
>  create mode 100644 include/linux/dma/dw_axi.h
>=20

Hi Herbert,
Could you please help review this patch series?

Thanks,
Jia Jie