pdr: Stabilise pldm_pdr_add_fru_record_set_check()
pldm_pdr_add_fru_record_set_check() is a replacement for
pldm_pdr_add_fru_record_set(). The latter uses assert() to sanitize its
arguments, while the former instead returns a value indicating success
or failure.
Use of pldm_pdr_add_fru_record_set_check() is demonstrated in the
following patch:
https://gerrit.openbmc.org/c/openbmc/pldm/+/64631
Additionally, deprecate pldm_pdr_add_fru_record_set() now that there is
a stable replacement.
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I706b2a8ed6c5468390b1e4a7b14fa60509dd9c37
diff --git a/abi/aarch64/gcc.dump b/abi/aarch64/gcc.dump
index b931b27..6b6ad30 100644
--- a/abi/aarch64/gcc.dump
+++ b/abi/aarch64/gcc.dump
@@ -6860,6 +6860,41 @@
'Return' => '88965',
'ShortName' => 'pldm_pdr_fru_record_set_find_by_rsi'
},
+ '89574' => {
+ 'Header' => 'pdr.h',
+ 'Param' => {
+ '0' => {
+ 'name' => 'repo',
+ 'type' => '81757'
+ },
+ '1' => {
+ 'name' => 'terminus_handle',
+ 'type' => '4895'
+ },
+ '2' => {
+ 'name' => 'fru_rsi',
+ 'type' => '4895'
+ },
+ '3' => {
+ 'name' => 'entity_type',
+ 'type' => '4895'
+ },
+ '4' => {
+ 'name' => 'entity_instance_num',
+ 'type' => '4895'
+ },
+ '5' => {
+ 'name' => 'container_id',
+ 'type' => '4895'
+ },
+ '6' => {
+ 'name' => 'bmc_record_handle',
+ 'type' => '2039'
+ }
+ },
+ 'Return' => '74',
+ 'ShortName' => 'pldm_pdr_add_fru_record_set_check'
+ },
'8973' => {
'Header' => 'bios.h',
'Param' => {
@@ -7779,6 +7814,7 @@
'pldm_pdr_add' => 1,
'pldm_pdr_add_check' => 1,
'pldm_pdr_add_fru_record_set' => 1,
+ 'pldm_pdr_add_fru_record_set_check' => 1,
'pldm_pdr_destroy' => 1,
'pldm_pdr_find_record' => 1,
'pldm_pdr_find_record_by_type' => 1,
diff --git a/abi/x86_64/gcc.dump b/abi/x86_64/gcc.dump
index fc68fa4..ddd10b6 100644
--- a/abi/x86_64/gcc.dump
+++ b/abi/x86_64/gcc.dump
@@ -7441,6 +7441,42 @@
'Return' => '90223',
'ShortName' => 'pldm_pdr_fru_record_set_find_by_rsi'
},
+ '90855' => {
+ 'Header' => 'pdr.h',
+ 'Param' => {
+ '0' => {
+ 'name' => 'repo',
+ 'type' => '82972'
+ },
+ '1' => {
+ 'name' => 'terminus_handle',
+ 'type' => '5053'
+ },
+ '2' => {
+ 'name' => 'fru_rsi',
+ 'type' => '5053'
+ },
+ '3' => {
+ 'name' => 'entity_type',
+ 'type' => '5053'
+ },
+ '4' => {
+ 'name' => 'entity_instance_num',
+ 'type' => '5053'
+ },
+ '5' => {
+ 'name' => 'container_id',
+ 'type' => '5053'
+ },
+ '6' => {
+ 'name' => 'bmc_record_handle',
+ 'offset' => '0',
+ 'type' => '2087'
+ }
+ },
+ 'Return' => '74',
+ 'ShortName' => 'pldm_pdr_add_fru_record_set_check'
+ },
'9094' => {
'Header' => 'bios.h',
'Param' => {
@@ -8275,6 +8311,7 @@
'pldm_pdr_add' => 1,
'pldm_pdr_add_check' => 1,
'pldm_pdr_add_fru_record_set' => 1,
+ 'pldm_pdr_add_fru_record_set_check' => 1,
'pldm_pdr_destroy' => 1,
'pldm_pdr_find_record' => 1,
'pldm_pdr_find_record_by_type' => 1,