HiHeikki On 3/26/2020 2:05 PM, Heikki Krogerus wrote: > Hi, > > On Sat, Feb 29, 2020 at 08:55:50AM +0530, Shah, Nehal-bakulchandra wrote: >> Hi >> On 2/27/2020 10:29 PM, Shah, Nehal-bakulchandra wrote: >>> Hi >>> On 2/27/2020 5:53 PM, Heikki Krogerus wrote: >>>> On Mon, Feb 24, 2020 at 02:38:12PM +0530, Shah, Nehal-bakulchandra wrote: >>>>> Hi >>>>> >>>>> On 2/14/2020 7:58 PM, Shah, Nehal-bakulchandra wrote: >>>>>> Hi >>>>>> >>>>>> On 2/13/2020 5:35 PM, Heikki Krogerus wrote: >>>>>>> On Thu, Feb 13, 2020 at 02:00:14PM +0200, Heikki Krogerus wrote: >>>>>>>>> I am using CCG based UCSI driver without any >>>>>>>>> modification.For I2C part i have written custom >>>>>>>>> driver. >>>>>>>>> >>>>>>>>> I have attached the trace out and dmesg crash log. >>>>>>>>> >>>>>>>>> Please have a look >>>>>>>> Thanks for the logs. Can you test the attached diff? >>>>>>> Actually, don't try that one. Try this one instead. >>>>>> Sure i will update on this on Monday. >>>>>> >>>>>> >>>>>> thanks >>>>>> >>>>>> Nehal >>>>> Patch is not solving the issue. I have attached both trace and dmesg output. >>>> How about if you try this (the attached patch) together with that >>>> previous diff? >>>> >>>> thanks, >>> Sure, infact i suspected that in first place and tried same logic but it was failed but now i will check with both patch combine and shall update. >>> >>> Thanks >>> >>> Nehal Shah >> This is still crashing . > Sorry about the slow process with this, and the late reply. > > Right now I'm out of ideas. I'll need to get my hands on the products > that allow me to reproduce the issue. Staring at the code does not > help anymore. > > I'm going to cleanup the code a little bit in any case. I'm attaching > a diff with my changes. I don't think it will fix this issue, but I > would appreciate if you tested it in any case, just to be sure. > > thanks, Thanks for the patch. But i will able to validate it after few days due to countrywide lock down. regards Nehal Shah