From: Vitaly Prosyak <vitaly.pros...@amd.com>
Signed-off-by: Vitaly Prosyak <vitaly.pros...@amd.com>
Reviewed-by: Tony Cheng <tony.ch...@amd.com>
Acked-by: Harry Wentland <harry.wentl...@amd.com>
---
drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c
b/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c
index 13f8b8c02212..169fe42f29cd 100644
--- a/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c
+++ b/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c
@@ -550,6 +550,9 @@ static inline uint32_t ux_dy(
result <<= fractional_bits;
/* 5. shrink fixed point fractional part to of fractional_bits width*/
fractional_part >>= FIXED31_32_BITS_PER_FRACTIONAL_PART -
fractional_bits;
+ /*handle the case when overflow, for example 1.0 in u14 is 0x3fff*/
+ if (value > 0 && result == 0 && integer_bits == 0 && fractional_part
== 0)
+ fractional_part = (1<<fractional_bits) - 1;
/* 6. merge the result */
return result | fractional_part;
}
@@ -569,11 +572,11 @@ uint32_t dal_fixed31_32_u0d19(
uint32_t dal_fixed31_32_u0d14(
struct fixed31_32 arg)
{
- return ux_dy(arg.value, 1, 14);
+ return ux_dy(arg.value, 0, 14);
}
uint32_t dal_fixed31_32_u0d10(
struct fixed31_32 arg)
{
- return ux_dy(arg.value, 1, 10);
+ return ux_dy(arg.value, 0, 10);
}
--
2.14.1
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
