ACK: [PATCH] cmos: just print drives + 1 with %d, fixes clang warning

Alex Hung alex.hung at canonical.com
Fri Nov 28 09:34:13 UTC 2014


On 14-11-27 08:02 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/cmos/cmosdump/cmosdump.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/cmos/cmosdump/cmosdump.c b/src/cmos/cmosdump/cmosdump.c
> index aff0373..1ab86d3 100644
> --- a/src/cmos/cmosdump/cmosdump.c
> +++ b/src/cmos/cmosdump/cmosdump.c
> @@ -265,7 +265,7 @@ static int cmosdump_test1(fwts_framework *fw)
>   		primary_display[(data[20] >> 4) & 3]);
>   	if (data[20] & 1) {
>   		int drives = (data[20] >> 6) & 3;
> -		fwts_log_info_verbatum(fw, "  Floppy Drives:          0x%2.2x (%hhu drive%s)",
> +		fwts_log_info_verbatum(fw, "  Floppy Drives:          0x%2.2x (%d drive%s)",
>   			drives, drives + 1, drives > 0 ? "s" : "");
>   	} else {
>   		fwts_log_info_verbatum(fw, "  Floppy Drives:          None.");

Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list