Re: [PATCH 3/4] Bluetooth: trailing statements

From: Marcel Holtmann
Date: Tue Mar 20 2018 - 04:09:45 EST


Hi Paul,

> trailing statements should be on next line
>
> Signed-off-by: Paul McQuade <paulmcquad@xxxxxxxxx>
> ---
> drivers/bluetooth/hci_ll.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/bluetooth/hci_ll.c b/drivers/bluetooth/hci_ll.c
> index 1b4417a623a4..06d859d82de8 100644
> --- a/drivers/bluetooth/hci_ll.c
> +++ b/drivers/bluetooth/hci_ll.c
> @@ -567,7 +567,8 @@ static int read_local_version(struct hci_dev *hdev)
> version = le16_to_cpu(ver->lmp_subver);
>
> out:
> - if (err) bt_dev_err(hdev, "Failed to read TI version info: %d", err);
> + if (err)
> + bt_dev_err(hdev, "Failed to read TI version info: %d", err);

actually this needs to be indented one further since it is the action of the if clause.

Regards

Marcel