[PATCH] score/or1k/cpu.c: discard linker-symbols.h include and extern the related variable

Gedare Bloom gedare at rtems.org
Tue Feb 10 20:30:31 UTC 2015


Yeah if you don't mind, when you now the ticket number, just include
something like

"closes #xyz"

On Tue, Feb 10, 2015 at 3:26 PM, Hesham Moustafa
<heshamelmatary at gmail.com> wrote:
> On Tue, Feb 10, 2015 at 8:21 PM, Joel Sherrill
> <joel.sherrill at oarcorp.com> wrote:
>> Thanks. One down.
>>
>> Crap. It should have had a comment that closed the PR. :(
> Who should add the comment? The ticket is now closed (fixed). In the
> future patches, should I tag the ticket/PR number part of the commit
> message?
>> On 2/10/2015 2:04 PM, Hesham ALMatary wrote:
>>> ---
>>>  cpukit/score/cpu/or1k/cpu.c | 7 ++++++-
>>>  1 file changed, 6 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/cpukit/score/cpu/or1k/cpu.c b/cpukit/score/cpu/or1k/cpu.c
>>> index 9d1ae49..69845ff 100644
>>> --- a/cpukit/score/cpu/or1k/cpu.c
>>> +++ b/cpukit/score/cpu/or1k/cpu.c
>>> @@ -14,9 +14,14 @@
>>>  #include <rtems/system.h>
>>>  #include <rtems/score/isr.h>
>>>  #include <rtems/score/wkspace.h>
>>> -#include <bsp/linker-symbols.h>
>>>  #include <rtems/score/cpu.h>
>>>
>>> +/* bsp_start_vector_table_begin is the start address of the vector table
>>> + * containing addresses to ISR Handlers. It's defined at the BSP linkcmds
>>> + * and may differ from one BSP to another.
>>> + */
>>> +extern char bsp_start_vector_table_begin[];
>>> +
>>>  /**
>>>   * @brief Performs processor dependent initialization.
>>>   */
>>> -- 2.1.0
>>
>> --
>> Joel Sherrill, Ph.D.             Director of Research & Development
>> joel.sherrill at OARcorp.com        On-Line Applications Research
>> Ask me about RTEMS: a free RTOS  Huntsville AL 35805
>> Support Available                (256) 722-9985
>>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel



More information about the devel mailing list