public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [AVR] Add new memory regions to script template
@ 2011-03-14 14:48 Weddington, Eric
  2011-03-30 15:31 ` Nick Clifton
  0 siblings, 1 reply; 6+ messages in thread
From: Weddington, Eric @ 2011-03-14 14:48 UTC (permalink / raw)
  To: binutils; +Cc: Anatoly Sokolov, Denis Chertykov, Joerg Wunsch

[-- Attachment #1: Type: text/plain, Size: 878 bytes --]

Hi All,

Attached is a patch to add new memory regions to the AVR linker script template. This adds support for fuse, lock and signature memory regions. This patch has been used extensively in the field for a while with no issues and allows this data to be added to the ELF output file. The fuse and lock information is needed for programming the AVR chip. So this allows the user to contain all information needed for programming an AVR chip in one single ELF file.

I have a question regarding approval for binutils: I'm a co-maintainer of the AVR port of GCC. Do I need approval for this patch before committing to binutils? I would like to be listed as co-maintainer of the AVR port of binutils.

Thanks,
Eric Weddington

/ld:
2011-03-14  Eric B. Weddington  <eric.weddington@atmel.com>

	* scripttempl/avr.sc: Add fuse, lock, and signature memory regions.



[-- Attachment #2: 32-binutils-2.20.1-new-sections.patch --]
[-- Type: application/octet-stream, Size: 1043 bytes --]

diff -ruwN ld/scripttempl/avr.sc ld/scripttempl/avr.sc
--- ld/scripttempl/avr.sc	2009-10-09 18:42:35.000000000 +0530
+++ ld/scripttempl/avr.sc	2010-03-11 12:26:00.563046000 +0530
@@ -7,6 +7,9 @@
   text   (rx)   : ORIGIN = 0, LENGTH = $TEXT_LENGTH
   data   (rw!x) : ORIGIN = $DATA_ORIGIN, LENGTH = $DATA_LENGTH
   eeprom (rw!x) : ORIGIN = 0x810000, LENGTH = 64K
+  fuse      (rw!x) : ORIGIN = 0x820000, LENGTH = 1K
+  lock      (rw!x) : ORIGIN = 0x830000, LENGTH = 1K
+  signature (rw!x) : ORIGIN = 0x840000, LENGTH = 1K
 }
 
 SECTIONS
@@ -196,6 +199,24 @@
     ${RELOCATING+ __eeprom_end = . ; }
   } ${RELOCATING+ > eeprom}
 
+  .fuse ${RELOCATING-0}:
+  {
+    KEEP(*(.fuse))
+    KEEP(*(.lfuse))
+    KEEP(*(.hfuse))
+    KEEP(*(.efuse))
+  } ${RELOCATING+ > fuse}
+
+  .lock ${RELOCATING-0}:
+  {
+    KEEP(*(.lock*))
+  } ${RELOCATING+ > lock}
+
+  .signature ${RELOCATING-0}:
+  {
+    KEEP(*(.signature*))
+  } ${RELOCATING+ > signature}
+
   /* Stabs debugging sections.  */
   .stab 0 : { *(.stab) }
   .stabstr 0 : { *(.stabstr) }

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

* Re: [AVR] Add new memory regions to script template
  2011-03-14 14:48 [AVR] Add new memory regions to script template Weddington, Eric
@ 2011-03-30 15:31 ` Nick Clifton
  2011-03-31  2:54   ` Weddington, Eric
  0 siblings, 1 reply; 6+ messages in thread
From: Nick Clifton @ 2011-03-30 15:31 UTC (permalink / raw)
  To: Weddington, Eric; +Cc: binutils, Anatoly Sokolov, Denis Chertykov, Joerg Wunsch

Hi Eric,

> I have a question regarding approval for binutils:
> I'm a co-maintainer of the AVR port of GCC.
> Do I need approval for this patch before committing to binutils?

No... :-)

> /ld:
> 2011-03-14  Eric B. Weddington<eric.weddington@atmel.com>
>
> 	* scripttempl/avr.sc: Add fuse, lock, and signature memory regions.

Go ahead.

Cheers
   Nick


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

* RE: [AVR] Add new memory regions to script template
  2011-03-30 15:31 ` Nick Clifton
@ 2011-03-31  2:54   ` Weddington, Eric
  0 siblings, 0 replies; 6+ messages in thread
From: Weddington, Eric @ 2011-03-31  2:54 UTC (permalink / raw)
  To: Nick Clifton; +Cc: binutils, Anatoly Sokolov, Denis Chertykov, Joerg Wunsch



> -----Original Message-----
> From: Nick Clifton [mailto:nickc@redhat.com]
> Sent: Wednesday, March 30, 2011 9:31 AM
> To: Weddington, Eric
> Cc: binutils@sourceware.org; Anatoly Sokolov; Denis Chertykov; Joerg
> Wunsch
> Subject: Re: [AVR] Add new memory regions to script template
> 
> Hi Eric,
> 
> > I have a question regarding approval for binutils:
> > I'm a co-maintainer of the AVR port of GCC.
> > Do I need approval for this patch before committing to binutils?
> 
> No... :-)

Thanks for the clarification, Nick.

> > /ld:
> > 2011-03-14  Eric B. Weddington<eric.weddington@atmel.com>
> >
> > 	* scripttempl/avr.sc: Add fuse, lock, and signature memory regions.
> 
> Go ahead.

Thanks, already committed, and other avr patches since then. I've got one more coming, hopefully tomorrow.

Eric

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

* Re: [AVR] Add new memory regions to script template
  2011-03-18  9:36 ` Joerg Wunsch
@ 2011-03-18 18:33   ` Denis Chertykov
  0 siblings, 0 replies; 6+ messages in thread
From: Denis Chertykov @ 2011-03-18 18:33 UTC (permalink / raw)
  To: Joerg Wunsch; +Cc: Joerg Wunsch, Weddington, Eric, binutils, Anatoly Sokolov

2011/3/18 Joerg Wunsch <j@uriah.heep.sax.de>:
> As Weddington, Eric wrote:
>
>> Should I just go ahead and commit this, then?
>
> No objections from my side.  The offsets used match those that
> have been in use in AVaRICE for about a decade now.

Please, commit.

Denis.

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

* Re: [AVR] Add new memory regions to script template
  2011-03-18  2:21 Weddington, Eric
@ 2011-03-18  9:36 ` Joerg Wunsch
  2011-03-18 18:33   ` Denis Chertykov
  0 siblings, 1 reply; 6+ messages in thread
From: Joerg Wunsch @ 2011-03-18  9:36 UTC (permalink / raw)
  To: Weddington, Eric; +Cc: binutils, Anatoly Sokolov, Denis Chertykov

As Weddington, Eric wrote:

> Should I just go ahead and commit this, then?

No objections from my side.  The offsets used match those that
have been in use in AVaRICE for about a decade now.
-- 
cheers, J"org               .-.-.   --... ...--   -.. .  DL8DTL

http://www.sax.de/~joerg/                        NIC: JW11-RIPE
Never trust an operating system you don't have sources for. ;-)

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

* RE: [AVR] Add new memory regions to script template
@ 2011-03-18  2:21 Weddington, Eric
  2011-03-18  9:36 ` Joerg Wunsch
  0 siblings, 1 reply; 6+ messages in thread
From: Weddington, Eric @ 2011-03-18  2:21 UTC (permalink / raw)
  To: binutils; +Cc: Anatoly Sokolov, Denis Chertykov, Joerg Wunsch

PING

Should I just go ahead and commit this, then?

Eric

> -----Original Message-----
> From: Weddington, Eric
> Sent: Monday, March 14, 2011 8:48 AM
> To: binutils@sourceware.org
> Cc: Anatoly Sokolov; Denis Chertykov; 'Joerg Wunsch'
> Subject: [AVR] Add new memory regions to script template
> 
> Hi All,
> 
> Attached is a patch to add new memory regions to the AVR linker script
> template. This adds support for fuse, lock and signature memory regions.
> This patch has been used extensively in the field for a while with no
> issues and allows this data to be added to the ELF output file. The fuse
> and lock information is needed for programming the AVR chip. So this
> allows the user to contain all information needed for programming an AVR
> chip in one single ELF file.
> 
> I have a question regarding approval for binutils: I'm a co-maintainer of
> the AVR port of GCC. Do I need approval for this patch before committing
> to binutils? I would like to be listed as co-maintainer of the AVR port of
> binutils.
> 
> Thanks,
> Eric Weddington
> 
> /ld:
> 2011-03-14  Eric B. Weddington  <eric.weddington@atmel.com>
> 
> 	* scripttempl/avr.sc: Add fuse, lock, and signature memory regions.
> 

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

end of thread, other threads:[~2011-03-31  2:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-14 14:48 [AVR] Add new memory regions to script template Weddington, Eric
2011-03-30 15:31 ` Nick Clifton
2011-03-31  2:54   ` Weddington, Eric
2011-03-18  2:21 Weddington, Eric
2011-03-18  9:36 ` Joerg Wunsch
2011-03-18 18:33   ` Denis Chertykov

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 read-only IMAP folder(s) and NNTP newsgroup(s).