A-Team_Digital_Solutions/android_kernel_motorola_sm6375
Watch
2
0
Fork
You've already forked android_kernel_motorola_sm6375
 0
mirror of https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git synced 2026-10-14 16:46:15 -04:00
Code Issues Projects Releases Packages Wiki Activity

greybus: camera: Improve module registration mechanism

Browse source 
Registering more then one module at same time was not
possible with previous implementation. Also unregistering
of the module was missing leading to many instability issues
when camera module is ejected when camera is still active.

Signed-off-by: Gjorgji Rosikopulos <grosikopulos@mm-sol.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
This commit is contained in:
Gjorgji Rosikopulos 2016-03-14 18:44:53 +02:00 • committed by Greg Kroah-Hartman
commit c3d77f7130
2 changed files with 29 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

25
drivers/staging/greybus/camera.c
View file

@ -38,6 +38,7 @@ struct gb_camera_debugfs_buffer {
* @connection: the greybus connection for camera control
* @data_connected: whether the data connection has been established
* @debugfs: debugfs entries for camera protocol operations testing
* @module: Greybus camera module registered to HOST processor.
*/
struct gb_camera {
struct gb_connection *connection;
@ -47,6 +48,8 @@ struct gb_camera {
struct dentry *root;
struct gb_camera_debugfs_buffer *buffers;
} debugfs;
struct gb_camera_module module;
};
struct gb_camera_stream_config {
@ -504,16 +507,20 @@ static int gb_camera_op_flush(void *priv, u32 *request_id)
return gb_camera_flush(priv, request_id);
}
struct gb_camera_ops gb_cam_ops = {
.capabilities = gb_camera_op_capabilities,
.configure_streams = gb_camera_op_configure_streams,
.capture = gb_camera_op_capture,
.flush = gb_camera_op_flush,
};
static int gb_camera_register_intf_ops(struct gb_camera *gcam)
{
return gb_camera_register(&gb_cam_ops, gcam);
gcam->module.priv = gcam;
gcam->module.ops.capabilities = gb_camera_op_capabilities;
gcam->module.ops.configure_streams = gb_camera_op_configure_streams;
gcam->module.ops.capture = gb_camera_op_capture;
gcam->module.ops.flush = gb_camera_op_flush;
return gb_camera_register(&gcam->module);
}
static int gb_camera_unregister_intf_ops(struct gb_camera *gcam)
{
return gb_camera_unregister(&gcam->module);
}
/* -----------------------------------------------------------------------------
@ -931,6 +938,8 @@ static void gb_camera_connection_exit(struct gb_connection *connection)
{
struct gb_camera *gcam = connection->private;
gb_camera_unregister_intf_ops(gcam);
gb_camera_cleanup(gcam);
}

15
drivers/staging/greybus/gb-camera.h
View file

@ -34,9 +34,18 @@ struct gb_camera_ops {
int (*flush)(void *priv, u32 *request_id);
};
#define gb_camera_call(f, p, op, args...) \
(((f)->op) ? (f)->op(p, ##args) : -ENOIOCTLCMD)
struct gb_camera_module {
void *priv;
struct gb_camera_ops ops;
int gb_camera_register(struct gb_camera_ops *ops, void *priv);
struct list_head list; /* Global list */
};
#define gb_camera_call(f, op, args...) \
((!(f) ? -ENODEV : ((f)->ops.op) ? \
(f)->ops.op((f)->priv, ##args) : -ENOIOCTLCMD))
int gb_camera_register(struct gb_camera_module *module);
int gb_camera_unregister(struct gb_camera_module *module);
#endif /* __GB_CAMERA_H */