From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Wed, 5 Jul 2017 12:04:15 +0200
Subject: [U-Boot] [PATCH v1] avr32: Retire AVR32 for good
In-Reply-To: <CAHp75Vev3XCdC+TkZu+1u5nCZ1gK7-aVq2_HmpGpLg09FO7+=w@mail.gmail.com>
References: <20170704162543.8267-1-andriy.shevchenko@linux.intel.com>
 <595C582A.8080202@denx.de>
 <CAHp75Vev3XCdC+TkZu+1u5nCZ1gK7-aVq2_HmpGpLg09FO7+=w@mail.gmail.com>
Message-ID: <595CB99F.7020802@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Andy,

Am 05.07.2017 um 11:49 schrieb Andy Shevchenko:
> On Wed, Jul 5, 2017 at 6:08 AM, Heiko Schocher <hs@denx.de> wrote:
>> Am 04.07.2017 um 18:25 schrieb Andy Shevchenko:
>>>
>>> AVR32 is gone. It's already more than two years for no support in
>>> Buildroot,
>>> even longer there is no support in GCC (last version is heavily patched
>>> 4.2.4).
>>>
>>> Linux kernel v4.12 got rid of it (and v4.11 didn't build successfully).
>>>
>>> There is no good point to keep this support in U-Boot either.
>
>> It seems you do not remove all avr32 specific defines here from this
>> list, for example:
>>
>> pollux:u-boot hs [master] $ grep -lr SYS_CLKDIV_CPU .
>> ./arch/avr32/cpu/at32ap700x/clk.c
>> ./arch/avr32/cpu/cpu.c
>> ./arch/avr32/include/asm/arch-at32ap700x/clk.h
>> ./scripts/config_whitelist.txt
>> ./include/configs/atstk1002.h
>> ./include/configs/grasshopper.h
>> ./include/configs/atngw100.h
>> ./include/configs/atngw100mkii.h
>> pollux:u-boot hs [master] $
>>
>> Can you please try to check here the defines you remove for
>> example in arch/avr32/cpu?
>
> Thanks, I would use advice from Tom.

Fine.

>> Oh... you do not remove the "grasshoper" board ... it is also
>> an AVR32 based board:
>>
>> pollux:u-boot hs [master] $ cat configs/grasshopper_defconfig | grep AVR32
>> CONFIG_AVR32=y
>> pollux:u-boot hs [master] $
>
>> Ok, you only missed the grasshoper board, can you please remove this
>> too?
>
>>>From the above patch:
> "delete mode 100644 configs/grasshopper_defconfig"
>
> What did you miss?

Ah, it was too early in the morning ...

You only remove configs/grasshopper_defconfig, but not:

./include/configs/grasshopper.h

nor the files in

./board/in-circuit/grasshopper

and as there is only one in-circuit board, please delete the complete
./board/in-circuit directory.

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany