[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/display/exynos4210_fimd: Fix bit-swapping co
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] hw/display/exynos4210_fimd: Fix bit-swapping code |
Date: |
Fri, 12 Jun 2015 15:08:34 +0100 |
Deafening silence, so I'm just going to apply this to target-arm.next.
-- PMM
On 5 June 2015 at 16:05, Peter Maydell <address@hidden> wrote:
> Ping for either patch review or suggestions
> for updating the MAINTAINERS section for this board?
>
> thanks
> -- PMM
>
> On 29 May 2015 at 16:16, Peter Maydell <address@hidden> wrote:
>> fimd_swap_data() includes code to reverse the bits in a
>> 64-bit integer, but an off-by-one error meant that it would
>> try to shift off the top of the integer. Correct the bug
>> (spotted by Coverity).
>>
>> Signed-off-by: Peter Maydell <address@hidden>
>> ---
>> Compile-tested only, I have no exynos images to test with.
>>
>> hw/display/exynos4210_fimd.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/hw/display/exynos4210_fimd.c b/hw/display/exynos4210_fimd.c
>> index 45c62af..3e59ee4 100644
>> --- a/hw/display/exynos4210_fimd.c
>> +++ b/hw/display/exynos4210_fimd.c
>> @@ -337,7 +337,7 @@ static inline void fimd_swap_data(unsigned int swap_ctl,
>> uint64_t *data)
>> if (swap_ctl & FIMD_WINCON_SWAP_BITS) {
>> res = 0;
>> for (i = 0; i < 64; i++) {
>> - if (x & (1ULL << (64 - i))) {
>> + if (x & (1ULL << (63 - i))) {
>> res |= (1ULL << i);
>> }
>> }
>> --
>> 1.9.1