build fails after latest 'git pull'
Rafal, After a 'git pull' last night, the build of the radeonhd driver failed with the following error. Recall, I have Rafal's downclocking patches installed and I had to do a commit -a before git pull would work properly. I was checking to see if any new packages addressed the heat issue with my laptop. Here is the error: /usr/include/xorg/xf86str.h:1098: warning: comma at end of enumerator list mv -f .deps/radeonhd_drv_la-rhd_pll.Tpo .deps/radeonhd_drv_la-rhd_pll.Plo /bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I.. -I./AtomBios/includes -I/usr/include/xorg -I/usr/include/pixman-1 -I/usr/include/drm -I/usr/include/X11/dri -Wall -Wextra -Wno-unused-parameter -Werror-implicit-function-declaration -Wstrict-aliasing -Wstrict-overflow -Wpointer-arith -Woverlength-strings -Wvolatile-register-var -Winit-self -Wbad-function-cast -Wstrict-prototypes -Wold-style-definition -Wnested-externs -Wdisabled-optimization -pedantic -Wno-long-long -Wno-variadic-macros -g -O2 -finline-functions -MT radeonhd_drv_la-rhd_pm.lo -MD -MP -MF .deps/radeonhd_drv_la-rhd_pm.Tpo -c -o radeonhd_drv_la-rhd_pm.lo `test -f 'rhd_pm.c' || echo './'`rhd_pm.c libtool: compile: gcc -DHAVE_CONFIG_H -I. -I.. -I./AtomBios/includes -I/usr/include/xorg -I/usr/include/pixman-1 -I/usr/include/drm -I/usr/include/X11/dri -Wall -Wextra -Wno-unused-parameter -Werror-implicit-function-declaration -Wstrict-aliasing -Wstrict-overflow -Wpointer-arith -Woverlength-strings -Wvolatile-register-var -Winit-self -Wbad-function-cast -Wstrict-prototypes -Wold-style-definition -Wnested-externs -Wdisabled-optimization -pedantic -Wno-long-long -Wno-variadic-macros -g -O2 -finline-functions -MT radeonhd_drv_la-rhd_pm.lo -MD -MP -MF .deps/radeonhd_drv_la-rhd_pm.Tpo -c rhd_pm.c -fPIC -DPIC -o .libs/radeonhd_drv_la-rhd_pm.o In file included from /usr/include/xorg/xf86.h:47, from rhd_pm.c:33: /usr/include/xorg/xf86str.h:1098: warning: comma at end of enumerator list rhd_pm.c: In function ‘RHDPmSetClock’: rhd_pm.c:108: error: expected expression before ‘<<’ token rhd_pm.c:131:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:206:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:283: error: expected declaration or statement at end of input make[3]: *** [radeonhd_drv_la-rhd_pm.lo] Error 1 make[3]: Leaving directory `/home/david/archlinux/apps/radeonhd/src' make[2]: *** [all] Error 2 make[2]: Leaving directory `/home/david/archlinux/apps/radeonhd/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/david/archlinux/apps/radeonhd' make: *** [all] Error 2 Before I start deleting commas, I thought I would check with the experts. What do I need to do or delete to get my radeonhd build directory back in shape ? -- David C. Rankin, J.D.,P.E. Rankin Law Firm, PLLC 510 Ochiltree Street Nacogdoches, Texas 75961 Telephone: (936) 715-9333 Facsimile: (936) 715-9339 www.rankinlawfirm.com -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
On Jun 16, 09 09:48:34 -0500, David C. Rankin wrote:
rhd_pm.c: In function ‘RHDPmSetClock’: rhd_pm.c:108: error: expected expression before ‘<<’ token rhd_pm.c:131:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:206:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:283: error: expected declaration or statement at end of input
Sounds like you committed a conflicted merge...
Matthias
--
Matthias Hopf
2009/6/16 Matthias Hopf
On Jun 16, 09 09:48:34 -0500, David C. Rankin wrote:
rhd_pm.c: In function ‘RHDPmSetClock’: rhd_pm.c:108: error: expected expression before ‘<<’ token rhd_pm.c:131:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:206:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:283: error: expected declaration or statement at end of input
Sounds like you committed a conflicted merge...
Matthias
Right, one of the ways may be to checkout clean branch, and just remove earlier: git checkout -t -b clean master/origin git branch -D name_of_old_one -- Rafał Miłecki -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
Matthias Hopf wrote:
On Jun 16, 09 09:48:34 -0500, David C. Rankin wrote:
rhd_pm.c: In function ‘RHDPmSetClock’: rhd_pm.c:108: error: expected expression before ‘<<’ token rhd_pm.c:131:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:206:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:283: error: expected declaration or statement at end of input
Sounds like you committed a conflicted merge...
Matthias
Yep, Now how do I get rid of it? Just delete and clone again? -- David C. Rankin, J.D.,P.E. Rankin Law Firm, PLLC 510 Ochiltree Street Nacogdoches, Texas 75961 Telephone: (936) 715-9333 Facsimile: (936) 715-9339 www.rankinlawfirm.com -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
On Tue, Jun 16, 2009 at 5:32 PM, David C.
Rankin
Matthias Hopf wrote:
On Jun 16, 09 09:48:34 -0500, David C. Rankin wrote:
rhd_pm.c: In function ‘RHDPmSetClock’: rhd_pm.c:108: error: expected expression before ‘<<’ token rhd_pm.c:131:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:206:9: error: invalid suffix "a3fa92b6fdcd86b502973c947e5aed1bd3e1ada" on integer constant rhd_pm.c:283: error: expected declaration or statement at end of input
Sounds like you committed a conflicted merge...
Now how do I get rid of it? Just delete and clone again?
Just checkout again the offending file. Luca -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
2009/6/16 Luca Tettamanti
On Tue, Jun 16, 2009 at 5:32 PM, David C.
Now how do I get rid of it? Just delete and clone again?
Just checkout again the offending file.
If this is the only messed file, then yes :) git checkout src/rhd_pm.c -- Rafał Miłecki -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
On or about Tuesday 16 June 2009 at approximately 10:47:13 Rafał Miłecki composed:
git checkout src/rhd_pm.c
Now what? I did that and I'm in vi looking at:
commit f0f2ae49762f2e9b78486bedf998281130156604
Merge: 0de8b48 8a3fa92
Author: David C. Rankin
2009/6/16 David C. Rankin, J.D.,P.E.
On or about Tuesday 16 June 2009 at approximately 10:47:13 Rafał Miłecki composed:
git checkout src/rhd_pm.c
Now what? I did that and I'm in vi looking at:
Er, is this already merged/commited...? Just checkout origin/master as new clean branch, as I said :) This will be the easiest. -- Rafał Miłecki -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
On or about Tuesday 16 June 2009 at approximately 12:29:46 Rafał Miłecki composed:
2009/6/16 David C. Rankin, J.D.,P.E.
: On or about Tuesday 16 June 2009 at approximately 10:47:13 Rafał Miłecki
composed:
git checkout src/rhd_pm.c
Now what? I did that and I'm in vi looking at:
Er, is this already merged/commited...? Just checkout origin/master as new clean branch, as I said :) This will be the easiest.
-- Rafał Miłecki
err, um ... Crap! I have got to "get" better with "git". Rafal, I'm stuck. Obviously the fix is simple and it is probably a simple as applying the randr patch. 12:53 alchemy:~/archlinux/apps/radeonhd> git pull Updating 08acc05..8a3fa92 Fast forward src/r600_reg_r6xx.h | 2 +- src/r6xx_accel.c | 1 + src/rhd_atombios.c | 135 ++++++++++++++++++++++++++++++++++++++++++++++- src/rhd_atombios.h | 5 ++ src/rhd_cs.c | 2 +- src/rhd_dig.c | 6 +- src/rhd_driver.c | 144 ++++++++++++++++----------------------------------- src/rhd_id.c | 16 ++++-- src/rhd_lvtma.c | 4 +- src/rhd_mc.c | 7 ++- src/rhd_pll.c | 12 ++++- src/rhd_pm.c | 119 +++++++++++++++++++++++++++++++++++------- src/rhd_pm.h | 6 ++- src/rhd_randr.c | 74 ++++++++++++++++++++++++-- src/rhd_regs.h | 2 - 15 files changed, 389 insertions(+), 146 deletions(-) 12:53 alchemy:~/archlinux/apps/radeonhd> git show-branch [master] randr: Reformat AtomBIOS parameter correctly when setting. 12:53 alchemy:~/archlinux/apps/radeonhd> git show 12:54 alchemy:~/archlinux/apps/radeonhd> git branch * master 12:55 alchemy:~/archlinux/apps/radeonhd> git checkout -t -b clean master/origin fatal: git checkout: updating paths is incompatible with switching branches. Did you intend to checkout 'master/origin' which can not be resolved as commit? Can you help a brother out? What do I need to do to get back in sync? Thanks. -- David C. Rankin, J.D.,P.E. Rankin Law Firm, PLLC 510 Ochiltree Street Nacogdoches, Texas 75961 Telephone: (936) 715-9333 Facsimile: (936) 715-9339 www.rankinlawfirm.com -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
2009/6/16 David C. Rankin, J.D.,P.E.
On or about Tuesday 16 June 2009 at approximately 12:29:46 Rafał Miłecki composed:
2009/6/16 David C. Rankin, J.D.,P.E.
: On or about Tuesday 16 June 2009 at approximately 10:47:13 Rafał Miłecki
composed:
git checkout src/rhd_pm.c
Now what? I did that and I'm in vi looking at:
Er, is this already merged/commited...? Just checkout origin/master as new clean branch, as I said :) This will be the easiest.
-- Rafał Miłecki
err, um ... Crap! I have got to "get" better with "git".
Rafal, I'm stuck.
Obviously the fix is simple and it is probably a simple as applying the randr patch.
12:53 alchemy:~/archlinux/apps/radeonhd> git pull Updating 08acc05..8a3fa92 Fast forward src/r600_reg_r6xx.h | 2 +- src/r6xx_accel.c | 1 + src/rhd_atombios.c | 135 ++++++++++++++++++++++++++++++++++++++++++++++- src/rhd_atombios.h | 5 ++ src/rhd_cs.c | 2 +- src/rhd_dig.c | 6 +- src/rhd_driver.c | 144 ++++++++++++++++----------------------------------- src/rhd_id.c | 16 ++++-- src/rhd_lvtma.c | 4 +- src/rhd_mc.c | 7 ++- src/rhd_pll.c | 12 ++++- src/rhd_pm.c | 119 +++++++++++++++++++++++++++++++++++------- src/rhd_pm.h | 6 ++- src/rhd_randr.c | 74 ++++++++++++++++++++++++-- src/rhd_regs.h | 2 - 15 files changed, 389 insertions(+), 146 deletions(-) 12:53 alchemy:~/archlinux/apps/radeonhd> git show-branch [master] randr: Reformat AtomBIOS parameter correctly when setting. 12:53 alchemy:~/archlinux/apps/radeonhd> git show 12:54 alchemy:~/archlinux/apps/radeonhd> git branch * master 12:55 alchemy:~/archlinux/apps/radeonhd> git checkout -t -b clean master/origin fatal: git checkout: updating paths is incompatible with switching branches. Did you intend to checkout 'master/origin' which can not be resolved as commit?
Can you help a brother out? What do I need to do to get back in sync? Thanks.
Try this: git commit -a -m garbages git checkout -t -b clean origin/master git pull make You (or me in instruction) mistaken origin/master with master/origin. -- Rafał Miłecki -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
On or about Tuesday 16 June 2009 at approximately 13:06:52 Rafał Miłecki composed:
2009/6/16 David C. Rankin, J.D.,P.E.
: On or about Tuesday 16 June 2009 at approximately 12:29:46 Rafał Miłecki
composed:
2009/6/16 David C. Rankin, J.D.,P.E.
: On or about Tuesday 16 June 2009 at approximately 10:47:13 Rafał Miłecki
composed:
git checkout src/rhd_pm.c
Now what? I did that and I'm in vi looking at:
Er, is this already merged/commited...? Just checkout origin/master as new clean branch, as I said :) This will be the easiest.
-- Rafał Miłecki
err, um ... Crap! I have got to "get" better with "git".
Rafal, I'm stuck.
Obviously the fix is simple and it is probably a simple as applying the randr patch.
12:53 alchemy:~/archlinux/apps/radeonhd> git pull Updating 08acc05..8a3fa92 Fast forward src/r600_reg_r6xx.h | 2 +- src/r6xx_accel.c | 1 + src/rhd_atombios.c | 135 ++++++++++++++++++++++++++++++++++++++++++++++- src/rhd_atombios.h | 5 ++ src/rhd_cs.c | 2 +- src/rhd_dig.c | 6 +- src/rhd_driver.c | 144 ++++++++++++++++----------------------------------- src/rhd_id.c | 16 ++++-- src/rhd_lvtma.c | 4 +- src/rhd_mc.c | 7 ++- src/rhd_pll.c | 12 ++++- src/rhd_pm.c | 119 +++++++++++++++++++++++++++++++++++------- src/rhd_pm.h | 6 ++- src/rhd_randr.c | 74 ++++++++++++++++++++++++-- src/rhd_regs.h | 2 - 15 files changed, 389 insertions(+), 146 deletions(-) 12:53 alchemy:~/archlinux/apps/radeonhd> git show-branch [master] randr: Reformat AtomBIOS parameter correctly when setting. 12:53 alchemy:~/archlinux/apps/radeonhd> git show 12:54 alchemy:~/archlinux/apps/radeonhd> git branch * master 12:55 alchemy:~/archlinux/apps/radeonhd> git checkout -t -b clean master/origin fatal: git checkout: updating paths is incompatible with switching branches. Did you intend to checkout 'master/origin' which can not be resolved as commit?
Can you help a brother out? What do I need to do to get back in sync? Thanks.
Try this: git commit -a -m garbages git checkout -t -b clean origin/master git pull make
You (or me in instruction) mistaken origin/master with master/origin.
-- Rafał Miłecki
Thanks Rafal! Before your last email, I got it to build fine: make[3]: Leaving directory `/home/david/archlinux/apps/radeonhd/utils/conntest' make[2]: Leaving directory `/home/david/archlinux/apps/radeonhd/utils/conntest' make[2]: Entering directory `/home/david/archlinux/apps/radeonhd' make[2]: Leaving directory `/home/david/archlinux/apps/radeonhd' make[1]: Leaving directory `/home/david/archlinux/apps/radeonhd' As for the master/origin: <quote> Right, one of the ways may be to checkout clean branch, and just remove earlier: git checkout -t -b clean master/origin git branch -D name_of_old_one -- Rafał Miłecki </quote> Man you really know how to drive a git newb crazy :-) 13:26 alchemy:~/archlinux/apps/radeonhd> git commit -a -m garbages # On branch master # Your branch is ahead of 'origin/master' by 2 commits. # nothing to commit (working directory clean) 13:29 alchemy:~/archlinux/apps/radeonhd> git checkout -t -b clean origin/master Branch clean set up to track remote branch master from origin. Switched to a new branch 'clean' We are up to date, compile is fine, and I'll report back on any changes/improvements to the exhaust or palm-rest temps. Thanks again! -- David C. Rankin, J.D.,P.E. Rankin Law Firm, PLLC 510 Ochiltree Street Nacogdoches, Texas 75961 Telephone: (936) 715-9333 Facsimile: (936) 715-9339 www.rankinlawfirm.com -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org
On Jun 16, 09 13:31:09 -0500, David C. Rankin, J.D.,P.E. wrote:
13:29 alchemy:~/archlinux/apps/radeonhd> git checkout -t -b clean origin/master Branch clean set up to track remote branch master from origin. Switched to a new branch 'clean'
Note that the branch you're now working on is 'clean' and not 'master'
anymore. Otherwise nothing changes.
If you want to get rid of all your changes lately, and just continue to
follow upstream again, the best is typically:
git checkout master (because you have 'clean' checked out)
git reset --hard origin (beware - git reset --hard *really* deletes content)
git pull
HTH
Matthias
--
Matthias Hopf
participants (5)
-
David C. Rankin
-
David C. Rankin, J.D.,P.E.
-
Luca Tettamanti
-
Matthias Hopf
-
Rafał Miłecki