On Wed, Jun 26, 2013 at 07:35:18PM -0300, Paulo Zanoni wrote: > 2013/6/12 Egbert Eich <[email protected]>: > > Daniel Vetter writes: > > > The usual pattern for our sub-function irq_handlers is that they check > > > for the no-irq case themselves. This results in more streamlined code > > > in the upper irq handlers. > > > > > > Cc: Egbert Eich <[email protected]> > > > Signed-off-by: Daniel Vetter <[email protected]> > > > --- > > > drivers/gpu/drm/i915/i915_irq.c | 33 +++++++++++++++++---------------- > > > 1 file changed, 17 insertions(+), 16 deletions(-) > > > > > > > [reviewed code deleted] > > > > Reviewed-by: Egbert Eich <[email protected]> > > Patches 4, 5, 6 and 7 (all reviewed by Egbert) are also Reviewed-by: > Paulo Zanoni <[email protected]> > > But I do have to notice they don't apply anymore due to > s/hpd_status_i965/hpd_status_i915/
Yeah, I'll fix up the patches and resend the first 7 anew. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
