X-Git-Url: http://cloudbase.mooo.com/gitweb/z180-stamp.git/blobdiff_plain/507d25e2cdac7489ca8706f6582e271fd1689816..a8eb521f94848a627a3fe470e34f62b13c157d34:/avr/debug.c?ds=sidebyside

diff --git a/avr/debug.c b/avr/debug.c
index 47b11b0..d4ae1f4 100644
--- a/avr/debug.c
+++ b/avr/debug.c
@@ -1,3 +1,9 @@
+/*
+ * (C) Copyright 2014 Leo C. <erbl259-lmu@yahoo.de>
+ *
+ * SPDX-License-Identifier:	GPL-2.0+
+ */
+
 #include "common.h"
 #include <stdlib.h>
 #include <string.h>
@@ -5,78 +11,14 @@
 #include <avr/eeprom.h>
 
 #include "command.h"
+#include "print-utils.h"
 #include "debug.h"
 
 /*
  * Debugging
  */
-#ifdef DEBUG
-
-static void print_blanks(uint_fast8_t count)
-{
-	while(count--)
-		putchar(' ');
-}
-
-static uint8_t ram_read_byte(const uint8_t *p)
-{
-	return *p;
-}
-
-void dump_mem(const uint8_t *startaddr, int len,
-		uint8_t (*readfkt)(const uint8_t *), char *title)
-{
-	uint8_t buf[16];
-	char *indent = NULL;
-	uint8_t llen = 16;
-	uint8_t pre = (size_t) startaddr % 16;
-	const uint8_t *addr = (uint8_t *) ((size_t) startaddr & ~0x0f);
-	len += pre;
-	uint8_t i;
-
-	if (title && *title) {
-		printf_P(PSTR("%s\n"),title);
-		indent = "    ";
-	}
-
-	while (len) {
-		if (len < 16)
-			llen = len;
 
-		for (i = pre; i < llen; i++)
-			buf[i] = readfkt(addr + i);
-
-		printf_P(PSTR("%s%04x:"),indent, addr);
-		for (i = 0; i < llen; i++) {
-			if ((i % 8) == 0)
-				putchar(' ');
-			if (i < pre)
-				printf_P(PSTR(".. "));
-			else
-				printf_P(PSTR("%.2x "), buf[i]);
-		}
-		/* fill line with whitespace for nice ASCII print */
-		print_blanks(3 * (16u - i) + (16u-i)/8 + 1 + pre);
-		/* Print data in ASCII characters */
-		for (i = pre; i < llen; i++)
-			printf_P(PSTR("%c"), isprint(buf[i]) ? buf[i] : '.');
-		putchar('\n');
-
-		pre = 0;
-		addr += 16;
-		len -= llen;
-	}
-}
-
-void dump_eep(const uint8_t *addr, unsigned int len, char *title)
-{
-	dump_mem(addr, len, eeprom_read_byte, title);
-}
-
-void dump_ram(const uint8_t *addr, unsigned int len, char *title)
-{
-	dump_mem(addr, len, ram_read_byte, title);
-}
+#ifdef DEBUG
 
 
 #if 0
@@ -84,7 +26,7 @@ void dump_heap(void)
 {
 	extern unsigned int __brkval;
 
-	dump_ram((uint8_t *) __malloc_heap_start,
+	dump_ram(__malloc_heap_start,
 		__brkval - (unsigned int) __malloc_heap_start,
 		"=== Heap:");
 }
@@ -97,34 +39,39 @@ void dump_heap(void)
  */
 command_ret_t do_dump_mem(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
-//	static const uint8_t *addr;
-//	static uint16_t length = 128;
-	uint8_t (*readhow)(const uint8_t *);
+	void (*readhow)(uint8_t *buf, uint32_t addr, uint8_t count);
 
 	(void) cmdtp; (void) flag;
 
 	if (argc < 2)
 		return CMD_RET_USAGE;
 
-	const uint8_t *addr;
-	uint16_t length = 128;
-
-	if (strchr(argv[0],'r') != NULL)
-		readhow = ram_read_byte;
-	else if (strchr(argv[0],'e') != NULL)
-		readhow = eeprom_read_byte;
-	else
+	uint32_t addr;
+	uint32_t length = 128;
+
+	switch (argv[0][3]) {
+	case 'r':
+		readhow = ram_read_buf;
+		break;
+	case 'e':
+		readhow = eeprom_read_buf;
+		break;
+	case 'f':
+		readhow = flash_read_buf;
+		break;
+	default:
 		return CMD_RET_USAGE;
+	}
 
 	/* Address is specified since argc > 1 */
-	addr = (const uint8_t *) (size_t) strtoul(argv[1], NULL, 16);
+	addr =  strtoul(argv[1], NULL, 16);
 
 	/* If another parameter, it is the length to display. */
 	if (argc > 2)
 		length = (uint16_t) strtoul(argv[2], NULL, 16);
 
 	/* Print the lines. */
-	dump_mem(addr, length, readhow, NULL);
+	dump_mem(addr, addr, length, readhow, NULL);
 
 	return CMD_RET_SUCCESS;
 }
@@ -179,8 +126,8 @@ command_ret_t do_eep_cp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[
 	return CMD_RET_SUCCESS;
 }
 
-/*------------------------------------------------------------------------------*/
 
+/*------------------------------------------------------------------------------*/
 
 #if 1
 
@@ -227,4 +174,3 @@ printfreelist(const char * title)
 #endif
 
 #endif /* DEBUG */
-