Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [RFC] full suspend/resume support for i915 DRM driver | Date | Wed, 24 Oct 2007 23:22:35 +0200 |
| |
On Wednesday, 24 October 2007 17:18, Jesse Barnes wrote: > On Wednesday, October 24, 2007 6:35:14 am Pavel Machek wrote: > > Hi! > > > > > We seem to see a lot of bug reports along the lines of, "my machine > > > resumes but I can't see X" or, "I can see X but only with a bright > > > flashlight", etc. These sorts of problems are due to the fact that > > > the X server isn't designed to do full state save/restore, and none > > > of the available kernel drivers do it on its behalf. > > > > > > Since intelfb and the rest of the Intel drivers are fairly incompatible, > > > this patch makes the DRM bind to the PCI device so it can register real > > > suspend/resume handlers. Those handlers take care of saving and > > > restoring enough state for X to come back reliably on at least one of my > > > problematic test machines, but text mode still has trouble (still > > > debugging VGA state save/restore, including trying to save/restore > > > actual VRAM contents for possible hibernate support). > > > > > > How does this approach look? Is a new DRM driver flag a good thing for > > > similar situations with other drivers? Thoughts? > > > > Looks okay to me... from very quick look. > > > > > + if (!i915_pipe_enabled(dev, pipe)) > > > + return; > > > + > > > + if (pipe == PIPE_A) > > > + array = dev_priv->savePaletteA; > > > > coding style, we probably want save_palette_A. > > Yeah, I tried not to pull over uglies from the X code but I guess I forgot > this bit. I should also update the copyright. > > > > + unsigned long reg = pipe == PIPE_A ? PALETTE_A : PALETTE_B; > > > > Uff. Mixing = and == and ? in one expression is evil. > > I could put parens around it if you think that would help, or just move it to > a new line...
I'd use parens around "pipe == PIPE_A".
Greetings, Rafael - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |