macro argument expansion bug
--0016e64ca970ad6a4804b0965894
Content-Type: text/plain; charset=ISO-8859-1
Hi,
The attached patch fixes a bug in the AMD powernow driver, the abs()
macro was defined in a way that may lead to unexpected results due to
the macro argument expansion vs operator precedence:
----------
#define abs(x) ( x < 0 ? -x : x )
[...]
while (abs(vco_fid - vco_cfid) > 2) {
----------
Cheers,
--
--------------------
Maurizio Lombardi
--0016e64ca970ad6a4804b0965894
Content-Type: application/octet-stream;
name="0001-Fix-a-macro-argument-expansion-bug.patch"
Content-Disposition: attachment;
filename="0001-Fix-a-macro-argument-expansion-bug.patch"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_gufdfc370
RnJvbSBhZTAwYjA4ZTdmYmNiY2EyN2RkM2Y5YTNkYTgwOTNmMDRhODcxNjJiIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBNYXVyaXppbyBMb21iYXJkaSA8bS5sb21iYXJkaTg1QGdtYWls
LmNvbT4KRGF0ZTogTW9uLCAzMSBPY3QgMjAxMSAxMjowMTo0OSArMDEwMApTdWJqZWN0OiBbUEFU
Q0hdIEZpeCBhIG1hY3JvIGFyZ3VtZW50IGV4cGFuc2lvbiBidWcKCi0tLQogc3lzL2Rldi9wb3dl
cm1uZy9wb3dlcm5vdy9wb3dlcm5vdy5jIHwgICAgMiArLQogMSBmaWxlcyBjaGFuZ2VkLCAxIGlu
c2VydGlvbnMoKyksIDEgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvc3lzL2Rldi9wb3dlcm1u
Zy9wb3dlcm5vdy9wb3dlcm5vdy5jIGIvc3lzL2Rldi9wb3dlcm1uZy9wb3dlcm5vdy9wb3dlcm5v
dy5jCmluZGV4IGNhN2JlMjYuLmM0NjhiNDMgMTAwNjQ0Ci0tLSBhL3N5cy9kZXYvcG93ZXJtbmcv
cG93ZXJub3cvcG93ZXJub3cuYworKysgYi9zeXMvZGV2L3Bvd2VybW5nL3Bvd2Vybm93L3Bvd2Vy
bm93LmMKQEAgLTg1LDcgKzg1LDcgQEAKICAgICAgICAgZG8geyAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBcCiAgICAgICAgICAgICAgICAgKHN0YXR1
cykgPSByZG1zcihNU1JfQU1ESzdfRklEVklEX1NUQVRVUyk7ICAgICAgXAogICAgICAgICB9IHdo
aWxlIChQTjhfU1RBX1BFTkRJTkcoc3RhdHVzKSkKLSNkZWZpbmUgYWJzKHgpICggeCA8IDAgPyAt
eCA6IHggKQorI2RlZmluZSBhYnMoeCkgKCAoeCkgPCAwID8gLSh4KSA6ICh4KSApCiAKICNkZWZp
bmUgUE9XRVJOT1dfTUFYX1NUQVRFUyAgICAgICAgICAgICAxNgogCi0tIAoxLjcuMQoK
--0016e64ca970ad6a4804b0965894--