[PATCH V2] drm/i915: gracefully bail out when init_clock_gating-pointer is not set
jbarnes at virtuousgeek.org
Sat Jul 2 11:00:07 PDT 2011
On Sat, 2 Jul 2011 17:55:53 +0200
Wolfram Sang <w.sang at pengutronix.de> wrote:
> On Mon, Jun 20, 2011 at 10:38:54AM -0700, Jesse Barnes wrote:
> > On Mon, 20 Jun 2011 19:36:11 +0200
> > Wolfram Sang <w.sang at pengutronix.de> wrote:
> > > Commit 6067aa (drm/i915: split clock gating init into per-chipset
> > > functions) introduces an init_clock_gating-pointer. There is one case,
> > > however, where it does not get set, so that caused an OOPS. Change the
> > > code to return -ENODEV in this case and propagate it to the upper
> > > layers.
> > >
> > > Signed-off-by: Wolfram Sang <w.sang at pengutronix.de>
> > > Cc: Jesse Barnes <jbarnes at virtuousgeek.org>
> > > Cc: Chris Wilson <chris at chris-wilson.co.uk>
> > > ---
> > Looks good, thanks Wolfram.
> > Reviewed-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> Just wondering: I don't see it in linux-next or the i915-branches I am aware
> of. Forgotten or intentional?
Bounce it over to Keith with the reviewed-by and he'll pick it up.
Jesse Barnes, Intel Open Source Technology Center
More information about the dri-devel