[U-Boot] [PATCH] Build system: Don't check for CONFIG_SYS_TEXT_BASE being set

Masahiro Yamada yamada.masahiro at socionext.com
Fri Feb 2 15:01:28 UTC 2018


2018-02-01 0:21 GMT+09:00 Alexey Brodkin <Alexey.Brodkin at synopsys.com>:
> Hi Masahiro-san,
>
> On Thu, 2018-02-01 at 00:18 +0900, Masahiro Yamada wrote:
>> 2018-01-31 1:28 GMT+09:00 Alexey Brodkin <Alexey.Brodkin at synopsys.com>:
>> > Hi MAsahiro-san, Tom,
>> >
>> > On Wed, 2018-01-31 at 01:23 +0900, Masahiro Yamada wrote:
>> > > 2018-01-31 0:23 GMT+09:00 Alexey Brodkin <Alexey.Brodkin at synopsys.com>:
>> > > >
>> > >
>> > > Please feel free to include historical reason:
>> > >
>> > > This ifneq conditional was added for Blackfin because it did not use
>> > > CONFIG_SYS_TEXT_BASE.
>> > > See.
>> > > https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_u-2Dboot_u-2Dboot_blob_v2017.03_arch_blackfin_config.mk-23L67&d=DwIBaQ&c=DPL6_X_
>> > > 6JkX
>> > > Fx7AXWqB0tg&r=lqdeeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I&m=mezwGj3E5EIPuE8Oln2gVeTJ-8q19KNepz7Jo-
>> > > Kcu1o&s=dmnDXQ05QLyAFKUgnsVyJOS8aybk3rhbDNsgiio6zvA&e=
>> > >
>> > > Then Blackfin is gone, so should be OK.
>> >
>> > Should I send a respin with this valuable addition or whoever's going to apply will
>> > fix-up this?
>> >
>> > -Alexey
>>
>>
>> Either will do for me.
>
> I'd prefer to let you do this fix-up so we don't spam both mailing list and patchwork with v2.
>
> -Alexey

I think your comment in another one is good
(http://patchwork.ozlabs.org/patch/868299/)

If you are reluctant on v2, you can ask Tom to
rephrase the log when this patch is applied.



-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list