[PATCH V2] drm/i915: gracefully bail out when init_clock_gating-pointer is not set

Jesse Barnes jbarnes at virtuousgeek.org
Mon Jun 20 10:38:54 PDT 2011


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>

-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the dri-devel mailing list