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

[Philby John <pjohn-k0rHJ+Hhz/SB+jHODAdFcQ@public.gmane.org>]

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-l0cyMroinI0@public.gmane.org
> 
> >-----Original Message-----
> >From: Philby John [mailto:pjohn-k0rHJ+Hhz/SB+jHODAdFcQ@public.gmane.org]
> >Sent: Thursday, December 03, 2009 11:50 AM
> >To: Karicheri, Muralidharan
> >Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
> >source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org; linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> >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-l0cyMroinI0@public.gmane.org
> >>
> >> >-----Original Message-----
> >> >From: Philby John [mailto:pjohn-k0rHJ+Hhz/SB+jHODAdFcQ@public.gmane.org]
> >> >Sent: Thursday, December 03, 2009 11:12 AM
> >> >To: Karicheri, Muralidharan
> >> >Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
> >> >source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org; linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> >> >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-l0cyMroinI0@public.gmane.org
> >> >>
> >> >> >-----Original Message-----
> >> >> >From: Philby John [mailto:pjohn-k0rHJ+Hhz/SB+jHODAdFcQ@public.gmane.org]
> >> >> >Sent: Monday, November 23, 2009 10:24 AM
> >> >> >To: Karicheri, Muralidharan
> >> >> >Cc: Kevin Hilman; Ben Dooks; davinci-linux-open-
> >> >> >source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org; linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> >> >> >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-fKfTphxkbEe3Z1R/gWBstg@public.gmane.org:/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
> >> >> >
> >> >>
> >>
>