[U-Boot] [PATCH v1 4/4] ARM: DRA7: fastboot: Implement reboot-bootloader command

Nishanth Menon nm at ti.com
Tue Mar 3 18:46:47 CET 2015


On 03/03/2015 11:31 AM, Dileep Katta wrote:
> On 3 March 2015 at 19:45, Rob Herring <robherring2 at gmail.com> wrote:
> 
>> On Tue, Mar 3, 2015 at 5:10 AM, Dileep Katta <dileep.katta at linaro.org>
>> wrote:

[...]
>>> diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
>>> index 48199bf..00791be 100644
>>> --- a/common/cmd_bootm.c
>>> +++ b/common/cmd_bootm.c
>>> @@ -729,6 +729,11 @@ int do_booti(cmd_tbl_t *cmdtp, int flag, int argc,
>> char * const argv[])
>>>  {
>>>         int ret;
>>>
>>> +#if defined(CONFIG_CMD_FASTBOOT)
>>> +       if (!fb_check_reboot_flag())
>>> +               do_fastboot(cmdtp, flag, argc, argv);
>>> +#endif
>>
>> Part of the booti command? First, booti is for arm64 Image files. How
>> does this work for you?
> 
> This was ported from an existing DRA7 implementation as per latest u-boot.

Ummm... I am curious and wonder why the authorship changed if it was
just a "port"? Might be nice to point to original source where this
was picked up from
http://git.omapzoom.org/?p=repo/u-boot.git;a=shortlog;h=refs/heads/p-ti-u-boot-2014.07

For example patch #2
http://git.omapzoom.org/?p=repo/u-boot.git;a=commit;h=5df0356d4c34611d32f905dea51e9c908075eec1
I cant see much of a change here to have an authorship change perhaps?

Do I understand that Angela is ok with the same? please confirm.

> Should have sent it
> as RFC patch, as this is not verified due to unavailability of board.

Hmm... interesting.... maybe Vish can help here?

[...]

-- 
Regards,
Nishanth Menon


More information about the U-Boot mailing list