2004-11-19 10:07:03

by Manfred Schwarb

[permalink] [raw]
Subject: [2.4.28] Build Error 2: build of pca200e.bin fails

Hi,

OK, I know I'm stupid...
I always forget to unset my GZIP options, as I have
"export GZIP='-9 -N'" in my .bashrc.

This results in the following:

objcopy -Iihex pca200e.data -Obinary pca200e.bin.gz
gzip -df pca200e.bin.gz
./fore200e_mkfirm -k -b _fore200e_pca_fw \
-i pca200e.bin -o fore200e_pca_fw.c
./fore200e_mkfirm: can't open pca200e.bin for reading
make[2]: *** [fore200e_pca_fw.c] Error 254
make[2]: Leaving directory `/usr/src/linux-2.4.28/drivers/atm'
make[1]: *** [_modsubdir_atm] Error 2
make[1]: Leaving directory `/usr/src/linux-2.4.28/drivers'
make: *** [_mod_drivers] Error 2


The following patch would correct this:

--- linux-2.4.28/drivers/atm/Makefile.orig 2004-11-19 09:33:21.000000000 +0000
+++ linux-2.4.28/drivers/atm/Makefile 2004-11-19 09:38:07.000000000 +0000
@@ -92,7 +92,7 @@
# deal with the various suffixes of the binary firmware images
%.bin %.bin1 %.bin2: %.data
objcopy -Iihex $< -Obinary [email protected]
- gzip -df [email protected]
+ gzip -n -df [email protected]

fore_200e.o: $(fore_200e-objs)
$(LD) -r -o $@ $(fore_200e-objs)


2004-11-28 19:21:26

by Adrian Bunk

[permalink] [raw]
Subject: [2.6 patch] Build Error 2: build of pca200e.bin fails

On Fri, Nov 19, 2004 at 05:03:29AM -0500, Manfred Schwarb wrote:
> Hi,
>
> OK, I know I'm stupid...
> I always forget to unset my GZIP options, as I have
> "export GZIP='-9 -N'" in my .bashrc.
>
> This results in the following:
>
> objcopy -Iihex pca200e.data -Obinary pca200e.bin.gz
> gzip -df pca200e.bin.gz
> ./fore200e_mkfirm -k -b _fore200e_pca_fw \
> -i pca200e.bin -o fore200e_pca_fw.c
> ./fore200e_mkfirm: can't open pca200e.bin for reading
> make[2]: *** [fore200e_pca_fw.c] Error 254
> make[2]: Leaving directory `/usr/src/linux-2.4.28/drivers/atm'
> make[1]: *** [_modsubdir_atm] Error 2
> make[1]: Leaving directory `/usr/src/linux-2.4.28/drivers'
> make: *** [_mod_drivers] Error 2
>
>
> The following patch would correct this:
>
> --- linux-2.4.28/drivers/atm/Makefile.orig 2004-11-19 09:33:21.000000000 +0000
> +++ linux-2.4.28/drivers/atm/Makefile 2004-11-19 09:38:07.000000000 +0000
> @@ -92,7 +92,7 @@
> # deal with the various suffixes of the binary firmware images
> %.bin %.bin1 %.bin2: %.data
> objcopy -Iihex $< -Obinary [email protected]
> - gzip -df [email protected]
> + gzip -n -df [email protected]
>
> fore_200e.o: $(fore_200e-objs)
> $(LD) -r -o $@ $(fore_200e-objs)


I have no problems with this patch, but shouldn't the same be done
in 2.6?


Signed-off-by: Adrian Bunk <[email protected]>

--- linux-2.6.10-rc2-mm3-full/drivers/atm/Makefile.old 2004-11-28 20:18:04.000000000 +0100
+++ linux-2.6.10-rc2-mm3-full/drivers/atm/Makefile 2004-11-28 20:18:15.000000000 +0100
@@ -68,4 +68,4 @@
# deal with the various suffixes of the binary firmware images
$(obj)/%.bin $(obj)/%.bin1 $(obj)/%.bin2: $(src)/%.data
objcopy -Iihex $< -Obinary [email protected]
- gzip -df [email protected]
+ gzip -n -df [email protected]

2004-12-01 00:35:23

by Marcelo Tosatti

[permalink] [raw]
Subject: Re: [2.6 patch] Build Error 2: build of pca200e.bin fails

On Sun, Nov 28, 2004 at 08:21:21PM +0100, Adrian Bunk wrote:
> On Fri, Nov 19, 2004 at 05:03:29AM -0500, Manfred Schwarb wrote:
> > Hi,
> >
> > OK, I know I'm stupid...
> > I always forget to unset my GZIP options, as I have
> > "export GZIP='-9 -N'" in my .bashrc.
> >
> > This results in the following:
> >
> > objcopy -Iihex pca200e.data -Obinary pca200e.bin.gz
> > gzip -df pca200e.bin.gz
> > ./fore200e_mkfirm -k -b _fore200e_pca_fw \
> > -i pca200e.bin -o fore200e_pca_fw.c
> > ./fore200e_mkfirm: can't open pca200e.bin for reading
> > make[2]: *** [fore200e_pca_fw.c] Error 254
> > make[2]: Leaving directory `/usr/src/linux-2.4.28/drivers/atm'
> > make[1]: *** [_modsubdir_atm] Error 2
> > make[1]: Leaving directory `/usr/src/linux-2.4.28/drivers'
> > make: *** [_mod_drivers] Error 2
> >
> >
> > The following patch would correct this:
> >
> > --- linux-2.4.28/drivers/atm/Makefile.orig 2004-11-19 09:33:21.000000000 +0000
> > +++ linux-2.4.28/drivers/atm/Makefile 2004-11-19 09:38:07.000000000 +0000
> > @@ -92,7 +92,7 @@
> > # deal with the various suffixes of the binary firmware images
> > %.bin %.bin1 %.bin2: %.data
> > objcopy -Iihex $< -Obinary [email protected]
> > - gzip -df [email protected]
> > + gzip -n -df [email protected]
> >
> > fore_200e.o: $(fore_200e-objs)
> > $(LD) -r -o $@ $(fore_200e-objs)
>
>
> I have no problems with this patch, but shouldn't the same be done
> in 2.6?
>
>
> Signed-off-by: Adrian Bunk <[email protected]>
>
> --- linux-2.6.10-rc2-mm3-full/drivers/atm/Makefile.old 2004-11-28 20:18:04.000000000 +0100
> +++ linux-2.6.10-rc2-mm3-full/drivers/atm/Makefile 2004-11-28 20:18:15.000000000 +0100
> @@ -68,4 +68,4 @@
> # deal with the various suffixes of the binary firmware images
> $(obj)/%.bin $(obj)/%.bin1 $(obj)/%.bin2: $(src)/%.data
> objcopy -Iihex $< -Obinary [email protected]
> - gzip -df [email protected]
> + gzip -n -df [email protected]

Isnt it exactly the same of what has been merged in v2.4?

I can't see any difference.

[marcelo@dmt atm]$ bk diffs -u -r1.8 -r1.9 Makefile
===== Makefile 1.8 vs 1.9 =====
--- 1.8/drivers/atm/Makefile Mon Jul 28 11:35:31 2003
+++ 1.9/drivers/atm/Makefile Mon Nov 22 21:54:09 2004
@@ -92,7 +92,7 @@
# deal with the various suffixes of the binary firmware images
%.bin %.bin1 %.bin2: %.data
objcopy -Iihex $< -Obinary [email protected]
- gzip -df [email protected]
+ gzip -n -df [email protected]

fore_200e.o: $(fore_200e-objs)
$(LD) -r -o $@ $(fore_200e-objs)

2004-12-01 02:36:48

by Adrian Bunk

[permalink] [raw]
Subject: Re: [2.6 patch] Build Error 2: build of pca200e.bin fails

On Tue, Nov 30, 2004 at 03:00:31PM -0200, Marcelo Tosatti wrote:
> On Sun, Nov 28, 2004 at 08:21:21PM +0100, Adrian Bunk wrote:
> >
> > I have no problems with this patch, but shouldn't the same be done
> > in 2.6?
> >
> >
> > Signed-off-by: Adrian Bunk <[email protected]>
> >
> > --- linux-2.6.10-rc2-mm3-full/drivers/atm/Makefile.old 2004-11-28 20:18:04.000000000 +0100
> > +++ linux-2.6.10-rc2-mm3-full/drivers/atm/Makefile 2004-11-28 20:18:15.000000000 +0100
> > @@ -68,4 +68,4 @@
> > # deal with the various suffixes of the binary firmware images
> > $(obj)/%.bin $(obj)/%.bin1 $(obj)/%.bin2: $(src)/%.data
> > objcopy -Iihex $< -Obinary [email protected]
> > - gzip -df [email protected]
> > + gzip -n -df [email protected]
>
> Isnt it exactly the same of what has been merged in v2.4?
>
> I can't see any difference.
>
> [marcelo@dmt atm]$ bk diffs -u -r1.8 -r1.9 Makefile
> ===== Makefile 1.8 vs 1.9 =====
> --- 1.8/drivers/atm/Makefile Mon Jul 28 11:35:31 2003
> +++ 1.9/drivers/atm/Makefile Mon Nov 22 21:54:09 2004
> @@ -92,7 +92,7 @@
> # deal with the various suffixes of the binary firmware images
> %.bin %.bin1 %.bin2: %.data
> objcopy -Iihex $< -Obinary [email protected]
> - gzip -df [email protected]
> + gzip -n -df [email protected]
>
> fore_200e.o: $(fore_200e-objs)
> $(LD) -r -o $@ $(fore_200e-objs)

It is exactly the same except for the patch context (note that in 2.6,
the gzip is the last line of the Makefile).

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed