tspd:FWU:Fix usage of SMC_RET0
authorAntonio Nino Diaz <antonio.ninodiaz@arm.com>
Tue, 4 Apr 2017 16:08:32 +0000 (17:08 +0100)
committerAntonio Nino Diaz <antonio.ninodiaz@arm.com>
Thu, 20 Apr 2017 08:54:59 +0000 (09:54 +0100)
commit7a317a70d4829feaeaf841584aff58162d13eb07
treea3a9813cabf783a46c11676f55b84addf3b30170
parent3d21c9452d4d5e17f545848bc4641274cdd6aa26
tspd:FWU:Fix usage of SMC_RET0

SMC_RET0 should only be used when the SMC code works as a function that
returns void. If the code of the SMC uses SMC_RET1 to return a value to
signify success and doesn't return anything in case of an error (or the
other way around) SMC_RET1 should always be used to return clearly
identifiable values.

This patch fixes two cases in which the code used SMC_RET0 instead of
SMC_RET1.

It also introduces the define SMC_OK to use when an SMC must return a
value to tell that it succeeded, the same way as SMC_UNK is used in case
of failure.

Change-Id: Ie4278b51559e4262aced13bbde4e844023270582
Signed-off-by: Antonio Nino Diaz <antonio.ninodiaz@arm.com>
bl1/bl1_fwu.c
include/lib/smcc.h
services/spd/tspd/tspd_main.c