[PATCH 1/5] ARM: tegra: rework fdt_serial_tag_setup_one

Tom Warren TWarren at nvidia.com
Tue Mar 17 18:42:52 CET 2020

-----Original Message-----
From: Stephen Warren <swarren at wwwdotorg.org> 
Sent: Tuesday, March 17, 2020 10:18 AM
To: Tom Warren <TWarren at nvidia.com>
Cc: u-boot at lists.denx.de; Stephen Warren <swarren at nvidia.com>
Subject: Re: [PATCH 1/5] ARM: tegra: rework fdt_serial_tag_setup_one

External email: Use caution opening links or attachments

On 3/17/20 11:16 AM, Stephen Warren wrote:
> On 3/16/20 1:40 PM, twarren at nvidia.com wrote:
>> From: Stephen Warren <swarren at nvidia.com>
>> Reword fdt_serial_tag_setup_one() so that the types it uses aren't 
>> tied to CONFIG_SERIAL_TAG, and rename the function to better indicate 
>> its purpose. This will allow it to be re-used by future board info 
>> related code.
>>  arch/arm/mach-tegra/ft_board_info.c | 77 
>> +++++++++++++++++++++++++++++++++++++
>>  arch/arm/mach-tegra/ft_board_info.h | 23 +++++++++++
> I don't think this series was based on upstream, or there are missing 
> patches in between upstream and this series; those files don't exist 
> upstream.

Oh. This patch adds those files. But the commit description says it's reworking some existing logic. I guess the description is from some squashed patch, and doesn't describe what the patch actually does? Also, the new file isn't added to any Makefile, and no edits are made to any other file in order to call the newly added functionality.
[Tom] Yeah, this got messed up when I squashed/edited the transition from nvtboot to cboot support files in arch/arm/mach-tegra. I'll have to revisit this, as the code isn't called anywhere currently in my upstream work, nor build by a Makefile. 

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.

More information about the U-Boot mailing list