On 06/13/2011 10:43 AM, Somebody in the thread at some point said:

Hi jserv -

Can you review Kan-Ru's patch?  Since Android assumes full-functional
PM, it is vital to improve the resuming behavior of OMAP framebuffer.

You're right about needing working suspend / resume... the idea sounds fine to my limited understanding of the since it's just setting again the mode already in the fb info. I don't understand the locking situation in console code but I guess the console can't wake until the frame buffer has resumed.

There are just some style comments from me.

+static int omapfb_resume(struct device *dev)
+{
+       struct platform_device *pdev = to_platform_device(dev);
+       struct omapfb2_device *fbdev = platform_get_drvdata(pdev);
+       int i;

lose all the { }

+       if (fbdev != NULL) {

                if (fbdev)

+               for (i = 0; i<  fbdev->num_fbs; i++) {
+                       omapfb_set_par(fbdev->fbs[i]);
+               }
+       }

Blank line here

+       return 0;
+}

-Andy

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to