Commit 86f06bcb authored by Samson Tam's avatar Samson Tam Committed by Alex Deucher
Browse files

drm/amd/display: Fix mismatch type comparison in custom_float 



[Why & How]
Passing uint into uchar function param.  Pass uint instead

Signed-off-by: default avatarSamson Tam <Samson.Tam@amd.com>
Reviewed-by: default avatarAlvin Lee <alvin.lee2@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 97b05c8c

drivers/gpu/drm/amd/display/dc/sspl/spl_fixpt31_32.c

+1 −1
Original line number Diff line number Diff line
@@ -346,7 +346,7 @@ struct spl_fixed31_32 spl_fixpt_exp(struct spl_fixed31_32 arg) 
		if (m > 0)

			return spl_fixpt_shl(

				spl_fixed31_32_exp_from_taylor_series(r),

				(unsigned char)m);

				(unsigned int)m);

		else

			return spl_fixpt_div_int(

				spl_fixed31_32_exp_from_taylor_series(r),

drivers/gpu/drm/amd/display/dc/sspl/spl_fixpt31_32.h

+2 −2
Original line number Diff line number Diff line
@@ -189,7 +189,7 @@ static inline struct spl_fixed31_32 spl_fixpt_clamp( 
 * @brief

 * result = arg << shift

 */

static inline struct spl_fixed31_32 spl_fixpt_shl(struct spl_fixed31_32 arg, unsigned char shift)

static inline struct spl_fixed31_32 spl_fixpt_shl(struct spl_fixed31_32 arg, unsigned int shift)

{

	SPL_ASSERT(((arg.value >= 0) && (arg.value <= LLONG_MAX >> shift)) ||

		((arg.value < 0) && (arg.value >= ~(LLONG_MAX >> shift))));
@@ -203,7 +203,7 @@ static inline struct spl_fixed31_32 spl_fixpt_shl(struct spl_fixed31_32 arg, uns 
 * @brief

 * result = arg >> shift

 */

static inline struct spl_fixed31_32 spl_fixpt_shr(struct spl_fixed31_32 arg, unsigned char shift)

static inline struct spl_fixed31_32 spl_fixpt_shr(struct spl_fixed31_32 arg, unsigned int shift)

{

	bool negative = arg.value < 0;