Revert "firmware: arm_scmi: Add a common helper to check if a message is supported"

This reverts commit 67a50f5721 which is
commit 637b6d6cae9c42db5a9525da67c991294924d2cd upstream.

It breaks the Android kernel abi and can be brought back in the future
in an abi-safe way if it is really needed.

Bug: 161946584
Change-Id: I82ce5a24c8c9d1459b6222c30c330b393340fdbe
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>

drivers/firmware/arm_scmi/driver.c

@@ -1892,7 +1892,6 @@ static const struct scmi_proto_helpers_ops helpers_ops = {
 	.get_max_msg_size = scmi_common_get_max_msg_size,
 	.iter_response_init = scmi_iterator_init,
 	.iter_response_run = scmi_iterator_run,
-	.protocol_msg_check = scmi_protocol_msg_check,
 	.fastchannel_init = scmi_common_fastchannel_init,
 	.fastchannel_db_ring = scmi_common_fastchannel_db_ring,
 };

drivers/firmware/arm_scmi/protocols.h

@@ -253,8 +253,6 @@ struct scmi_fc_info {
  *			provided in @ops.
  * @iter_response_run: A common helper to trigger the run of a previously
  *		       initialized iterator.
- * @protocol_msg_check: A common helper to check is a specific protocol message
- *			is supported.
  * @fastchannel_init: A common helper used to initialize FC descriptors by
  *		      gathering FC descriptions from the SCMI platform server.
  * @fastchannel_db_ring: A common helper to ring a FC doorbell.
@@ -269,8 +267,6 @@ struct scmi_proto_helpers_ops {
 				    unsigned int max_resources, u8 msg_id,
 				    size_t tx_size, void *priv);
 	int (*iter_response_run)(void *iter);
-	int (*protocol_msg_check)(const struct scmi_protocol_handle *ph,
-				  u32 message_id, u32 *attributes);
 	void (*fastchannel_init)(const struct scmi_protocol_handle *ph,
 				 u8 describe_id, u32 message_id,
 				 u32 valid_size, u32 domain,