V4L/DVB (8028): Improve error messages for tda1004x attach

Previous thread: V4L/DVB (8027): saa7134: Avermedia A700: only s-video and composite input are working by Linux Kernel Mailing List on Sunday, June 29, 2008 - 1:03 pm. (1 message)

Next thread: V4L/DVB (8029): Improve error message at tda1004x_attach by Linux Kernel Mailing List on Sunday, June 29, 2008 - 1:03 pm. (1 message)
From: Linux Kernel Mailing List
Date: Sunday, June 29, 2008 - 1:03 pm

Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=bc36ec...
Commit:     bc36ec746409e4e4719b94a86dc0d8cbeb6f439f
Parent:     b25fed115a8bd8d39d14ce0d03e808272a401d67
Author:     Mauro Carvalho Chehab <mchehab@infradead.org>
AuthorDate: Sat Jun 14 10:44:04 2008 -0300
Committer:  Mauro Carvalho Chehab <mchehab@infradead.org>
CommitDate: Thu Jun 26 15:58:48 2008 -0300

    V4L/DVB (8028): Improve error messages for tda1004x attach
    
    A fresh copy of v.29 firmware, using get_firmware, is leading to an invalid
    firmware:
    
    DVB: registering new adapter (saa7133[0])
    DVB: registering frontend 0 (Philips TDA10046H DVB-T)...
    tda1004x: setting up plls for 48MHz sampling clock
    tda1004x: found firmware revision ff -- invalid
    tda1004x: trying to boot from eeprom
    tda1004x: found firmware revision ff -- invalid
    tda1004x: waiting for firmware upload...
    tda1004x: Error during firmware upload
    tda1004x: found firmware revision ff -- invalid
    tda1004x: firmware upload failed
    
    Sometimes, loading/unloading this firmware makes tda1004x to return an invalid
    ID. However, there were no printk messages to help to identify what were the
    cause for the error.
    
    With this patch, it will now print:
    
    Invalid tda1004x ID = 0xff. Can't proceed
    saa7133[0]/dvb: failed to attach tda10046
    saa7133[0]/dvb: frontend initialization failed
    
    Tested with LifeView FlyDVB-T Hybrid Cardbus/MSI TV @nywhere A/D NB
    
    Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
---
 drivers/media/dvb/frontends/tda1004x.c    |   18 ++++++++++++++----
 drivers/media/video/saa7134/saa7134-dvb.c |    3 ++-
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/drivers/media/dvb/frontends/tda1004x.c b/drivers/media/dvb/frontends/tda1004x.c
index 4997384..3993d1c 100644
--- a/drivers/media/dvb/frontends/tda1004x.c
+++ ...
Previous thread: V4L/DVB (8027): saa7134: Avermedia A700: only s-video and composite input are working by Linux Kernel Mailing List on Sunday, June 29, 2008 - 1:03 pm. (1 message)

Next thread: V4L/DVB (8029): Improve error message at tda1004x_attach by Linux Kernel Mailing List on Sunday, June 29, 2008 - 1:03 pm. (1 message)