On Wed, Dec 13, 2017 at 10:18 AM, Dhaval Shah <dhaval.shah@xxxxxxxxxxxxxxx> wrote: > Resolved Block comments use * on subsequent lines checkpatch warnings. > Issue found by checkpatch. // indeed will look better in these cases. Please, get blessing from author(s) / maintainer(s) of the module first before sending any patches like this. > > Signed-off-by: Dhaval Shah <dhaval.shah@xxxxxxxxxxxxxxx> > --- > drivers/platform/x86/dell-wmi-aio.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/platform/x86/dell-wmi-aio.c b/drivers/platform/x86/dell-wmi-aio.c > index 50c2078715d6..3842b32a729c 100644 > --- a/drivers/platform/x86/dell-wmi-aio.c > +++ b/drivers/platform/x86/dell-wmi-aio.c > @@ -36,7 +36,8 @@ MODULE_LICENSE("GPL"); > struct dell_wmi_event { > u16 length; > /* 0x000: A hot key pressed or an event occurred > - * 0x00F: A sequence of hot keys are pressed */ > + * 0x00F: A sequence of hot keys are pressed > + */ > u16 type; > u16 event[]; > }; > @@ -115,7 +116,8 @@ static void dell_wmi_aio_notify(u32 value, void *context) > scancode = event->event[0]; > } else { > /* Broken machines return the scancode in a > - buffer */ > + * buffer > + */ > if (obj->buffer.pointer && > obj->buffer.length > 0) > scancode = obj->buffer.pointer[0]; > -- > 2.11.0 > -- With Best Regards, Andy Shevchenko