ACK: [PATCH 09/12] hpet/hpet_check: re-arrange code to avoid declarations after code

Alex Hung alex.hung at canonical.com
Tue Jun 10 18:24:08 UTC 2014


On 06/08/2014 09:42 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/hpet/hpet_check/hpet_check.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/hpet/hpet_check/hpet_check.c b/src/hpet/hpet_check/hpet_check.c
> index 6d7403c..8f37c2f 100644
> --- a/src/hpet/hpet_check/hpet_check.c
> +++ b/src/hpet/hpet_check/hpet_check.c
> @@ -142,11 +142,11 @@ static int hpet_check_deinit(fwts_framework *fw)
>  
>  static int hpet_check_test1(fwts_framework *fw)
>  {
> +	fwts_list_link *item;
> +
>  	if (klog == NULL)
>  		return FWTS_ERROR;
>  
> -	fwts_list_link *item;
> -
>  	fwts_log_info(fw,
>  		"This test checks the HPET PCI BAR for each timer block "
>  		"in the timer. The base address is passed by the firmware "
> 

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



More information about the fwts-devel mailing list