On Fri, 10 Jul 2020 at 04:51, Rayagonda Kokatanur
<[email protected]> wrote:
>
> From: Vikas Gupta <[email protected]>
>
> Add optee based bnxt fw load driver.
> bnxt is Broadcom NetXtreme controller Ethernet card.
> This driver is used to load bnxt firmware binary using OpTEE.
>
> Signed-off-by: Vikas Gupta <[email protected]>
> Signed-off-by: Rayagonda Kokatanur <[email protected]>
> ---
> Changes from v3:
>  -Address review comments from Simon,
>   Rearrange code and remove while loop,
>   Add comments for function.
>
> Changes from v2:
>  -Address review comments from Simon,
>   Remove own return code and use standard error code.
>   Take out common lines from different functions and move them
>   into common static function.
>   Remove include <common.h> as its not required.
>   Move functions with printf from header file into c file.
>
>  drivers/tee/Kconfig                |   1 +
>  drivers/tee/Makefile               |   1 +
>  drivers/tee/broadcom/Kconfig       |   7 ++
>  drivers/tee/broadcom/Makefile      |   3 +
>  drivers/tee/broadcom/chimp_optee.c | 180 +++++++++++++++++++++++++++++
>  include/broadcom/chimp.h           |  43 +++++++
>  6 files changed, 235 insertions(+)
>  create mode 100644 drivers/tee/broadcom/Kconfig
>  create mode 100644 drivers/tee/broadcom/Makefile
>  create mode 100644 drivers/tee/broadcom/chimp_optee.c
>  create mode 100644 include/broadcom/chimp.h

Reviewed-by: Simon Glass <[email protected]>

Blank line before last 'return' please.

Reply via email to