Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, Feb 13, 2011 at 7:16 PM, Andy Walls wrote: > Devin, > > I just checked. The CX23885 driver *is* setting up to allow slaves to > stretch the clock. > > By analysis, I have confirmed that Jean's sugguested patch that I moved > forward was wrong for the hardware's behavior. When the cx23885 I2C > routines decide to set the I2C_EXTEND flag (and maybe the I2C_NOSTOP > flag), we most certainly should *not* be expecting an ACK from the > particular hardware register. The original commit should certainly be > reverted. > > Checking for slave ACK/NAK will need to be done with a little more care; > so for now, I'll settle for ignoring them. Ok, that makes sense. I just threw out the possibility of it being related to clock stretch because I've seen that with other bridges and the xc5000. I haven't really reviewed the code in question or the cx23885 datasheet. Cheers, Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
Hi Mark, On Sun, 13 Feb 2011 07:47:59 -0700, Mark Zimmerman wrote: > Clearly my previous bisection went astray; I think I have a more > sensible result this time. > > qpc$ git bisect good > 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit > commit 44835f197bf1e3f57464f23dfb239fef06cf89be > Author: Jean Delvare > Date: Sun Jul 18 16:52:05 2010 -0300 > > V4L/DVB: cx23885: Check for slave nack on all transactions > > Don't just check for nacks on zero-length transactions. Check on > other transactions too. > > Signed-off-by: Jean Delvare > Signed-off-by: Andy Walls > Signed-off-by: Mauro Carvalho Chehab I'm sorry for the trouble :( I should probably refrain from writing patches for media drivers I don't own. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, Feb 13, 2011 at 04:26:50PM -0500, Andy Walls wrote: > On Sun, 2011-02-13 at 13:26 -0700, Mark Zimmerman wrote: > > On Sun, Feb 13, 2011 at 09:52:25AM -0500, Devin Heitmueller wrote: > > > On Sun, Feb 13, 2011 at 9:47 AM, Mark Zimmerman wrote: > > > > Clearly my previous bisection went astray; I think I have a more > > > > sensible result this time. > > > > > > > > qpc$ git bisect good > > > > 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit > > > > commit 44835f197bf1e3f57464f23dfb239fef06cf89be > > > > Author: Jean Delvare > > > > Date: ? Sun Jul 18 16:52:05 2010 -0300 > > > > > > > > ? ?V4L/DVB: cx23885: Check for slave nack on all transactions > > > > > > > > ? ?Don't just check for nacks on zero-length transactions. Check on > > > > ? ?other transactions too. > > > > > > This could be a combination of the xc5000 doing clock stretching and > > > the cx23885 i2c master not properly implementing clock stretch. In > > > the past I've seen i2c masters broken in their handling of clock > > > stretching where they treat it as a NAK. > > > > > > The xc5000 being one of the few devices that actually does i2c clock > > > stretching often exposes cases where it is improperly implemented in > > > the i2c master driver (I've had to fix this with several bridges). > > > > > > > Thanks for your insight. I am looking at cx23885-i2c.c and there is no > > clock stretching logic in i2c_slave_did_ack(). Would this be the > > right place for it to be? Can you point me to an example of another > > driver that does it correctly? I really don't know what I am doing... > > > Mark, > > You don't have much hope of getting that right without the CX23885 > datasheet. > > Let's just get the bad commit reverted and into 2.6.38, and fix what > used to work for you. Doing a git bisect is enough work for anyone. > > I'll do a patch to revert the commit and ask it to be pulled for > 2.6.38-rc-whatever. I'll be sure to add a > > Bisected-by: Mark Zimmerman > > tag to the patch. (The Linux Kernel devs understand the work involved > to do a bisection.) > > > Later, if I can work up a patch to deal with clock stretching properly, > I may ask you to test. > Thanks, that would be great. Meanwhile, I have built a 2.6.37 with the offending commit removed: git bisect reset git checkout v2.6.37 git revert 44835f197bf1e3f57464f23dfb239fef06cf89be and it seems to be working fine using both tuners: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: firmware upload complete... xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: firmware upload complete... Thanks again -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, 2011-02-13 at 16:26 -0500, Andy Walls wrote: > On Sun, 2011-02-13 at 13:26 -0700, Mark Zimmerman wrote: > > On Sun, Feb 13, 2011 at 09:52:25AM -0500, Devin Heitmueller wrote: > > > On Sun, Feb 13, 2011 at 9:47 AM, Mark Zimmerman wrote: > > > > Clearly my previous bisection went astray; I think I have a more > > > > sensible result this time. > > > > > > > > qpc$ git bisect good > > > > 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit > > > > commit 44835f197bf1e3f57464f23dfb239fef06cf89be > > > > Author: Jean Delvare > > > > Date: ? Sun Jul 18 16:52:05 2010 -0300 > > > > > > > > ? ?V4L/DVB: cx23885: Check for slave nack on all transactions > > > > > > > > ? ?Don't just check for nacks on zero-length transactions. Check on > > > > ? ?other transactions too. > > > > > > This could be a combination of the xc5000 doing clock stretching and > > > the cx23885 i2c master not properly implementing clock stretch. In > > > the past I've seen i2c masters broken in their handling of clock > > > stretching where they treat it as a NAK. > > > > > > The xc5000 being one of the few devices that actually does i2c clock > > > stretching often exposes cases where it is improperly implemented in > > > the i2c master driver (I've had to fix this with several bridges). > > > Devin, I just checked. The CX23885 driver *is* setting up to allow slaves to stretch the clock. By analysis, I have confirmed that Jean's sugguested patch that I moved forward was wrong for the hardware's behavior. When the cx23885 I2C routines decide to set the I2C_EXTEND flag (and maybe the I2C_NOSTOP flag), we most certainly should *not* be expecting an ACK from the particular hardware register. The original commit should certainly be reverted. Checking for slave ACK/NAK will need to be done with a little more care; so for now, I'll settle for ignoring them. Regards, Andy -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, 2011-02-13 at 13:26 -0700, Mark Zimmerman wrote: > On Sun, Feb 13, 2011 at 09:52:25AM -0500, Devin Heitmueller wrote: > > On Sun, Feb 13, 2011 at 9:47 AM, Mark Zimmerman wrote: > > > Clearly my previous bisection went astray; I think I have a more > > > sensible result this time. > > > > > > qpc$ git bisect good > > > 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit > > > commit 44835f197bf1e3f57464f23dfb239fef06cf89be > > > Author: Jean Delvare > > > Date: ? Sun Jul 18 16:52:05 2010 -0300 > > > > > > ? ?V4L/DVB: cx23885: Check for slave nack on all transactions > > > > > > ? ?Don't just check for nacks on zero-length transactions. Check on > > > ? ?other transactions too. > > > > This could be a combination of the xc5000 doing clock stretching and > > the cx23885 i2c master not properly implementing clock stretch. In > > the past I've seen i2c masters broken in their handling of clock > > stretching where they treat it as a NAK. > > > > The xc5000 being one of the few devices that actually does i2c clock > > stretching often exposes cases where it is improperly implemented in > > the i2c master driver (I've had to fix this with several bridges). > > > > Thanks for your insight. I am looking at cx23885-i2c.c and there is no > clock stretching logic in i2c_slave_did_ack(). Would this be the > right place for it to be? Can you point me to an example of another > driver that does it correctly? I really don't know what I am doing... Mark, You don't have much hope of getting that right without the CX23885 datasheet. Let's just get the bad commit reverted and into 2.6.38, and fix what used to work for you. Doing a git bisect is enough work for anyone. I'll do a patch to revert the commit and ask it to be pulled for 2.6.38-rc-whatever. I'll be sure to add a Bisected-by: Mark Zimmerman tag to the patch. (The Linux Kernel devs understand the work involved to do a bisection.) Later, if I can work up a patch to deal with clock stretching properly, I may ask you to test. Regards, Andy -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, Feb 13, 2011 at 09:52:25AM -0500, Devin Heitmueller wrote: > On Sun, Feb 13, 2011 at 9:47 AM, Mark Zimmerman wrote: > > Clearly my previous bisection went astray; I think I have a more > > sensible result this time. > > > > qpc$ git bisect good > > 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit > > commit 44835f197bf1e3f57464f23dfb239fef06cf89be > > Author: Jean Delvare > > Date: ? Sun Jul 18 16:52:05 2010 -0300 > > > > ? ?V4L/DVB: cx23885: Check for slave nack on all transactions > > > > ? ?Don't just check for nacks on zero-length transactions. Check on > > ? ?other transactions too. > > This could be a combination of the xc5000 doing clock stretching and > the cx23885 i2c master not properly implementing clock stretch. In > the past I've seen i2c masters broken in their handling of clock > stretching where they treat it as a NAK. > > The xc5000 being one of the few devices that actually does i2c clock > stretching often exposes cases where it is improperly implemented in > the i2c master driver (I've had to fix this with several bridges). > Thanks for your insight. I am looking at cx23885-i2c.c and there is no clock stretching logic in i2c_slave_did_ack(). Would this be the right place for it to be? Can you point me to an example of another driver that does it correctly? I really don't know what I am doing... -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, Feb 13, 2011 at 9:47 AM, Mark Zimmerman wrote: > Clearly my previous bisection went astray; I think I have a more > sensible result this time. > > qpc$ git bisect good > 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit > commit 44835f197bf1e3f57464f23dfb239fef06cf89be > Author: Jean Delvare > Date: Sun Jul 18 16:52:05 2010 -0300 > > V4L/DVB: cx23885: Check for slave nack on all transactions > > Don't just check for nacks on zero-length transactions. Check on > other transactions too. This could be a combination of the xc5000 doing clock stretching and the cx23885 i2c master not properly implementing clock stretch. In the past I've seen i2c masters broken in their handling of clock stretching where they treat it as a NAK. The xc5000 being one of the few devices that actually does i2c clock stretching often exposes cases where it is improperly implemented in the i2c master driver (I've had to fix this with several bridges). Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[corrected get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, Feb 12, 2011 at 08:29:54AM -0700, Mark Zimmerman wrote: > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > Greetings: > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but > > which fails to initialize with the latest 2.6.36 kernel. The firmware > > fails to load due to an i2c failure. A search of the archives indicates > > that this is not the first time this issue has occurred. > > > > What can I do to help get this problem fixed? > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: firmware upload complete... > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: firmware upload complete.. > > > > and here is what happens with 2.6.36: > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: I2C write failed (len=3) > > xc5000: firmware upload complete... > > xc5000: Unable to initialise tuner > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: I2C write failed (len=3) > > xc5000: firmware upload complete... > > > > I did a git bisect on this and finally reached the end of the line. > Blah blah blah... > Clearly my previous bisection went astray; I think I have a more sensible result this time. qpc$ git bisect good 44835f197bf1e3f57464f23dfb239fef06cf89be is the first bad commit commit 44835f197bf1e3f57464f23dfb239fef06cf89be Author: Jean Delvare Date: Sun Jul 18 16:52:05 2010 -0300 V4L/DVB: cx23885: Check for slave nack on all transactions Don't just check for nacks on zero-length transactions. Check on other transactions too. Signed-off-by: Jean Delvare Signed-off-by: Andy Walls Signed-off-by: Mauro Carvalho Chehab :04 04 e48c9f6efc6186800e8d711c05987c0ad9445c09 1ba37458c6a5fc22d19271f09cde2f336887c616 M drivers git bisect start # good: [9fe6206f400646a2322096b56c59891d530e8d51] Linux 2.6.35 git bisect good 9fe6206f400646a2322096b56c59891d530e8d51 # bad: [18a87becf85d50e7f3d547f1b7a75108b151374d] V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check for < 0 return value git bisect bad 18a87becf85d50e7f3d547f1b7a75108b151374d # good: [03da30986793385af57eeca3296253c887b742e6] Merge git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6 git bisect good 03da30986793385af57eeca3296253c887b742e6 # good: [ab69bcd66fb4be64edfc767365cb9eb084961246] Merge git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core-2.6 git bisect good ab69bcd66fb4be64edfc767365cb9eb084961246 # good: [a57f9a3e811cf1246b394f0cc667c6bc5a52e099] Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/ryusuke/nilfs2 git bisect good a57f9a3e811cf1246b394f0cc667c6bc5a52e099 # good: [9e50ab91d025afc17ca14a1764be2e1d0c24245d] Merge branch 'acpica' of git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6 git bisect good 9e50ab91d025afc17ca14a1764be2e1d0c24245d # good: [d71048e22f47725a5808ea2e4e1e72fa36c1a788] Merge branch 'omap-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap-2.6 git bisect good d71048e22f47725a5808ea2e4e1e72fa36c1a788 # good: [4fd6c6bf83cb16321e9902b00e2af79054f4e0d6] Merge branch 'for-linus' of git://android.kernel.org/kernel/tegra git bisect good 4fd6c6bf83cb16321e9902b00e2af79054f4e0d6 # good: [c1c8f558749cbf2a7ed16b6ae6e19a4238b6fa33] CRIS: Return something from profile write git bisect good c1c8f558749cbf2a7ed16b6ae6e19a4238b6fa33 # good: [ab11b487402f97975f3ac1eeea09c82f4431481e] Merge branch 'master' into for-linus git bisect good ab11b487402f97975f3ac1eeea09c82f4431481e # good: [30d4554a02d3ad6f9928767c9f98214775f4dcb2] V4L/DVB: gspca - main: Version change git bisect good 30d4554a02d3ad6f9928767c9f98214775f4dcb2 # good: [6e80cc51b4419ca0f8162024ee2497d7ec8ba31c] V4L/DVB: gspca - sq930x: Cleanup source, add comments git bisect good 6e80cc51b4419ca0f8162024ee2497d7ec8ba31c # good: [3d217c8656842c77d6f33329a034102157363c8d] V4L/DVB: gspca - vc032x: Force main register write at probe time (po) git bisect good 3d217c8656842c77d6f33329a034102157363c8d # bad: [44835f197bf1e3f57464f23dfb239fef06cf89be] V4L/DVB: cx23885: Check for slave nack on all transactions git bisect bad 44835f197bf1e3f57464f23dfb239fef06cf89be # good: [f4acb3c4ccca74f5448354308f917e87ce83505a] V4L/DVB: cx23885: Return -ENXIO on slave nack git bisect good f4acb3c4ccca74f5448354308f917e87ce83505a -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://
Re: [get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, 2011-02-12 at 12:05 -0700, Mark Zimmerman wrote: > On Sat, Feb 12, 2011 at 11:46:13AM -0500, Andy Walls wrote: > > On Sat, 2011-02-12 at 09:36 -0700, Mark Zimmerman wrote: > > > On Sat, Feb 12, 2011 at 11:27:27AM -0500, Andy Walls wrote: > > > > On Sat, 2011-02-12 at 08:29 -0700, Mark Zimmerman wrote: > > > > > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > > > > > Greetings: > > > > > > > > > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 > > > > > > but > > > > > > which fails to initialize with the latest 2.6.36 kernel. The > > > > > > firmware > > > > > > fails to load due to an i2c failure. A search of the archives > > > > > > indicates > > > > > > that this is not the first time this issue has occurred. > > > > > > > > > > > > What can I do to help get this problem fixed? > > > > > > > > > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > > xc5000: firmware read 12401 bytes. > > > > > > xc5000: firmware uploading... > > > > > > xc5000: firmware upload complete... > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > > xc5000: firmware read 12401 bytes. > > > > > > xc5000: firmware uploading... > > > > > > xc5000: firmware upload complete.. > > > > > > > > > > > > and here is what happens with 2.6.36: > > > > > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > > xc5000: firmware read 12401 bytes. > > > > > > xc5000: firmware uploading... > > > > > > xc5000: I2C write failed (len=3) > > > > > > xc5000: firmware upload complete... > > > > > > xc5000: Unable to initialise tuner > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > > xc5000: firmware read 12401 bytes. > > > > > > xc5000: firmware uploading... > > > > > > xc5000: I2C write failed (len=3) > > > > > > xc5000: firmware upload complete... > > > > > > > > > > > > > > > > I did a git bisect on this and finally reached the end of the line. > > > > > Here is what it said: > > > > > > > > > > qpc$ git bisect bad > > > > > 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit > > > > > commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 > > > > > Author: Jarod Wilson > > > > > Date: Thu Jul 29 18:20:44 2010 -0300 > > > > > > > > > > V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage > > > > > > > > > > Fix when CONFIG_MODULES is not enabled: > > > > > > > > > > drivers/staging/lirc/lirc_parallel.c:243: error: implicit > > > > > declaration of function 'module_refcount' > > > > > drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration > > > > > of function 'module_refcount' > > > > > drivers/built-in.o: In function `it87_probe': > > > > > lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' > > > > > lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' > > > > > drivers/built-in.o: In function `lirc_it87_exit': > > > > > lirc_it87.c:(.exit.text+0x38a5): undefined reference to > > > > > `drop_chrdev' > > > > > > > > > > Its a quick hack and untested beyond building, since I don't have > > > > > the > > > > > hardware, but it should do the trick. > > > > > > > > > > Acked-by: Randy Dunlap > > > > > Signed-off-by: Jarod Wilson > > > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > > > > > :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 > > > > > 49e50945ccf8e1c8567c049908890d2752443b72 M drivers > > > > > > > > Hmm. git log --patch 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 shows the > > > > commit is completely unrealted. > > > > > > > > Please try and see if things are good or bad at commit > > > > 18a87becf85d50e7f3d547f1b7a75108b151374d: > > > > > > > > commit 18a87becf85d50e7f3d547f1b7a75108b151374d > > > > Author: Jean Delvare > > > > Date: Sun Jul 18 17:05:17 2010 -0300 > > > > > > > > V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check > > > > for < 0 return v > > > > > > > > Function i2c_wait_done() never returns negative values, so > > > > there is no > > > > point in checking for them. > > > > > > > > Signed-off-by: Jean Delvare > > > > Signed-off-by: Andy Walls > > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > > > Which is the first commit, prior to the one you found, that seems to me > > > > to have any direct bearing to I2C transactions. > > > > > > > > If that commit is good, then these commits in between would be my next > > > > likely suspects: > > > > e5514f104d875b3d28cbcd5d4f2b96ab2fca1e29 > > > > dbe83a3b921328e12b2abe894fc692afba293d7f > > > > > > > > Regards, > > > > Andy > > > > > > > > > > Sorry to require so
Re: [get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, Feb 12, 2011 at 11:46:13AM -0500, Andy Walls wrote: > On Sat, 2011-02-12 at 09:36 -0700, Mark Zimmerman wrote: > > On Sat, Feb 12, 2011 at 11:27:27AM -0500, Andy Walls wrote: > > > On Sat, 2011-02-12 at 08:29 -0700, Mark Zimmerman wrote: > > > > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > > > > Greetings: > > > > > > > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 > > > > > but > > > > > which fails to initialize with the latest 2.6.36 kernel. The firmware > > > > > fails to load due to an i2c failure. A search of the archives > > > > > indicates > > > > > that this is not the first time this issue has occurred. > > > > > > > > > > What can I do to help get this problem fixed? > > > > > > > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: firmware upload complete... > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: firmware upload complete.. > > > > > > > > > > and here is what happens with 2.6.36: > > > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: I2C write failed (len=3) > > > > > xc5000: firmware upload complete... > > > > > xc5000: Unable to initialise tuner > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: I2C write failed (len=3) > > > > > xc5000: firmware upload complete... > > > > > > > > > > > > > I did a git bisect on this and finally reached the end of the line. > > > > Here is what it said: > > > > > > > > qpc$ git bisect bad > > > > 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit > > > > commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 > > > > Author: Jarod Wilson > > > > Date: Thu Jul 29 18:20:44 2010 -0300 > > > > > > > > V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage > > > > > > > > Fix when CONFIG_MODULES is not enabled: > > > > > > > > drivers/staging/lirc/lirc_parallel.c:243: error: implicit > > > > declaration of function 'module_refcount' > > > > drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration > > > > of function 'module_refcount' > > > > drivers/built-in.o: In function `it87_probe': > > > > lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' > > > > lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' > > > > drivers/built-in.o: In function `lirc_it87_exit': > > > > lirc_it87.c:(.exit.text+0x38a5): undefined reference to > > > > `drop_chrdev' > > > > > > > > Its a quick hack and untested beyond building, since I don't have > > > > the > > > > hardware, but it should do the trick. > > > > > > > > Acked-by: Randy Dunlap > > > > Signed-off-by: Jarod Wilson > > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > > > :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 > > > > 49e50945ccf8e1c8567c049908890d2752443b72 M drivers > > > > > > Hmm. git log --patch 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 shows the > > > commit is completely unrealted. > > > > > > Please try and see if things are good or bad at commit > > > 18a87becf85d50e7f3d547f1b7a75108b151374d: > > > > > > commit 18a87becf85d50e7f3d547f1b7a75108b151374d > > > Author: Jean Delvare > > > Date: Sun Jul 18 17:05:17 2010 -0300 > > > > > > V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check > > > for < 0 return v > > > > > > Function i2c_wait_done() never returns negative values, so > > > there is no > > > point in checking for them. > > > > > > Signed-off-by: Jean Delvare > > > Signed-off-by: Andy Walls > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > Which is the first commit, prior to the one you found, that seems to me > > > to have any direct bearing to I2C transactions. > > > > > > If that commit is good, then these commits in between would be my next > > > likely suspects: > > > e5514f104d875b3d28cbcd5d4f2b96ab2fca1e29 > > > dbe83a3b921328e12b2abe894fc692afba293d7f > > > > > > Regards, > > > Andy > > > > > > > Sorry to require so much hand holding, but I am new to all of this git > > gymnastics. Would you mind sending me the correct git command to get > > to a specific commit? > > It should just be > > $ git checkout 18a87becf85d50e7f3d547f1b7a75108b151374d > > or whatever the commit number git log shows you for t
Re: [get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, Feb 12, 2011 at 11:46:13AM -0500, Andy Walls wrote: > On Sat, 2011-02-12 at 09:36 -0700, Mark Zimmerman wrote: > > On Sat, Feb 12, 2011 at 11:27:27AM -0500, Andy Walls wrote: > > > On Sat, 2011-02-12 at 08:29 -0700, Mark Zimmerman wrote: > > > > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > > > > Greetings: > > > > > > > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 > > > > > but > > > > > which fails to initialize with the latest 2.6.36 kernel. The firmware > > > > > fails to load due to an i2c failure. A search of the archives > > > > > indicates > > > > > that this is not the first time this issue has occurred. > > > > > > > > > > What can I do to help get this problem fixed? > > > > > > > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: firmware upload complete... > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: firmware upload complete.. > > > > > > > > > > and here is what happens with 2.6.36: > > > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: I2C write failed (len=3) > > > > > xc5000: firmware upload complete... > > > > > xc5000: Unable to initialise tuner > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > > xc5000: firmware read 12401 bytes. > > > > > xc5000: firmware uploading... > > > > > xc5000: I2C write failed (len=3) > > > > > xc5000: firmware upload complete... > > > > > > > > > > > > > I did a git bisect on this and finally reached the end of the line. > > > > Here is what it said: > > > > > > > > qpc$ git bisect bad > > > > 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit > > > > commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 > > > > Author: Jarod Wilson > > > > Date: Thu Jul 29 18:20:44 2010 -0300 > > > > > > > > V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage > > > > > > > > Fix when CONFIG_MODULES is not enabled: > > > > > > > > drivers/staging/lirc/lirc_parallel.c:243: error: implicit > > > > declaration of function 'module_refcount' > > > > drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration > > > > of function 'module_refcount' > > > > drivers/built-in.o: In function `it87_probe': > > > > lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' > > > > lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' > > > > drivers/built-in.o: In function `lirc_it87_exit': > > > > lirc_it87.c:(.exit.text+0x38a5): undefined reference to > > > > `drop_chrdev' > > > > > > > > Its a quick hack and untested beyond building, since I don't have > > > > the > > > > hardware, but it should do the trick. > > > > > > > > Acked-by: Randy Dunlap > > > > Signed-off-by: Jarod Wilson > > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > > > :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 > > > > 49e50945ccf8e1c8567c049908890d2752443b72 M drivers > > > > > > Hmm. git log --patch 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 shows the > > > commit is completely unrealted. > > > > > > Please try and see if things are good or bad at commit > > > 18a87becf85d50e7f3d547f1b7a75108b151374d: > > > > > > commit 18a87becf85d50e7f3d547f1b7a75108b151374d > > > Author: Jean Delvare > > > Date: Sun Jul 18 17:05:17 2010 -0300 > > > > > > V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check > > > for < 0 return v > > > > > > Function i2c_wait_done() never returns negative values, so > > > there is no > > > point in checking for them. > > > > > > Signed-off-by: Jean Delvare > > > Signed-off-by: Andy Walls > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > Which is the first commit, prior to the one you found, that seems to me > > > to have any direct bearing to I2C transactions. > > > > > > If that commit is good, then these commits in between would be my next > > > likely suspects: > > > e5514f104d875b3d28cbcd5d4f2b96ab2fca1e29 > > > dbe83a3b921328e12b2abe894fc692afba293d7f > > > > > > Regards, > > > Andy > > > > > > > Sorry to require so much hand holding, but I am new to all of this git > > gymnastics. Would you mind sending me the correct git command to get > > to a specific commit? > > It should just be > > $ git checkout 18a87becf85d50e7f3d547f1b7a75108b151374d > > or whatever the commit number git log shows you for t
Re: [get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, 2011-02-12 at 09:36 -0700, Mark Zimmerman wrote: > On Sat, Feb 12, 2011 at 11:27:27AM -0500, Andy Walls wrote: > > On Sat, 2011-02-12 at 08:29 -0700, Mark Zimmerman wrote: > > > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > > > Greetings: > > > > > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but > > > > which fails to initialize with the latest 2.6.36 kernel. The firmware > > > > fails to load due to an i2c failure. A search of the archives indicates > > > > that this is not the first time this issue has occurred. > > > > > > > > What can I do to help get this problem fixed? > > > > > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > xc5000: firmware read 12401 bytes. > > > > xc5000: firmware uploading... > > > > xc5000: firmware upload complete... > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > xc5000: firmware read 12401 bytes. > > > > xc5000: firmware uploading... > > > > xc5000: firmware upload complete.. > > > > > > > > and here is what happens with 2.6.36: > > > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > xc5000: firmware read 12401 bytes. > > > > xc5000: firmware uploading... > > > > xc5000: I2C write failed (len=3) > > > > xc5000: firmware upload complete... > > > > xc5000: Unable to initialise tuner > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > > xc5000: firmware read 12401 bytes. > > > > xc5000: firmware uploading... > > > > xc5000: I2C write failed (len=3) > > > > xc5000: firmware upload complete... > > > > > > > > > > I did a git bisect on this and finally reached the end of the line. > > > Here is what it said: > > > > > > qpc$ git bisect bad > > > 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit > > > commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 > > > Author: Jarod Wilson > > > Date: Thu Jul 29 18:20:44 2010 -0300 > > > > > > V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage > > > > > > Fix when CONFIG_MODULES is not enabled: > > > > > > drivers/staging/lirc/lirc_parallel.c:243: error: implicit declaration > > > of function 'module_refcount' > > > drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration of > > > function 'module_refcount' > > > drivers/built-in.o: In function `it87_probe': > > > lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' > > > lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' > > > drivers/built-in.o: In function `lirc_it87_exit': > > > lirc_it87.c:(.exit.text+0x38a5): undefined reference to `drop_chrdev' > > > > > > Its a quick hack and untested beyond building, since I don't have the > > > hardware, but it should do the trick. > > > > > > Acked-by: Randy Dunlap > > > Signed-off-by: Jarod Wilson > > > Signed-off-by: Mauro Carvalho Chehab > > > > > > :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 > > > 49e50945ccf8e1c8567c049908890d2752443b72 M drivers > > > > Hmm. git log --patch 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 shows the > > commit is completely unrealted. > > > > Please try and see if things are good or bad at commit > > 18a87becf85d50e7f3d547f1b7a75108b151374d: > > > > commit 18a87becf85d50e7f3d547f1b7a75108b151374d > > Author: Jean Delvare > > Date: Sun Jul 18 17:05:17 2010 -0300 > > > > V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check for > > < 0 return v > > > > Function i2c_wait_done() never returns negative values, so > > there is no > > point in checking for them. > > > > Signed-off-by: Jean Delvare > > Signed-off-by: Andy Walls > > Signed-off-by: Mauro Carvalho Chehab > > > > Which is the first commit, prior to the one you found, that seems to me > > to have any direct bearing to I2C transactions. > > > > If that commit is good, then these commits in between would be my next > > likely suspects: > > e5514f104d875b3d28cbcd5d4f2b96ab2fca1e29 > > dbe83a3b921328e12b2abe894fc692afba293d7f > > > > Regards, > > Andy > > > > Sorry to require so much hand holding, but I am new to all of this git > gymnastics. Would you mind sending me the correct git command to get > to a specific commit? It should just be $ git checkout 18a87becf85d50e7f3d547f1b7a75108b151374d or whatever the commit number git log shows you for the change I suspect is the problem. > Also, do I need to do a bisect reset? I wouldn't reset the git bisect yet. If you test a commit and it is good, you will want to mark it with 'git bisect good ', and if it is bad, you will want to mark it with 'git bisect bad ' BTW, can you provide the output of 'git bisect log' ? R
Re: [get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, Feb 12, 2011 at 11:27:27AM -0500, Andy Walls wrote: > On Sat, 2011-02-12 at 08:29 -0700, Mark Zimmerman wrote: > > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > > Greetings: > > > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but > > > which fails to initialize with the latest 2.6.36 kernel. The firmware > > > fails to load due to an i2c failure. A search of the archives indicates > > > that this is not the first time this issue has occurred. > > > > > > What can I do to help get this problem fixed? > > > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > xc5000: firmware read 12401 bytes. > > > xc5000: firmware uploading... > > > xc5000: firmware upload complete... > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > xc5000: firmware read 12401 bytes. > > > xc5000: firmware uploading... > > > xc5000: firmware upload complete.. > > > > > > and here is what happens with 2.6.36: > > > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > xc5000: firmware read 12401 bytes. > > > xc5000: firmware uploading... > > > xc5000: I2C write failed (len=3) > > > xc5000: firmware upload complete... > > > xc5000: Unable to initialise tuner > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > > xc5000: firmware read 12401 bytes. > > > xc5000: firmware uploading... > > > xc5000: I2C write failed (len=3) > > > xc5000: firmware upload complete... > > > > > > > I did a git bisect on this and finally reached the end of the line. > > Here is what it said: > > > > qpc$ git bisect bad > > 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit > > commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 > > Author: Jarod Wilson > > Date: Thu Jul 29 18:20:44 2010 -0300 > > > > V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage > > > > Fix when CONFIG_MODULES is not enabled: > > > > drivers/staging/lirc/lirc_parallel.c:243: error: implicit declaration > > of function 'module_refcount' > > drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration of > > function 'module_refcount' > > drivers/built-in.o: In function `it87_probe': > > lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' > > lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' > > drivers/built-in.o: In function `lirc_it87_exit': > > lirc_it87.c:(.exit.text+0x38a5): undefined reference to `drop_chrdev' > > > > Its a quick hack and untested beyond building, since I don't have the > > hardware, but it should do the trick. > > > > Acked-by: Randy Dunlap > > Signed-off-by: Jarod Wilson > > Signed-off-by: Mauro Carvalho Chehab > > > > :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 > > 49e50945ccf8e1c8567c049908890d2752443b72 M drivers > > Hmm. git log --patch 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 shows the > commit is completely unrealted. > > Please try and see if things are good or bad at commit > 18a87becf85d50e7f3d547f1b7a75108b151374d: > > commit 18a87becf85d50e7f3d547f1b7a75108b151374d > Author: Jean Delvare > Date: Sun Jul 18 17:05:17 2010 -0300 > > V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check for < > 0 return v > > Function i2c_wait_done() never returns negative values, so there > is no > point in checking for them. > > Signed-off-by: Jean Delvare > Signed-off-by: Andy Walls > Signed-off-by: Mauro Carvalho Chehab > > Which is the first commit, prior to the one you found, that seems to me > to have any direct bearing to I2C transactions. > > If that commit is good, then these commits in between would be my next > likely suspects: > e5514f104d875b3d28cbcd5d4f2b96ab2fca1e29 > dbe83a3b921328e12b2abe894fc692afba293d7f > > Regards, > Andy > Sorry to require so much hand holding, but I am new to all of this git gymnastics. Would you mind sending me the correct git command to get to a specific commit? Also, do I need to do a bisect reset? Thanks, -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Sat, 2011-02-12 at 08:29 -0700, Mark Zimmerman wrote: > On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > > Greetings: > > > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but > > which fails to initialize with the latest 2.6.36 kernel. The firmware > > fails to load due to an i2c failure. A search of the archives indicates > > that this is not the first time this issue has occurred. > > > > What can I do to help get this problem fixed? > > > > Here is the dmesg from 2.6.35, for the two tuners: > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: firmware upload complete... > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: firmware upload complete.. > > > > and here is what happens with 2.6.36: > > > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: I2C write failed (len=3) > > xc5000: firmware upload complete... > > xc5000: Unable to initialise tuner > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > > xc5000: firmware read 12401 bytes. > > xc5000: firmware uploading... > > xc5000: I2C write failed (len=3) > > xc5000: firmware upload complete... > > > > I did a git bisect on this and finally reached the end of the line. > Here is what it said: > > qpc$ git bisect bad > 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit > commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 > Author: Jarod Wilson > Date: Thu Jul 29 18:20:44 2010 -0300 > > V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage > > Fix when CONFIG_MODULES is not enabled: > > drivers/staging/lirc/lirc_parallel.c:243: error: implicit declaration of > function 'module_refcount' > drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration of > function 'module_refcount' > drivers/built-in.o: In function `it87_probe': > lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' > lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' > drivers/built-in.o: In function `lirc_it87_exit': > lirc_it87.c:(.exit.text+0x38a5): undefined reference to `drop_chrdev' > > Its a quick hack and untested beyond building, since I don't have the > hardware, but it should do the trick. > > Acked-by: Randy Dunlap > Signed-off-by: Jarod Wilson > Signed-off-by: Mauro Carvalho Chehab > > :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 > 49e50945ccf8e1c8567c049908890d2752443b72 M drivers Hmm. git log --patch 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 shows the commit is completely unrealted. Please try and see if things are good or bad at commit 18a87becf85d50e7f3d547f1b7a75108b151374d: commit 18a87becf85d50e7f3d547f1b7a75108b151374d Author: Jean Delvare Date: Sun Jul 18 17:05:17 2010 -0300 V4L/DVB: cx23885: i2c_wait_done returns 0 or 1, don't check for < 0 return v Function i2c_wait_done() never returns negative values, so there is no point in checking for them. Signed-off-by: Jean Delvare Signed-off-by: Andy Walls Signed-off-by: Mauro Carvalho Chehab Which is the first commit, prior to the one you found, that seems to me to have any direct bearing to I2C transactions. If that commit is good, then these commits in between would be my next likely suspects: e5514f104d875b3d28cbcd5d4f2b96ab2fca1e29 dbe83a3b921328e12b2abe894fc692afba293d7f Regards, Andy > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[get-bisect results]: DViCO FusionHDTV7 Dual Express I2C write failed
On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > Greetings: > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but > which fails to initialize with the latest 2.6.36 kernel. The firmware > fails to load due to an i2c failure. A search of the archives indicates > that this is not the first time this issue has occurred. > > What can I do to help get this problem fixed? > > Here is the dmesg from 2.6.35, for the two tuners: > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: firmware upload complete... > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: firmware upload complete.. > > and here is what happens with 2.6.36: > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: I2C write failed (len=3) > xc5000: firmware upload complete... > xc5000: Unable to initialise tuner > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: I2C write failed (len=3) > xc5000: firmware upload complete... > I did a git bisect on this and finally reached the end of the line. Here is what it said: qpc$ git bisect bad 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 is the first bad commit commit 82ce67bf262b3f47ecb5a0ca31cace8ac72b7c98 Author: Jarod Wilson Date: Thu Jul 29 18:20:44 2010 -0300 V4L/DVB: staging/lirc: fix non-CONFIG_MODULES build horkage Fix when CONFIG_MODULES is not enabled: drivers/staging/lirc/lirc_parallel.c:243: error: implicit declaration of function 'module_refcount' drivers/staging/lirc/lirc_it87.c:150: error: implicit declaration of function 'module_refcount' drivers/built-in.o: In function `it87_probe': lirc_it87.c:(.text+0x4079b0): undefined reference to `init_chrdev' lirc_it87.c:(.text+0x4079cc): undefined reference to `drop_chrdev' drivers/built-in.o: In function `lirc_it87_exit': lirc_it87.c:(.exit.text+0x38a5): undefined reference to `drop_chrdev' Its a quick hack and untested beyond building, since I don't have the hardware, but it should do the trick. Acked-by: Randy Dunlap Signed-off-by: Jarod Wilson Signed-off-by: Mauro Carvalho Chehab :04 04 f645b46a07b7ff87a2c11ac9296a5ff56e89a0d0 49e50945ccf8e1c8567c049908890d2752443b72 M drivers -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Mon, Jan 24, 2011 at 10:57:02AM -0500, Devin Heitmueller wrote: > On Mon, Jan 24, 2011 at 10:49 AM, Mark Zimmerman wrote: > > From looking at the code and a dump of the firmware file, the first > > i2c write would have a length of 3; so this error: > > > > xc5000: I2C write failed (len=3) > > > > tells me that there were probably no successful i2c transactions on > > this device. The i2c write call looks the same as that in other > > drivers, so I wonder if there is an initialization step that is now > > necessary but which is missing. > > > > Still hoping for suggestions... > > My guess would be that somebody screwed up either the GPIO config int > the cx88 board profile, or the i2c gate, which is resulting in not > being able to reach the tuner at all. > > Do you have an oscilloscope? If so, I bet you will find that the > xc5000 pin is being held in reset. If I had an oscilloscope I probably wouldn't know where to stick the probe. > > I would probably take a hard look at the board profile in cx88-cards.c > as well as whether there have been any changes to the GPIO setup and > power management code. cx23885-cards.c, actually. I'll see what I can find in there. Thanks, -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Mon, Jan 24, 2011 at 10:49 AM, Mark Zimmerman wrote: > From looking at the code and a dump of the firmware file, the first > i2c write would have a length of 3; so this error: > > xc5000: I2C write failed (len=3) > > tells me that there were probably no successful i2c transactions on > this device. The i2c write call looks the same as that in other > drivers, so I wonder if there is an initialization step that is now > necessary but which is missing. > > Still hoping for suggestions... My guess would be that somebody screwed up either the GPIO config int the cx88 board profile, or the i2c gate, which is resulting in not being able to reach the tuner at all. Do you have an oscilloscope? If so, I bet you will find that the xc5000 pin is being held in reset. I would probably take a hard look at the board profile in cx88-cards.c as well as whether there have been any changes to the GPIO setup and power management code. Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Wed, Jan 19, 2011 at 10:39:46AM -0700, Mark Zimmerman wrote: > On Wed, Jan 19, 2011 at 09:22:28AM -0800, VDR User wrote: > > On Wed, Jan 19, 2011 at 8:13 AM, Devin Heitmueller > > wrote: > > >> Can someone please look into this and possibly provide a fix for the > > >> bug? ??I'm surprised it hasn't happened yet after all this time but > > >> maybe it's been forgotten the bug existed. > > > > > > You shouldn't be too surprised. ??In many cases device support for more > > > obscure products comes not from the maintainer of the actual driver > > > but rather from some random user who hacked in an additional board > > > profile (in many cases, not doing it correctly but good enough so it > > > "works for them"). ??In cases like that, the changes get committed, the > > > original submitter disappears, and then when things break there is > > > nobody with the appropriate knowledge and the hardware to debug the > > > problem. > > > > Good point. My understanding is that this is a fairly common card so > > I wouldn't think that would be the case. At any rate, hopefully we'll > > be able to narrow down the cause of the problem and get it fixed. > > > > Were there changes to i2c between 2.6.35 and 2.6.36 that are missing > from the xc5000 driver? If so, is there another driver that has the > required updates so I can look at what changed? I would like to get > some traction on this but I really don't know where to start. > >From looking at the code and a dump of the firmware file, the first i2c write would have a length of 3; so this error: xc5000: I2C write failed (len=3) tells me that there were probably no successful i2c transactions on this device. The i2c write call looks the same as that in other drivers, so I wonder if there is an initialization step that is now necessary but which is missing. Still hoping for suggestions... -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
So what would a "mainstream" dual (or more) tuner card be? I've found these Fusions to be flaky. Had one die and another went flaky when I enabled the sleep mode. Can't really afford any more now, but am always watching. A company called Ceton seems to havea quad, but it's a cable card tuner costing $450. On 1/19/2011 9:13 AM, Devin Heitmueller wrote: On Wed, Jan 19, 2011 at 10:59 AM, VDR User wrote: Can someone please look into this and possibly provide a fix for the bug? I'm surprised it hasn't happened yet after all this time but maybe it's been forgotten the bug existed. You shouldn't be too surprised. In many cases device support for more obscure products comes not from the maintainer of the actual driver but rather from some random user who hacked in an additional board profile (in many cases, not doing it correctly but good enough so it "works for them"). In cases like that, the changes get committed, the original submitter disappears, and then when things break there is nobody with the appropriate knowledge and the hardware to debug the problem. Devin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Wed, Jan 19, 2011 at 09:22:28AM -0800, VDR User wrote: > On Wed, Jan 19, 2011 at 8:13 AM, Devin Heitmueller > wrote: > >> Can someone please look into this and possibly provide a fix for the > >> bug? ??I'm surprised it hasn't happened yet after all this time but > >> maybe it's been forgotten the bug existed. > > > > You shouldn't be too surprised. ??In many cases device support for more > > obscure products comes not from the maintainer of the actual driver > > but rather from some random user who hacked in an additional board > > profile (in many cases, not doing it correctly but good enough so it > > "works for them"). ??In cases like that, the changes get committed, the > > original submitter disappears, and then when things break there is > > nobody with the appropriate knowledge and the hardware to debug the > > problem. > > Good point. My understanding is that this is a fairly common card so > I wouldn't think that would be the case. At any rate, hopefully we'll > be able to narrow down the cause of the problem and get it fixed. > Were there changes to i2c between 2.6.35 and 2.6.36 that are missing from the xc5000 driver? If so, is there another driver that has the required updates so I can look at what changed? I would like to get some traction on this but I really don't know where to start. Thanks, -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Wed, Jan 19, 2011 at 8:13 AM, Devin Heitmueller wrote: >> Can someone please look into this and possibly provide a fix for the >> bug? I'm surprised it hasn't happened yet after all this time but >> maybe it's been forgotten the bug existed. > > You shouldn't be too surprised. In many cases device support for more > obscure products comes not from the maintainer of the actual driver > but rather from some random user who hacked in an additional board > profile (in many cases, not doing it correctly but good enough so it > "works for them"). In cases like that, the changes get committed, the > original submitter disappears, and then when things break there is > nobody with the appropriate knowledge and the hardware to debug the > problem. Good point. My understanding is that this is a fairly common card so I wouldn't think that would be the case. At any rate, hopefully we'll be able to narrow down the cause of the problem and get it fixed. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Wed, Jan 19, 2011 at 10:59 AM, VDR User wrote: > Can someone please look into this and possibly provide a fix for the > bug? I'm surprised it hasn't happened yet after all this time but > maybe it's been forgotten the bug existed. You shouldn't be too surprised. In many cases device support for more obscure products comes not from the maintainer of the actual driver but rather from some random user who hacked in an additional board profile (in many cases, not doing it correctly but good enough so it "works for them"). In cases like that, the changes get committed, the original submitter disappears, and then when things break there is nobody with the appropriate knowledge and the hardware to debug the problem. Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Sun, Jan 9, 2011 at 6:14 PM, Mark Zimmerman wrote: >> I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but >> which fails to initialize with the latest 2.6.36 kernel. The firmware >> fails to load due to an i2c failure. A search of the archives indicates >> that this is not the first time this issue has occurred. >> >> What can I do to help get this problem fixed? >> >> Here is the dmesg from 2.6.35, for the two tuners: >> >> xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... >> xc5000: firmware read 12401 bytes. >> xc5000: firmware uploading... >> xc5000: firmware upload complete... >> xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... >> xc5000: firmware read 12401 bytes. >> xc5000: firmware uploading... >> xc5000: firmware upload complete.. >> >> and here is what happens with 2.6.36: >> >> xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... >> xc5000: firmware read 12401 bytes. >> xc5000: firmware uploading... >> xc5000: I2C write failed (len=3) >> xc5000: firmware upload complete... >> xc5000: Unable to initialise tuner >> xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... >> xc5000: firmware read 12401 bytes. >> xc5000: firmware uploading... >> xc5000: I2C write failed (len=3) >> xc5000: firmware upload complete... >> > > More information about this: I tried 2.6.37 (vanilla source from > kernel.org) and the problem persisted. So, I enabled these options: > CONFIG_I2C_DEBUG_CORE=y > CONFIG_I2C_DEBUG_ALGO=y > CONFIG_I2C_DEBUG_BUS=y > hoping to get more information but this time the firmware loaded > successfully and the tuner works properly. > > This leads me to suspect a race condition somewhere, or maybe a > tunable parameter that can be adjusted. The fact that the 'write > failed' message occurs before the 'upload complete' message would tend > to support this. Can anyone suggest something I might try? Can someone please look into this and possibly provide a fix for the bug? I'm surprised it hasn't happened yet after all this time but maybe it's been forgotten the bug existed. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
I just tried to update from kernel 2.6.34 and in doing so had to switch to git for v4l. I noticed the chip in this this post and saved it to look at latter. now I'm glad I did. I ran into the same problem. Driver seems to load ok, but when I try to start vdr I get thet same messages in syslog. Jan 16 21:50:48 LLLx64-32 vdr: [6170] saved setup to /usr/local/dvb/VDR/config/setup.conf Jan 16 21:50:49 LLLx64-32 kernel: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... Jan 16 21:50:49 LLLx64-32 kernel: xc5000: firmware read 12401 bytes. Jan 16 21:50:49 LLLx64-32 kernel: xc5000: firmware uploading... Jan 16 21:50:49 LLLx64-32 vdr: [6176] section handler thread ended (pid=6170, tid=6176) Jan 16 21:50:49 LLLx64-32 kernel: xc5000: I2C write failed (len=3) Jan 16 21:50:49 LLLx64-32 kernel: xc5000: firmware upload complete... Jan 16 21:50:50 LLLx64-32 vdr: [6175] tuner on frontend 0/0 thread ended (pid=6170, tid=6175) Jan 16 21:50:50 LLLx64-32 kernel: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... Jan 16 21:50:50 LLLx64-32 kernel: xc5000: firmware read 12401 bytes. Jan 16 21:50:50 LLLx64-32 kernel: xc5000: firmware uploading... Jan 16 21:50:50 LLLx64-32 vdr: [6174] CI adapter on device 0 thread ended (pid=6170, tid=6174) ... On 12/7/2010 12:07 PM, Mark Zimmerman wrote: Greetings: I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but which fails to initialize with the latest 2.6.36 kernel. The firmware fails to load due to an i2c failure. A search of the archives indicates that this is not the first time this issue has occurred. What can I do to help get this problem fixed? Here is the dmesg from 2.6.35, for the two tuners: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: firmware upload complete... xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: firmware upload complete.. and here is what happens with 2.6.36: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: I2C write failed (len=3) xc5000: firmware upload complete... xc5000: Unable to initialise tuner xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: I2C write failed (len=3) xc5000: firmware upload complete... -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: DViCO FusionHDTV7 Dual Express I2C write failed
On Tue, Dec 07, 2010 at 12:07:53PM -0700, Mark Zimmerman wrote: > Greetings: > > I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but > which fails to initialize with the latest 2.6.36 kernel. The firmware > fails to load due to an i2c failure. A search of the archives indicates > that this is not the first time this issue has occurred. > > What can I do to help get this problem fixed? > > Here is the dmesg from 2.6.35, for the two tuners: > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: firmware upload complete... > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: firmware upload complete.. > > and here is what happens with 2.6.36: > > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: I2C write failed (len=3) > xc5000: firmware upload complete... > xc5000: Unable to initialise tuner > xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... > xc5000: firmware read 12401 bytes. > xc5000: firmware uploading... > xc5000: I2C write failed (len=3) > xc5000: firmware upload complete... > More information about this: I tried 2.6.37 (vanilla source from kernel.org) and the problem persisted. So, I enabled these options: CONFIG_I2C_DEBUG_CORE=y CONFIG_I2C_DEBUG_ALGO=y CONFIG_I2C_DEBUG_BUS=y hoping to get more information but this time the firmware loaded successfully and the tuner works properly. This leads me to suspect a race condition somewhere, or maybe a tunable parameter that can be adjusted. The fact that the 'write failed' message occurs before the 'upload complete' message would tend to support this. Can anyone suggest something I might try? -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
DViCO FusionHDTV7 Dual Express I2C write failed
Greetings: I have a DViCO FusionHDTV7 Dual Express card that works with 2.6.35 but which fails to initialize with the latest 2.6.36 kernel. The firmware fails to load due to an i2c failure. A search of the archives indicates that this is not the first time this issue has occurred. What can I do to help get this problem fixed? Here is the dmesg from 2.6.35, for the two tuners: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: firmware upload complete... xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: firmware upload complete.. and here is what happens with 2.6.36: xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: I2C write failed (len=3) xc5000: firmware upload complete... xc5000: Unable to initialise tuner xc5000: waiting for firmware upload (dvb-fe-xc5000-1.6.114.fw)... xc5000: firmware read 12401 bytes. xc5000: firmware uploading... xc5000: I2C write failed (len=3) xc5000: firmware upload complete... -- Mark -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
