patch kernel 2.4.21 (including the arm-linux patch rmk2) with the latest CVS snapshot

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

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?

<snip warnings>

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

HTH

Martin Donnelly

And it helps if the patches are attached :-)

Cheers

Martin Donnelly

@@ -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

@@ -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

Committed. Thanks.

Thayne Harbaugh
Linux Networx