Elyes Haouas has submitted this change. ( https://review.coreboot.org/c/coreboot/+/85782?usp=email )
Change subject: arch/x86: Replace 'unsigned long int' by 'unsigned long' ......................................................................
arch/x86: Replace 'unsigned long int' by 'unsigned long'
As suggested by the linter:
Prefer 'unsigned long' over 'unsigned long int' as the int is unnecessary
Link: https://qa.coreboot.org/job/coreboot-untested-files/lastSuccessfulBuild/arti... Cc: "Jérémy Compostella" jeremy.compostella@intel.com Change-Id: Ida1de23830b0b67ab7fac635b02a4e99c65746f8 Signed-off-by: Ariel Otilibili otilibil@eurecom.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/85782 Reviewed-by: Maximilian Brune maximilian.brune@9elements.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Elyes Haouas ehaouas@noos.fr --- M src/arch/x86/memset.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Elyes Haouas: Looks good to me, approved Angel Pons: Looks good to me, but someone else must approve build bot (Jenkins): Verified Maximilian Brune: Looks good to me, approved
diff --git a/src/arch/x86/memset.c b/src/arch/x86/memset.c index 142dda3..4ab07ed 100644 --- a/src/arch/x86/memset.c +++ b/src/arch/x86/memset.c @@ -12,7 +12,7 @@ void *memset(void *dstpp, int c, size_t len) { int d0; - unsigned long int dstp = (unsigned long int)dstpp; + unsigned long dstp = (unsigned long)dstpp;
#if (ENV_SEPARATE_ROMSTAGE && CONFIG(ASAN_IN_ROMSTAGE)) || \ (ENV_RAMSTAGE && CONFIG(ASAN_IN_RAMSTAGE))