Messages in this thread | | | Subject | Re: [PATCH 3/4] Bluetooth: trailing statements | From | Marcel Holtmann <> | Date | Tue, 20 Mar 2018 09:09:31 +0100 |
| |
Hi Paul,
> trailing statements should be on next line > > Signed-off-by: Paul McQuade <paulmcquad@gmail.com> > --- > 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
| |