[PATCH] Add EXA support

Michel Dänzer michel at daenzer.net
Wed Jul 11 08:13:14 PDT 2012


On Mon, 2012-07-09 at 16:44 -0700, Connor Behan wrote:
> On 06/07/12 03:52 AM, Michel Dänzer wrote:
> > On Don, 2012-07-05 at 17:51 -0700, Connor Behan wrote: 
> >> For real this time. This allows the r128 driver to continue having 2D
> > That this patch is for the r128 driver should be mentioned on the
> > subject line (even if it was submitted to the xorg-driver-ati list :),
> > e.g. as '[PATCH xf86-video-r128] ...'
> >
> > OTOH it doesn't need to be mentioned in the Git commit log, which is
> > everything between the Subject: line and the '---' line. You can include
> > additional information for the patch submission between the latter and
> > the actual patch.
> I just uploaded a patch to the bug tracker which gets rid of the
> warnings and uses better conventions for the subject line and commit
> summary. Should I send it to this list or xorg-driver-ati?

I think the latter would be better, but either is fine.

BTW, extra points for stripping the unrelated PLL and other
whitespace-only changes from the patch. :) Otherwise it looks pretty
good at a quick glance. 


> > BTW, as you seem to be the most active r128 driver developer ATM, you
> > should consider applying for a freedesktop.org account and maintaining
> > the driver in Git.
> >
> Thanks for the encouragement, I think I will do this. As I'm sure you've
> noticed, I'm not as knowledgeable as the other module maintainers. So
> even if I get full git access in the future, I will still make sure any
> large patches I write get reviewed before being pushed.

Sounds good.


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the xorg-devel mailing list