Re: [PATCH 4/5] drm/amdgpu: fix build_coefficients() argument
Applied. Thanks! Alex On Fri, Oct 30, 2020 at 1:04 PM Harry Wentland wrote: > > > > On 2020-10-29 11:53 p.m., Alex Deucher wrote: > > On Mon, Oct 26, 2020 at 5:01 PM Arnd Bergmann wrote: > >> > >> From: Arnd Bergmann > >> > >> gcc -Wextra warns about a function taking an enum argument > >> being called with a bool: > >> > >> drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c: In > >> function 'apply_degamma_for_user_regamma': > >> drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c:1617:29: > >> warning: implicit conversion from 'enum ' to 'enum > >> dc_transfer_func_predefined' [-Wenum-conversion] > >> 1617 | build_coefficients(, true); > >> > >> It appears that a patch was added using the old calling conventions > >> after the type was changed, and the value should actually be 0 > >> (TRANSFER_FUNCTION_SRGB) here instead of 1 (true). > > > > This looks correct to me. Harry, Leo? > > > > Confirmed with Kruno, this is correct. > > Reviewed-by: Harry Wentland > > Harry > > > Alex > > > > > >> > >> Fixes: 55a01d4023ce ("drm/amd/display: Add user_regamma to color module") > >> Signed-off-by: Arnd Bergmann > >> --- > >> drivers/gpu/drm/amd/display/modules/color/color_gamma.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > >> b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > >> index b8695660b480..09bc2c249e1a 100644 > >> --- a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > >> +++ b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > >> @@ -1614,7 +1614,7 @@ static void apply_degamma_for_user_regamma(struct > >> pwl_float_data_ex *rgb_regamma > >> struct pwl_float_data_ex *rgb = rgb_regamma; > >> const struct hw_x_point *coord_x = coordinates_x; > >> > >> - build_coefficients(, true); > >> + build_coefficients(, TRANSFER_FUNCTION_SRGB); > >> > >> i = 0; > >> while (i != hw_points_num + 1) { > >> -- > >> 2.27.0 > >> > >> ___ > >> amd-gfx mailing list > >> amd-gfx@lists.freedesktop.org > >> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfxdata=04%7C01%7Charry.wentland%40amd.com%7C3b50cfb318a04e2708e308d87c875c07%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637396268091128887%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QGijLrlFTXI3xx2sGx1iNczHBezfWdu%2FP2xkfoq%2FMB0%3Dreserved=0 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 4/5] drm/amdgpu: fix build_coefficients() argument
On 2020-10-29 11:53 p.m., Alex Deucher wrote: On Mon, Oct 26, 2020 at 5:01 PM Arnd Bergmann wrote: From: Arnd Bergmann gcc -Wextra warns about a function taking an enum argument being called with a bool: drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c: In function 'apply_degamma_for_user_regamma': drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c:1617:29: warning: implicit conversion from 'enum ' to 'enum dc_transfer_func_predefined' [-Wenum-conversion] 1617 | build_coefficients(, true); It appears that a patch was added using the old calling conventions after the type was changed, and the value should actually be 0 (TRANSFER_FUNCTION_SRGB) here instead of 1 (true). This looks correct to me. Harry, Leo? Confirmed with Kruno, this is correct. Reviewed-by: Harry Wentland Harry Alex Fixes: 55a01d4023ce ("drm/amd/display: Add user_regamma to color module") Signed-off-by: Arnd Bergmann --- drivers/gpu/drm/amd/display/modules/color/color_gamma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c index b8695660b480..09bc2c249e1a 100644 --- a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c +++ b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c @@ -1614,7 +1614,7 @@ static void apply_degamma_for_user_regamma(struct pwl_float_data_ex *rgb_regamma struct pwl_float_data_ex *rgb = rgb_regamma; const struct hw_x_point *coord_x = coordinates_x; - build_coefficients(, true); + build_coefficients(, TRANSFER_FUNCTION_SRGB); i = 0; while (i != hw_points_num + 1) { -- 2.27.0 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfxdata=04%7C01%7Charry.wentland%40amd.com%7C3b50cfb318a04e2708e308d87c875c07%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637396268091128887%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QGijLrlFTXI3xx2sGx1iNczHBezfWdu%2FP2xkfoq%2FMB0%3Dreserved=0 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 4/5] drm/amdgpu: fix build_coefficients() argument
On Mon, Oct 26, 2020 at 5:01 PM Arnd Bergmann wrote: > > From: Arnd Bergmann > > gcc -Wextra warns about a function taking an enum argument > being called with a bool: > > drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c: In > function 'apply_degamma_for_user_regamma': > drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c:1617:29: > warning: implicit conversion from 'enum ' to 'enum > dc_transfer_func_predefined' [-Wenum-conversion] > 1617 | build_coefficients(, true); > > It appears that a patch was added using the old calling conventions > after the type was changed, and the value should actually be 0 > (TRANSFER_FUNCTION_SRGB) here instead of 1 (true). This looks correct to me. Harry, Leo? Alex > > Fixes: 55a01d4023ce ("drm/amd/display: Add user_regamma to color module") > Signed-off-by: Arnd Bergmann > --- > drivers/gpu/drm/amd/display/modules/color/color_gamma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > index b8695660b480..09bc2c249e1a 100644 > --- a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > +++ b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c > @@ -1614,7 +1614,7 @@ static void apply_degamma_for_user_regamma(struct > pwl_float_data_ex *rgb_regamma > struct pwl_float_data_ex *rgb = rgb_regamma; > const struct hw_x_point *coord_x = coordinates_x; > > - build_coefficients(, true); > + build_coefficients(, TRANSFER_FUNCTION_SRGB); > > i = 0; > while (i != hw_points_num + 1) { > -- > 2.27.0 > > ___ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH 4/5] drm/amdgpu: fix build_coefficients() argument
From: Arnd Bergmann gcc -Wextra warns about a function taking an enum argument being called with a bool: drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c: In function 'apply_degamma_for_user_regamma': drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c:1617:29: warning: implicit conversion from 'enum ' to 'enum dc_transfer_func_predefined' [-Wenum-conversion] 1617 | build_coefficients(, true); It appears that a patch was added using the old calling conventions after the type was changed, and the value should actually be 0 (TRANSFER_FUNCTION_SRGB) here instead of 1 (true). Fixes: 55a01d4023ce ("drm/amd/display: Add user_regamma to color module") Signed-off-by: Arnd Bergmann --- drivers/gpu/drm/amd/display/modules/color/color_gamma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c index b8695660b480..09bc2c249e1a 100644 --- a/drivers/gpu/drm/amd/display/modules/color/color_gamma.c +++ b/drivers/gpu/drm/amd/display/modules/color/color_gamma.c @@ -1614,7 +1614,7 @@ static void apply_degamma_for_user_regamma(struct pwl_float_data_ex *rgb_regamma struct pwl_float_data_ex *rgb = rgb_regamma; const struct hw_x_point *coord_x = coordinates_x; - build_coefficients(, true); + build_coefficients(, TRANSFER_FUNCTION_SRGB); i = 0; while (i != hw_points_num + 1) { -- 2.27.0 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx