pldmfw: Don't require send_package_data or send_component_table to be defined

Not all drivers require send_package_data or send_component_table when
updating firmware. Instead of forcing drivers to implement a stub allow
these functions to go undefined.

Signed-off-by: Lee Trager <lee@trager.us>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Reviewed-by: Simon Horman <horms@kernel.org>
Acked-by: Jacob Keller <jacob.e.keller@intel.com>
Link: https://patch.msgid.link/20250512190109.2475614-2-lee@trager.us
Signed-off-by: Paolo Abeni <pabeni@redhat.com>

diff --git a/lib/pldmfw/pldmfw.c b/lib/pldmfw/pldmfw.c
index 6264e2013f25ee1f9f729b5689c5542e8e4c2b78..b45ceb7257801aea6b7b665316fbe1b1248b97b1 100644 (file)
--- a/lib/pldmfw/pldmfw.c
+++ b/lib/pldmfw/pldmfw.c
@@ -728,6 +728,9 @@ pldm_send_package_data(struct pldmfw_priv *data)
        struct pldmfw_record *record = data->matching_record;
        const struct pldmfw_ops *ops = data->context->ops;
 
+       if (!ops->send_package_data)
+               return 0;
+
        return ops->send_package_data(data->context, record->package_data,
                                      record->package_data_len);
 }
@@ -755,6 +758,9 @@ pldm_send_component_tables(struct pldmfw_priv *data)
                if (!test_bit(index, bitmap))
                        continue;
 
+               if (!data->context->ops->send_component_table)
+                       continue;
+
                /* determine whether this is the start, middle, end, or both
                 * the start and end of the component tables
                 */