linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* Patch for zsrec.c
@ 2000-02-01  0:12 Pavel Roskin
  2000-02-01  0:56 ` Download timeout problems on Embedded Planet CLLF (was Patch for zsrec.c) Graham Stoney
  0 siblings, 1 reply; 7+ messages in thread
From: Pavel Roskin @ 2000-02-01  0:12 UTC (permalink / raw)
  To: linuxppc-embedded


Hello!

File ftp://linuxppc.cs.nmt.edu/pub/linuxppc/embedded/zsrec.c has a bug.
It uses an S9 closing record, but specifies 3 address bytes instead of 2
required by the standard.

RPX/Lite boards report checksum error on such records, which is not good
either, but the real bug is zsrec.c

Since data are represented by S3 records with 4-byte addresses, the safest
way to fix it would be to use S7 records that also use 4-byte addresses.
It is also possible to use 0x200000 as the starting address, so that the
"t" command in the boot prompt works!

The sentence about buggy RPX boards should be removed from
getting_started.rpx

This is my patch

=== cut here ===
--- zsrec.c	Mon Jan 31 10:57:19 2000
+++ zsrec.c	Mon Jan 31 18:41:55 2000
@@ -69,7 +69,7 @@
 		lev1();
 	if (l1total)
 		lev1();
-	printf("S904000000FB\n");
+	printf("S70500200000DA\n");
 }

 lev1()
=== cut here ===

Pavel Roskin


** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/

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

end of thread, other threads:[~2000-02-01  7:20 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2000-02-01  0:12 Patch for zsrec.c Pavel Roskin
2000-02-01  0:56 ` Download timeout problems on Embedded Planet CLLF (was Patch for zsrec.c) Graham Stoney
2000-02-01  1:07   ` Alan Mimms
2000-02-01  1:13   ` Matthew Locke
2000-02-01  1:53     ` Graham Stoney
2000-02-01  2:05       ` Matthew Locke
2000-02-01  7:20   ` Dan Malek

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).