]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
staging: vc04_services: Replace VCHIQ_INSTANCE_T typedef with struct vchiq_instance
authorJamal Shareef <jamal.k.shareef@gmail.com>
Tue, 5 Nov 2019 22:55:30 +0000 (14:55 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 7 Nov 2019 15:25:23 +0000 (16:25 +0100)
Replaces  VCHIQ_INSTANCE_T typedef with struct vchiq_instance to match
kernel code style. Issue found by checkpatch.

Additionally, as part of the process renames "struct vchiq_instance_struct" to "struct vchiq_instance".

Signed-off-by: Jamal Shareef <jamal.k.shareef@gmail.com>
Link: https://lore.kernel.org/r/ed2b3076f93a920149716687b48e0c5e3ddf0569.1572994235.git.jamal.k.shareef@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.h
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_if.h
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c

index add552226eaaf90adee71a0716fbd9af7f2f9386..be8b2a71da2d375a55fc25cd8d9a37930a86767e 100644 (file)
@@ -84,7 +84,7 @@ static void suspend_timer_callback(struct timer_list *t);
 struct user_service {
        struct vchiq_service *service;
        void *userdata;
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
        char is_vchi;
        char dequeue_pending;
        char close_pending;
@@ -103,7 +103,7 @@ struct bulk_waiter_node {
        struct list_head list;
 };
 
-struct vchiq_instance_struct {
+struct vchiq_instance {
        struct vchiq_state *state;
        struct vchiq_completion_data completions[MAX_COMPLETIONS];
        int completion_insert;
@@ -177,11 +177,11 @@ vchiq_blocking_bulk_transfer(unsigned int handle, void *data,
        unsigned int size, enum vchiq_bulk_dir dir);
 
 #define VCHIQ_INIT_RETRIES 10
-enum vchiq_status vchiq_initialise(VCHIQ_INSTANCE_T *instance_out)
+enum vchiq_status vchiq_initialise(struct vchiq_instance **instance_out)
 {
        enum vchiq_status status = VCHIQ_ERROR;
        struct vchiq_state *state;
-       VCHIQ_INSTANCE_T instance = NULL;
+       struct vchiq_instance *instance = NULL;
        int i;
 
        vchiq_log_trace(vchiq_core_log_level, "%s called", __func__);
@@ -230,7 +230,7 @@ failed:
 }
 EXPORT_SYMBOL(vchiq_initialise);
 
-enum vchiq_status vchiq_shutdown(VCHIQ_INSTANCE_T instance)
+enum vchiq_status vchiq_shutdown(struct vchiq_instance *instance)
 {
        enum vchiq_status status;
        struct vchiq_state *state = instance->state;
@@ -267,12 +267,12 @@ enum vchiq_status vchiq_shutdown(VCHIQ_INSTANCE_T instance)
 }
 EXPORT_SYMBOL(vchiq_shutdown);
 
-static int vchiq_is_connected(VCHIQ_INSTANCE_T instance)
+static int vchiq_is_connected(struct vchiq_instance *instance)
 {
        return instance->connected;
 }
 
-enum vchiq_status vchiq_connect(VCHIQ_INSTANCE_T instance)
+enum vchiq_status vchiq_connect(struct vchiq_instance *instance)
 {
        enum vchiq_status status;
        struct vchiq_state *state = instance->state;
@@ -302,7 +302,7 @@ failed:
 EXPORT_SYMBOL(vchiq_connect);
 
 enum vchiq_status vchiq_add_service(
-       VCHIQ_INSTANCE_T              instance,
+       struct vchiq_instance             *instance,
        const struct vchiq_service_params *params,
        unsigned int       *phandle)
 {
@@ -341,7 +341,7 @@ enum vchiq_status vchiq_add_service(
 EXPORT_SYMBOL(vchiq_add_service);
 
 enum vchiq_status vchiq_open_service(
-       VCHIQ_INSTANCE_T              instance,
+       struct vchiq_instance             *instance,
        const struct vchiq_service_params *params,
        unsigned int       *phandle)
 {
@@ -433,7 +433,7 @@ static enum vchiq_status
 vchiq_blocking_bulk_transfer(unsigned int handle, void *data,
        unsigned int size, enum vchiq_bulk_dir dir)
 {
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
        struct vchiq_service *service;
        enum vchiq_status status;
        struct bulk_waiter_node *waiter = NULL;
@@ -516,7 +516,7 @@ vchiq_blocking_bulk_transfer(unsigned int handle, void *data,
 ***************************************************************************/
 
 static enum vchiq_status
-add_completion(VCHIQ_INSTANCE_T instance, enum vchiq_reason reason,
+add_completion(struct vchiq_instance *instance, enum vchiq_reason reason,
               struct vchiq_header *header, struct user_service *user_service,
               void *bulk_userdata)
 {
@@ -593,7 +593,7 @@ service_callback(enum vchiq_reason reason, struct vchiq_header *header,
        */
        struct user_service *user_service;
        struct vchiq_service *service;
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
        bool skip_completion = false;
 
        DEBUG_INITIALISE(g_state.local)
@@ -804,7 +804,7 @@ vchiq_ioc_queue_message(unsigned int handle,
 static long
 vchiq_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
 {
-       VCHIQ_INSTANCE_T instance = file->private_data;
+       struct vchiq_instance *instance = file->private_data;
        enum vchiq_status status = VCHIQ_SUCCESS;
        struct vchiq_service *service = NULL;
        long ret = 0;
@@ -1919,7 +1919,7 @@ vchiq_compat_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
 static int vchiq_open(struct inode *inode, struct file *file)
 {
        struct vchiq_state *state = vchiq_get_state();
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
 
        vchiq_log_info(vchiq_arm_log_level, "vchiq_open");
 
@@ -1951,7 +1951,7 @@ static int vchiq_open(struct inode *inode, struct file *file)
 
 static int vchiq_release(struct inode *inode, struct file *file)
 {
-       VCHIQ_INSTANCE_T instance = file->private_data;
+       struct vchiq_instance *instance = file->private_data;
        struct vchiq_state *state = vchiq_get_state();
        struct vchiq_service *service;
        int ret = 0;
@@ -2130,7 +2130,7 @@ vchiq_dump_platform_instances(void *dump_context)
 
        for (i = 0; i < state->unused_service; i++) {
                struct vchiq_service *service = state->services[i];
-               VCHIQ_INSTANCE_T instance;
+               struct vchiq_instance *instance;
 
                if (service && (service->base.callback == service_callback)) {
                        instance = service->instance;
@@ -2141,7 +2141,7 @@ vchiq_dump_platform_instances(void *dump_context)
 
        for (i = 0; i < state->unused_service; i++) {
                struct vchiq_service *service = state->services[i];
-               VCHIQ_INSTANCE_T instance;
+               struct vchiq_instance *instance;
 
                if (service && (service->base.callback == service_callback)) {
                        instance = service->instance;
@@ -2288,7 +2288,7 @@ vchiq_keepalive_thread_func(void *v)
        struct vchiq_arm_state *arm_state = vchiq_platform_get_arm_state(state);
 
        enum vchiq_status status;
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
        unsigned int ka_handle;
 
        struct vchiq_service_params params = {
@@ -2911,13 +2911,13 @@ vchiq_release_service_internal(struct vchiq_service *service)
 }
 
 struct vchiq_debugfs_node *
-vchiq_instance_get_debugfs_node(VCHIQ_INSTANCE_T instance)
+vchiq_instance_get_debugfs_node(struct vchiq_instance *instance)
 {
        return &instance->debugfs_node;
 }
 
 int
-vchiq_instance_get_use_count(VCHIQ_INSTANCE_T instance)
+vchiq_instance_get_use_count(struct vchiq_instance *instance)
 {
        struct vchiq_service *service;
        int use_count = 0, i;
@@ -2932,19 +2932,19 @@ vchiq_instance_get_use_count(VCHIQ_INSTANCE_T instance)
 }
 
 int
-vchiq_instance_get_pid(VCHIQ_INSTANCE_T instance)
+vchiq_instance_get_pid(struct vchiq_instance *instance)
 {
        return instance->pid;
 }
 
 int
-vchiq_instance_get_trace(VCHIQ_INSTANCE_T instance)
+vchiq_instance_get_trace(struct vchiq_instance *instance)
 {
        return instance->trace;
 }
 
 void
-vchiq_instance_set_trace(VCHIQ_INSTANCE_T instance, int trace)
+vchiq_instance_set_trace(struct vchiq_instance *instance, int trace)
 {
        struct vchiq_service *service;
        int i;
index 229dd262ec7df0a49d01fb18a26f4b5d1ffa4b90..19d2a2eefb6a473a919eeedcbb5adeae93dbb4e3 100644 (file)
@@ -162,19 +162,19 @@ vchiq_release_internal(struct vchiq_state *state,
                       struct vchiq_service *service);
 
 extern struct vchiq_debugfs_node *
-vchiq_instance_get_debugfs_node(VCHIQ_INSTANCE_T instance);
+vchiq_instance_get_debugfs_node(struct vchiq_instance *instance);
 
 extern int
-vchiq_instance_get_use_count(VCHIQ_INSTANCE_T instance);
+vchiq_instance_get_use_count(struct vchiq_instance *instance);
 
 extern int
-vchiq_instance_get_pid(VCHIQ_INSTANCE_T instance);
+vchiq_instance_get_pid(struct vchiq_instance *instance);
 
 extern int
-vchiq_instance_get_trace(VCHIQ_INSTANCE_T instance);
+vchiq_instance_get_trace(struct vchiq_instance *instance);
 
 extern void
-vchiq_instance_set_trace(VCHIQ_INSTANCE_T instance, int trace);
+vchiq_instance_set_trace(struct vchiq_instance *instance, int trace);
 
 extern void
 set_suspend_state(struct vchiq_arm_state *arm_state,
index 7cdb5d50fb4e4415f7d3123ccdeb536153a7dcab..b9d94f6b9beff25e770c4e07d0f9384dace9efd6 100644 (file)
@@ -177,7 +177,7 @@ find_service_by_port(struct vchiq_state *state, int localport)
 }
 
 struct vchiq_service *
-find_service_for_instance(VCHIQ_INSTANCE_T instance,
+find_service_for_instance(struct vchiq_instance *instance,
        unsigned int handle)
 {
        struct vchiq_service *service;
@@ -201,7 +201,7 @@ find_service_for_instance(VCHIQ_INSTANCE_T instance,
 }
 
 struct vchiq_service *
-find_closed_service_for_instance(VCHIQ_INSTANCE_T instance,
+find_closed_service_for_instance(struct vchiq_instance *instance,
        unsigned int handle)
 {
        struct vchiq_service *service;
@@ -227,7 +227,7 @@ find_closed_service_for_instance(VCHIQ_INSTANCE_T instance,
 }
 
 struct vchiq_service *
-next_service_by_instance(struct vchiq_state *state, VCHIQ_INSTANCE_T instance,
+next_service_by_instance(struct vchiq_state *state, struct vchiq_instance *instance,
                         int *pidx)
 {
        struct vchiq_service *service = NULL;
@@ -2280,7 +2280,7 @@ fail_free_handler_thread:
 struct vchiq_service *
 vchiq_add_service_internal(struct vchiq_state *state,
                           const struct vchiq_service_params *params,
-                          int srvstate, VCHIQ_INSTANCE_T instance,
+                          int srvstate, struct vchiq_instance *instance,
                           vchiq_userdata_term userdata_term)
 {
        struct vchiq_service *service;
@@ -2775,7 +2775,7 @@ vchiq_free_service_internal(struct vchiq_service *service)
 }
 
 enum vchiq_status
-vchiq_connect_internal(struct vchiq_state *state, VCHIQ_INSTANCE_T instance)
+vchiq_connect_internal(struct vchiq_state *state, struct vchiq_instance *instance)
 {
        struct vchiq_service *service;
        int i;
@@ -2811,7 +2811,7 @@ vchiq_connect_internal(struct vchiq_state *state, VCHIQ_INSTANCE_T instance)
 }
 
 enum vchiq_status
-vchiq_shutdown_internal(struct vchiq_state *state, VCHIQ_INSTANCE_T instance)
+vchiq_shutdown_internal(struct vchiq_state *state, struct vchiq_instance *instance)
 {
        struct vchiq_service *service;
        int i;
index 0339503d57d70e90579d2341970e0f89a96a93ce..05b39c3564f6f22204d850e8edbdfa9c73ac0fa4 100644 (file)
@@ -268,7 +268,7 @@ struct vchiq_service {
        short peer_version;
 
        struct vchiq_state *state;
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
 
        int service_use_count;
 
@@ -382,7 +382,7 @@ struct vchiq_state {
 
        /* Mutex protecting services */
        struct mutex mutex;
-       VCHIQ_INSTANCE_T *instance;
+       struct vchiq_instance **instance;
 
        /* Processes incoming messages */
        struct task_struct *slot_handler_thread;
@@ -495,12 +495,12 @@ extern enum vchiq_status
 vchiq_init_state(struct vchiq_state *state, struct vchiq_slot_zero *slot_zero);
 
 extern enum vchiq_status
-vchiq_connect_internal(struct vchiq_state *state, VCHIQ_INSTANCE_T instance);
+vchiq_connect_internal(struct vchiq_state *state, struct vchiq_instance *instance);
 
 extern struct vchiq_service *
 vchiq_add_service_internal(struct vchiq_state *state,
                           const struct vchiq_service_params *params,
-                          int srvstate, VCHIQ_INSTANCE_T instance,
+                          int srvstate, struct vchiq_instance *instance,
                           vchiq_userdata_term userdata_term);
 
 extern enum vchiq_status
@@ -516,7 +516,7 @@ extern void
 vchiq_free_service_internal(struct vchiq_service *service);
 
 extern enum vchiq_status
-vchiq_shutdown_internal(struct vchiq_state *state, VCHIQ_INSTANCE_T instance);
+vchiq_shutdown_internal(struct vchiq_state *state, struct vchiq_instance *instance);
 
 extern void
 remote_event_pollall(struct vchiq_state *state);
@@ -560,15 +560,15 @@ extern struct vchiq_service *
 find_service_by_port(struct vchiq_state *state, int localport);
 
 extern struct vchiq_service *
-find_service_for_instance(VCHIQ_INSTANCE_T instance,
+find_service_for_instance(struct vchiq_instance *instance,
        unsigned int handle);
 
 extern struct vchiq_service *
-find_closed_service_for_instance(VCHIQ_INSTANCE_T instance,
+find_closed_service_for_instance(struct vchiq_instance *instance,
        unsigned int handle);
 
 extern struct vchiq_service *
-next_service_by_instance(struct vchiq_state *state, VCHIQ_INSTANCE_T instance,
+next_service_by_instance(struct vchiq_state *state, struct vchiq_instance *instance,
                         int *pidx);
 
 extern void
index f217b78d95a0457e5a4cc089cb1869e2651e0d0f..89cc52211de44f79c480e5fd5d47c2b66c4630ad 100644 (file)
@@ -117,7 +117,7 @@ static const struct file_operations debugfs_log_fops = {
 
 static int debugfs_usecount_show(struct seq_file *f, void *offset)
 {
-       VCHIQ_INSTANCE_T instance = f->private;
+       struct vchiq_instance *instance = f->private;
        int use_count;
 
        use_count = vchiq_instance_get_use_count(instance);
@@ -129,7 +129,7 @@ DEFINE_SHOW_ATTRIBUTE(debugfs_usecount);
 
 static int debugfs_trace_show(struct seq_file *f, void *offset)
 {
-       VCHIQ_INSTANCE_T instance = f->private;
+       struct vchiq_instance *instance = f->private;
        int trace;
 
        trace = vchiq_instance_get_trace(instance);
@@ -148,7 +148,7 @@ static ssize_t debugfs_trace_write(struct file *file,
        size_t count, loff_t *ppos)
 {
        struct seq_file *f = (struct seq_file *)file->private_data;
-       VCHIQ_INSTANCE_T instance = f->private;
+       struct vchiq_instance *instance = f->private;
        char firstchar;
 
        if (copy_from_user(&firstchar, buffer, 1))
@@ -184,7 +184,7 @@ static const struct file_operations debugfs_trace_fops = {
 };
 
 /* add an instance (process) to the debugfs entries */
-void vchiq_debugfs_add_instance(VCHIQ_INSTANCE_T instance)
+void vchiq_debugfs_add_instance(struct vchiq_instance *instance)
 {
        char pidstr[16];
        struct dentry *top;
@@ -201,7 +201,7 @@ void vchiq_debugfs_add_instance(VCHIQ_INSTANCE_T instance)
        vchiq_instance_get_debugfs_node(instance)->dentry = top;
 }
 
-void vchiq_debugfs_remove_instance(VCHIQ_INSTANCE_T instance)
+void vchiq_debugfs_remove_instance(struct vchiq_instance *instance)
 {
        struct vchiq_debugfs_node *node =
                                vchiq_instance_get_debugfs_node(instance);
@@ -242,11 +242,11 @@ void vchiq_debugfs_deinit(void)
 {
 }
 
-void vchiq_debugfs_add_instance(VCHIQ_INSTANCE_T instance)
+void vchiq_debugfs_add_instance(struct vchiq_instance *instance)
 {
 }
 
-void vchiq_debugfs_remove_instance(VCHIQ_INSTANCE_T instance)
+void vchiq_debugfs_remove_instance(struct vchiq_instance *instance)
 {
 }
 
index 9b563d105fdb8d5465f90e52d93600af370247a4..ec2f033cdf321364168f976b07e7c642ce4941af 100644 (file)
@@ -14,8 +14,8 @@ void vchiq_debugfs_init(void);
 
 void vchiq_debugfs_deinit(void);
 
-void vchiq_debugfs_add_instance(VCHIQ_INSTANCE_T instance);
+void vchiq_debugfs_add_instance(struct vchiq_instance *instance);
 
-void vchiq_debugfs_remove_instance(VCHIQ_INSTANCE_T instance);
+void vchiq_debugfs_remove_instance(struct vchiq_instance *instance);
 
 #endif /* VCHIQ_DEBUGFS_H */
index dc3d204e76984495fd7923a07c7c69c6e1d66512..07c6a3db5ab6af7710726c7d8cb822e765953c74 100644 (file)
@@ -90,16 +90,16 @@ struct vchiq_config {
        short version_min;  /* The minimum compatible version of VCHIQ */
 };
 
-typedef struct vchiq_instance_struct *VCHIQ_INSTANCE_T;
+struct vchiq_instance;
 typedef void (*vchiq_remote_callback)(void *cb_arg);
 
-extern enum vchiq_status vchiq_initialise(VCHIQ_INSTANCE_T *pinstance);
-extern enum vchiq_status vchiq_shutdown(VCHIQ_INSTANCE_T instance);
-extern enum vchiq_status vchiq_connect(VCHIQ_INSTANCE_T instance);
-extern enum vchiq_status vchiq_add_service(VCHIQ_INSTANCE_T instance,
+extern enum vchiq_status vchiq_initialise(struct vchiq_instance **pinstance);
+extern enum vchiq_status vchiq_shutdown(struct vchiq_instance *instance);
+extern enum vchiq_status vchiq_connect(struct vchiq_instance *instance);
+extern enum vchiq_status vchiq_add_service(struct vchiq_instance *instance,
        const struct vchiq_service_params *params,
        unsigned int *pservice);
-extern enum vchiq_status vchiq_open_service(VCHIQ_INSTANCE_T instance,
+extern enum vchiq_status vchiq_open_service(struct vchiq_instance *instance,
        const struct vchiq_service_params *params,
        unsigned int *pservice);
 extern enum vchiq_status vchiq_close_service(unsigned int service);
@@ -133,9 +133,9 @@ extern void vchiq_get_config(struct vchiq_config *config);
 extern enum vchiq_status vchiq_set_service_option(unsigned int service,
        enum vchiq_service_option option, int value);
 
-extern enum vchiq_status vchiq_remote_use(VCHIQ_INSTANCE_T instance,
+extern enum vchiq_status vchiq_remote_use(struct vchiq_instance *instance,
        vchiq_remote_callback callback, void *cb_arg);
-extern enum vchiq_status vchiq_remote_release(VCHIQ_INSTANCE_T instance);
+extern enum vchiq_status vchiq_remote_release(struct vchiq_instance *instance);
 
 extern enum vchiq_status vchiq_dump_phys_mem(unsigned int service,
        void *ptr, size_t num_bytes);
index 3f052cae5bb76671e5adb8f7bdcb235a6d4a1d94..3055e94a53dae19d1d0fe0fee060be6c67627b4f 100644 (file)
@@ -440,7 +440,7 @@ EXPORT_SYMBOL(vchi_msg_hold);
 
 int32_t vchi_initialise(struct vchi_instance_handle **instance_handle)
 {
-       VCHIQ_INSTANCE_T instance;
+       struct vchiq_instance *instance;
        enum vchiq_status status;
 
        status = vchiq_initialise(&instance);
@@ -464,7 +464,7 @@ EXPORT_SYMBOL(vchi_initialise);
  ***********************************************************/
 int32_t vchi_connect(struct vchi_instance_handle *instance_handle)
 {
-       VCHIQ_INSTANCE_T instance = (VCHIQ_INSTANCE_T)instance_handle;
+       struct vchiq_instance *instance = (struct vchiq_instance *)instance_handle;
 
        return vchiq_connect(instance);
 }
@@ -483,7 +483,7 @@ EXPORT_SYMBOL(vchi_connect);
  ***********************************************************/
 int32_t vchi_disconnect(struct vchi_instance_handle *instance_handle)
 {
-       VCHIQ_INSTANCE_T instance = (VCHIQ_INSTANCE_T)instance_handle;
+       struct vchiq_instance *instance = (struct vchiq_instance *)instance_handle;
 
        return vchiq_status_to_vchi(vchiq_shutdown(instance));
 }
@@ -565,7 +565,7 @@ done:
        return VCHIQ_SUCCESS;
 }
 
-static struct shim_service *service_alloc(VCHIQ_INSTANCE_T instance,
+static struct shim_service *service_alloc(struct vchiq_instance *instance,
        struct service_creation *setup)
 {
        struct shim_service *service = kzalloc(sizeof(struct shim_service), GFP_KERNEL);
@@ -597,7 +597,7 @@ int32_t vchi_service_open(struct vchi_instance_handle *instance_handle,
        struct service_creation *setup,
        struct vchi_service_handle **handle)
 {
-       VCHIQ_INSTANCE_T instance = (VCHIQ_INSTANCE_T)instance_handle;
+       struct vchiq_instance *instance = (struct vchiq_instance *)instance_handle;
        struct shim_service *service = service_alloc(instance, setup);
 
        *handle = (struct vchi_service_handle *)service;