Hi Marek, On 11/07/2013 20:31, Marek Vasut wrote: >> +void display_event(uint8_t *event_data, size_t bytes) >> +{ >> + uint32_t i; >> + >> + if ((event_data) && (bytes > 0)) { > > if (!<cond>) > return; > > <The loop goes here>
Fix in V2, thanks. >> + if (is_hab_enabled()) >> + printf("\nSecure boot enabled\n"); >> + else >> + printf("\nSecure boot disabled\n"); > > Use puts() instead of printf() with no args. > Right, and I will fix also other similar cases in this file. Regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot