[PATCH 01/16] staging: vchiq_core: rework vchiq_get_config

[Nicolas Saenz Julienne]

The function is overly complicated for what it's ultimately achieving.
It's simply filling up a structure.

Signed-off-by: Nicolas Saenz Julienne <nsaenzjulie...@suse.de>
---
 .../interface/vchiq_arm/vchiq_arm.c           | 12 ++++----
 .../interface/vchiq_arm/vchiq_core.c          | 30 +++++--------------
 .../interface/vchiq_arm/vchiq_if.h            |  3 +-
 3 files changed, 14 insertions(+), 31 deletions(-)

diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c 
b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index ea789376de0f..6d503392341e 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -1480,13 +1480,11 @@ vchiq_ioctl(struct file *file, unsigned int cmd, 
unsigned long arg)
                        ret = -EINVAL;
                        break;
                }
-               status = vchiq_get_config(instance, args.config_size, &config);
-               if (status == VCHIQ_SUCCESS) {
-                       if (copy_to_user((void __user *)args.pconfig,
-                                   &config, args.config_size) != 0) {
-                               ret = -EFAULT;
-                               break;
-                       }
+
+               vchiq_get_config(&config);
+               if (copy_to_user(args.pconfig, &config, args.config_size)) {
+                       ret = -EFAULT;
+                       break;
                }
        } break;
 
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c 
b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
index 7642ced31436..89f1ccdc3b98 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
@@ -3583,28 +3583,14 @@ vchiq_get_peer_version(VCHIQ_SERVICE_HANDLE_T handle, 
short *peer_version)
        return status;
 }
 
-VCHIQ_STATUS_T
-vchiq_get_config(VCHIQ_INSTANCE_T instance,
-       int config_size, VCHIQ_CONFIG_T *pconfig)
-{
-       VCHIQ_CONFIG_T config;
-
-       (void)instance;
-
-       config.max_msg_size           = VCHIQ_MAX_MSG_SIZE;
-       config.bulk_threshold         = VCHIQ_MAX_MSG_SIZE;
-       config.max_outstanding_bulks  = VCHIQ_NUM_SERVICE_BULKS;
-       config.max_services           = VCHIQ_MAX_SERVICES;
-       config.version                = VCHIQ_VERSION;
-       config.version_min            = VCHIQ_VERSION_MIN;
-
-       if (config_size > sizeof(VCHIQ_CONFIG_T))
-               return VCHIQ_ERROR;
-
-       memcpy(pconfig, &config,
-               min(config_size, (int)(sizeof(VCHIQ_CONFIG_T))));
-
-       return VCHIQ_SUCCESS;
+void vchiq_get_config(VCHIQ_CONFIG_T *config)
+{
+       config->max_msg_size           = VCHIQ_MAX_MSG_SIZE;
+       config->bulk_threshold         = VCHIQ_MAX_MSG_SIZE;
+       config->max_outstanding_bulks  = VCHIQ_NUM_SERVICE_BULKS;
+       config->max_services           = VCHIQ_MAX_SERVICES;
+       config->version                = VCHIQ_VERSION;
+       config->version_min            = VCHIQ_VERSION_MIN;
 }
 
 VCHIQ_STATUS_T
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_if.h 
b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_if.h
index e4109a83e628..87829a244465 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_if.h
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_if.h
@@ -164,8 +164,7 @@ extern VCHIQ_STATUS_T 
vchiq_bulk_receive_handle(VCHIQ_SERVICE_HANDLE_T service,
 extern int   vchiq_get_client_id(VCHIQ_SERVICE_HANDLE_T service);
 extern void *vchiq_get_service_userdata(VCHIQ_SERVICE_HANDLE_T service);
 extern int   vchiq_get_service_fourcc(VCHIQ_SERVICE_HANDLE_T service);
-extern VCHIQ_STATUS_T vchiq_get_config(VCHIQ_INSTANCE_T instance,
-       int config_size, VCHIQ_CONFIG_T *pconfig);
+extern void vchiq_get_config(VCHIQ_CONFIG_T *config);
 extern VCHIQ_STATUS_T vchiq_set_service_option(VCHIQ_SERVICE_HANDLE_T service,
        VCHIQ_SERVICE_OPTION_T option, int value);
 
-- 
2.19.1