[U-Boot] [PATCH 30/60] ARM: tegra: remove pmu.h
Stephen Warren
swarren at wwwdotorg.org
Fri Apr 22 22:42:40 CEST 2016
On 04/22/2016 12:32 PM, Simon Glass wrote:
> Hi Stephen,
>
> On 19 April 2016 at 14:59, Stephen Warren <swarren at wwwdotorg.org> wrote:
>> From: Stephen Warren <swarren at nvidia.com>
>>
>> This header is duplicated many times, and does nothing but prototype a
>> single function that's used solely by mach-tegra code. Move the proto-
>> type of mach-tegra/cpu.h. That's not an awesome location for it, but
>> other similar functions like pmic_enable_cpu_vdd() are already there.
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> I wonder if it would be better to retain the header name and move
> pmic_enable_cpu_vdd()?
I think that pmu_set_nominal() (the function moved into cpu.h by this
patch) and pmic_enable_cpu_vdd() serve essentially the same semantic
purpose, but are used on different sets of SoCs via ifdefs or weak
functions. I expect that early in the part 2 series, I'll get rid of
more code in board2.c's board_init() and put it into something like
tegra_board_init_soc(), prototyped in board_init.h or soc_init.h.
Or put another way, I expect I'll clean up the "not a great location"
issue mentioned in this patch description pretty soon.
More information about the U-Boot
mailing list