On Wed, Mar 04, 2015 at 07:02:07PM +0530, Darshana Padmadas wrote:
> cpu/intel_cacheinfo.c defines two functions amd_get_l3_disable_slot
> and amd_set_l3_disable_slot. No other file uses these functions.
> So make these static.
> 
> This eliminates the following warnings:
> 
> arch/x86/kernel/cpu/intel_cacheinfo.c:349:5: warning: no previous prototype 
> for ‘amd_get_l3_disable_slot’ [-Wmissing-prototypes]
> arch/x86/kernel/cpu/intel_cacheinfo.c:427:5: warning: no previous prototype 
> for ‘amd_set_l3_disable_slot’ [-Wmissing-prototypes]

Please correct your commit message - this patch is making functions
static and not fixing -Wmissing-prototypes warnings. Also, go through
the rest of the patchset as I see other cases of this too.

Thanks.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
--
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/

Reply via email to