linux-kernel - Re: [PATCH] firmware: xilinx: Move FIRMWARE_VERSION

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5a7d3d78-f152-466f-90de-e4a781577bb1@amd.com>
Date: Mon, 3 Jun 2024 13:04:36 +0200
From: Michal Simek <michal.simek@....com>
To: Ronak Jain <ronak.jain@....com>, linux-arm-kernel@...ts.infradead.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] firmware: xilinx: Move FIRMWARE_VERSION_MASK to
 xlnx-zynqmp.h



On 4/25/24 11:59, Ronak Jain wrote:
> Move FIRMWARE_VERSION_MASK macro to xlnx-zynqmp.h so that other
> drivers can use it for verifying the supported firmware version.
> 
> Signed-off-by: Ronak Jain <ronak.jain@....com>
> Signed-off-by: Anand Ashok Dumbre <anand.ashok.dumbre@....com>
> Signed-off-by: Radhey Shyam Pandey <radhey.shyam.pandey@....com>
> ---
>   drivers/firmware/xilinx/zynqmp.c        | 3 ---
>   drivers/soc/xilinx/xlnx_event_manager.c | 1 -
>   include/linux/firmware/xlnx-zynqmp.h    | 3 +++
>   3 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/firmware/xilinx/zynqmp.c b/drivers/firmware/xilinx/zynqmp.c
> index 9bc45357e1a8..add8acf66a9c 100644
> --- a/drivers/firmware/xilinx/zynqmp.c
> +++ b/drivers/firmware/xilinx/zynqmp.c
> @@ -41,9 +41,6 @@
>   /* IOCTL/QUERY feature payload size */
>   #define FEATURE_PAYLOAD_SIZE		2
>   
> -/* Firmware feature check version mask */
> -#define FIRMWARE_VERSION_MASK		GENMASK(15, 0)
> -
>   static bool feature_check_enabled;
>   static DEFINE_HASHTABLE(pm_api_features_map, PM_API_FEATURE_CHECK_MAX_ORDER);
>   static u32 ioctl_features[FEATURE_PAYLOAD_SIZE];
> diff --git a/drivers/soc/xilinx/xlnx_event_manager.c b/drivers/soc/xilinx/xlnx_event_manager.c
> index 253299e4214d..9d89b5a40f97 100644
> --- a/drivers/soc/xilinx/xlnx_event_manager.c
> +++ b/drivers/soc/xilinx/xlnx_event_manager.c
> @@ -35,7 +35,6 @@ static int event_manager_availability = -EACCES;
>   
>   #define MAX_BITS	(32U) /* Number of bits available for error mask */
>   
> -#define FIRMWARE_VERSION_MASK			(0xFFFFU)
>   #define REGISTER_NOTIFIER_FIRMWARE_VERSION	(2U)
>   
>   static DEFINE_HASHTABLE(reg_driver_map, REGISTERED_DRIVER_MAX_ORDER);
> diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
> index 1a069a56c961..d7d07afc0532 100644
> --- a/include/linux/firmware/xlnx-zynqmp.h
> +++ b/include/linux/firmware/xlnx-zynqmp.h
> @@ -52,6 +52,9 @@
>   #define API_ID_MASK		GENMASK(7, 0)
>   #define MODULE_ID_MASK		GENMASK(11, 8)
>   
> +/* Firmware feature check version mask */
> +#define FIRMWARE_VERSION_MASK		0xFFFFU
> +
>   /* ATF only commands */
>   #define TF_A_PM_REGISTER_SGI		0xa04
>   #define PM_GET_TRUSTZONE_VERSION	0xa03

Applied.
M