Re: [U-Boot] [PATCH v5 03/24] arm: imx: hab: Optimise flow of authenticate_image on is_enabled fail

2018-01-12 Thread Bryan O'Donoghue
On 11/01/18 19:34, Breno Matheus Lima wrote: Hi Bryan, 2018-01-08 15:36 GMT-02:00 Bryan O'Donoghue : There is no need to call is_enabled() twice in authenticate_image - it does nothing but add an additional layer of indentation. We can check for is_enabled() at

Re: [U-Boot] [PATCH v5 03/24] arm: imx: hab: Optimise flow of authenticate_image on is_enabled fail

2018-01-11 Thread Breno Matheus Lima
Hi Bryan, 2018-01-08 15:36 GMT-02:00 Bryan O'Donoghue : > There is no need to call is_enabled() twice in authenticate_image - it does > nothing but add an additional layer of indentation. > > We can check for is_enabled() at the start of the function and return the >

[U-Boot] [PATCH v5 03/24] arm: imx: hab: Optimise flow of authenticate_image on is_enabled fail

2018-01-08 Thread Bryan O'Donoghue
There is no need to call is_enabled() twice in authenticate_image - it does nothing but add an additional layer of indentation. We can check for is_enabled() at the start of the function and return the result code directly. Signed-off-by: Bryan O'Donoghue Cc: Stefano