Skip to content

Commit 363bad0

Browse files
SebastianBoecfriedt
authored andcommitted
soc: nordic: ironside: Clean up error code docs
Clean up error code docs. Signed-off-by: Sebastian Bøe <[email protected]>
1 parent 377a18c commit 363bad0

File tree

4 files changed

+19
-6
lines changed

4 files changed

+19
-6
lines changed

soc/nordic/ironside/include/nrf_ironside/cpuconf.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,10 +63,10 @@ BUILD_ASSERT(IRONSIDE_CPUCONF_NUM_ARGS <= NRF_IRONSIDE_CALL_NUM_ARGS);
6363
* If the given msg_size is less than that, the remaining bytes are set to zero.
6464
*
6565
* @retval 0 on success or if the CPU has already booted.
66-
* @retval Positive non-0 error status if reported by IronSide call.
67-
* @retval -IRONSIDE_CPUCONF_ERROR_WRONG_CPU if cpu is unrecognized
66+
* @retval -IRONSIDE_CPUCONF_ERROR_WRONG_CPU if cpu is unrecognized.
6867
* @retval -IRONSIDE_CPUCONF_ERROR_MESSAGE_TOO_LARGE if msg_size is greater than
6968
* IRONSIDE_CPUCONF_SERVICE_MSG_MAX_SIZE.
69+
* @retval Positive error status if reported by IronSide call (see error codes in @ref call.h).
7070
*/
7171
int ironside_cpuconf(NRF_PROCESSORID_Type cpu, const void *vector_table, bool cpu_wait,
7272
const uint8_t *msg, size_t msg_size);

soc/nordic/ironside/include/nrf_ironside/dvfs.h

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,19 @@ enum ironside_dvfs_oppoint {
6868
* specified value. It will block until the change is applied.
6969
*
7070
* @param dvfs_oppoint The new DVFS oppoint to set.
71-
* @return int 0 on success, negative error code on failure.
71+
*
72+
* @retval 0 on success.
73+
* @retval -IRONSIDE_DVFS_ERROR_WRONG_OPPOINT if the requested DVFS oppoint is not allowed.
74+
* @retval -IRONSIDE_DVFS_ERROR_BUSY if waiting for mutex lock timed out, or hardware is busy.
75+
* @retval -IRONSIDE_DVFS_ERROR_OPPOINT_DATA if there is configuration error in the DVFS service.
76+
* @retval -IRONSIDE_DVFS_ERROR_PERMISSION if the caller does not have permission to change the DVFS
77+
* oppoint.
78+
* @retval -IRONSIDE_DVFS_ERROR_NO_CHANGE_NEEDED if the requested DVFS oppoint is already set.
79+
* @retval -IRONSIDE_DVFS_ERROR_TIMEOUT if the operation timed out, possibly due to a hardware
80+
* issue.
81+
* @retval -IRONSIDE_DVFS_ERROR_ISR_NOT_ALLOWED if the DVFS oppoint change operation is not allowed
82+
* in the ISR context.
83+
* @retval Positive error status if reported by IronSide call (see error codes in @ref call.h).
7284
*/
7385
int ironside_dvfs_change_oppoint(enum ironside_dvfs_oppoint dvfs_oppoint);
7486

soc/nordic/ironside/include/nrf_ironside/tdd.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@ enum ironside_se_tdd_config {
3131
* @param config The configuration to be applied.
3232
*
3333
* @retval 0 on success.
34-
* @retval -IRONSIDE_SE_TDD_ERROR_EINVAL on invalid argument.
34+
* @retval -IRONSIDE_SE_TDD_SERVICE_ERROR_INVALID_CONFIG if the configuration is invalid.
35+
* @retval Positive error status if reported by IronSide call (see error codes in @ref call.h).
3536
*/
3637
int ironside_se_tdd_configure(const enum ironside_se_tdd_config config);
3738

soc/nordic/ironside/include/nrf_ironside/update.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,10 @@ struct ironside_update_blob {
6161
*
6262
* @param update Pointer to update blob
6363
*
64+
* @retval 0 on a successful request (although the update itself may still fail).
6465
* @retval -IRONSIDE_UPDATE_ERROR_NOT_PERMITTED if missing access to the update candidate.
6566
* @retval -IRONSIDE_UPDATE_ERROR_SICR_WRITE_FAILED if writing update parameters to SICR failed.
66-
* @returns Positive non-0 error status if reported by IronSide call.
67-
* @returns 0 on a successful request (although the update itself may still fail).
67+
* @retval Positive error status if reported by IronSide call (see error codes in @ref call.h).
6868
*
6969
*/
7070
int ironside_update(const struct ironside_update_blob *update);

0 commit comments

Comments
 (0)