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

2011-07-02 Thread Wolfram Sang
On Mon, Jun 20, 2011 at 10:38:54AM -0700, Jesse Barnes wrote: > On Mon, 20 Jun 2011 19:36:11 +0200 > Wolfram Sang 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 no

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

2011-07-02 Thread Jesse Barnes
On Sat, 2 Jul 2011 17:55:53 +0200 Wolfram Sang 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 wrote: > > > > > Commit 6067aa (drm/i915: split clock gating init into per-chipset > > > functions) introduces an init_cl

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

2011-07-02 Thread Jesse Barnes
On Sat, 2 Jul 2011 17:55:53 +0200 Wolfram Sang 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 wrote: > > > > > Commit 6067aa (drm/i915: split clock gating init into per-chipset > > > functions) introduces an init_cl

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

2011-07-02 Thread Wolfram Sang
On Mon, Jun 20, 2011 at 10:38:54AM -0700, Jesse Barnes wrote: > On Mon, 20 Jun 2011 19:36:11 +0200 > Wolfram Sang 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 no

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

2011-06-20 Thread Wolfram Sang
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: W

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

2011-06-20 Thread Jesse Barnes
On Mon, 20 Jun 2011 19:36:11 +0200 Wolfram Sang 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

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

2011-06-20 Thread Jesse Barnes
On Mon, 20 Jun 2011 19:36:11 +0200 Wolfram Sang 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

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

2011-06-20 Thread Wolfram Sang
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: W