On Wed 2014-02-05 13:44:58, Jenny Tc wrote: > On Tue, Feb 04, 2014 at 12:36:30PM +0100, Pavel Machek wrote: > > > +struct psy_charger_context { > > > + bool is_usb_cable_evt_reg; > > > + int psyc_cnt; > > > + int batt_status; > > > + /*cache battery and charger properties */ > > > > Comment coding style. Please run you patches through checkpatch. > > checkpatch doesn't throw any error/warning. /* ... */ not allowed for single line > comments? (will fix missing space after /*) I meant the space after /*. Maybe checkpatch does not report anything here, but I guess there are other places where it will comment :-). Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html