Hi Linus,
Can you pull the drm-linus branch from
git://git.kernel.org/pub/scm/linux/kernel/git/airlied/drm-2.6.git
It contains a bunch of cleanup patches (mutex conversion, static function,
sparse) along with new i945GM chipset support (pciid and some minor
changes) which Intel recently pushed to me, there should be nothing
contentious in here ....
Dave.
ati_pcigart.c | 17 +++++-------
drmP.h | 5 ++-
drm_auth.c | 20 +++++++-------
drm_bufs.c | 80 +++++++++++++++++++++++++++++-----------------------------
drm_context.c | 52 ++++++++++++++++++-------------------
drm_drv.c | 4 +-
drm_fops.c | 12 ++++----
drm_ioctl.c | 18 ++++++-------
drm_irq.c | 16 +++++------
drm_pciids.h | 2 +
drm_proc.c | 28 ++++++++++----------
drm_stub.c | 4 +-
drm_vm.c | 12 ++++----
i810_dma.c | 2 -
i810_drv.h | 2 -
i830_dma.c | 2 -
i830_drv.h | 3 --
i915_dma.c | 42 +++++++++++++++++++++---------
i915_drm.h | 33 +++++++++++++++++++++++
i915_drv.h | 6 ++--
i915_mem.c | 31 ++++++++++++++++++++++
radeon_cp.c | 2 -
savage_bci.c | 4 ++
savage_drv.h | 1
via_dma.c | 10 +++----
via_dmablit.c | 6 ++--
via_drv.h | 7 -----
via_irq.c | 2 -
28 files changed, 248 insertions(+), 175 deletions(-)
commit 30e2fb188194908e48d3f27a53ccea6740eb1e98
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Thu Feb 2 19:37:46 2006 +1100
sem2mutex: drivers/char/drm/
From: Arjan van de Ven <[email protected]>
Semaphore to mutex conversion.
The conversion was generated via scripts, and the result was validated
automatically via a script as well.
Signed-off-by: Arjan van de Ven <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit ce60fe02fbe737cbce09e2ba5a2ef1efd20eff73
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Thu Feb 2 19:21:38 2006 +1100
drm: drivers/char/drm/: make some functions static
From: Adrian Bunk <[email protected]>
This patch makes some needlessly global functions static.
Signed-off-by: Adrian Bunk <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit 339363c4c6fe01043c51e7d6e9fbeb8feee00841
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Thu Jan 26 08:32:14 2006 +1100
drm: Fixes sparse warnings in via_dmablit.c
Fixes the following sparse warnings:
drivers/char/drm/via_dmablit.c:111:35: warning: Using plain integer as NULL pointer
drivers/char/drm/via_dmablit.c:584:23: warning: Using plain integer as NULL pointer
Signed-off-by: Luiz Capitulino <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit de227f5f32775d86e5c780a7cffdd2e08574f7fb
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Wed Jan 25 15:31:43 2006 +1100
drm: i915 patches from Tungsten Graphics
Fix CMDBUFFER path, add heap destroy and flesh out sarea for rotation
(Tungsten Graphics)
From: Alan Hourihane <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit 507d256bae9eef7acd5049af6e3f67c24904a1e4
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Wed Jan 25 14:58:58 2006 +1100
drm: ati_pcigart: simplify page_count manipulations
From: Nick Piggin <[email protected]>
Allocate a compound page for the user mapping instead of tweaking the page
refcounts.
Signed-off-by: Nick Piggin <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit f1e5c03d34c39394781ae13543cd3355976e4812
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Wed Jan 25 14:54:15 2006 +1100
drm: use NULL instead of 0
From: Randy Dunlap <[email protected]>
Use NULL instead of 0 (sparse warnings):
drivers/char/drm/ati_pcigart.c:64:10: warning: Using plain integer as NULL
pointer
drivers/char/drm/ati_pcigart.c:130:21: warning: Using plain integer as NULL
pointer
drivers/char/drm/ati_pcigart.c:171:14: warning: Using plain integer as NULL
pointer
Signed-off-by: Randy Dunlap <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit 2fed3bd7436e8988980989493c16b4983be1a800
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Wed Jan 25 14:52:43 2006 +1100
drm: add X600 PCI IDs
From: Brice Goglin <[email protected]>
Now that Xorg 6.9/7.0 has been released, DRI is supported on more Radeon
cards without ATI proprietary drivers. I got my X300 to work without
problem. But, another Radeon X600 required to add its PCI ids to the
Radeon driver. Patch is attached.
I can't be sure about the "CHIP_RV350", I copied it from the X300 entry
(from http://dri.freedesktop.org/wiki/ATIRadeon, X600 is a rv380 chip while
X300 is a rv370). But, at least it works now.
Signed-off-by: Brice Goglin <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit 5457f38e01ae2d296ff49db42254679018f13fa9
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Wed Jan 25 14:34:33 2006 +1100
drm: add i945GM PCI ID
From: Charles F. Johnson <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
commit d59cc22f7ce48bf5454f12eec8603bff81c34cdb
Author: Dave Airlie <airlied@starflyer.(none)>
Date: Wed Jan 25 14:31:45 2006 +1100
drm: Fix sparce warning in radeon driver
From: Luiz Fernando Capitulino <[email protected]>
drivers/char/drm/radeon_cp.c:1643:31: warning: Using plain integer as NULL
pointer
Signed-off-by: Luiz Capitulino <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
On Thu, 2 Feb 2006, Dave Airlie wrote:
>
> commit 30e2fb188194908e48d3f27a53ccea6740eb1e98
> Author: Dave Airlie <airlied@starflyer.(none)>
> Date: Thu Feb 2 19:37:46 2006 +1100
>
> sem2mutex: drivers/char/drm/
>
> From: Arjan van de Ven <[email protected]>
A lot of your commits have this structure.
What do you use to apply these emails? It _looks_ like the emails are
well-behaved ("From:" at the top), yet your Author: information is wrong
and whatever script you used to do it missed it.
Linus
> >
> > commit 30e2fb188194908e48d3f27a53ccea6740eb1e98
> > Author: Dave Airlie <airlied@starflyer.(none)>
> > Date: Thu Feb 2 19:37:46 2006 +1100
> >
> > sem2mutex: drivers/char/drm/
> >
> > From: Arjan van de Ven <[email protected]>
>
> A lot of your commits have this structure.
>
> What do you use to apply these emails? It _looks_ like the emails are
> well-behaved ("From:" at the top), yet your Author: information is wrong
> and whatever script you used to do it missed it.
>
> Linus
>
I mostly apply the patches + any cleanups (most of the patches I get from
DRM CVS needs whitespace cleanups - damn X hackers) so I usually just
write the commit message by hand from the mail when I check the stuff in,
I don't know of any way when doing hand commits to easily change the
author without messing with environment variables...
Dave.
--
David Airlie, Software Engineer
http://www.skynet.ie/~airlied / airlied at skynet.ie
Linux kernel - DRI, VAX / pam_smb / ILUG
On Thu, Feb 02 2006, Dave Airlie wrote:
>
> > >
> > > commit 30e2fb188194908e48d3f27a53ccea6740eb1e98
> > > Author: Dave Airlie <airlied@starflyer.(none)>
> > > Date: Thu Feb 2 19:37:46 2006 +1100
> > >
> > > sem2mutex: drivers/char/drm/
> > >
> > > From: Arjan van de Ven <[email protected]>
> >
> > A lot of your commits have this structure.
> >
> > What do you use to apply these emails? It _looks_ like the emails are
> > well-behaved ("From:" at the top), yet your Author: information is wrong
> > and whatever script you used to do it missed it.
> >
> > Linus
> >
>
> I mostly apply the patches + any cleanups (most of the patches I get from
> DRM CVS needs whitespace cleanups - damn X hackers) so I usually just
> write the commit message by hand from the mail when I check the stuff in,
>
> I don't know of any way when doing hand commits to easily change the
> author without messing with environment variables...
I have a set of really cheasy ag-* git scripts to help with this sort of
thing. This is ag-commit. If you just do ag-commit, it'll use the
defined author/committer. If I need to commit a patch that eg Linus did
(hah, fat chance!), I'll just do ag-commit -a "Linus Torvalds
<[email protected]>". -c is for committer (which I never ended up using,
since that's always me), and you can write the changelog up front as
changelog.txt if you choose (which I never ended up using either).
I'm sure there are better ways, but it works fine for me.
#!/bin/bash
TEMP=`getopt -o a:c: --long a-long:,c-long:: -- "$@"`
eval set -- "$TEMP"
AUTHOR="Jens Axboe"
AUTHOR_EMAIL="[email protected]"
COMMITTER="Jens Axboe"
COMMITTER_EMAIL="[email protected]"
while true; do
case "$1" in
-a|--a-long)
AUTHOR=`echo $2 | cut -d '<' -f 1`
AUTHOR_EMAIL=`echo $2 | cut -d '<' -f 2 | sed s/">"//g`
shift 2;;
-c|--c-long)
COMMITTER=`echo $2 | cut -d '<' -f 1`
COMMITTER_EMAIL=`echo $2 | cut -d '<' -f 2 | sed s/">"//g`
shift 2;;
--) shift; break;;
*) echo "error"; exit 1;;
esac
done
if [ x"$AUTHOR" == 'x' ]; then
echo bad author
exit 1
fi
if [ x"$AUTHOR_EMAIL" == 'x' ]; then
echo bad author email
exit 1
fi
if [ x"$COMMITTER" == 'x' ]; then
echo bad committer
exit 1
fi
if [ x"$COMMITTER_EMAIL" == 'x' ]; then
echo bad commiter email
exit 1
fi
GIT_AUTHOR_NAME="$AUTHOR"
GIT_AUTHOR_EMAIL="$AUTHOR_EMAIL"
GIT_COMMITTER_NAME="$COMMITTER"
GIT_COMMITTER_EMAIL="$COMMITTER_EMAIL"
export GIT_AUTHOR_NAME GIT_AUTHOR_EMAIL GIT_COMMITTER_NAME GIT_COMMITTER_EMAIL
echo "Committing: $GIT_AUTHOR_NAME/$GIT_AUTHOR_EMAIL, $GIT_COMMITTER_NAME/$GIT_COMMITTER_EMAIL"
#git-commit-tree `git-write-tree` -p $(cat .git/HEAD) < changelog.txt > .git/HEAD
if [ -f changelog.txt ]; then
git commit -m changelog.txt
rm -rf changelog.txt
else
git commit
fi
unset GIT_AUTHOR_NAME GIT_AUTHOR_EMAIL GIT_COMMITTER_NAME GIT_COMMITTER_EMAIL
--
Jens Axboe