RE: [avr-libc-dev] [RFC] New eeprom.h

[Weddington, Eric]

> -----Original Message-----
> From: 
> address@hidden 
> [mailto:address@hidden
> org] On Behalf Of Dmitry K.
> Sent: Saturday, March 01, 2008 12:43 AM
> To: address@hidden
> Subject: Re: [avr-libc-dev] [RFC] New eeprom.h
> 
> Hi all.
> 
> 
>       New variant of EEPROM functions: inline byte procedures 
> and library
>       multibyte envelopes.
>       * include/avr/eeprom.h: Rewrite on the base of project 
> by Eric B.
>       Weddington.
>       * libc/misc/eeprom.S: Remove file.
>       * libc/misc/ee_rb.S: Remove file.
>       * libc/misc/ee_rblk.S: Remove file.
>       * libc/misc/ee_rw.S: Remove file.
>       * libc/misc/ee_wb.S: Remove file.
>       * libc/misc/ee_wblk.S: Remove file.
>       * libc/misc/ee_ww.S: Remove file.
>       * libc/misc/eerd_block.c: New file.
>       * libc/misc/eerd_dword.c: New file.
>       * libc/misc/eerd_word.c: New file.
>       * libc/misc/eewr_block.c: New file.
>       * libc/misc/eewr_dword.c: New file.
>       * libc/misc/eewr_word.c: New file.
>       * libc/misc/Files.am: Remove old sources, add new.
>       * libc/misc/Makefile.am: Include $(misc_a_c_sources) to 
> EXTRA_DIST.
>       * libc/misc/Rules.am: Remove rules to build 
> chip-specific objects.
>       * tests/simulate/avr: New directory (for AVR-specific 
> functions).
>       * tests/simulate/avr/eeprom-1.c: New file.
> 
> Eric, I wait yours approval to commit into CVS HEAD.
> 

Hi Dmitry,

Thanks for taking the initiative on this and optimizing the code space.

I'm ok to commit to HEAD if these conditions are met:
- builds for all current AVR devices that have EEPROM
- fixes avr-libc bug #21410
- fixes gcc bug #31644

I would like Joerg's opinion on whether this should be committed to 1.6
branch. Joerg?

Thanks,
Eric