]> cloudbase.mooo.com Git - z180-stamp.git/blobdiff - avr/env.c
env.c needs getopt-min.h
[z180-stamp.git] / avr / env.c
index 48413f3c5064f74ec6a8eda419eabaa57109d12f..70209ca00fae6e170f5cfaeeddaf856a77c6e5c9 100644 (file)
--- a/avr/env.c
+++ b/avr/env.c
+/*
+ * (C) Copyright 2014 Leo C. <erbl259-lmu@yahoo.de>
+ *
+ * SPDX-License-Identifier:    GPL-2.0
+ */
+
 #include "common.h"
 #include <string.h>
 #include <stdlib.h>
-
+#include <stdbool.h>
 #include <avr/eeprom.h>
 
 #include "config.h"
 #include "debug.h"
 #include "xmalloc.h"
 #include "crc.h"
+#include "getopt-min.h"
 #include "command.h"
-
 #include "env.h"
 
 
+#define ENV_SIZE       (CONFIG_ENV_SIZE - sizeof(uint16_t) -1)
+#define ACTIVE_FLAG    1
+#define OBSOLETE_FLAG  0
 
-#define DELIM          "\0"
-
-
-#define ENV_GET_VAL    (1<<0)
+#define ENVLIST_DELETE (1<<0)
 
 
 /*
  * Default Environment
  */
+
+#define DELIM          "\0"
+
 const FLASH char default_env[] = {
-       "bootdelay="    "3"             DELIM
-       "bootcmd="      "reset; loadf; go $(startaddr)" DELIM
-       "baudrate="     "115200"        DELIM
-       "startaddr="    "0"             DELIM
+       ENV_BAUDRATE      "=" "115200" DELIM
+       ENV_BOOTDELAY     "=" "3" DELIM
+       ENV_BOOTCMD               "=" "pin ${pins}; loadcpm3; go ${startaddress}" DELIM
+       ENV_CPM3_SYSFILE  "=" CONFIG_CPM3_SYSFILE DELIM
+       ENV_PINALIAS      "=" "0:PG5,1:PG4,2:PB4,3:PB5,4:PB6,5:PB7,"
+                                                 "6:PG3,7:PG2,8:PG1,9:PG0,10:PE7" DELIM
+       ENV_STARTADDRESS  "=" "0" DELIM
+       "pins"                    "=" "2,8 low 9 high 3 2" DELIM
        DELIM
 };
 
 
-/*
- * Debugging
- *
- * TODO: move elsewhere
- */
-#include <ctype.h>
-
-static void print_blanks(uint_fast8_t count)
-{
-       while(count--)
-               putchar(' ');
-}
-
-
-void dump_ram(const uint8_t *startaddr, int len, char *title)
-{
-       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);
-               
-       while (len) {
-               if (len < 16)
-                       llen = len;
-                       
-               printf_P(PSTR("    %.4x:"), (size_t) addr);
-               print_blanks(3 * pre);
-               for (i = pre; i < llen; i++)
-                       printf_P(PSTR(" %.2x"), addr[i]);
-               print_blanks(3 * (16 - i + 1) + pre);
-               for (i = pre; i < llen; i++)
-                       printf_P(PSTR("%c"), isprint(addr[i]) ? addr[i] : '.');
-               putchar('\n');
-
-               pre = 0;
-               addr += 16;
-               len -= llen;
-       }
-}
-
-void dump_heap(void)
-{
-       extern unsigned int __brkval;
-
-       dump_ram((uint8_t *) __malloc_heap_start, 
-               __brkval - (unsigned int) __malloc_heap_start,
-               "=== Heap:");
-}
-
-/* TODO: combine with dump_ram() */
-void dump_eep(const uint8_t *addr, int len)
-{
-       int i;
-       uint8_t buf[16];
-       
-       printf_P(PSTR("eeprom dump:"));
-       while (len) {
-               printf_P(PSTR("\n    0x%.4x:"), (unsigned int) addr);
-               for (i = 0; i<16; i++)
-                       buf[i] = eeprom_read_byte(addr + i);
-               for (i = 0; i<16; i++)
-                       printf_P(PSTR(" %.2x"), buf[i]);
-               printf_P(PSTR("   "));
-               for (i = 0; i<16; i++) 
-                       printf_P(PSTR("%c"), isprint(buf[i]) ? buf[i] : '.');
-                       
-               addr += 16;
-               len -= len > 16 ? 16 : len;
-       }
-       putchar('\n');
-}
-
-#define ENV_SIZE       CONFIG_ENV_SIZE-2
+/* EEPROM storage */
 typedef struct environment_s {
        uint16_t        crc;            /* CRC16 over data bytes        */
+       uint8_t         flags;          /* active/obsolete flags        */
        char            data[ENV_SIZE]; /* Environment data             */
 } env_t;
 
 
-
+/* */
 typedef struct env_item_s {
-#define EF_N_EEP       (1<<7)          /* Variable name is in EEPROM */
-#define EF_V_EEP       (1<<6)          /* Variable value is in EEPROM */
-#define EF_DIRTY       (1<<0)          /* Variable is new or value changed */
-       uint8_t flags;
-       union {
-               uint16_t eep;
-               char     *ram;
-       } name;
-       union {
-               uint16_t eep;
-               char     *ram;
-       } val;
+       char * envvar;
 } env_item_t;
 
 
+static uint8_t env_valid;
 static env_item_t env_list[CONFIG_ENVVAR_MAX];
 static int entrycount;
 
 
-static char env_get_char(uint16_t index)
+static
+char env_get_char(uint16_t index)
 {
        unsigned int off = CONFIG_ENV_OFFSET;
+       char ret;
 
-       return (char) eeprom_read_byte((const uint8_t *)off + index + 
+       switch (env_valid) {
+       case 2:
+               off += CONFIG_ENV_SIZE;
+       case 1:
+               ret = (char) eeprom_read_byte((const uint8_t *)off + index +
                                offsetof(env_t, data));
-}
-
-
-uint16_t ee_name_get(char *buf, env_item_t *ep)
-{
-       int i = 0;
-       char c;
-       
-       if (ep->flags & EF_N_EEP) {
-
-               while ((c = env_get_char(ep->name.eep + i)) != '=' &&
-                               c != '\0' && i < CONFIG_SYS_ENV_NAMELEN) {
+               break;
 
-                       buf[i] = c;
-                       i++;
-               }
-               if (i > 0 && c != '=') {
-                       debug("** ee_name: '%s' not '=' terminated!\n", buf);
-               }
-       } else {
-               strncpy(buf, ep->name.ram, CONFIG_SYS_ENV_NAMELEN);
-               i = strnlen(buf, CONFIG_SYS_ENV_NAMELEN);
+       default:
+               ret = default_env[index];
        }
-       buf[i] = '\0';
-       return i;
-}
-
 
-uint16_t ee_val_get(char *buf, uint16_t index, int len)
-{
-       int i = 0;
-       char c;
-       
-       while ((c = env_get_char (index + i)) != '\0' && i < len) {
-               buf[i] = c;
-               i++;
-       };
-               
-       buf[i] = '\0';
-       
-       /* TODO: len check */
-       
-       return i;
+       return ret;
 }
 
-static
-int comp_env_key_item(const void *key, const void *item)
-{
-       char buf[CONFIG_SYS_ENV_NAMELEN+1];
-       env_item_t *ep = (env_item_t *) item;
-       
-       ee_name_get(buf, ep);
-       
-       return strcmp((char *) key, buf);
-}
+
+static const FLASH char *comp_key;
 
 static
 int comp_env_items(const void *m1, const void *m2)
 {
-       char b1[CONFIG_SYS_ENV_NAMELEN+1];
-       char b2[CONFIG_SYS_ENV_NAMELEN+1];
-       
        env_item_t *ep1 = (env_item_t *) m1;
        env_item_t *ep2 = (env_item_t *) m2;
-       
-       ee_name_get(b1, ep1);
-       ee_name_get(b2, ep2);
-       
-       return strcmp(b1, b2);
-}
 
-#if 0
-env_item_t * ee_entry_get(const char *s, env_item_t *ep, uint_fast8_t flag)
-{
-       char name[CONFIG_SYS_ENV_NAMELEN+1];
-       uint16_t idx = 0;
-       int nlen;
-       
-//printf_P(PSTR("*** ee_entry_get:        >>>>>>>>> ENTER <<<<<<<<<\n"));
-
-
-       while ((nlen = ee_name_get(name, idx)) != 0 && idx < CONFIG_ENV_SIZE) {
-       
-//printf_P(PSTR("**  idx: %d, name: '%s', s: '%s', nlen: %d, cpres:%d\n"), 
-//             idx, name, s, nlen, strncmp(name, s, nlen));
-       
-               if (strncmp(name, s, nlen) == 0)
-                       break;
-                       
-               idx += nlen + 1;
-               while (env_get_char(idx++) != 0)
-                       ;
-       }
-       
-       if (nlen) {
-               char *vp;
-               ep->flags = EF_N_EEP;
-               ep->name.eep = idx;
-               if (flag & ENV_GET_VAL) {
-                       vp = xmalloc(CONFIG_SYS_CBSIZE);
-                       nlen = ee_val_get(vp, idx + nlen + 1, CONFIG_SYS_CBSIZE);
-                       ep->val = xrealloc(vp, nlen + 1);
-               } else
-                       ep->val = NULL;
-
-
-//printf_P(PSTR("*** ee_entry_get:        >>>>>> LEAVE 0x%.4x <<<<<\n"), 
-//                     (unsigned int) ep);
-               return ep;
-       }
-
-//printf_P(PSTR("*** ee_entry_get:        >>>>>> LEAVE <NULL> <<<<<\n"));
-       return NULL;
-}                      
-#endif
-
-#if 0
-static char p_env_name_buf[CONFIG_SYS_ENV_NAMELEN+1];
-
-static char *dbg_p_env_name(env_item_t *p)
-{
-       if (p->flags & EF_N_EEP) {
-               if (ee_name_get(p_env_name_buf, p) != 0)
-                       return p_env_name_buf;
-               else
-                       return "<NULL>";
-       }
-       return "<NO EEP_NAME>";
+       if (ep1 == NULL)
+               return - strcmp_P(ep2->envvar, comp_key);
+       else
+               return strcmp(ep1->envvar, ep2->envvar);
 }
-#endif
 
-int env_item_print(env_item_t *ep)
+
+env_item_t *envlist_search(const MEMX char *name)
 {
-       char buf[CONFIG_SYS_ENV_NAMELEN+1];
-       int len;
-       env_item_t e = *ep;
-       
-       ee_name_get(buf, ep);
-       len = printf_P(PSTR("%s="), buf);
-       
-       if (e.val.ram != NULL) {
-               while (1) {
-                       char c;
-                       if (e.flags & EF_V_EEP) 
-                               c = env_get_char(e.val.eep++);
-                       else
-                               c = *e.val.ram++;
-                       
-                       if (c != '\0') {
-                               putchar(c);
-                               len++;
-                       } else
-                               break;
-               }
-       }
-       putchar('\n');
-       len ++;
+#ifdef __MEMX
+       if (__builtin_avr_flash_segment(name) != -1) {
+               comp_key = name;
+               return bsearch(0, env_list, entrycount,
+                               sizeof(env_item_t), comp_env_items);
+       } else {
 
-       return len;
-}      
+               env_item_t e;
+               e.envvar = (char *) name;
 
+               return bsearch(&e, env_list, entrycount,
+                               sizeof(env_item_t), comp_env_items);
+       }
+#else
+       env_item_t e;
+       e.envvar = (char *) name;
 
-static env_item_t *envlist_search(const char *name)
-{
-       return bsearch(name, env_list, entrycount, 
-                               sizeof(env_item_t), comp_env_key_item);
+       return bsearch(&e, env_list, entrycount,
+                               sizeof(env_item_t), comp_env_items);
+#endif /* __MEMX */
 }
-                       
-#if 0
-env_item_t *envlist_insert(const char *key, env_item_t *e)
-{
-       const size_t size = sizeof(env_item_t);
-
-       if (entrycount < CONFIG_ENVVAR_MAX) {
-               env_list[entrycount++] = *e;
-               qsort(env_list, entrycount, size, comp_env_items);
 
-               return bsearch(key, env_list, entrycount, 
-                                       size, comp_env_key_item);
-               
-       } else
-               return NULL;
-}      
-#endif
 
+static
 env_item_t *envlist_enter(env_item_t *e)
 {
-       char *key = e->name.ram;
        const size_t size = sizeof(env_item_t);
        env_item_t *ep;
-       
-       ep = bsearch(key, env_list, entrycount, 
-                       size, comp_env_key_item);
-                                       
+
+       ep = bsearch(e, env_list, entrycount,
+                       size, comp_env_items);
+
        if (ep == NULL) {
                if (entrycount < CONFIG_ENVVAR_MAX) {
-                       
+
                        env_list[entrycount++] = *e;
                        qsort(env_list, entrycount, size, comp_env_items);
-                       ep = bsearch(key, env_list, entrycount, 
-                                               size, comp_env_key_item);
-               }
-       } else { 
-               if ((ep->flags & EF_V_EEP) == 0) {
-                       free(ep->val.ram);
-               }
-               ep->val.ram = e->val.ram;
-       }               
-
-       if (ep != NULL) {
-               ep->flags |= EF_DIRTY;
-               ep->flags &= ~EF_V_EEP;
-               
-               if ((ep->flags & EF_N_EEP) == 0) {
-                       int nlen = strnlen(key, CONFIG_SYS_ENV_NAMELEN);
-                       char *name = xmalloc(nlen + 1);
-                       if (name == NULL) {
-                               printf_P(PSTR("## Can't malloc %d bytes\n"), 
-                                       nlen + 1);
-                               return NULL;
-                       }
-                       strcpy(name, key);
-                       name[nlen] = '\0';
-                       ep->name.ram = name;
+                       ep = bsearch(e, env_list, entrycount,
+                                               size, comp_env_items);
                }
+       } else {
+               free(ep->envvar);
+               ep->envvar = e->envvar;
        }
 
-       return ep;              
-}      
-
-
-static env_item_t *envlist_get(const char *name, uint_fast8_t flag)
-{
-       env_item_t *ep;
-       
-       ep = envlist_search(name);
-
-       if (ep != NULL && (flag & ENV_GET_VAL)) {
-               if (ep->flags & EF_V_EEP) {
-                       char *vp;
-                       uint_fast8_t len;
-                       vp = xmalloc(CONFIG_SYS_CBSIZE);
-                       len = ee_val_get(vp, ep->val.eep, CONFIG_SYS_CBSIZE);
-                       ep->val.ram = xrealloc(vp, len + 1);
-                       ep->flags &= ~EF_V_EEP;
-               }
-       }                       
-       
        return ep;
 }
 
 
-static int envlist_delete(const char *name)
+static
+int env_item_delete(env_item_t *ep)
 {
-       size_t size = sizeof(env_item_t);
-       env_item_t *ep = bsearch(name, env_list, entrycount, 
-                               sizeof(env_item_t), comp_env_key_item);
-       int rc = 0;
+       if (entrycount == 0)
+               return -1;
 
-       if (ep != NULL) {
-       
-               if ((ep->flags & EF_V_EEP) == 0) 
-                       free(ep->val.ram);
-               if ((ep->flags & EF_N_EEP) == 0) 
-                       free(ep->name.ram); 
+       free(ep->envvar);
 
-               entrycount--;
-               memmove(ep, ep + 1, (env_list - ep)*size + entrycount*size);
-               
-               rc = 1;
-       }
-#if 0
-       dump_ram((uint8_t *) &env_list[0], entrycount * sizeof(env_item_t),
-                       "=== env_list:");
-       dump_heap();
-       debug("** entrycount: %d\n", entrycount);
-       for (int i=0; i<entrycount; i++) {
-               printf_P(PSTR("** env var [%d] "), i);
-               env_item_print(&env_list[i]);
-       }
-#endif
+       entrycount--;
+       size_t size = sizeof(env_item_t);
+       memmove(ep, ep + 1, (env_list - ep)*size + entrycount*size);
 
-       return rc;
+       return 0;
 }
-                       
-char *getenv(const char *name)
+
+static
+int envlist_delete(const MEMX char *name)
 {
-       env_item_t *ep;
-       char *ret = NULL;
+       env_item_t *ep = envlist_search(name);
 
-       debug("** getenv: %s\n", name);
-       
-       ep = envlist_get(name, ENV_GET_VAL);
        if (ep != NULL)
-               ret = ep->val.ram;
-#if 0
-       dump_ram((uint8_t *) &env_list[0], entrycount * sizeof(env_item_t),
-                       "=== env_list:");
-       dump_heap();
-       debug("** entrycount: %d\n", entrycount);
-       for (int i=0; i<entrycount; i++) {
-               printf_P(PSTR("** env var [%d] "), i);
-               env_item_print(&env_list[i]);
-       }
-#endif
-       return ret;
+               return env_item_delete(ep);
+
+       return 1;
 }
 
-/* TODO: implement saveenv() */
-int saveenv(void)
-{
-       int     rc = 0;
 
-//     rc = env_export(&env_new);
-       if (rc)
-               return rc;
 
-//     rc = eeprom_bus_write(CONFIG_SYS_DEF_EEPROM_ADDR,
-//                           off, (uchar *)&env_new, CONFIG_ENV_SIZE);
-       return rc;
-}
+static
+int envlist_import(uint8_t flags)
+{
+       uint16_t index;
 
+       int len;
+       env_item_t e;
+       char *np, c;
+       uint_fast8_t ef;
 
-/*-----------------------------------------------------------------------------*/
+       if ((flags & ENVLIST_DELETE) != 0)
+               while (entrycount != 0)
+                       env_item_delete(&env_list[entrycount-1]);
 
+       for (index = 0; env_get_char(index) != '\0'; ) {
+               for (len = 0; env_get_char(index + len) != '\0'; ++len) {
+                       if ((index + len) >= ENV_SIZE)
+                               return -1;
+               }
 
-int set_default_env(void)
-{
-       char buf[64];
-       uint16_t crc = 0xffff;
-       uint16_t eep = CONFIG_ENV_OFFSET + offsetof(env_t, data);
-       unsigned int len = ENV_SIZE;
-       unsigned int i, src = 0;
-       char c = 0xff, c0 = c;
-#if 0
-printf_P(PSTR("\n\n** set_default_env()\n"));
-#endif
-       while (len)  {
-       
-               memcpy_P(buf, default_env+src, sizeof(buf));
-               
-               for (i=0; i < (len < sizeof(buf) ? len : sizeof(buf)) &&
-                    !(c == 0 && c0 == 0);
-                   i++)
-               {
-                       c0 = c; c = buf[i];
-                       crc = crc16(crc, c);
+               np = (char *) xmalloc(len+1);
+               if (np == NULL) {
+                       printf_P(PSTR("## Can't malloc %d bytes\n"), len+1);
+                       return 1;
                }
-               
-               eeprom_update_block(buf, (char *) eep, i);
-#if 0
-               printf_P(PSTR("eeprom_update_block: eep: 0x%.4x, i:%d\n"), 
-                       (unsigned int) eep, i);
-               dump_ram((uint8_t *) buf, i, "=== buf:");
-               dump_eep((const uint8_t *) eep, i);
-#endif
-               if (c == 0 && c0 == 0)
-                       len = 0;
-               if (len > sizeof(buf))
-                       len -= sizeof(buf);
-               src += sizeof(buf);
-               eep += sizeof(buf);
+               e.envvar = np;
+               ef = 0;
+               while ((c = env_get_char(index++)) != '\0') {
+                       if (!ef && (c == '=')) {
+                               *np = '\0';
+                               ef = 1;
+                       } else
+                               *np = c;
+                       np++;
+               }
+               *np = '\0';
+               envlist_enter(&e);
        }
 
-       eeprom_update_word(
-               (uint16_t *) CONFIG_ENV_OFFSET + offsetof(env_t, crc), 
-               crc);
-#if 0
-       dump_eep(0, 128);
-#endif
+
        return 0;
 }
 
 
-int env_check(void)
+static
+uint16_t env_crc(uint16_t data_offset)
 {
-       uint16_t crc, stored_crc; 
+       uint16_t crc;
        uint16_t i;
        char c, c0;
 
-       debug("\n\n** env_check()\n");
-       
-
-       /* read old CRC */
-       stored_crc = eeprom_read_word((const uint16_t *) CONFIG_ENV_OFFSET + 
-                                       offsetof(env_t, crc));
        crc = 0xffff;
        c = 0xff;
-       for (i = offsetof(env_t, data); 
-               !(c == 0 && c0 == 0) && i < ENV_SIZE;
-               i++)
+       for (i = 0; !(c == 0 && c0 == 0) && i < ENV_SIZE; i++)
        {
-               c0 = c; 
-               c = eeprom_read_byte((const uint8_t *) CONFIG_ENV_OFFSET + i);
+               c0 = c;
+               c = eeprom_read_byte((const uint8_t *) data_offset + i);
                crc = crc16(crc, c);
        }
-       debug_cond((crc != stored_crc),
-               "** crc eep: 0x%.4x, crc new: 0x%.4x\n", stored_crc, crc);
-       
-       return crc == stored_crc;
+       return crc ;
 }
 
+
+/**
+ * return valid env
+ */
+static
+int env_check_valid(void)
+{
+       const uint16_t offset[2] = {CONFIG_ENV_OFFSET,
+                                   CONFIG_ENV_OFFSET + CONFIG_ENV_SIZE};
+       uint_fast8_t flags[2], crc_ok[2];
+       int rc;
+
+       /* read FLAGS */
+       flags[0] = eeprom_read_byte ((uint8_t *) offset[0] +
+                               offsetof(env_t, flags));
+       flags[1] = eeprom_read_byte ((uint8_t *) offset[1] +
+                               offsetof(env_t, flags));
+
+       /* check CRC */
+       crc_ok[0] = (
+               eeprom_read_word((uint16_t *) offset[0] +
+                               offsetof(env_t, crc))
+               == env_crc(offset[0] + offsetof(env_t, data))
+       );
+       crc_ok[1] = (
+               eeprom_read_word((uint16_t *) offset[1] +
+                               offsetof(env_t, crc))
+               == env_crc(offset[1] + offsetof(env_t, data))
+       );
+
+       if (!crc_ok[0] && !crc_ok[1]) {
+               rc = 0;
+
+       } else if (crc_ok[0] && !crc_ok[1]) {
+               rc = 1;
+       } else if (!crc_ok[0] && crc_ok[1]) {
+               rc = 2;
+       } else {
+               /* both ok - check serial */
+#if 1
+               if      (flags[1] == ACTIVE_FLAG && flags[0] != ACTIVE_FLAG)
+                       rc = 2;
+               else if (flags[1] == OBSOLETE_FLAG && flags[0] == 0xFF)
+                       rc = 2;
+               else
+                       rc = 1;
+#else
+               if (flags[0] == ACTIVE_FLAG && flags[1] == OBSOLETE_FLAG)
+                       rc = 1;
+               else if (flags[0] == OBSOLETE_FLAG && flags[1] == ACTIVE_FLAG)
+                       rc = 2;
+               else if (flags[0] == 0xFF && flags[1] == 0)
+                       rc = 2;
+               else if (flags[1] == 0xFF && flags[0] == 0)
+                       rc = 1;
+               else /* flags are equal - almost impossible */
+                       rc = 1;
+#endif
+       }
+
+       return rc;
+}
+
+
 int env_init(void)
 {
-       char name[CONFIG_SYS_ENV_NAMELEN+1];
-       uint16_t idx = 0;
-       int nlen;
-       env_item_t e;
-       
-       e.flags = EF_N_EEP | EF_V_EEP;
-       e.name.eep = idx;
-       while ((nlen = ee_name_get(name, &e)) != 0 && idx < ENV_SIZE) {
-       
-               if (entrycount <= CONFIG_ENVVAR_MAX) {
-                       e.val.eep = idx + nlen + 1;
-       
-                       env_list[entrycount++] = e;
-
-                       idx += nlen + 1;
-                       while (env_get_char(idx++) != 0 && idx < ENV_SIZE)
-                               ;
-                       e.name.eep = idx;
-               } else {
-                       debug("** Too many environment variables!\n");
-                       break;
-               }
+       env_valid = env_check_valid();
+       if (env_valid == 0) {
+               printf_P(PSTR("*** Warning - bad CRC, "
+                               "using default environment\n\n"));
        }
-       qsort(env_list, entrycount, sizeof(env_item_t), comp_env_items);
 
+       envlist_import(ENVLIST_DELETE);
        return 0;
 }
 
 
+char *getenv_str(const MEMX char *name)
+{
+       env_item_t *ep;
+       char *ret = NULL;
+
+       ep = envlist_search(name);
+       if (ep != NULL)
+               ret = ep->envvar + strlen(ep->envvar) +1;
+       return ret;
+}
+
+static
+int env_item_save(env_item_t *ep, uint16_t offset, int space_left)
+{
+       int nlen, vlen;
+       char *var = ep->envvar;
+
+       nlen = strlen(var);
+       if (nlen == 0)
+               return 0;
+       vlen = strlen(var + nlen + 1);
+       if (vlen == 0)
+               return 0;
+       if (nlen + vlen + 2 > space_left)
+               return 0;
+
+       eeprom_update_block(var, (uint8_t *) offset, nlen);
+       offset += nlen;
+       eeprom_update_byte((uint8_t *) offset++, '=');
+       eeprom_update_block(var + nlen +1, (uint8_t *) offset, vlen+1);
+
+       return nlen + vlen + 2;
+}
+
+
+static
+int saveenv(void)
+{
+       unsigned int off     = CONFIG_ENV_OFFSET + CONFIG_ENV_SIZE;
+       unsigned int off_red = CONFIG_ENV_OFFSET;
+       unsigned int pos;
+       int len, left;
+       uint16_t crc;
+       int rc = 0;
+
+       if (env_valid == 2) {
+               off     = CONFIG_ENV_OFFSET;
+               off_red = CONFIG_ENV_OFFSET + CONFIG_ENV_SIZE;
+       }
+
+       eeprom_update_byte((uint8_t *) off + offsetof(env_t, flags), 0xff);
+
+       pos = off + offsetof(env_t, data);
+       left = ENV_SIZE - 1;
+       for (int i = 0 ; i < entrycount; i++) {
+               len = env_item_save(&env_list[i], pos, left);
+               if (len == 0) {
+                       return 1;
+               }
+               pos += len;
+               left -= len;
+       }
+       /* terminate list */
+       eeprom_update_byte((uint8_t *) pos, 0);
+       crc = env_crc(off + offsetof(env_t, data));
+       eeprom_update_word((uint16_t *) off + offsetof(env_t, crc), crc);
+       eeprom_update_byte((uint8_t *) off + offsetof(env_t, flags),
+                                       ACTIVE_FLAG);
+
+       if (rc == 0) {
+               eeprom_update_byte((uint8_t *) off_red + offsetof(env_t, flags),
+                                       OBSOLETE_FLAG);
+               env_valid = (env_valid == 2) ? 1 : 2;
+       }
+
+       return rc;
+}
+
+
+static
+int env_item_print(env_item_t *ep, bool mode)
+{
+       int len;
+       char *ev = ep->envvar;
+
+       if (mode) {
+               len = printf_P(PSTR("setenv %s "), ev) - 7;
+               len += printf_P(PSTR("'%s'\n"), ev + len) - 2;
+       } else {
+       len = printf_P(PSTR("%s="), ev);
+       len += printf_P(PSTR("%s\n"), ev + len);
+       }
+       return len;
+}
+
+
 /*
  * Command interface: print one or all environment variables
  *
  * Returns -1 in case of error, or length of printed string
  */
-static int env_print(char *name)
+static
+int env_print(const MEMX char *name, bool mode)
 {
        int len = -1;
 
        if (name) {             /* print a single name */
-       
+
                env_item_t *ep = envlist_search(name);
                if (ep != NULL)
-                       len = env_item_print(ep);
+                       len = env_item_print(ep, mode);
 
        } else {                /* print whole list */
                len = 0;
                for (int i = 0 ; i < entrycount; i++) {
-                       len += env_item_print(&env_list[i]);
+                       len += env_item_print(&env_list[i], mode);
                }
        }
        return len;
 }
 
-int env_print_ramsize(void)
-{
-       int size = 0;
-       uint8_t name_cnt = 0;
-       uint8_t val_cnt = 0;
-       
-       for (int i = 0 ; i < entrycount; i++) {
-               if ((env_list[i].flags & EF_N_EEP) == 0 &&
-                               (env_list[i].name.ram != NULL)) {
-                       name_cnt++;
-                       size += strlen(env_list[i].name.ram) + 3;
-               }
-               if ((env_list[i].flags & EF_V_EEP) == 0 &&
-                               (env_list[i].val.ram != NULL)) {
-                       val_cnt++;
-                       size += strlen(env_list[i].val.ram) + 3;
-               }
-       }
-       printf_P(PSTR("%d bytes RAM used for %u names and %u values\n"),
-                       size, name_cnt, val_cnt);
-       return size;
-}
-
-int do_env_print(cmd_tbl_t *cmdtp, int flag, int argc,
-                       char * const argv[])
-{
-       int i;
-       int rcode = 0;
-
-       (void) cmdtp; (void) flag;
-
-       if (argc == 1) {
-               /* print all env vars */
-               rcode = env_print(NULL);
-               if (rcode < 0)
-                       return 1;
-               printf_P(PSTR("\nEnvironment size: %d/%d bytes\n"),
-                       rcode, ENV_SIZE);
-               env_print_ramsize();
-               return 0;
-       }
-
-       /* print selected env vars */
-       for (i = 1; i < argc; ++i) {
-               int rc = env_print(argv[i]);
-               if (rc < 0) {
-                       printf_P(PSTR("## Error: \"%s\" not defined\n"), argv[i]);
-                       ++rcode;
-               }
-       }
-
-       return rcode;
-}
-
 
 /**
  * Set or delete environment variable
@@ -668,14 +455,14 @@ int do_env_print(cmd_tbl_t *cmdtp, int flag, int argc,
  *
  * @return 0 if ok, 1 on error
  */
-static int _do_env_set(int flag, int argc, char * const argv[])
+static
+command_ret_t _do_env_set(int flag, int argc, char * const argv[])
 {
        int i, len;
-       char *name, *value, *s;
+       char *name, *value, *valp, *p;
        env_item_t e, *ep;
 
        (void) flag;
-       debug("Initial value for argc=%d\n", argc);
 
        name = argv[1];
        value = argv[2];
@@ -683,12 +470,13 @@ static int _do_env_set(int flag, int argc, char * const argv[])
        if (strchr(name, '=')) {
                printf_P(PSTR("## Error: illegal character '='"
                       "in variable name \"%s\"\n"), name);
-               return 1;
+               return CMD_RET_FAILURE;
        }
-       if (strlen(name) > CONFIG_SYS_ENV_NAMELEN) {
+       len = strlen(name);
+       if (len > CONFIG_SYS_ENV_NAMELEN) {
                printf_P(PSTR("## Error: Variable name \"%s\" too long. "
                       "(max %d characters)\n"), name, CONFIG_SYS_ENV_NAMELEN);
-               return 1;
+               return CMD_RET_FAILURE;
        }
 /*
        env_id++;
@@ -696,41 +484,41 @@ static int _do_env_set(int flag, int argc, char * const argv[])
        /* Delete only ? */
        if (argc < 3 || argv[2] == NULL) {
                int rc = envlist_delete(name);
-               return !rc;
+               return rc ? CMD_RET_FAILURE : CMD_RET_SUCCESS;
        }
 
        /*
         * Insert / replace new value
         */
-       for (i = 2, len = 0; i < argc; ++i)
+       for (i = 2, len += 1; i < argc; ++i)
                len += strlen(argv[i]) + 1;
 
        value = xmalloc(len);
        if (value == NULL) {
                printf_P(PSTR("## Can't malloc %d bytes\n"), len);
-               return 1;
+               return CMD_RET_FAILURE;
        }
-       for (i = 2, s = value; i < argc; ++i) {
+       strcpy(value, name);
+       valp = value + strlen(name) + 1;
+       for (i = 2, p = valp; i < argc; ++i) {
                char *v = argv[i];
 
-               while ((*s++ = *v++) != '\0')
+               while ((*p++ = *v++) != '\0')
                        ;
-               *(s - 1) = ' ';
+               *(p - 1) = ' ';
        }
-       if (s != value)
-               *--s = '\0';
+       if (p != valp)
+               *--p = '\0';
 
-       e.flags = EF_DIRTY;
-       e.name.ram = name;
-       e.val.ram = value;
+       e.envvar = value;
        ep = envlist_enter(&e);
        if (!ep) {
                printf_P(PSTR("## Error inserting \"%s\" variable.\n"),
                        name);
-               return 1;
+               return CMD_RET_FAILURE;
        }
 
-       return 0;
+       return CMD_RET_SUCCESS;
 }
 
 /**
@@ -740,73 +528,174 @@ static int _do_env_set(int flag, int argc, char * const argv[])
  * @param varvalue     Value to set it to
  * @return 0 if ok, 1 on error
  */
-int setenv(const char *varname, const char *varvalue)
+static
+int setenv(const MEMX char *varname, const char *varvalue)
 {
-       const char * const argv[3] = { NULL, varname, varvalue };
+       int rc;
+
+#ifdef __MEMX
+       char *tmpname = NULL;
+       if (__builtin_avr_flash_segment(varname) != -1) {
+               tmpname = malloc(strlen_P(varname)+1);
+               if (tmpname == NULL) {
+                       printf_P(PSTR("setenv: Out of Memory!\n"));
+                       return 1;
+               }
+               strcpy_P(tmpname, varname);
+       } else
+               tmpname = (char *) varname;
+#endif
+
+       const char * const argv[3] = { NULL, tmpname, varvalue };
        int argc = 3;
 
        if (varvalue == NULL || varvalue[0] == '\0')
                --argc;
-               
-       return _do_env_set(0, argc, (char * const *)argv);
+
+       rc = (int) _do_env_set(0, argc, (char * const *)argv);
+
+#ifdef __MEMX
+       free(tmpname);
+#endif
+       return rc;
 }
 
-#if 0
 /**
  * Set an environment variable to an integer value
  *
- * @param varname      Environment variable to set
- * @param value                Value to set it to
+ * @param name Environment variable to set
+ * @param value        Value to set it to
+ * @param radix        Base
  * @return 0 if ok, 1 on error
  */
-int setenv_ulong(const char *varname, unsigned long value)
+static
+int setenv_intval(const MEMX char *name, unsigned long value, int radix)
 {
-       /* TODO: this should be unsigned */
-       char *str = simple_itoa(value);
+       char buf[11];
 
-       return setenv(varname, str);
-}
-#endif
+       ultoa(value, buf, radix);
 
+       return setenv(name, buf);
+}
 
 /**
- * Set an environment variable to an value in hex
+ * Set an environment variable to a decimal integer value
  *
- * @param varname      Environment variable to set
- * @param value                Value to set it to
+ * @param name Environment variable to set
+ * @param value        Value to set it to
  * @return 0 if ok, 1 on error
  */
-int setenv_hex(const char *varname, unsigned long value)
+int setenv_ulong(const MEMX char *name, unsigned long value)
 {
-       char str[sizeof(unsigned long) *2 + 1];
+       return setenv_intval(name, value, 10);
+}
+
 
-       sprintf_P(str, PSTR("%lx"), value);
-       return setenv(varname, str);
+/**
+ * Set an environment variable to a value in hex
+ *
+ * @param name Environment variable to set
+ * @param value        Value to set it to
+ * @return 0 if ok, 1 on error
+ */
+int setenv_hex(const MEMX char *name, unsigned long value)
+{
+       return setenv_intval(name, value, 16);
 }
 
+
 /**
- * Get an environment variable as a hex value
+ * Decode the integer value of an environment variable and return it.
  *
- * @param varname      Environment variable to get
- * @param default_val  Return this, if variable does not exist
- * @return hex value of variable or default_val
+ * @param name         Name of environemnt variable
+ * @param base         Number base to use (normally 10, or 16 for hex)
+ * @param default_val  Default value to return if the variable is not
+ *                     found
+ * @return the decoded value, or default_val if not found
  */
-unsigned long getenv_hex(const char *varname, unsigned long default_val)
+unsigned long getenv_ulong(const MEMX char *name, int base, unsigned long default_val)
 {
-       const char *s;
        unsigned long value;
-       char *endp;
+       char *vp, *endp;
+
+       env_item_t *ep = envlist_search(name);
 
-       s = getenv(varname);
-       if (s)
-               value = strtoul(s, &endp, 16);
-       if (!s || endp == s)
-               return default_val;
+       if (ep != NULL ) {
+               vp = ep->envvar + strlen(ep->envvar) +1;
+               value = strtoul(vp, &endp, base);
+               if (endp != vp)
+                       return value;
+       }
 
-       return value;
+       return default_val;
 }
 
-int do_env_set(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+
+/*
+ * Read an environment variable as a boolean
+ */
+bool getenv_yesno(const MEMX char *name)
+{
+       char *s = getenv_str(name);
+
+       if (s == NULL)
+               return false;
+
+       return strchr_P(PSTR("1yYtT"), *s) != NULL;
+
+/*
+       return *s == '1' || *s == 'y' || *s == 'Y' || *s == 't' || *s == 'T' ?
+               1 : 0;
+*/
+}
+
+command_ret_t do_env_print(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+{
+       (void) cmdtp; (void) flag;
+
+       bool mode = 0;
+       command_ret_t rc = CMD_RET_SUCCESS;
+
+       /* reset getopt() */
+       optind = 0;
+
+       int opt;
+       while ((opt = getopt(argc, argv, PSTR("s"))) != -1) {
+               switch (opt) {
+               case 's':
+                       mode = 1;
+                       break;
+               default: /* '?' */
+                       return CMD_RET_USAGE;
+               }
+       }
+
+       if (optind == argc) {
+               /* print all env vars */
+               int size = env_print(NULL, mode);
+               if (size < 0)
+                       return CMD_RET_FAILURE;
+               if (mode == 0)
+                       printf_P(PSTR("\nEnvironment size: %d/%d bytes\n"),
+                               size, ENV_SIZE);
+               return CMD_RET_SUCCESS;
+       }
+
+       /* print selected env vars */
+       while (optind < argc) {
+               int rc = env_print(argv[optind], mode);
+               if (rc < 0) {
+                       printf_P(PSTR("## Error: \"%s\" not defined\n"), argv[optind]);
+                       rc = CMD_RET_FAILURE;
+               }
+               optind++;
+       }
+
+       return rc;
+}
+
+
+command_ret_t do_env_set(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
        (void) cmdtp;
 
@@ -817,7 +706,34 @@ int do_env_set(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 }
 
 
-#if defined(CONFIG_AUTO_COMPLETE) 
+command_ret_t do_env_default(cmd_tbl_t *cmdtp, int flag,
+                         int argc, char * const argv[])
+{
+       (void) cmdtp; (void) flag; (void) argc; (void) argv;
+
+       uint8_t tmp = env_valid;
+       env_valid = 0;
+
+       /* Reset the whole environment */
+       printf_P(PSTR("## Resetting to default environment\n"));
+       envlist_import(ENVLIST_DELETE);
+
+       env_valid = tmp;
+
+       return CMD_RET_SUCCESS;
+}
+
+
+command_ret_t do_env_save(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+{
+       (void) cmdtp; (void) flag; (void) argc; (void) argv;
+
+       printf_P(PSTR("Saving Environment ...\n"));
+       return saveenv() ? CMD_RET_FAILURE : CMD_RET_SUCCESS;
+}
+
+
+#if defined(CONFIG_AUTO_COMPLETE)
 int env_complete(char *var, int maxv, char *cmdv[], int bufsz, char *buf)
 {
        ENTRY *match;
@@ -848,4 +764,3 @@ int env_complete(char *var, int maxv, char *cmdv[], int bufsz, char *buf)
        return found;
 }
 #endif
-