linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [2.4.28] Build Error 2: build of pca200e.bin fails
@ 2004-11-19 10:03 Manfred Schwarb
  2004-11-28 19:21 ` [2.6 patch] " Adrian Bunk
  0 siblings, 1 reply; 4+ messages in thread
From: Manfred Schwarb @ 2004-11-19 10:03 UTC (permalink / raw)
  To: chas, Marcelo Tosatti, linux-kernel; +Cc: Manfred Schwarb

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 $@.gz
-	gzip -df $@.gz
+	gzip -n -df $@.gz
 
 fore_200e.o: $(fore_200e-objs)
 	$(LD) -r -o $@ $(fore_200e-objs)

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

* [2.6 patch] Build Error 2: build of pca200e.bin fails
  2004-11-19 10:03 [2.4.28] Build Error 2: build of pca200e.bin fails Manfred Schwarb
@ 2004-11-28 19:21 ` Adrian Bunk
  2004-11-30 17:00   ` Marcelo Tosatti
  0 siblings, 1 reply; 4+ messages in thread
From: Adrian Bunk @ 2004-11-28 19:21 UTC (permalink / raw)
  To: Manfred Schwarb; +Cc: chas, Marcelo Tosatti, linux-kernel, linux-atm-general

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 $@.gz
> -	gzip -df $@.gz
> +	gzip -n -df $@.gz
>  
>  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 <bunk@stusta.de>

--- 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 $@.gz
-	gzip -df $@.gz
+	gzip -n -df $@.gz


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

* Re: [2.6 patch] Build Error 2: build of pca200e.bin fails
  2004-11-28 19:21 ` [2.6 patch] " Adrian Bunk
@ 2004-11-30 17:00   ` Marcelo Tosatti
  2004-12-01  2:36     ` Adrian Bunk
  0 siblings, 1 reply; 4+ messages in thread
From: Marcelo Tosatti @ 2004-11-30 17:00 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: Manfred Schwarb, chas, linux-kernel, linux-atm-general

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 $@.gz
> > -	gzip -df $@.gz
> > +	gzip -n -df $@.gz
> >  
> >  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 <bunk@stusta.de>
> 
> --- 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 $@.gz
> -	gzip -df $@.gz
> +	gzip -n -df $@.gz

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 $@.gz
-       gzip -df $@.gz
+       gzip -n -df $@.gz
 
 fore_200e.o: $(fore_200e-objs)
        $(LD) -r -o $@ $(fore_200e-objs)

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

* Re: [2.6 patch] Build Error 2: build of pca200e.bin fails
  2004-11-30 17:00   ` Marcelo Tosatti
@ 2004-12-01  2:36     ` Adrian Bunk
  0 siblings, 0 replies; 4+ messages in thread
From: Adrian Bunk @ 2004-12-01  2:36 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: Manfred Schwarb, chas, linux-kernel, linux-atm-general

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 <bunk@stusta.de>
> > 
> > --- 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 $@.gz
> > -	gzip -df $@.gz
> > +	gzip -n -df $@.gz
> 
> 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 $@.gz
> -       gzip -df $@.gz
> +       gzip -n -df $@.gz
>  
>  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


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

end of thread, other threads:[~2004-12-01  2:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-11-19 10:03 [2.4.28] Build Error 2: build of pca200e.bin fails Manfred Schwarb
2004-11-28 19:21 ` [2.6 patch] " Adrian Bunk
2004-11-30 17:00   ` Marcelo Tosatti
2004-12-01  2:36     ` Adrian Bunk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).