Re: [PATCH] scsi: esas2r: Fix inconsistent of format with argument type

["Martin K. Petersen" <martin.petersen@xxxxxxxxxx>]

Hello Ye,

> @@ -310,7 +310,7 @@ static void esas2r_complete_vda_ioctl(struct esas2r_adapter *a,
>  				le32_to_cpu(rsp->vda_version);
>  			cfg->data.init.fw_build = rsp->fw_build;
>  
> -			snprintf(buf, sizeof(buf), "%1.1u.%2.2u",
> +			snprintf(buf, sizeof(buf), "%1.1d.%2.2d",
>  				 (int)LOBYTE(le16_to_cpu(rsp->fw_release)),
>  				 (int)HIBYTE(le16_to_cpu(rsp->fw_release)));

I doubt the firmware release is a negative number.

-- 
Martin K. Petersen	Oracle Linux Engineering