The subject line should give information about what part of the kernel is being changed. See other patches or the tutorial for examples. On Wed, 11 Mar 2020, Enrique Vargas wrote: > put brace at the end of line following coding conventions > > Signed-off-by: Enrique Vargas <jevargas@xxxxxxxxxxxxxx> > --- > drivers/atm/adummy.c | 3 +-- Is there a reason why you are working on drivers/atm? Normally for the application period you should only do cleanups on drivers/staging. julia > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/atm/adummy.c b/drivers/atm/adummy.c > index 8157925af824..6eeaa1b21f3b 100644 > --- a/drivers/atm/adummy.c > +++ b/drivers/atm/adummy.c > @@ -131,8 +131,7 @@ adummy_proc_read(struct atm_dev *dev, loff_t *pos, char *page) > return 0; > } > > -static const struct atmdev_ops adummy_ops = > -{ > +static const struct atmdev_ops adummy_ops = { > .open = adummy_open, > .close = adummy_close, > .send = adummy_send, > -- > 2.17.1 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@xxxxxxxxxxxxxxxx. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20200311125507.GA9347%40evX1. >