RE: [PATCH 0/2] davinci i2c fixes for 2.6.31

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Philby,

My MT9T031 sensor has some problem (i2c stopped responding). So I need to get a sensor to test your fix.  Let you know if I could test it.

Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
phone: 301-407-9583
email: m-karicheri2@xxxxxx

>-----Original Message-----
>From: Philby John [mailto:pjohn@xxxxxxxxxxxxx]
>Sent: Thursday, December 03, 2009 1:11 PM
>To: Karicheri, Muralidharan
>Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
>source@xxxxxxxxxxxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx
>Subject: RE: [PATCH 0/2] davinci i2c fixes for 2.6.31
>
>Yes, it did (on DM355).
>Let me know the result and I could also use the log in case of a kernel
>dump.
>
>Thanks.
>Philby
>
>On Thu, 2009-12-03 at 12:01 -0600, Karicheri, Muralidharan wrote:
>> I can verify on DM355. But I will be running latest kernel on Arago. I
>have manually merged your patch. Did it worked on DM355?
>>
>> Murali Karicheri
>> Software Design Engineer
>> Texas Instruments Inc.
>> Germantown, MD 20874
>> phone: 301-407-9583
>> email: m-karicheri2@xxxxxx
>>
>> >-----Original Message-----
>> >From: Philby John [mailto:pjohn@xxxxxxxxxxxxx]
>> >Sent: Thursday, December 03, 2009 11:50 AM
>> >To: Karicheri, Muralidharan
>> >Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
>> >source@xxxxxxxxxxxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx
>> >Subject: RE: [PATCH 0/2] davinci i2c fixes for 2.6.31
>> >
>> >Murali,
>> >
>> >	It would be excellent if you could verify the patch on a DM6446 first
>> >and perhaps then make changes for DM365.
>> >
>> >Regards,
>> >Philby
>> >
>> >
>> >
>> >On Thu, 2009-12-03 at 10:33 -0600, Karicheri, Muralidharan wrote:
>> >> Philby,
>> >>
>> >> So where is the patch for adding davinci_i2c_platform_data in the
>> >platform
>> >> file? Could you send it please?
>> >>
>> >> Murali Karicheri
>> >> Software Design Engineer
>> >> Texas Instruments Inc.
>> >> Germantown, MD 20874
>> >> phone: 301-407-9583
>> >> email: m-karicheri2@xxxxxx
>> >>
>> >> >-----Original Message-----
>> >> >From: Philby John [mailto:pjohn@xxxxxxxxxxxxx]
>> >> >Sent: Thursday, December 03, 2009 11:12 AM
>> >> >To: Karicheri, Muralidharan
>> >> >Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
>> >> >source@xxxxxxxxxxxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx
>> >> >Subject: RE: [PATCH 0/2] davinci i2c fixes for 2.6.31
>> >> >
>> >> >Murali,
>> >> >
>> >> >	The patch works for DM6446 and DM355 only. To get it to work
>for
>> >> >DM365
>> >> >one must update the gpio pin number variables (sda_pin and scl_pin)
>in
>> >> >struct davinci_i2c_platform_data for each platform.
>> >> >
>> >> >I do not have the pin number information for other Davinci platforms.
>> >> >This was mentioned in the comment section of the patch v2...
>> >> >
>> >> >"TODO: Need to add SDA and SCL pin numbers to the respective
>> >> >platforms such as dm355-leopard, dm365, dm646x, da8xx etc.
>> >> >What I have info on is limited to just dm355 and dm6446."
>> >> >
>> >> >If someone can provide me with the relevant information, I could
>update
>> >> >for all.
>> >> >
>> >> >I used "./i2cset -y 0 0x50 0x00" cause its the only way to kill the
>i2c
>> >> >bus on a 2.6.18 kernel with DVSDK 2.0. FYI, I do not know how to
>> >> >generate cmem.ko for the Arago tree kernel that would get
>> >> >"fbdev_loopback" running.
>> >> >
>> >> >Regards,
>> >> >Philby
>> >> >
>> >> >
>> >> >On Thu, 2009-12-03 at 09:45 -0600, Karicheri, Muralidharan wrote:
>> >> >> Philby,
>> >> >>
>> >> >> Is this a generic solution which works on DM365 as well?
>> >> >>
>> >> >> Why did you do the following command?
>> >> >>
>> >> >> #./i2cset -y 0 0x50 0x00
>> >> >>
>> >> >> Using the Arago tree kernel I was able to reproduce the issue
>> >> >> by doing
>> >> >>
>> >> >> ./fbdev_loopback -i2 -d0
>> >> >>
>> >> >> and then press Control C. I am not sure why you run the command
>> >> >> ./i2cset -y 0 0x50 0x00
>> >> >>
>> >> >>
>> >> >> Murali Karicheri
>> >> >> Software Design Engineer
>> >> >> Texas Instruments Inc.
>> >> >> Germantown, MD 20874
>> >> >> phone: 301-407-9583
>> >> >> email: m-karicheri2@xxxxxx
>> >> >>
>> >> >> >-----Original Message-----
>> >> >> >From: Philby John [mailto:pjohn@xxxxxxxxxxxxx]
>> >> >> >Sent: Monday, November 23, 2009 10:24 AM
>> >> >> >To: Karicheri, Muralidharan
>> >> >> >Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
>> >> >> >source@xxxxxxxxxxxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx
>> >> >> >Subject: RE: [PATCH 0/2] davinci i2c fixes for 2.6.31
>> >> >> >
>> >> >> >Hello Murali,
>> >> >> >
>> >> >> >On Thu, 2009-07-16 at 16:22 -0500, Karicheri, Muralidharan wrote:
>> >> >> >> Philby,
>> >> >> >>
>> >> >> >> I tried running this patch against 2.6.31.rc2 (davinci kernel
>tree).
>> >> >But
>> >> >> >I got a crash as I run the application which is given below....
>> >> >> >>
>> >> >> >> In my case, time out happens when I kill the capture application
>> >using
>> >> >> >Cntrl-C. After Cnrtl-C, the mt9t031 i2c device couldn't be
>contacted
>> >by
>> >> >the
>> >> >> >host (during cleanup, i2c commands fails). When the application is
>> >run
>> >> >> >again, I get following outputs..
>> >> >> >>
>> >> >> >> i2c_davinci i2c_davinci.1: timeout waiting for bus ready
>> >> >> >> i2c_davinci i2c_davinci.1: timeout waiting for bus ready
>> >> >> >>
>> >> >> >> Here are the complete log....
>> >> >> >>
>> >> >> >> application running and pressed Cntrl-C ====>
>> >> >> >>
>> >> >> >> time:105    frame:33
>> >> >> >> time:105    frame:34vpfe-capture vpfe-capture: stream off failed
>in
>> >> >> >subdev
>> >> >> >>
>> >> >> >> Running it again .....
>> >> >> >>
>> >> >> >> root@xxxxxxxxxxxxxxx:/opt/git# ./fbdev_loopback -i2 -d0
>> >> >> >
>> >> >> >
>> >> >> >I have now verified the code on a DM6446 and DM355, with
>> >> >> >the ./fbdev_loopback test. I had to first do a #./i2cset -y 0 0x50
>> >0x00
>> >> >> >0x0000 w to kill the bus and then run the ./fbdev_loopback test to
>> >> >> >recreate the issue. Recovery seems to happen with the new patch.
>> >Could
>> >> >> >you try this out at your end, with the new patch?
>> >> >> >
>> >> >> >Thanks and regards,
>> >> >> >Philby
>> >> >> >
>> >> >>
>> >>
>>

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux