Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 431694 - x11-drivers/ati-drivers-12.8 version bump
Summary: x11-drivers/ati-drivers-12.8 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Luca Barbato
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2012-08-17 03:55 UTC by zhoujingmiller
Modified: 2012-09-09 22:48 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Updated ati-powermode-opt-path Patch for 12.8 onwards (ati-powermode-opt-path-3.patch,1.78 KB, patch)
2012-08-17 13:50 UTC, zhoujingmiller
Details | Diff
Updated Patch for ati-powermode-opt-path (ati-powermode-opt-path-3.patch,1.89 KB, patch)
2012-08-17 13:55 UTC, zhoujingmiller
Details | Diff
Catalyst 12.8 ebuild (ati-drivers-12.8.ebuild,19.38 KB, text/plain)
2012-08-17 14:12 UTC, zhoujingmiller
Details
ati-powermode-opt-path-3.patch (ati-powermode-opt-path-3.patch,1.44 KB, patch)
2012-08-17 14:45 UTC, zhoujingmiller
Details | Diff
ati-drivers-12.8.ebuild (ati-drivers-12.8.ebuild,19.52 KB, text/plain)
2012-08-17 15:00 UTC, zhoujingmiller
Details
ati-drivers-12.8.ebuild (ati-drivers-12.8.ebuild,19.51 KB, text/plain)
2012-08-17 15:07 UTC, zhoujingmiller
Details

Note You need to log in before you can comment on or make changes to this bug.
Description zhoujingmiller 2012-08-17 03:55:16 UTC
Per http://support.amd.com/us/gpudownload/linux/Pages/radeon_linux.aspx , Catalyst 12.8 has been released. However, do note that the URL to the driver file has changed. The correct link is http://www2.ati.com/drivers/linux/amd-driver-installer-12-8-x86.x86_64.zip , which ends with a .zip suffix, as opposed to .run. Please update the ebuild accordantly.

Please update x11-drivers/ati-drivers to 12.8. Thanks.
Comment 1 zhoujingmiller 2012-08-17 09:20:59 UTC
Also, please drop ati-drivers-old_rsp.patch since it is already covered in the updated module source. Thanks.
Comment 2 zhoujingmiller 2012-08-17 13:50:59 UTC
Created attachment 321556 [details, diff]
Updated ati-powermode-opt-path Patch for 12.8 onwards
Comment 3 zhoujingmiller 2012-08-17 13:55:57 UTC
Created attachment 321558 [details, diff]
Updated Patch for ati-powermode-opt-path

Please ignore the patch committed before this one.
Comment 4 zhoujingmiller 2012-08-17 13:57:05 UTC
Comment on attachment 321556 [details, diff]
Updated ati-powermode-opt-path Patch for 12.8 onwards

>@@ -4,10 +4,12 @@
> # Control script for ACPI lid state and AC adapter state
> #
> 
>+aticonfig='/opt/bin/aticonfig'
>+
> getXuser() {
>-        user=`finger| grep -m1 ":$displaynum " | awk '{print $1}'`
>+        user=`who| grep -m1 ":$displaynum " | awk '{print $1}'`
>         if [ x"$user" = x"" ]; then
>-                user=`finger| grep -m1 ":$displaynum" | awk '{print $1}'`
>+                user=`who| grep -m1 ":$displaynum" | awk '{print $1}'`
>         fi
>         if [ x"$user" != x"" ]; then
>                 userhome=`getent passwd $user | cut -d: -f6`
>@@ -47,7 +49,7 @@
> done
> 
> #If PPLIB is enabled
>-su $user -c '/usr/bin/aticonfig --pplib-cmd="get version"' | grep PPLIB
>+su $user -c '$aticonfig --pplib-cmd="get version"' | grep PPLIB
> if [ $? = 0 ]; then
>    echo "Has PPLIB"
>    has_pplib=1
>@@ -61,15 +63,15 @@
> if [ ${lid_closed} -eq 1 -o ${on_dc} -eq 1 ]; then
>     echo "Low power"
>     if [ ${has_pplib} -eq 1 ]; then
>-        su $user -c '/usr/bin/aticonfig --pplib-cmd="notify psrc dc"'
>+        su $user -c '$aticonfig --pplib-cmd="notify psrc dc"'
>     else
>-        su $user -c "/usr/bin/aticonfig --set-powerstate=1 --effective=now"
>+        su $user -c "$aticonfig --set-powerstate=1 --effective=now"
>     fi
> else
>     echo "high power"
>     if [ ${has_pplib} -eq 1 ]; then
>-        su $user -c '/usr/bin/aticonfig --pplib-cmd="notify psrc ac"'
>+        su $user -c '$aticonfig --pplib-cmd="notify psrc ac"'
>     else
>-        su $user -c "/usr/bin/aticonfig --set-powerstate=3 --effective=now"
>+	su $user -c "$aticonfig --set-powerstate=3 --effective=now"
>     fi
> fi
Comment 5 zhoujingmiller 2012-08-17 14:12:28 UTC
Created attachment 321560 [details]
Catalyst 12.8 ebuild

My first ebuild.

Please be brave to point out any mistake/pitfall encountered. Thanks.
Comment 6 zhoujingmiller 2012-08-17 14:14:33 UTC
Disclaimer: Every ebuild/patch herein is based on some Gentoo's ebuild/patch and I claim no copyright over them, period.
Comment 7 zhoujingmiller 2012-08-17 14:45:06 UTC
Created attachment 321564 [details, diff]
ati-powermode-opt-path-3.patch

Please ignore all previous patches with the same file name.
Comment 8 zhoujingmiller 2012-08-17 15:00:32 UTC
Created attachment 321570 [details]
ati-drivers-12.8.ebuild

Please ignore any previous ebuild commits.
Comment 9 zhoujingmiller 2012-08-17 15:07:22 UTC
Created attachment 321572 [details]
ati-drivers-12.8.ebuild

Spaces and tabs . . .
Comment 10 Jeroen Roovers (RETIRED) gentoo-dev 2012-08-17 16:10:24 UTC
Comment on attachment 321572 [details]
ati-drivers-12.8.ebuild

--- ati-drivers-12.6.ebuild     2012-08-01 02:49:16.000000000 +0200
+++ -   2012-08-17 18:10:14.154710722 +0200
@@ -1,6 +1,6 @@
 # Copyright 1999-2012 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/x11-drivers/ati-drivers/ati-drivers-12.6.ebuild,v 1.3 2012/07/26 15:08:38 johu Exp $
+# $Header: /var/cvsroot/gentoo-x86/x11-drivers/ati-drivers/ati-drivers-12.8.ebuild,v 1.3 2012/07/26 15:08:38 johu Exp $
 
 EAPI=4

@@ -11,11 +11,11 @@
 MY_V=( $(get_version_components) )
 if [[ ${MY_V[2]} != beta ]]; then
        ATI_URL="http://www2.ati.com/drivers/linux/"
-       SRC_URI="${ATI_URL}/amd-driver-installer-${PV/./-}-x86.x86_64.run"
+       SRC_URI="${ATI_URL}/amd-driver-installer-${PV/./-}-x86.x86_64.zip"
        FOLDER_PREFIX="common/"
 else   
        #SRC_URI="https://launchpad.net/ubuntu/natty/+source/fglrx-installer/2:${PV}-0ubuntu1/+files/fglrx-installer_${PV}.orig.tar.gz"
-       SRC_URI="http://www2.ati.com/drivers/hotfix/catalyst_12.6_hotfixes/amd-driver-installer-8.98-x86.x86_64.zip"
+       #SRC_URI="http://www2.ati.com/drivers/hotfix/catalyst_12.6_hotfixes/amd-driver-installer-8.98-x86.x86_64.zip"
        FOLDER_PREFIX="common/"
 fi
 IUSE="debug +modules multilib qt4 static-libs"
@@ -289,7 +289,8 @@
                unpack ${A}
                RUN="${S}/${A/%.zip/.run}"
        else
-               RUN="${DISTDIR}/${A}"
+               unpack ${A}
+               RUN="${S}/amd-driver-installer-8.982-x86.x86_64.run"
        fi
        sh ${RUN} --extract "${S}" # 2>&1 > /dev/null || die
 }
@@ -328,14 +329,11 @@
                || die "Replacing 'finger' with 'who' failed."
        # Adjust paths in the script from /usr/X11R6/bin/ to /opt/bin/ and
        # add function to detect default state.
-       epatch "${FILESDIR}"/ati-powermode-opt-path-2.patch
+       epatch "${FILESDIR}"/ati-powermode-opt-path-3.patch

        #fixes bug #420751
        epatch "${FILESDIR}"/ati-drivers-do_mmap.patch

-       # see http://ati.cchtml.com/show_bug.cgi?id=495
-       epatch "${FILESDIR}"/ati-drivers-old_rsp.patch
-       
        cd "${MODULE_DIR}"

        # bugged fglrx build system, this file should be copied by hand
Comment 11 Jim Tupper 2012-08-25 18:20:50 UTC
I can confirm I've just installed this for my 7950 and it works fine.
Comment 12 Rick Farina (Zero_Chaos) gentoo-dev 2012-09-09 22:47:31 UTC
+  09 Sep 2012; Rick Farina <zerochaos@gentoo.org> +ati-drivers-12.8.ebuild,
+  +files/ati-drivers-x32_something_something.patch,
+  +files/ati-powermode-opt-path-3.patch:

At this time this commit is a merger between this bug, the ebuild in the x11 overlay, and my own works. The gentoo tree is the most up to date as of this time.

Thanks to zhoujingmiller for the powermode patch update, I like it and accepted it.

Of special note, although this bug and x11 overlay dropped old_rsp patch I *did not*.  Only part of this patch was merged upstream and I believe the second part is still required (and may fix x32 users). So crossed my fingers and commited.

thanks to Ke for helping me understand the changes in the x11 overlay and special thanks to mrueg for actually testing the driver works (I lack the hardware).