Commit 30f2d2e4 authored by Tales Aparecida's avatar Tales Aparecida Committed by Alex Deucher

drm/amd/display: fix minor codestyle problems

Fixes five checkpatch warnings:

CHECK: Please don't use multiple blank lines
+
+

ERROR: Macros with complex values should be enclosed in parentheses
+#define MAX_I64 \
+       (int64_t)((1ULL << 63) - 1)

WARNING: Missing a blank line after declarations
+       struct bw_fixed res;
+       ASSERT(value < BW_FIXED_MAX_I32 && value > BW_FIXED_MIN_I32);

ERROR: that open brace { should be on the previous line
+               do
+               {

ERROR: that open brace { should be on the previous line
+                       if (remainder >= arg2_value)
+                       {
Signed-off-by: default avatarTales Aparecida <tales.aparecida@gmail.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6ae0632d
...@@ -25,9 +25,8 @@ ...@@ -25,9 +25,8 @@
#include "dm_services.h" #include "dm_services.h"
#include "bw_fixed.h" #include "bw_fixed.h"
#define MAX_I64 \ #define MAX_I64 \
(int64_t)((1ULL << 63) - 1) ((int64_t)((1ULL << 63) - 1))
#define MIN_I64 \ #define MIN_I64 \
(-MAX_I64 - 1) (-MAX_I64 - 1)
...@@ -49,6 +48,7 @@ static uint64_t abs_i64(int64_t arg) ...@@ -49,6 +48,7 @@ static uint64_t abs_i64(int64_t arg)
struct bw_fixed bw_int_to_fixed_nonconst(int64_t value) struct bw_fixed bw_int_to_fixed_nonconst(int64_t value)
{ {
struct bw_fixed res; struct bw_fixed res;
ASSERT(value < BW_FIXED_MAX_I32 && value > BW_FIXED_MIN_I32); ASSERT(value < BW_FIXED_MAX_I32 && value > BW_FIXED_MIN_I32);
res.value = value << BW_FIXED_BITS_PER_FRACTIONAL_PART; res.value = value << BW_FIXED_BITS_PER_FRACTIONAL_PART;
return res; return res;
...@@ -78,14 +78,12 @@ struct bw_fixed bw_frc_to_fixed(int64_t numerator, int64_t denominator) ...@@ -78,14 +78,12 @@ struct bw_fixed bw_frc_to_fixed(int64_t numerator, int64_t denominator)
{ {
uint32_t i = BW_FIXED_BITS_PER_FRACTIONAL_PART; uint32_t i = BW_FIXED_BITS_PER_FRACTIONAL_PART;
do do {
{
remainder <<= 1; remainder <<= 1;
res_value <<= 1; res_value <<= 1;
if (remainder >= arg2_value) if (remainder >= arg2_value) {
{
res_value |= 1; res_value |= 1;
remainder -= arg2_value; remainder -= arg2_value;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment