[PATCH] greybus: Fix coding stye error
Greg KH
gregkh at linuxfoundation.org
Fri Jan 27 10:17:14 UTC 2017
On Fri, Jan 27, 2017 at 11:11:21AM +0100, mrossibellom at gmail.com wrote:
> From: Maxime Rossi Bellom <mrossibellom at gmail.com>
>
> Split line before funtion argument.
>
> Error was reported by checkpatch.pl as
> WARNING: line over 80 characters
>
> Signed-off-by: Maxime Rossi Bellom <mrossibellom at gmail.com>
> ---
> drivers/staging/greybus/fw-management.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/greybus/fw-management.c b/drivers/staging/greybus/fw-management.c
> index 3cd6cf0..d88117e 100644
> --- a/drivers/staging/greybus/fw-management.c
> +++ b/drivers/staging/greybus/fw-management.c
> @@ -516,7 +516,8 @@ static int fw_mgmt_ioctl(struct fw_mgmt *fw_mgmt, unsigned int cmd,
> */
> fw_mgmt->mode_switch_started = true;
>
> - ret = gb_interface_request_mode_switch(fw_mgmt->connection->intf);
> + ret = gb_interface_request_mode_switch(
> + fw_mgmt->connection->intf);
That's horrid looking, please don't make code look worse, this is fine
as-is.
thanks,
greg k-h
More information about the devel
mailing list