Re: [PATCH] drm/i915: add check for valid init_clock_gating-pointer

On Thu, 16 Jun 2011 00:24:39 +0200 Wolfram Sang w.sang@pengutronix.de wrote:

Commit 6067aa (drm/i915: split clock gating init into per-chipset functions) unconditionally calls the newly created init_clock_gating-pointer. There is one case, however, where it does not get set:

if (HAS_PCH_SPLIT(dev)) { ... } else dev_priv->display.update_wm = NULL; }

We'll only hit this path on non-existent hardware. Since a clock gating routine is required I'd rather just see the panic and add a new routine at that time (i.e. what we normally do during bringup).