On 12/11/2011 10:30 PM, Mauro Carvalho Chehab wrote:
On 11-12-2011 13:34, Antti Palosaari wrote:
On 12/11/2011 12:59 PM, Mauro Carvalho Chehab wrote:
On 30-11-2011 15:24, Antti Palosaari wrote:
Morjens Mauro,
I rewrote whole af9013 demodulator driver in order to decrease I2C
load. Please pull that to the next Kernel merge window.
Antti
The following changes since commit
a235af24a74a0fa03ece0a9f5e28a72e4d1e2cad:
ce168: remove experimental from Kconfig (2011-11-19 23:07:54 +0200)
are available in the git repository at:
git://linuxtv.org/anttip/media_tree.git misc
Antti Palosaari (1):
af9013: rewrite whole driver
drivers/media/dvb/dvb-usb/af9015.c | 82 +-
drivers/media/dvb/frontends/af9013.c | 1756
++++++++++++++---------------
drivers/media/dvb/frontends/af9013.h | 113 +-
drivers/media/dvb/frontends/af9013_priv.h | 93 +-
4 files changed, 1017 insertions(+), 1027 deletions(-)
There was a minor context change here:
@@ -1156,7 +1158,7 @@ static int af9015_af9013_sleep(struct dvb_frontend
*fe)
if (mutex_lock_interruptible(&adap->dev->usb_mutex))
return -EAGAIN;
- ret = priv->init[adap->id](fe);
+ ret = priv->sleep[adap->id](fe);
Correct, that is bug fix for the earlier patch I made. As a result we
call demod .init() in case we should call .sleep() resulting demod
will never sleep. I found that very late phase, when af9013 rewrite
was almost complete. At that time I was too lazy at that point to made
separate patch for fixing it because I had made so many changes for
af9015 already.
But if you like, I can rebase whole thing and move that fix as own patch.
Basically, the current code doesn't have that mutex_lock_interruptible
logic. It
may be into the fixes we'll send for 3.2.
Do you mean I should change all mutex_lock_interruptible => mutex_lock
and send as bugfix to 3.2?
No. I just meant to say that the current code (without the patches I'm
preparing for -rc6) doesn't
have the mutex_lock_interruptible() at the context.
Anyway, I asked you to push those mutex_lock_interruptible things to
3.3 and I think you haven't send those 3.2 so not fixes for 3.2
hopefully.
Need to check on my linux-media tree, in order to be sure if those are
there or not. Maybe I've
merged it as a bug fix for 3.2.
There is one old mutex_lock_interruptible inside af9015_rw_udev(). It
have been there many years and it is copied from dvb_usb_generic_rw().
I think better to not change it as bugfix.
It is not even clear for me when to use mutex_lock_interruptible or
mutex_lock. I suspect mutex_lock is cheaper and that's why it should
be used when possible? I am happy to hear reasons and learn (and too
lazy to look through docs and codes...).
The mutex_lock_interruptible will return -EINTR if a signal is received
while
trying to handle it. Userspace application needs to be prepared for that.
Both are handled by the same routine at kernel/mutex.c.
However, after this patch, compilation broke:
drivers/media/dvb/dvb-usb/af9015.c: In function ‘af9015_rc_query’:
drivers/media/dvb/dvb-usb/af9015.c:1089:12: error: ‘struct af9015_state’
has no member named ‘sleep’
drivers/media/dvb/dvb-usb/af9015.c:1089:20: error: ‘adap’ undeclared
(first use in this function)
drivers/media/dvb/dvb-usb/af9015.c:1089:20: note: each undeclared
identifier is reported only once for each function it appears in
drivers/media/dvb/dvb-usb/af9015.c:1089:30: error: ‘fe’ undeclared
(first use in this function)
Arg, how the I hell I missed af9015.h file from that change-set.
After all, I hope it was not TLDR :) What you think I should made in
order to fix these issues correctly?
Rebase the patch ;) compilation errors break git bisect.
I suspect there is now something else wrong. Two person have reported
that patch is working and no mention about compile errors you have.
So you have missing now some af9015 patches I have sent earlier. There
patches are needed before that last one can be applied:
http://patchwork.linuxtv.org/patch/8410/
http://patchwork.linuxtv.org/patch/8411/
If you PULL all those starting from:
2011-11-19 af9015: limit I2C access to keep FW happy
http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/misc
you will get all what is needed. Some of those are applied but some not.
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
