On Sun, Sep 08, 2019 at 05:40:40PM +0100, Greg Kroah-Hartman wrote: > On Sun, Sep 08, 2019 at 04:10:15PM +0000, Valentin Vidic wrote: > > void fat_set_entry_time(struct dentry_t *p_entry, struct timestamp_t *tp, > > u8 mode) > > { > > + u8 ms; > > u16 t, d; > > struct dos_dentry_t *ep = (struct dos_dentry_t *)p_entry; > > > > t = (tp->hour << 11) | (tp->min << 5) | (tp->sec >> 1); > > d = (tp->year << 9) | (tp->mon << 5) | tp->day; > > > > + ms = tp->millisec; > > + if (tp->sec & 1) { > > + ms += 1000; > > + } > > checkpatch didn't complain about this { } not being needed? > > Same in other parts of this patch, please fix up. No warnings from checkpatch here, will update the code. -- Valentin _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel