Re: [PATCH 13/16] resolve warnings found on ARM compile

From: Grant Likely
Date: Fri Aug 30 2013 - 09:41:05 EST


On Fri, 9 Aug 2013 16:26:14 -0700, Roy Franz <roy.franz@xxxxxxxxxx> wrote:
> 2 unused labels
> 1 "value computed is not used"
>
>
> Signed-off-by: Roy Franz <roy.franz@xxxxxxxxxx>

When posting a patch to fix compile errors or warnings, it is helpful
inlude the output from the compiler with the error/warning message.

g.

> ---
> drivers/firmware/efi/efi-stub-helper.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/firmware/efi/efi-stub-helper.c b/drivers/firmware/efi/efi-stub-helper.c
> index 4bb542f..3e82cb0 100644
> --- a/drivers/firmware/efi/efi-stub-helper.c
> +++ b/drivers/firmware/efi/efi-stub-helper.c
> @@ -166,7 +166,6 @@ again:
> *addr = max_addr;
> }
>
> -free_pool:
> efi_call_phys1(sys_table_arg->boottime->free_pool, map);
>
> fail:
> @@ -242,7 +241,6 @@ static efi_status_t efi_low_alloc(efi_system_table_t *sys_table_arg,
> if (i == map_size / desc_size)
> status = EFI_NOT_FOUND;
>
> -free_pool:
> efi_call_phys1(sys_table_arg->boottime->free_pool, map);
> fail:
> return status;
> @@ -357,7 +355,7 @@ static efi_status_t handle_cmdline_files(efi_system_table_t *sys_table_arg,
>
> if (*str == '/') {
> *p++ = '\\';
> - *str++;
> + str++;
> } else {
> *p++ = *str++;
> }
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/