From mboxrd@z Thu Jan 1 00:00:00 1970 From: J Freyensee Subject: Re: [PATCH] PTI implicit declaration fix. Date: Mon, 11 Jul 2011 09:10:53 -0700 Message-ID: <4E1B208D.1000708@linux.intel.com> References: <1310160851-26415-1-git-send-email-james_p_freyensee@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com ([192.55.52.93]:7916 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758009Ab1GKQKd (ORCPT ); Mon, 11 Jul 2011 12:10:33 -0400 In-Reply-To: Sender: linux-next-owner@vger.kernel.org List-ID: To: Geert Uytterhoeven Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, randy.dunlap@oracle.com, sfr@canb.auug.org.au, christophe.guerard@intel.com, pranav.k.sanghadia@intel.com On 07/09/2011 01:08 AM, Geert Uytterhoeven wrote: > On Fri, Jul 8, 2011 at 23:34, wrote: >> From: J Freyensee >> >> In a certain compile configuration, functions like >> 'kmalloc' and 'copy_from_user' were coming up as >> implicit errors. This patch adds slab.h and >> uaccess.h to the driver. >> >> Reported-by: Geert Uytterhoeven >> Acked-by: Geert Uytterhoeven > > Please don't add "Acked-by" lines before people have ack'ed your patches. > Okay...so should I re-send this patch out without the Acked-by in it? Or is it okay this time and next time please do this? >> Signed-off-by: J Freyensee >> --- >> drivers/misc/pti.c | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/misc/pti.c b/drivers/misc/pti.c >> index 8653bd0..4ba4e5f 100644 >> --- a/drivers/misc/pti.c >> +++ b/drivers/misc/pti.c >> @@ -22,6 +22,8 @@ >> */ >> >> #include >> +#include >> +#include >> #include >> #include >> #include > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds