avb: Fix build when CONFIG_OPTEE_TA_AVB is disabled
authorSam Protsenko <semen.protsenko@linaro.org>
Wed, 31 Jul 2019 16:59:08 +0000 (19:59 +0300)
committerTom Rini <trini@konsulko.com>
Wed, 7 Aug 2019 19:31:04 +0000 (15:31 -0400)
When having only these AVB related configs enabled:

    CONFIG_AVB_VERIFY=y
    CONFIG_CMD_AVB=y
    CONFIG_LIBAVB=y

build fails with next errors:

    common/avb_verify.c: In function 'read_persistent_value':
    common/avb_verify.c:867:6: warning: implicit declaration of function
                                        'get_open_session'
    common/avb_verify.c:870:45: error: 'struct AvbOpsData' has no member
                                        named 'tee'
    common/avb_verify.c:894:7: warning: implicit declaration of function
                                        'invoke_func'
    common/avb_verify.c: In function 'write_persistent_value':
    common/avb_verify.c:931:45: error: 'struct AvbOpsData' has no member
                                       named 'tee'

Guard read_persistent_value() and write_persistent_value() functions
by checking if CONFIG_OPTEE_TA_AVB is enabled (as those are only used in
that case) to fix the build with mentioned configuration.

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
common/avb_verify.c

index 32034d927cf98cf087249d7f8805c854897f2bcc..36898a610f8cd1e31642cd7ba31f25dbcf2adf45 100644 (file)
@@ -851,6 +851,7 @@ static AvbIOResult get_size_of_partition(AvbOps *ops,
        return AVB_IO_RESULT_OK;
 }
 
+#ifdef CONFIG_OPTEE_TA_AVB
 static AvbIOResult read_persistent_value(AvbOps *ops,
                                         const char *name,
                                         size_t buffer_size,
@@ -968,6 +969,8 @@ free_name:
 
        return rc;
 }
+#endif
+
 /**
  * ============================================================================
  * AVB2.0 AvbOps alloc/initialisation/free