ACK: [PATCH 28/30] acpi: spcr: add missing white space in error messages

Alex Hung alex.hung at canonical.com
Tue Jun 23 06:02:32 UTC 2015


On 06/18/2015 04:49 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> A few error messages are badly formatted and need an extra
> white space. Detected during testing.
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/spcr/spcr.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/acpi/spcr/spcr.c b/src/acpi/spcr/spcr.c
> index 63c3189..f4e22e3 100644
> --- a/src/acpi/spcr/spcr.c
> +++ b/src/acpi/spcr/spcr.c
> @@ -85,7 +85,7 @@ static int spcr_test1(fwts_framework *fw)
>  		fwts_failed(fw, LOG_LEVEL_HIGH,
>  			"SPCRInterfaceReserved",
>  			"SPCR Serial interface type 0x%2.2" PRIx8
> -			"is a reserved interface", spcr->interface_type);
> +			" is a reserved interface", spcr->interface_type);
>  	}
>  
>  	if ((spcr->reserved1[0] |
> @@ -155,7 +155,7 @@ static int spcr_test1(fwts_framework *fw)
>  		fwts_failed(fw, LOG_LEVEL_HIGH,
>  			"SPCRBaudRateReserved",
>  			"SPCR Serial baud rate type 0x%2.2" PRIx8
> -			"is a reserved baud rate", spcr->baud_rate);
> +			" is a reserved baud rate", spcr->baud_rate);
>  	}
>  
>  	if (spcr->parity & ~1) {
> @@ -207,7 +207,7 @@ static int spcr_test1(fwts_framework *fw)
>  		fwts_failed(fw, LOG_LEVEL_HIGH,
>  			"SPCRTerminalTypeReserved",
>  			"SPCR terminal type type 0x%2.2" PRIx8
> -			"is a reserved terminal type", spcr->terminal_type);
> +			" is a reserved terminal type", spcr->terminal_type);
>  	}
>  
>  	if (spcr->reserved2) {
> 

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



More information about the fwts-devel mailing list