Welcome to Soft32 Linux Forums!
FAQFAQ    SearchSearch      ProfileProfile    Private MessagesPrivate Messages   Log inLog in

[PATCH] drivers/char/nvram.c: Undo removal of duplicated i..

 
   Soft32 Home -> Linux -> Kernel RSS
Next:  [PATCH] sched: Fix rq lock use through wrong spin..  
Author Message
Etienne Le Sueur

External


Since: Dec 10, 2009
Posts: 1



(Msg. 1) Posted: Wed Dec 09, 2009 9:20 pm
Post subject: [PATCH] drivers/char/nvram.c: Undo removal of duplicated include
Archived from groups: linux>kernel (more info?)

elesueur@bitburger:/usr/src/linux-2.6-git$ make
CHK include/linux/version.h
CHK include/linux/utsrelease.h
SYMLINK include/asm -> include/asm-x86
CALL scripts/checksyscalls.sh
CHK include/linux/compile.h
CC drivers/char/consolemap_deftbl.o
CC drivers/char/defkeymap.o
LD drivers/char/built-in.o
CC [M] drivers/char/nvram.o
drivers/char/nvram.c: In function 'nvram_open':
drivers/char/nvram.c:332: error: implicit declaration of function
'lock_kernel'
drivers/char/nvram.c:339: error: implicit declaration of function
'unlock_kernel'
make[2]: *** [drivers/char/nvram.o] Error 1
make[1]: *** [drivers/char] Error 2
make: *** [drivers] Error 2

a check shows commit id: 4d8f54d9e4323baae8c8ca753db91563f4b34ddb
removed a 'duplicated include' which I believe is erroneous.




diff --git a/drivers/char/nvram.c b/drivers/char/nvram.c
index 2100a8f..3cfa0f6 100644
--- a/drivers/char/nvram.c
+++ b/drivers/char/nvram.c
@@ -38,6 +38,7 @@
#define NVRAM_VERSION "1.3"

#include <linux/module.h>
+#include <linux/smp_lock.h>
#include <linux/nvram.h>

#define PC 1


Regards,
Etienne Le Sueur
www.ertos.nicta.com.au
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo DeleteThis @vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to top
Login to vote
Jean Delvare

External


Since: Nov 13, 2006
Posts: 295



(Msg. 2) Posted: Thu Dec 10, 2009 3:24 am
Post subject: Re: [PATCH] drivers/char/nvram.c: Undo removal of duplicated include [Login to view extended thread Info.]
Archived from groups: per prev. post (more info?)

Hi Etienne,

On Thu, 10 Dec 2009 13:04:53 +1100, Etienne Le Sueur wrote:
> elesueur@bitburger:/usr/src/linux-2.6-git$ make
> CHK include/linux/version.h
> CHK include/linux/utsrelease.h
> SYMLINK include/asm -> include/asm-x86
> CALL scripts/checksyscalls.sh
> CHK include/linux/compile.h
> CC drivers/char/consolemap_deftbl.o
> CC drivers/char/defkeymap.o
> LD drivers/char/built-in.o
> CC [M] drivers/char/nvram.o
> drivers/char/nvram.c: In function 'nvram_open':
> drivers/char/nvram.c:332: error: implicit declaration of function
> 'lock_kernel'
> drivers/char/nvram.c:339: error: implicit declaration of function
> 'unlock_kernel'
> make[2]: *** [drivers/char/nvram.o] Error 1
> make[1]: *** [drivers/char] Error 2
> make: *** [drivers] Error 2
>
> a check shows commit id: 4d8f54d9e4323baae8c8ca753db91563f4b34ddb
> removed a 'duplicated include' which I believe is erroneous.
>
>
>
>
> diff --git a/drivers/char/nvram.c b/drivers/char/nvram.c
> index 2100a8f..3cfa0f6 100644
> --- a/drivers/char/nvram.c
> +++ b/drivers/char/nvram.c
> @@ -38,6 +38,7 @@
> #define NVRAM_VERSION "1.3"
>
> #include <linux/module.h>
> +#include <linux/smp_lock.h>
> #include <linux/nvram.h>
>
> #define PC 1

Your patch got corrupted so it won't apply.

Anyway it seems this build error is already fixed upstream:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=com...diff;h=

--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo.RemoveThis@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to top
Login to vote
Display posts from previous:   
Related Topics:
[RFC][PATCH] Removal of duplicated include net/wanrouter/w.. - Hi, There is no need to include linux/init.h twice Regards, Michal -- LOG http://www.stardust.webpages.pl/log/ ..

[RFC][PATCH] Removal of duplicated include arch/sh64/kerne.. - Hi, There is no need to include linux/console.h twice Regards, Michal -- LOG http://www.stardust.webpages.pl/log/ ..

[RFC][PATCH] Removal of duplicated include arch/x86_64/ker.. - Hi, There is no need to include asm/hpet.h twice Regards, Michal -- LOG http://www.stardust.webpages.pl/log/ ..

[RFC][PATCH] Removal of duplicated include arch/x86_64/vds.. - Hi, There is no need to include asm/vgtod.h twice Regards, Michal -- LOG http://www.stardust.webpages.pl/log/ ..

[RFC][PATCH] Removal of duplicated include arch/x86_64/ker.. - Hi, There is no need to include linux/init.h twice Regards, Michal -- LOG http://www.stardust.webpages.pl/log/ ..

[PATCH] extract duplicated constants out of PIIX like driv.. - Each PIIX driver has two copies of a tiny 10 byte table. Jeff asked that it was moved to a common place in each driver....
       Soft32 Home -> Linux -> Kernel All times are: Pacific Time (US & Canada) (change)
Page 1 of 1

 
You can post new topics in this forum
You can reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum

Categories:
 Windows
  Linux
 Mac
 PDA


[ Contact us | Terms of Service/Privacy Policy ]