[PATCH sis 00/15] a bunch of cleanups, and a taste of things to come

Jeremy Huddleston jeremyhu at apple.com
Sun Oct 30 12:27:59 PDT 2011


Wow, I wish I knew about this series before I started cleaning up the drivers...

Where was this series sent?  I don't see it in my xorg-devel log.

By all means, take it over.  Do you want to maintain it?  If so, get yourself an fd.o account, so you can get yourself commit access to the driver.  See http://www.freedesktop.org/wiki/AccountRequests

--Jeremy

On Oct 30, 2011, at 01:44, Tormod Volden wrote:

> On Wed, Apr 27, 2011 at 10:59 PM, TImo Aaltonen <tjaalton at ubuntu.com> wrote:
>> On 13.04.2011 18:22, TImo Aaltonen wrote:
>>> On 06.04.2011 19:50, Timo Aaltonen wrote:
>>>> Here's a first set of patches to xf86-video-sis, hoping to narrow down
>>>> the diff against Thomas Winischofer's 'sisfree' (*) package and the
>>>> forked versions from Intel and SiS (and Mandriva).
>>>> 
>>>> The first 13 are more or less self-explanatory, but the two final ones
>>>> are moving things around to make the code more readable, among other things.
>>>> 
>>>> Every commit is build-tested against xserver 1.10 headers.
>>>> 
>>>> * http://www.winischhofer.net/linuxsispart4.shtml#download
>>>> 
>>>> Timo Aaltonen (15):
>>>>   Write out remaining IS_SIS* macros.
>>>>   Replace deprecated x(c)alloc/xfree with m/calloc/free
>>>>   Fix "format not a string literal and no format arguments" compile
>>>>     warnings
>>>>   Fix "suggest parentheses around operand of ‘!’" compiler warnings.
>>>>   Fix compile warnings of uninitialized variables
>>>>   Declare mmioFlags only if XSERVER_LIBPCIACCESS isn't defined
>>>>   Purge obsolete and unused SIS_CP checks
>>>>   Drop useless checks and relevant code for old XFree86 releases
>>>>   Drop unused and non-working XAA trapezoid support for 310 series and
>>>>     later
>>>>   Remove unused code from sis_video.c
>>>>   Remove unused code from SISPutImageBlit()
>>>>   Remove unused & non-working Xv deinterlacer code
>>>>   Remove deprecated XV SD interface
>>>>   Move configurable values from sis.h to sis_config.h
>>>>   Move all TV related functions fron sis_driver.c to sis_vb.c
>>>> 
>>>>  src/init.c          |   35 +-
>>>>  src/init.h          |    3 -
>>>>  src/init301.c       |   62 +-
>>>>  src/init301.h       |    3 -
>>>>  src/initdef.h       |   19 +-
>>>>  src/initextx.c      |   18 +-
>>>>  src/sis.h           |   86 +--
>>>>  src/sis310_accel.c  |  754 ---------------
>>>>  src/sis6326_video.c |   60 +--
>>>>  src/sis_accel.c     |  128 ---
>>>>  src/sis_config.h    |   71 ++
>>>>  src/sis_cursor.c    |    8 +-
>>>>  src/sis_dac.h       |    2 +
>>>>  src/sis_dga.c       |    6 +-
>>>>  src/sis_dri.c       |   38 +-
>>>>  src/sis_driver.c    | 2625 ++++-----------------------------------------------
>>>>  src/sis_driver.h    |  813 +----------------
>>>>  src/sis_memcpy.c    |   10 +-
>>>>  src/sis_opt.c       |   49 +-
>>>>  src/sis_utility.c   |  527 +----------
>>>>  src/sis_vb.c        | 2259 ++++++++++++++++++++++++++++++++++++++++++--
>>>>  src/sis_vb.h        |  931 ++++++++++++++++++
>>>>  src/sis_vga.c       |    7 +-
>>>>  src/sis_video.c     |  291 +------
>>>>  src/sis_video.h     |  195 ----
>>>>  src/sis_videostr.h  |    3 -
>>>>  src/vgatypes.h      |    5 -
>>>>  27 files changed, 3490 insertions(+), 5518 deletions(-)
>>>>  create mode 100644 src/sis_config.h
>>>>  create mode 100644 src/sis_vb.h
>>>> 
>>> 
>>> Ping? Patch 04/15 got reviewed, but the rest are still waiting for input.
>> 
>> And another ping.
>> 
>> Patch 03 is apparently controversial, I could drop it from the series..
>> 
> 
> What about this patch series? If no one wants to review it, let's give
> Timo free hands on it. Since April, other people have been reinventing
> some of these patches, which is kind of stupid.
> 
> Timo, you are still interested in having this applied and willing to
> fix up any reported regressions, right? Do you have it in a public
> tree somewhere?
> 
> Regards,
> Tormod
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 



More information about the xorg-devel mailing list