Ping: Re: [patch, avr] Add flash size to device info and make wrap around default

[Georg-Johann Lay]

On 30.11.2016 07:27, Pitchumani Sivanupandi wrote:
> On Tuesday 29 November 2016 10:06 PM, Denis Chertykov wrote:
>> 2016-11-28 10:17 GMT+03:00 Pitchumani Sivanupandi
>> <pitchumani.sivanupandi@microchip.com>:
>>> On Saturday 26 November 2016 12:11 AM, Denis Chertykov wrote:
>>>> I'm sorry for delay.
>>>>
>>>> I have a problem with the patch:
>>>> (Stripping trailing CRs from patch; use --binary to disable.)
>>>> patching file avr-arch.h
>>>> (Stripping trailing CRs from patch; use --binary to disable.)
>>>> patching file avr-devices.c
>>>> (Stripping trailing CRs from patch; use --binary to disable.)
>>>> patching file avr-mcus.def
>>>> Hunk #1 FAILED at 62.
>>>> 1 out of 1 hunk FAILED -- saving rejects to file avr-mcus.def.rej
>>>> (Stripping trailing CRs from patch; use --binary to disable.)
>>>> patching file gen-avr-mmcu-specs.c
>>>> Hunk #1 succeeded at 215 (offset 5 lines).
>>>> (Stripping trailing CRs from patch; use --binary to disable.)
>>>> patching file specs.h
>>>> Hunk #1 succeeded at 58 (offset 1 line).
>>>> Hunk #2 succeeded at 66 (offset 1 line).
>>>
>>> There are changes in avr-mcus.def after this patch is submitted.
>>> Now, I have incorporated the changes and attached the resolved patch.
>>>
>>> Regards,
>>> Pitchumani
>>>
>>> gcc/ChangeLog
>>>
>>> 2016-11-09  Pitchumani Sivanupandi <pitchumani.s@atmel.com>
>>>
>>>      * config/avr/avr-arch.h (avr_mcu_t): Add flash_size member.
>>>      * config/avr/avr-devices.c(avr_mcu_types): Add flash size info.
>>>      * config/avr/avr-mcu.def: Likewise.
>>>      * config/avr/gen-avr-mmcu-specs.c (print_mcu): Remove hard-coded
>>> prefix
>>>      check to find wrap-around value, instead use MCU flash size. For 8k
>>> flash
>>>      devices, update link_pmem_wrap spec string to add
>>> --pmem-wrap-around=8k.
>>>      * config/avr/specs.h: Remove link_pmem_wrap from LINK_RELAX_SPEC
>>> and
>>>      add to linker specs (LINK_SPEC) directly.
>> Committed.
> It looks like only avr-mcus.def and ChangeLog are committed.
> Without the other changes trunk build is broken.
>
> Regards,
> Pitchumani

Hi, I allowed me to commit the missing files.

http://gcc.gnu.org/r243033

Johann