From f7c66766b75f99c802d740f34b52b3377a3ed11a Mon Sep 17 00:00:00 2001 From: pripatel Date: Mon, 10 Jul 2023 19:45:54 +0000 Subject: [PATCH] firmware-api:changed macro definition Change is done to resolve AUTOSAR C++14 M5-0-10 violations Rule: AUTOSAR C++14 M5-0-10 Rule description : If the bitwise operators ~ and << are applied to an operand with an underlying type of unsigned char or unsigned short, the result shall be immediately cast to the underlying type of the operand. JIRA CAMERASW-12899 Change-Id: Ide1491e00094299d3785dbb1cd3366a043b3de0e Signed-off-by: Mika Liljeberg Reviewed-on: https://git-master.nvidia.com/r/c/tegra/camera/firmware-api/+/2933961 Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/2988965 Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/2988977 GVS: Gerrit_Virtual_Submit Reviewed-by: Matti Ryttylainen Reviewed-by: Viktor Horsmanheimo --- include/soc/tegra/camrtc-capture.h | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/include/soc/tegra/camrtc-capture.h b/include/soc/tegra/camrtc-capture.h index b798df58..42b95ebf 100644 --- a/include/soc/tegra/camrtc-capture.h +++ b/include/soc/tegra/camrtc-capture.h @@ -325,20 +325,20 @@ typedef struct syncpoint_info { /**@}*/ #define VI_NUM_GOS_TABLES MK_U32(12) -#define VI_NUM_ATOMP_SURFACES 4 -#define VI_NUM_STATUS_SURFACES 1 -#define VI_NUM_VI_PFSD_SURFACES 2 +#define VI_NUM_ATOMP_SURFACES 4U +#define VI_NUM_STATUS_SURFACES 1U +#define VI_NUM_VI_PFSD_SURFACES 2U /** * @defgroup ViAtompSurface VI ATOMP surface related defines * @{ */ /** Output surface plane 0 */ -#define VI_ATOMP_SURFACE0 0 +#define VI_ATOMP_SURFACE0 0U /** Output surface plane 1 */ -#define VI_ATOMP_SURFACE1 1 +#define VI_ATOMP_SURFACE1 1U /** Output surface plane 2 */ -#define VI_ATOMP_SURFACE2 2 +#define VI_ATOMP_SURFACE2 2U /** Sensor embedded data */ #define VI_ATOMP_SURFACE_EMBEDDED 3