staging: vchiq_arm: pass vchiq instance to 'vchiq_get_service_userdata'
authorAdrien Thierry <athierry@redhat.com>
Wed, 18 May 2022 19:11:17 +0000 (15:11 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 6 Jun 2022 06:03:46 +0000 (08:03 +0200)
In order to remove the 'vchiq_states' global array, we need to pass the
vchiq_instance reference to the 'handle_to_service' function, as well as
to all functions that call 'handle_to_service'. This will allow
accessing the vchiq state through the vchiq instance instead of through
the global array.

'handle_to_service' is called by 'vchiq_get_service_userdata'.
Therefore, pass the vchiq instance reference to it.

Tested-by: Stefan Wahren <stefan.wahren@i2se.com>
Signed-off-by: Adrien Thierry <athierry@redhat.com>
Link: https://lore.kernel.org/r/20220518191126.60396-7-athierry@redhat.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
drivers/staging/vc04_services/include/linux/raspberrypi/vchiq.h
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c

index aa2ab0d..f4c2c95 100644 (file)
@@ -96,7 +96,8 @@ static enum vchiq_status audio_vchi_callback(struct vchiq_instance *vchiq_instan
                                             struct vchiq_header *header,
                                             unsigned int handle, void *userdata)
 {
-       struct bcm2835_audio_instance *instance = vchiq_get_service_userdata(handle);
+       struct bcm2835_audio_instance *instance = vchiq_get_service_userdata(vchiq_instance,
+                                                                            handle);
        struct vc_audio_msg *m;
 
        if (reason != VCHIQ_MESSAGE_AVAILABLE)
index 0d3833e..db1441c 100644 (file)
@@ -109,7 +109,7 @@ extern enum vchiq_status vchiq_bulk_transmit(struct vchiq_instance *instance, un
 extern enum vchiq_status vchiq_bulk_receive(struct vchiq_instance *instance, unsigned int service,
                                            void *data, unsigned int size, void *userdata,
                                            enum vchiq_bulk_mode mode);
-extern void *vchiq_get_service_userdata(unsigned int service);
+extern void *vchiq_get_service_userdata(struct vchiq_instance *instance, unsigned int service);
 extern enum vchiq_status vchiq_get_peer_version(struct vchiq_instance *instance,
                                                unsigned int handle,
                                                short *peer_version);
index d6853e2..d4fed18 100644 (file)
@@ -411,7 +411,7 @@ vchiq_get_client_id(struct vchiq_instance *instance, unsigned int handle)
 }
 
 void *
-vchiq_get_service_userdata(unsigned int handle)
+vchiq_get_service_userdata(struct vchiq_instance *instance, unsigned int handle)
 {
        void *userdata;
        struct vchiq_service *service;
index 4c29cb7..cb921c9 100644 (file)
@@ -553,7 +553,7 @@ static enum vchiq_status service_callback(struct vchiq_instance *vchiq_instance,
                                          struct vchiq_header *header,
                                          unsigned int handle, void *bulk_ctx)
 {
-       struct vchiq_mmal_instance *instance = vchiq_get_service_userdata(handle);
+       struct vchiq_mmal_instance *instance = vchiq_get_service_userdata(vchiq_instance, handle);
        u32 msg_len;
        struct mmal_msg *msg;
        struct mmal_msg_context *msg_context;