On Tue, Feb 24, 2015 at 11:37:09AM +0100, Borislav Petkov wrote: > @@ -153,16 +152,15 @@ int get_matching_sig(unsigned int csig, int cpf, void > *mc, int rev) > } > > /* > - * return 0 - no update found > - * return 1 - found update > + * Returns 1 if update has been found, 0 otherwise. > */ > -int get_matching_microcode(unsigned int csig, int cpf, void *mc, int rev) > +int get_matching_microcode(unsigned int csig, int cpf, int rev, void *mc)
Why not rename the function to is_microcode_matching() while at it? :) Quentin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/