CVS compile errors

CVS compile errors

[Kalev Lember]

Hi,

I just wanted to notify that current CVS code as of 04/04/04 does not 
compile. Last snapshot that does compile is mtd-snapshot-20040327.

You probably know that, but I am reporting anyhow.

I have set the following options:
CONFIG_MTD=y
CONFIG_INFTL=y
CONFIG_MTD_DOC2001PLUS=y
CONFIG_MTD_DOCPROBE=y
CONFIG_MTD_DOCPROBE_ADDRESS=0
CONFIG_MTD_NAND=y
CONFIG_MTD_NAND_IDS=y

Linux kernel version is 2.6.4.
Output from make is:

make -f scripts/Makefile.build obj=drivers/mtd/devices
  gcc -Wp,-MD,drivers/mtd/devices/.doc2001plus.o.d -nostdinc -iwithprefix 
include -D__KERNEL__ -Iinclude  -D__KERNEL__ -Iin
clude  -Wall -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing 
-fno-common -pipe -mpreferred-stack-boundary=2  -march
=pentium-mmx -Iinclude/asm-i386/mach-default -O2 -fomit-frame-pointer 
-Wdeclaration-after-statement    -DKBUILD_BASENAME=do
c2001plus -DKBUILD_MODNAME=doc2001plus -c -o 
drivers/mtd/devices/doc2001plus.o drivers/mtd/devices/doc2001plus.c
drivers/mtd/devices/doc2001plus.c: In function `DoC_IdentChip':
drivers/mtd/devices/doc2001plus.c:339: error: structure has no member 
named `chipshift'
make[3]: *** [drivers/mtd/devices/doc2001plus.o] Error 1
make[2]: *** [drivers/mtd/devices] Error 2
make[1]: *** [drivers/mtd] Error 2
make: *** [drivers] Error 2

And indeed, there is no member named chipshift in nand.h.

-- 
Best regards,
Kalev Lember

Re: CVS compile errors

[Thomas Gleixner]

On Sunday 04 April 2004 14:21, Kalev Lember wrote:
> Hi,
>
> I just wanted to notify that current CVS code as of 04/04/04 does not
> compile. Last snapshot that does compile is mtd-snapshot-20040327.
>
> You probably know that, but I am reporting anyhow.
>
> I have set the following options:
> CONFIG_MTD=y
> CONFIG_INFTL=y
> CONFIG_MTD_DOC2001PLUS=y
> CONFIG_MTD_DOCPROBE=y
> CONFIG_MTD_DOCPROBE_ADDRESS=0
> CONFIG_MTD_NAND=y
> CONFIG_MTD_NAND_IDS=y
>
> Linux kernel version is 2.6.4.
> Output from make is:
>
> make -f scripts/Makefile.build obj=drivers/mtd/devices
>   gcc -Wp,-MD,drivers/mtd/devices/.doc2001plus.o.d -nostdinc -iwithprefix
> include -D__KERNEL__ -Iinclude  -D__KERNEL__ -Iin
> clude  -Wall -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing
> -fno-common -pipe -mpreferred-stack-boundary=2  -march
> =pentium-mmx -Iinclude/asm-i386/mach-default -O2 -fomit-frame-pointer
> -Wdeclaration-after-statement    -DKBUILD_BASENAME=do
> c2001plus -DKBUILD_MODNAME=doc2001plus -c -o
> drivers/mtd/devices/doc2001plus.o drivers/mtd/devices/doc2001plus.c
> drivers/mtd/devices/doc2001plus.c: In function `DoC_IdentChip':
> drivers/mtd/devices/doc2001plus.c:339: error: structure has no member
> named `chipshift'
> make[3]: *** [drivers/mtd/devices/doc2001plus.o] Error 1
> make[2]: *** [drivers/mtd/devices] Error 2
> make[1]: *** [drivers/mtd] Error 2
> make: *** [drivers] Error 2
>
> And indeed, there is no member named chipshift in nand.h.

I'm in the middle of an overhaul of nand stuff. I modified the id structure 
and forgot to adjust the doc stuff. I have a look into it

-- 
Thomas
________________________________________________________________________
"Free software" is a matter of liberty, not price. To understand the concept,
you should think of "free" as in "free speech,'' not as in "free beer".
________________________________________________________________________
linutronix - competence in embedded & realtime linux
http://www.linutronix.de
mail: tglx@linutronix.de

Re: CVS compile errors

[Thomas Gleixner]

On Sunday 04 April 2004 14:21, Thomas Gleixner wrote:
> On Sunday 04 April 2004 14:21, Kalev Lember wrote:
> >
> > And indeed, there is no member named chipshift in nand.h.
>
> I'm in the middle of an overhaul of nand stuff. I modified the id structure
> and forgot to adjust the doc stuff. I have a look into it

Fixed in CVS. cvs up

-- 
Thomas
________________________________________________________________________
"Free software" is a matter of liberty, not price. To understand the concept,
you should think of "free" as in "free speech,'' not as in "free beer".
________________________________________________________________________
linutronix - competence in embedded & realtime linux
http://www.linutronix.de
mail: tglx@linutronix.de

Re: CVS compile errors

[Kalev Lember]

Thomas Gleixner wrote:
> On Sunday 04 April 2004 14:21, Thomas Gleixner wrote:
> 
>>On Sunday 04 April 2004 14:21, Kalev Lember wrote:
>>
>>>And indeed, there is no member named chipshift in nand.h.
>>
>>I'm in the middle of an overhaul of nand stuff. I modified the id structure
>>and forgot to adjust the doc stuff. I have a look into it
> 
> 
> Fixed in CVS. cvs up
> 

Thank you! Builds ok now.

By the way, what is it that you are overhauling?

-- 
Kalev Lember

Re: CVS compile errors

[Jon Schlueter]

Thomas Gleixner wrote:

 >
> I'm in the middle of an overhaul of nand stuff. I modified the id structure 
> and forgot to adjust the doc stuff. I have a look into it
> 

Hello,


I think I found another missed update but I'm not sure.


I used CVS to pull down the whole mtd tree on 4-5-2004 after those 
changes were posted.

I used the automatic kernel patch script

When I get the following error when I compile the kernel modules

I am using the 2.4.25 kernel


make -C devices modules
make[3]: Entering directory `/usr/src/linux-2.4.25/drivers/mtd/devices'
gcc -D__KERNEL__ -I/usr/src/linux-2.4.25/include -Wall 
-Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
-fomit-frame-pointer -pipe -mpreferred-stack-boundary=2 -march=i486 
-DMODULE  -nostdinc -iwithprefix include -DKBUILD_BASENAME=doc2000  -c 
-o doc2000.o doc2000.c
doc2000.c: In function `DoC_IdentChip':
doc2000.c:415: structure has no member named `page256'
doc2000.c: In function `DoC2k_init':
doc2000.c:601: warning: assignment from incompatible pointer type
doc2000.c:602: warning: assignment from incompatible pointer type
doc2000.c: In function `doc_writev_ecc':
doc2000.c:1026: warning: passing arg 7 of `doc_write_ecc' makes integer 
from pointer without a cast
make[3]: *** [doc2000.o] Error 1
make[3]: Leaving directory `/usr/src/linux-2.4.25/drivers/mtd/devices'
make[2]: *** [_modsubdir_devices] Error 2
make[2]: Leaving directory `/usr/src/linux-2.4.25/drivers/mtd'
make[1]: *** [_modsubdir_mtd] Error 2
make[1]: Leaving directory `/usr/src/linux-2.4.25/drivers'
make: *** [_mod_drivers] Error 2



CONFIG_MTD=y
CONFIG_NFTL=m
CONFIG_NFTL_RW=m
CONFIG_INFTL=m
CONFIG_MTD_DOC2000=m
CONFIG_MTD_DOC2001=m
CONFIG_MTD_DOC2001PLUS=m
CONFIG_MTD_DOCPROBE=m
CONFIG_MTD_NAND=y
CONFIG_MTD_NAND_IDS=y



Side Note:

I haven't been able to determine exactly what I need to compile to 
support my DiskOnChip  M02202-D256 chip with the 5.1.2 firmware from 
M-Systems.  Am I on the right path to getting it working?


Thanks,

Jon Schlueter

Re: CVS compile errors

[andreas.huebner]

Hi all,

On Tue, Apr 06, 2004 at 09:13:47PM -0400, Jon Schlueter wrote:
> 
> I think I found another missed update but I'm not sure.
> 
> make -C devices modules
> make[3]: Entering directory `/usr/src/linux-2.4.25/drivers/mtd/devices'
> gcc -D__KERNEL__ -I/usr/src/linux-2.4.25/include -Wall 
> -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
> -fomit-frame-pointer -pipe -mpreferred-stack-boundary=2 -march=i486 
> -DMODULE  -nostdinc -iwithprefix include -DKBUILD_BASENAME=doc2000  -c 
> -o doc2000.o doc2000.c
> doc2000.c: In function `DoC_IdentChip':
> doc2000.c:415: structure has no member named `page256'
> doc2000.c: In function `DoC2k_init':
> doc2000.c:601: warning: assignment from incompatible pointer type
> doc2000.c:602: warning: assignment from incompatible pointer type
> doc2000.c: In function `doc_writev_ecc':
> doc2000.c:1026: warning: passing arg 7 of `doc_write_ecc' makes integer 
> from pointer without a cast
> make[3]: *** [doc2000.o] Error 1
> make[3]: Leaving directory `/usr/src/linux-2.4.25/drivers/mtd/devices'
> make[2]: *** [_modsubdir_devices] Error 2
> make[2]: Leaving directory `/usr/src/linux-2.4.25/drivers/mtd'
> make[1]: *** [_modsubdir_mtd] Error 2
> make[1]: Leaving directory `/usr/src/linux-2.4.25/drivers'
> make: *** [_mod_drivers] Error 2

I can confirm this error.


Andreas

Re: CVS compile errors

[Thomas Gleixner]

On Wednesday 07 April 2004 07:47, andreas.huebner@hp.com wrote:
> Hi all,
>
> I can confirm this error.

That's nice, if everybody can confirm this. 

I can confirm this too.

-- 
Thomas
________________________________________________________________________
"Free software" is a matter of liberty, not price. To understand the concept,
you should think of "free" as in "free speech,'' not as in "free beer".
________________________________________________________________________
linutronix - competence in embedded & realtime linux
http://www.linutronix.de
mail: tglx@linutronix.de

Re: CVS compile errors

[Thomas Gleixner]

On Wednesday 07 April 2004 03:13, Jon Schlueter wrote:
> Thomas Gleixner wrote:
> > I'm in the middle of an overhaul of nand stuff. I modified the id
> > structure and forgot to adjust the doc stuff. I have a look into it
>
> doc2000.c:415: structure has no member named `page256'

Thanks for pointing it out. I will fix it.

-- 
Thomas
________________________________________________________________________
"Free software" is a matter of liberty, not price. To understand the concept,
you should think of "free" as in "free speech,'' not as in "free beer".
________________________________________________________________________
linutronix - competence in embedded & realtime linux
http://www.linutronix.de
mail: tglx@linutronix.de

Re: CVS compile errors

[Thomas Gleixner]

On Wednesday 07 April 2004 03:13, Jon Schlueter wrote:
>
> doc2000.c:415: structure has no member named `page256'

Fixed in CVS. CVS up

-- 
Thomas
________________________________________________________________________
"Free software" is a matter of liberty, not price. To understand the concept,
you should think of "free" as in "free speech,'' not as in "free beer".
________________________________________________________________________
linutronix - competence in embedded & realtime linux
http://www.linutronix.de
mail: tglx@linutronix.de