On Wed, Oct 29, 2014 at 11:28 AM, Jeroen Hofstee <jer...@myspectrum.nl> wrote: > On 29-10-14 14:08, Nikita Kiryanov wrote: >> >> Introduce board specific function board_preboot_os() to allow for board >> specific config before we boot, and use it on cm_fx6 to power off sata >> so that Linux will be able to setup it correctly. >> >> Signed-off-by: Nikita Kiryanov <nik...@compulab.co.il> >> Cc: Igor Grinberg <grinb...@compulab.co.il> >> Cc: Stefano Babic <sba...@denx.de> >> Cc: Tom Rini <tr...@ti.com> > > .. >> >> + >> +void board_preboot_os(void) >> +{ >> + cm_fx6_sata_power(0); >> +} >> +/* Allow for board specific config before we boot */ >> +__weak void board_preboot_os(void) >> +{ >> + /* please define board specific board_preboot_os() */ >> +} >> + >> int boot_selected_os(int argc, char * const argv[], int state, >> bootm_headers_t *images, boot_os_fn *boot_fn) >> { >> arch_preboot_os(); >> + board_preboot_os(); >> boot_fn(state, argc, argv, images); > > > Can you also add a prototype for the board_preboot_os in say > bootm.h or similiar and make sure those headers are included. > (it will warn otherwise with W=1).
and also split the patch in two. -- Otavio Salvador O.S. Systems http://www.ossystems.com.br http://code.ossystems.com.br Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot