From: Limeng <meng...@windriver.com>

There are 3 kernel modules(bcm2835_v4l2, bcm2835_codec,
bcm2835_isp) to invoke finction vchiq_mmal_init(). In this function,
there is static pointer variable vchi_instance pointing a region of
memory. Because the 3 kernel modules are loaded without dependency,
there is no fixed order. In this way, the static pointer variable may be
changed by one module when another module is initializing it.
Therefore, remove the useless static keyword.

Signed-off-by: Meng Li <meng...@windriver.com>
---
 drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c 
b/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c
index 61a3593cdcfc..cade780d90c6 100644
--- a/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c
+++ b/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c
@@ -2103,7 +2103,7 @@ int vchiq_mmal_init(struct vchiq_mmal_instance 
**out_instance)
 {
        int status;
        struct vchiq_mmal_instance *instance;
-       static VCHI_INSTANCE_T vchi_instance;
+       VCHI_INSTANCE_T vchi_instance;
        struct service_creation params = {
                .version                = VCHI_VERSION_EX(VC_MMAL_VER, 
VC_MMAL_MIN_VER),
                .service_id             = VC_MMAL_SERVER_NAME,
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#8888): 
https://lists.yoctoproject.org/g/linux-yocto/message/8888
Mute This Topic: https://lists.yoctoproject.org/mt/75840754/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to