diff options
author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2011-11-21 11:35:40 +1100 |
---|---|---|
committer | Alex Deucher <alexdeucher@gmail.com> | 2011-11-21 11:48:40 -0500 |
commit | d669c34f140c000f88c4b4e464e44e6c8694f581 (patch) | |
tree | fbcd881f46fb7c304bc95e11f3e8e2e055375fb2 /src/evergreen_accel.c | |
parent | 422bdd4fe6cb728e1dd08a56f6ee2d0f009cbfcb (diff) |
ddx/evergreen: Fix endian of ALU constants
The constants are written directly into a buffer object shared with the
card and we "forget" to swap them. This patch fixes it by doing the swap
in evergreen_set_alu_consts() in-place (ie, it modifies the buffer),
which should be fine with the way we use it in the ddx.
This makes everything work fine on my caicos card on a G5 including some
quik tests with Xv, gnome3 shell, etc...
Thanks a lot to Jerome Glisse for holding my hand through debugging that
(and finding the actual bug).
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Alex Deucher <alexdeucher@gmail.com>
Diffstat (limited to 'src/evergreen_accel.c')
-rw-r--r-- | src/evergreen_accel.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/src/evergreen_accel.c b/src/evergreen_accel.c index 5c95e201..83320c83 100644 --- a/src/evergreen_accel.c +++ b/src/evergreen_accel.c @@ -479,6 +479,17 @@ evergreen_set_alu_consts(ScrnInfoPtr pScrn, const_config_t *const_conf, uint32_t if (size == 0) size = 1; +#if X_BYTE_ORDER == X_BIG_ENDIAN + { + uint32_t count = size << 4, *p = const_conf->cpu_ptr; + + while(count--) { + *p = cpu_to_le32(*p); + p++; + } + } +#endif + /* flush SQ cache */ evergreen_cp_set_surface_sync(pScrn, SH_ACTION_ENA_bit, const_conf->size_bytes, const_conf->const_addr, |