[Buildroot] [PATCH v3 2/7] arch: Add blackfin CPU choice.

Sonic Zhang sonic.adi at gmail.com
Wed Apr 10 07:20:14 UTC 2013


On Wed, Apr 10, 2013 at 3:19 PM, Sonic Zhang <sonic.adi at gmail.com> wrote:
> Hi Arnout,
>
> On Wed, Apr 10, 2013 at 2:20 PM, Arnout Vandecappelle <arnout at mind.be> wrote:
>> On 09/04/13 08:29, Sonic Zhang wrote:
>>>
>>> From: Sonic Zhang <sonic.zhang at analog.com>
>>>
>>> v3-changes:
>>> none
>>>
>>> v2-changes:
>>> none
>>>
>>> v1-changes:
>>> - is patch adds a Target CPU configuration option and uses it to select a
>>> -m
>>> option for gcc.
>>>
>>> Signed-off-by: Sonic Zhang <sonic.zhang at analog.com>
>>> ---
>>>   arch/Config.in.bfin |   97
>>> +++++++++++++++++++++++++++++++++++++++++++++++----
>>>   1 files changed, 90 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/arch/Config.in.bfin b/arch/Config.in.bfin
>>> index 0b137ae..ac96620 100644
>>> --- a/arch/Config.in.bfin
>>> +++ b/arch/Config.in.bfin
>>> @@ -1,12 +1,65 @@
>>>   choice
>>> -       prompt "Target ABI"
>>> +       prompt "Target CPU"
>>>         depends on BR2_bfin
>>> -       default BR2_BFIN_FDPIC
>>> -config BR2_BFIN_FDPIC
>>> -       bool "FDPIC"
>>> -config BR2_BFIN_FLAT
>>> -       bool "FLAT"
>>> -       select BR2_PREFER_STATIC_LIB
>>
>>
>>  These should have been removed in the previous patch.
>>
>>> +       default BR2_bf609
>>
>>
>>  I could be wrong, but wasn't bf60x introduced relatively recently? Do all
>> toolchains support it? If not, it would be better to default to something
>> more traditional. Also, the toolchains will need conditionals to check if
>> they support the selected architecture.
>>
>
> The Blackfin toochain releases before 2012R1 should be removed from
> the buildroot toolchain makefile.
> ADI supports buildroot for blackfin only since the 2012R1 release and
> bf60x is enabled in the 2012R2 Blackfin toolchain release.
>
Sorry, should be "bf60x is enabled in the 2012R1 Blackfin toolchain release".

Regards,

Sonic


More information about the buildroot mailing list