[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