patchAVR C Runtime Library - Patches: patch #8610, Remove eeprom function name...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #8610: Remove eeprom function name mangling

Submitter:  Pitchumani <pitchumani>
Submitted:  Mon 23 Feb 2015 09:58:21 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  pitchumani Open/Closed:  Closed

Wed 25 Feb 2015 01:01:35 PM UTC, comment #2: 
Pitchumani <pitchumani>
Group Member
Mon 23 Feb 2015 12:02:15 PM UTC, comment #1: 

Some device headers do not define EEARL if the EEAR is one byte. But EEPROM functions refers EEARL assuming it is defined even if the length is one byte.

Following will define EEARL if device header defines only EEAR, not EEARL/EEARH.

(snip to be added to include/avr/eeprom.h)

#if defined (EEAR) && !defined (EEARL) && !defined (EEARH)
#define EEARL EEAR
#endif


Pitchumani <pitchumani>
Group Member
Mon 23 Feb 2015 09:58:21 AM UTC, original submission:  

Continuation to patch #8587 (https://savannah.nongnu.org/patch/?8587)

EEPROM function names are mangled with _EEPROM_SUFFIX to make them device specific. This is done to make the functions unique as they are added into same library (libc.a).

Now EEPROM functions are added into UNIQUE device specific libary (libdev.a, patch #8587). So, function name mangling is not required. Attached patch will remove that name mangling.

ChangeLog

2015-02-23  Joern Rennecke  <joern.rennecke@embecosm.com>

    * include/avr/eeprom.h: Remove _EEPROM_SUFFIX definition and eeprom
    function name mangling.
    * libc/misc/eedef.h: Remove eeprom function name mangling.


This is part of Joern Rennecke's patch.
https://github.com/embecosm/avr-libc/commit/93f15d167faf4ee3019ddefbd520740c04bb3123

Pitchumani <pitchumani>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #33162:  remove-eeprom-function-name-mangling.patch added by pitchumani (18KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by pitchumani (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-02-25 pitchumani StatusReady For Test Done
        Open/ClosedOpen Closed
    2015-02-23 pitchumani Attached File- Added remove-eeprom-function-name-mangling.patch, #33162

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code