On Tue, Nov 02, 2010 at 09:06:56PM +0000, Haiyang Zhang wrote: > > From: Brandon Philips [mailto:brandon@xxxxxxxx] > > Sent: Tuesday, November 02, 2010 1:04 PM > > > -static int HvQueryHypervisorInfo(void) > > > +static int hvquery_hypervisor_info(void) > > > -static u64 HvDoHypercall(u64 control, void *input, void *output) > > > +static u64 hvdo_hypercall(u64 control, void *input, void *output) > > > > Should these be hv_do_hypercall and hv_query_hypervisor? That seems to > > be the style in the rest of the patch: > > I didn't insert underscore between every word, for example, it's not > added between two short words. But you should. Remember, be consistent. Care to redo these again? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
