On 04/22/2016 12:32 PM, Simon Glass wrote:
Hi Stephen,
On 19 April 2016 at 14:59, Stephen Warren <swar...@wwwdotorg.org> wrote:
From: Stephen Warren <swar...@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 <s...@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.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
