On Tuesday 12 July 2011 03:21:05 Ming Lei wrote: > On Mon, Jul 11, 2011 at 6:44 PM, Laurent Pinchart wrote: > > That's unfortunately not acceptable as-is. If two cameras are connected > > to the system, and only one of them doesn't support suspend/resume, the > > other will be affected by your patch. > > Yes, other cameras may be affected, but they still can work well, so > the patch still makes sense. They can still work, but not optimally, as they will be reset instead of suspended/resumed. That's not acceptable. > > Have you tried to investigate why suspend/resume fails for the > > above-mentioned camera, instead of working around the problem ? > > I have investigated the problem, and not found failures in the > suspend/resume path, > either .suspend or .resume callback return successful, but no stream > packets are received from camera any longer after resume from sleep. Once > doing a unbind& bind will make the camera work again. > > Could you give any suggestions to help to find the root cause of the > problem? -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
