[Intel-gfx] [PATCH 1/2] drm/i915:merge ring_put/get_irq into bsd_ring_put/get_irq

Feng, Boqun boqun.feng at intel.com
Thu Apr 28 05:23:40 CEST 2011


I got it!
Thanks for reminding me. :)

-----Original Message-----
From: Keith Packard [mailto:keithp at keithp.com] 
Sent: Thursday, April 28, 2011 10:41 AM
To: Feng, Boqun; intel-gfx at lists.freedesktop.org
Subject: RE: [Intel-gfx] [PATCH 1/2] drm/i915:merge ring_put/get_irq into bsd_ring_put/get_irq

On Thu, 28 Apr 2011 10:06:51 +0800, "Feng, Boqun" <boqun.feng at intel.com> wrote:
> I have discussed this with Chris in my earlier patch.
> 
> This change is a clean-up, since ring_put_irq and ring_get_irq are 
> only used by bsd_ring_put_irq and bsd_ring_get_irq.
> 
> And once this change is made, it is more clear to see the difference 
> between g4x and ironlake BSD interrupt control interface, because they 
> are handled in a single function and they are different at the 
> interrupt mask reg addresss as well as the interrupt flag

please put important details like that in the commit message; having some way to evaluate the utility of the patch is very important for something which doesn't actually change how the code works.

--
keith.packard at intel.com



More information about the Intel-gfx mailing list