lowRISC / lowRISC/opentitan

[sw/dif] Replace `dif_result_t` with `status_t`

Open
#14,089 2 comments 0 reactions 1 assignee Claimed by @timothytrippel View on GitHub
Component:Software Earlgrey-PROD Candidate SW:DIF Type:Cleanup
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

#14052 adds `INTO_STATUS` to convert different error types to `status_t`. Since `dif_result_t` does not have the same layout as `status_t`, `INTO_STATUS` has to handle the `dif_result_t` to `status_t` conversion separately. This also introduces a dependency from `lib:base` to `lib:dif`, which we would like to avoid.

We should
- [ ] Replace `dif_result_t` with `status_t`,
- [ ] Remove `__builtin_types_compatible_p` from `INTO_STATUS`, and
- [ ] Remove `dif:base` dependency from `lib:base`.

cc @cfrantz @arunthomas

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.