[Skiboot] [PATCH 3/6] gard: enable building with skiboot HOSTCFLAGS warnings

Cyril Bur cyrilbur at gmail.com
Tue Jul 18 14:25:24 AEST 2017


On Mon, 2017-07-17 at 19:13 +1000, Stewart Smith wrote:
> Mostly unused parameter warnings due to callbacks
> 
> Signed-off-by: Stewart Smith <stewart at linux.vnet.ibm.com>

Reviewed-by: Cyril Bur <cyril.bur at au1.ibm.com>

> ---
>  external/gard/gard.c | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
> 
> diff --git a/external/gard/gard.c b/external/gard/gard.c
> index 32a0003f4811..c5cb93b8830a 100644
> --- a/external/gard/gard.c
> +++ b/external/gard/gard.c
> @@ -260,6 +260,9 @@ static int do_iterate(struct gard_ctx *ctx,
>  
>  static int get_largest_pos_i(struct gard_ctx *ctx, int pos, struct gard_record *gard, void *priv)
>  {
> +	(void)ctx;
> +	(void)gard;
> +
>  	if (!priv)
>  		return -1;
>  
> @@ -272,6 +275,8 @@ static int get_largest_pos(struct gard_ctx *ctx)
>  {
>  	int rc, largest = -1;
>  
> +	(void)ctx;
> +
>  	rc = do_iterate(ctx, &get_largest_pos_i, &largest);
>  	if (rc)
>  		return -1;
> @@ -281,6 +286,9 @@ static int get_largest_pos(struct gard_ctx *ctx)
>  
>  static int count_valid_records_i(struct gard_ctx *ctx, int pos, struct gard_record *gard, void *priv)
>  {
> +	(void)ctx;
> +	(void)pos;
> +
>  	if (!gard || !priv)
>  		return -1;
>  
> @@ -303,6 +311,10 @@ static int count_valid_records(struct gard_ctx *ctx)
>  
>  static int do_list_i(struct gard_ctx *ctx, int pos, struct gard_record *gard, void *priv)
>  {
> +	(void)ctx;
> +	(void)pos;
> +	(void)priv;
> +
>  	if (!gard)
>  		return -1;
>  
> @@ -317,6 +329,9 @@ static int do_list(struct gard_ctx *ctx, int argc, char **argv)
>  {
>  	int rc;
>  
> +	(void)argc;
> +	(void)argv;
> +
>  	/* No entries */
>  	if (count_valid_records(ctx) == 0) {
>  		printf("No GARD entries to display\n");
> @@ -335,6 +350,9 @@ static int do_show_i(struct gard_ctx *ctx, int pos, struct gard_record *gard, vo
>  {
>  	uint32_t id;
>  
> +	(void)ctx;
> +	(void)pos;
> +
>  	if (!priv || !gard)
>  		return -1;
>  
> @@ -477,7 +495,7 @@ static int do_clear(struct gard_ctx *ctx, int argc, char **argv)
>  	return rc;
>  }
>  
> -int check_gard_partition(struct gard_ctx *ctx)
> +static int check_gard_partition(struct gard_ctx *ctx)
>  {
>  	int rc;
>  	struct gard_record gard;
> @@ -520,6 +538,8 @@ int check_gard_partition(struct gard_ctx *ctx)
>  __attribute__ ((unused))
>  static int do_nop(struct gard_ctx *ctx, int argc, char **argv)
>  {
> +	(void)ctx;
> +	(void)argc;
>  	fprintf(stderr, "Unimplemented action '%s'\n", argv[0]);
>  	return EXIT_SUCCESS;
>  }


More information about the Skiboot mailing list