[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180827175213.GA27361@kroah.com>
Date: Mon, 27 Aug 2018 19:52:13 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Arkadiusz Lis <areklis909@...il.com>
Cc: jfrederich@...il.com, dsd@...top.org, jon.nettleton@...il.com,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Staging: One Laptop Per Child: add function argument
names
On Thu, Aug 09, 2018 at 06:23:59PM +0200, Arkadiusz Lis wrote:
> Add function argument names to pointers to functions
> in struct dcon_platform_data: init, set_dconload and read_status
> This commit removes checkpatch.pl warning.
>
> Signed-off-by: Arkadiusz Lis <areklis909@...il.com>
> ---
> drivers/staging/olpc_dcon/olpc_dcon.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/staging/olpc_dcon/olpc_dcon.h b/drivers/staging/olpc_dcon/olpc_dcon.h
> index fa89bb9..0ce1a7a 100644
> --- a/drivers/staging/olpc_dcon/olpc_dcon.h
> +++ b/drivers/staging/olpc_dcon/olpc_dcon.h
> @@ -91,10 +91,10 @@ struct dcon_priv {
> };
>
> struct dcon_platform_data {
> - int (*init)(struct dcon_priv *);
> + int (*init)(struct dcon_priv *dcon);
> void (*bus_stabilize_wiggle)(void);
> - void (*set_dconload)(int);
> - int (*read_status)(u8 *);
> + void (*set_dconload)(int val);
> + int (*read_status)(u8 *status);
> };
>
> #include <linux/interrupt.h>
> --
> 1.9.1
>
> _______________________________________________
> devel mailing list
> devel@...uxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Patch did not apply to my tree at all. Please rebase against the latest
linux-next and resend.
thanks,
greg k-h
Powered by blists - more mailing lists