| Summary: | Support for RageTheatre200 added | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | xorg | Reporter: | Bogdan Diaconescu <b_diaconescu> | ||||||
| Component: | Driver/Radeon | Assignee: | Bogdan Diaconescu <b_diaconescu> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||
| Severity: | normal | ||||||||
| Priority: | high | CC: | ajax, alexdeucher, mharris | ||||||
| Version: | git | ||||||||
| Hardware: | x86 (IA32) | ||||||||
| OS: | Linux (All) | ||||||||
| Whiteboard: | |||||||||
| i915 platform: | i915 features: | ||||||||
| Attachments: |
|
||||||||
|
Description
Bogdan Diaconescu
2005-03-20 07:53:58 UTC
Created attachment 2163 [details] [review] RageTheatre200 patch Created attachment 2164 [details]
New files for rt200 support
this patch has some whitespace issues. it looks like you're setting your tab stops to a weird width and then using ^I to do indentation. in general explicit ^I characters to mean anything other than "indent eight spaces" are a mistake. beyond that, how much testing has this patch had? I'm using vim as favourite editor with a diferent than 8 tabstop. The patch is created with cvs diff command which seems to expand the tab character to 8 spaces. The wrong indentation can be easily fixed with expand tool. I'll do that when cvs acces will be granted. The RT200 code in GATOS cvs was tested with success on couple of boards (see the GATOS mailing lists). The ideea is to broaden the testing by integrating the code into X.org. Didn't this recently get added to CVS head already? The patches have been added into cvs head. The patch has been added into cvs head. They were tested only on my card AIW9600 PRO (PAL). |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.