public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
* patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot
@ 2004-04-15  7:57 Tobias Arp
  2004-04-15  8:58 ` Martin Donnelly
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Arp @ 2004-04-15  7:57 UTC (permalink / raw)
  To: linux-mtd

 Hi 
 
i tried to patch my arm linux kernel 2.4.21 (rmk2 patch) with the latest CVS snapshot. I had 
follwing errors: 
 
1. While making xconfig after patching the kernel (Options: Copy files, update fs) i had a 
problem making xconfig. This kernel release seems not to support elif in Config-files. I 
comment these lines out because they don't affect my configuration. After this it works. 
 
2. I get some warnings and errors messages while make zImage: 
 
cfi_cmdset_0002.c: In function `do_write_oneword': 
cfi_cmdset_0002.c:848: warning: label `op_failed' defined but not used 
cfi_cmdset_0002.c: In function `do_erase_chip': 
cfi_cmdset_0002.c:1385: warning: label `op_failed' defined but not used 
cfi_cmdset_0002.c: In function `do_erase_oneblock': 
cfi_cmdset_0002.c:1575: warning: label `op_failed' defined but not used 
physmap.c: In function `init_physmap': 
physmap.c:68: warning: passing arg 2 of `parse_mtd_partitions' from incompatible pointer type 
debug.c:147: warning: `page_info_op' defined but not used 
nodelist.c: In function `jffs2_get_inode_nodes': 
nodelist.c:283: warning: unknown conversion type character `z' in format 
nodelist.c:283: warning: too many arguments for format 
read.c: In function `jffs2_read_dnode': 
read.c:44: warning: unknown conversion type character `z' in format 
read.c:44: warning: unknown conversion type character `z' in format 
read.c:44: warning: too many arguments for format 
nodemgmt.c: In function `jffs2_mark_node_obsolete': 
nodemgmt.c:530: warning: unknown conversion type character `z' in format 
nodemgmt.c:530: warning: too many arguments for format 
nodemgmt.c:549: warning: unknown conversion type character `z' in format 
nodemgmt.c:549: warning: too many arguments for format 
readinode.c: In function `jffs2_do_read_inode_internal': 
readinode.c:587: warning: unknown conversion type character `z' in format 
readinode.c:587: warning: unknown conversion type character `z' in format 
readinode.c:587: warning: too many arguments for format 
write.c: In function `writecheck': 
write.c:66: warning: unknown conversion type character `z' in format 
write.c:66: warning: too many arguments for format 
write.c: In function `jffs2_write_dnode': 
write.c:112: warning: unknown conversion type character `z' in format 
write.c:112: warning: too many arguments for format 
write.c:143: warning: unknown conversion type character `z' in format 
write.c:143: warning: unknown conversion type character `z' in format 
write.c:143: warning: too many arguments for format 
write.c: In function `jffs2_write_dirent': 
write.c:284: warning: unknown conversion type character `z' in format 
write.c:284: warning: unknown conversion type character `z' in format 
write.c:284: warning: too many arguments for format 
scan.c: In function `jffs2_scan_medium': 
scan.c:91: warning: unknown conversion type character `z' in format 
scan.c:91: warning: too many arguments for format 
scan.c: In function `jffs2_fill_scan_buf': 
scan.c:271: warning: unknown conversion type character `z' in format 
scan.c:271: warning: too many arguments for format 
scan.c: In function `jffs2_scan_eraseblock': 
scan.c:383: warning: unknown conversion type character `z' in format 
scan.c:383: warning: unknown conversion type character `z' in format 
scan.c:383: warning: too many arguments for format 
scan.c:391: warning: unknown conversion type character `z' in format 
scan.c:391: warning: too many arguments for format 
scan.c:509: warning: unknown conversion type character `z' in format 
scan.c:509: warning: too many arguments for format 
gc.c: In function `jffs2_garbage_collect_pristine': 
gc.c:588: warning: unknown conversion type character `z' in format 
gc.c:588: warning: too many arguments for format 
gc.c: In function `jffs2_garbage_collect_metadata': 
gc.c:690: warning: unknown conversion type character `z' in format 
gc.c:690: warning: too many arguments for format 
gc.c: In function `jffs2_garbage_collect_dirent': 
gc.c:757: warning: unknown conversion type character `z' in format 
gc.c:757: warning: too many arguments for format 
gc.c: In function `jffs2_garbage_collect_deletion_dirent': 
gc.c:827: warning: unknown conversion type character `z' in format 
gc.c:827: warning: unknown conversion type character `z' in format 
gc.c:827: warning: too many arguments for format 
gc.c: In function `jffs2_garbage_collect_hole': 
gc.c:901: warning: unknown conversion type character `z' in format 
gc.c:901: warning: too many arguments for format 
gc.c:913: warning: unknown conversion type character `z' in format 
gc.c:913: warning: too many arguments for format 
gc.c:959: warning: unknown conversion type character `z' in format 
gc.c:959: warning: too many arguments for format 
gc.c: In function `jffs2_garbage_collect_dnode': 
gc.c:1185: warning: unknown conversion type character `z' in format 
gc.c:1185: warning: too many arguments for format 
erase.c: In function `jffs2_mark_erased_block': 
erase.c:322: warning: unknown conversion type character `z' in format 
erase.c:322: warning: too many arguments for format 
erase.c:385: warning: unknown conversion type character `z' in format 
erase.c:385: warning: too many arguments for format 
fs.c: In function `jffs2_gc_fetch_inode': 
fs.c:550: warning: implicit declaration of function `ilookup' 
fs.c:550: warning: assignment makes pointer from integer without a cast 
fs/fs.o: In function `jffs2_gc_fetch_inode': 
fs/fs.o(.text+0x61f10): undefined reference to `ilookup' 
make: *** [vmlinux] Fehler 1 
 
Has someone made the same experience ? 
 
I have another question: Before i made the patch i have configured my flash partitions in 
physmap.c where do i configure it now (after applying the patch) ? 
 
Regards 
 
Tobias Arp 
 
_______________________________________________________________________
... and the winner is... WEB.DE FreeMail! - Deutschlands beste E-Mail
ist zum 39. Mal Testsieger (PC Praxis 03/04) http://f.web.de/?mc=021191

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot
  2004-04-15  7:57 patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot Tobias Arp
@ 2004-04-15  8:58 ` Martin Donnelly
  2004-04-15  9:00   ` Martin Donnelly
  0 siblings, 1 reply; 4+ messages in thread
From: Martin Donnelly @ 2004-04-15  8:58 UTC (permalink / raw)
  To: Tobias Arp; +Cc: linux-mtd

On Thu, 2004-04-15 at 08:57, Tobias Arp wrote: 
> i tried to patch my arm linux kernel 2.4.21 (rmk2 patch) with the latest CVS snapshot. I had 
> follwing errors: 
>  
> 1. While making xconfig after patching the kernel (Options: Copy files, update fs) i had a 
> problem making xconfig. This kernel release seems not to support elif in Config-files. I 
> comment these lines out because they don't affect my configuration. After this it works. 
>  

Apply the two attached patches chips-config.patch to
'drivers/mtd/chips/Config.in' and nand-config.patch to
'drivers/mtd/nand/Config.in' which should sort out a couple of syntax
errors. e.g `cd linux-2.4.21; patch -p1 -i chips-config.patch`

Maybe someone can apply these to CVS?

> 2. I get some warnings and errors messages while make zImage: 
>  

<snip warnings>

> fs.c: In function `jffs2_gc_fetch_inode': 
> fs.c:550: warning: implicit declaration of function `ilookup' 
> fs.c:550: warning: assignment makes pointer from integer without a cast 
> fs/fs.o: In function `jffs2_gc_fetch_inode': 
> fs/fs.o(.text+0x61f10): undefined reference to `ilookup' 
> make: *** [vmlinux] Fehler 1 
>  
> Has someone made the same experience ? 

You need a patch to the kernel for this you can download it from
http://linux.bkbits.net:8080/linux-2.4/cset@1.1136.44.19?nav=index.html|ChangeSet@-9M

HTH
-- 
Martin Donnelly

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot
  2004-04-15  8:58 ` Martin Donnelly
@ 2004-04-15  9:00   ` Martin Donnelly
  2004-04-15 14:18     ` Thayne Harbaugh
  0 siblings, 1 reply; 4+ messages in thread
From: Martin Donnelly @ 2004-04-15  9:00 UTC (permalink / raw)
  To: Tobias Arp; +Cc: linux-mtd

[-- Attachment #1: Type: text/plain, Size: 376 bytes --]

On Thu, 2004-04-15 at 09:58, Martin Donnelly wrote:
> Apply the two attached patches chips-config.patch to
> 'drivers/mtd/chips/Config.in' and nand-config.patch to
> 'drivers/mtd/nand/Config.in' which should sort out a couple of syntax
> errors. e.g `cd linux-2.4.21; patch -p1 -i chips-config.patch`

And it helps if the patches are attached :-)

Cheers
-- 
Martin Donnelly


[-- Attachment #2: chips-config.patch --]
[-- Type: text/plain, Size: 1371 bytes --]

--- linux-2.4.22/drivers/mtd/chips/Config.in.orig	2004-04-13 16:40:33.000000000 +0100
+++ linux-2.4.22/drivers/mtd/chips/Config.in	2004-04-13 16:41:20.000000000 +0100
@@ -11,10 +11,12 @@
 
 if [ "$CONFIG_MTD_CFI" = "y" -o "$CONFIG_MTD_JEDECPROBE" = "y" ]; then
    define_bool CONFIG_MTD_GEN_PROBE y
-elif [ "$CONFIG_MTD_CFI" = "m" -o "$CONFIG_MTD_JEDECPROBE" = "m" ]; then
-   define_bool CONFIG_MTD_GEN_PROBE m
 else
-   define_bool CONFIG_MTD_GEN_PROBE n
+   if [ "$CONFIG_MTD_CFI" = "m" -o "$CONFIG_MTD_JEDECPROBE" = "m" ]; then
+      define_bool CONFIG_MTD_GEN_PROBE m
+   else
+      define_bool CONFIG_MTD_GEN_PROBE n
+   fi
 fi
 
 if [ "$CONFIG_MTD_GEN_PROBE" = "y" -o "$CONFIG_MTD_GEN_PROBE" = "m" ]; then
@@ -49,12 +51,14 @@
 	-o "$CONFIG_MTD_CFI_AMDSTD" = "y" \
 	-o "$CONFIG_MTD_CFI_STAA" = "y" ]; then
    define_bool CONFIG_MTD_CFI_UTIL y
-elif [ "$CONFIG_MTD_CFI_INTELEXT" = "m" \
-	-o "$CONFIG_MTD_CFI_AMDSTD" = "m" \
-	-o "$CONFIG_MTD_CFI_STAA" = "m" ]; then
-   define_bool CONFIG_MTD_CFI_UTIL m
 else
-   define_bool CONFIG_MTD_CFI_UTIL n
+   if [ "$CONFIG_MTD_CFI_INTELEXT" = "m" \
+	 -o "$CONFIG_MTD_CFI_AMDSTD" = "m" \
+	 -o "$CONFIG_MTD_CFI_STAA" = "m" ]; then
+      define_bool CONFIG_MTD_CFI_UTIL m
+   else
+      define_bool CONFIG_MTD_CFI_UTIL n
+   fi
 fi
 
 dep_tristate '  Support for RAM chips in bus mapping' CONFIG_MTD_RAM $CONFIG_MTD

[-- Attachment #3: nand-config.patch --]
[-- Type: text/plain, Size: 477 bytes --]

--- linux-2.4.22/drivers/mtd/nand/Config.in.orig	2004-04-13 16:47:02.000000000 +0100
+++ linux-2.4.22/drivers/mtd/nand/Config.in	2004-04-13 16:47:23.000000000 +0100
@@ -35,7 +35,6 @@
 fi
 
 if [ "$CONFIG_PPCHAMELEONEVB" = "y" ]; then
-    dep_tristate '  NAND Flash device on PPChameleonEVB board' 
-    CONFIG_MTD_NAND_PPCHAMELEONEVB $CONFIG_MTD_NAND
+    dep_tristate '  NAND Flash device on PPChameleonEVB board' CONFIG_MTD_NAND_PPCHAMELEONEVB $CONFIG_MTD_NAND
 fi
 endmenu

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot
  2004-04-15  9:00   ` Martin Donnelly
@ 2004-04-15 14:18     ` Thayne Harbaugh
  0 siblings, 0 replies; 4+ messages in thread
From: Thayne Harbaugh @ 2004-04-15 14:18 UTC (permalink / raw)
  Cc: linux-mtd, Tobias Arp

On Thu, 2004-04-15 at 03:00, Martin Donnelly wrote:
> On Thu, 2004-04-15 at 09:58, Martin Donnelly wrote:
> > Apply the two attached patches chips-config.patch to
> > 'drivers/mtd/chips/Config.in' and nand-config.patch to
> > 'drivers/mtd/nand/Config.in' which should sort out a couple of syntax
> > errors. e.g `cd linux-2.4.21; patch -p1 -i chips-config.patch`
> 
> And it helps if the patches are attached :-)

Committed.  Thanks.

-- 
Thayne Harbaugh
Linux Networx

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2004-04-15 14:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-04-15  7:57 patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot Tobias Arp
2004-04-15  8:58 ` Martin Donnelly
2004-04-15  9:00   ` Martin Donnelly
2004-04-15 14:18     ` Thayne Harbaugh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox