-----Original Message----- From: Stephen Warren swarren@wwwdotorg.org Sent: Tuesday, March 17, 2020 10:18 AM To: Tom Warren TWarren@nvidia.com Cc: u-boot@lists.denx.de; Stephen Warren swarren@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@nvidia.com wrote:
From: Stephen Warren swarren@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. -----------------------------------------------------------------------------------