On 13.03.2014 15:57, Mauro Carvalho Chehab wrote:
Em Thu, 27 Feb 2014 02:30:11 +0200
Antti Palosaari <crope@xxxxxx> escreveu:
Implement gain and bandwidth controls using v4l2 control framework.
Pointer to control handler is provided by exported symbol.
Cc: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx>
Cc: Hans Verkuil <hverkuil@xxxxxxxxx>
Signed-off-by: Antti Palosaari <crope@xxxxxx>
---
drivers/media/tuners/e4000.c | 210 +++++++++++++++++++++++++++++++++++++-
drivers/media/tuners/e4000.h | 14 +++
drivers/media/tuners/e4000_priv.h | 75 ++++++++++++++
3 files changed, 298 insertions(+), 1 deletion(-)
...
diff --git a/drivers/media/tuners/e4000.h b/drivers/media/tuners/e4000.h
index e74b8b2..989f2ea 100644
--- a/drivers/media/tuners/e4000.h
+++ b/drivers/media/tuners/e4000.h
@@ -40,4 +40,18 @@ struct e4000_config {
u32 clock;
};
+#if IS_ENABLED(CONFIG_MEDIA_TUNER_E4000)
+extern struct v4l2_ctrl_handler *e4000_get_ctrl_handler(
+ struct dvb_frontend *fe
+);
+#else
+static inline struct v4l2_ctrl_handler *e4000_get_ctrl_handler(
+ struct dvb_frontend *fe
+)
+{
+ pr_warn("%s: driver disabled by Kconfig\n", __func__);
+ return NULL;
+}
+#endif
+
#endif
There are two things to be noticed here:
1) Please don't add any EXPORT_SYMBOL() on a pure I2C module. You
should, instead, use the subdev calls, in order to callback a
function provided by the module;
That means, I have to implement it as a V4L subdev driver then...
Is there any problem to leave as it is? It just only provides control
handler using that export. If you look those existing dvb frontend or
tuner drivers there is many kind of resources exported just similarly
(example DibCom PID filters, af9033 pid filters), so I cannot see why
that should be different.
2) As you're now using request_module(), you don't need to use
#if IS_ENABLED() anymore. It is up to the module to register
itself as a V4L2 subdevice. The caller module should use the
subdevice interface to run the callbacks.
If you don't to that, you'll have several issues with the
building system.
So basically you are saying I should implement that driver as a V4L
subdev too?
regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
