diff options
Diffstat (limited to 'meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2913-drm-amd-display-Add-disclaimer-to-BW-and-DML-code-pr.patch')
| -rw-r--r-- | meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2913-drm-amd-display-Add-disclaimer-to-BW-and-DML-code-pr.patch | 189 |
1 files changed, 189 insertions, 0 deletions
diff --git a/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2913-drm-amd-display-Add-disclaimer-to-BW-and-DML-code-pr.patch b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2913-drm-amd-display-Add-disclaimer-to-BW-and-DML-code-pr.patch new file mode 100644 index 00000000..b2f997fc --- /dev/null +++ b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2913-drm-amd-display-Add-disclaimer-to-BW-and-DML-code-pr.patch @@ -0,0 +1,189 @@ +From 58531149e1796cb877134e7eeb612c040f78f5a3 Mon Sep 17 00:00:00 2001 +From: Harry Wentland <harry.wentland@amd.com> +Date: Wed, 29 Nov 2017 10:28:29 -0500 +Subject: [PATCH 2913/4131] drm/amd/display: Add disclaimer to BW and DML code + provided by HW + +This code can sometimes look troubling but we trust it as it comes from +HW teams with a guarantee of correctness. Add a note to these files to +explain this. + +v2: thing -> things + +Signed-off-by: Harry Wentland <harry.wentland@amd.com> +Reviewed-by: Alex Deucher <alexander.deucher@amd.com> +--- + drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_auto.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_math.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/dml/display_mode_vba.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/dml/display_rq_dlg_calc.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/dml/dml1_display_rq_dlg_calc.c | 9 +++++++++ + drivers/gpu/drm/amd/display/dc/dml/soc_bounding_box.c | 10 ++++++++++ + 8 files changed, 73 insertions(+) + +diff --git a/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c b/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c +index 4f8a953..ea9c08a 100644 +--- a/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c ++++ b/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c +@@ -29,6 +29,15 @@ + #include "core_types.h" + #include "dal_asic_id.h" + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + /******************************************************************************* + * Private Functions + ******************************************************************************/ +diff --git a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_auto.c b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_auto.c +index 626f9cf..366aace 100644 +--- a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_auto.c ++++ b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_auto.c +@@ -27,6 +27,15 @@ + #include "dcn_calc_auto.h" + #include "dcn_calc_math.h" + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + /*REVISION#250*/ + void scaler_settings_calculation(struct dcn_bw_internal_vars *v) + { +diff --git a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_math.c b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_math.c +index ae38cdb..7600a4a 100644 +--- a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_math.c ++++ b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calc_math.c +@@ -27,6 +27,15 @@ + + #define isNaN(number) ((number) != (number)) + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + float dcn_bw_mod(const float arg1, const float arg2) + { + if (isNaN(arg1)) +diff --git a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c +index a4fbca3..b5bc915 100644 +--- a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c ++++ b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c +@@ -33,6 +33,15 @@ + #include "dcn10/dcn10_resource.h" + #include "dcn_calc_math.h" + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + /* Defaults from spreadsheet rev#247 */ + const struct dcn_soc_bounding_box dcn10_soc_defaults = { + /* latencies */ +diff --git a/drivers/gpu/drm/amd/display/dc/dml/display_mode_vba.c b/drivers/gpu/drm/amd/display/dc/dml/display_mode_vba.c +index 1f337ec..260e113 100644 +--- a/drivers/gpu/drm/amd/display/dc/dml/display_mode_vba.c ++++ b/drivers/gpu/drm/amd/display/dc/dml/display_mode_vba.c +@@ -28,6 +28,15 @@ + + #include "dml_inline_defs.h" + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + #define BPP_INVALID 0 + #define BPP_BLENDED_PIPE 0xffffffff + static const unsigned int NumberOfStates = DC__VOLTAGE_STATES; +diff --git a/drivers/gpu/drm/amd/display/dc/dml/display_rq_dlg_calc.c b/drivers/gpu/drm/amd/display/dc/dml/display_rq_dlg_calc.c +index 8ba962d..325dd2b 100644 +--- a/drivers/gpu/drm/amd/display/dc/dml/display_rq_dlg_calc.c ++++ b/drivers/gpu/drm/amd/display/dc/dml/display_rq_dlg_calc.c +@@ -27,6 +27,15 @@ + #include "display_mode_vba.h" + #include "display_rq_dlg_calc.h" + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + static void calculate_ttu_cursor(struct display_mode_lib *mode_lib, + double *refcyc_per_req_delivery_pre_cur, + double *refcyc_per_req_delivery_cur, +diff --git a/drivers/gpu/drm/amd/display/dc/dml/dml1_display_rq_dlg_calc.c b/drivers/gpu/drm/amd/display/dc/dml/dml1_display_rq_dlg_calc.c +index 1e4b1e3..c2037da 100644 +--- a/drivers/gpu/drm/amd/display/dc/dml/dml1_display_rq_dlg_calc.c ++++ b/drivers/gpu/drm/amd/display/dc/dml/dml1_display_rq_dlg_calc.c +@@ -28,6 +28,15 @@ + + #include "dml_inline_defs.h" + ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + static unsigned int get_bytes_per_element(enum source_format_class source_format, bool is_chroma) + { + unsigned int ret_val = 0; +diff --git a/drivers/gpu/drm/amd/display/dc/dml/soc_bounding_box.c b/drivers/gpu/drm/amd/display/dc/dml/soc_bounding_box.c +index bc7d8c7..324239c 100644 +--- a/drivers/gpu/drm/amd/display/dc/dml/soc_bounding_box.c ++++ b/drivers/gpu/drm/amd/display/dc/dml/soc_bounding_box.c +@@ -27,6 +27,16 @@ + #include "dc_features.h" + + #include "dml_inline_defs.h" ++ ++/* ++ * NOTE: ++ * This file is gcc-parseable HW gospel, coming straight from HW engineers. ++ * ++ * It doesn't adhere to Linux kernel style and sometimes will do things in odd ++ * ways. Unless there is something clearly wrong with it the code should ++ * remain as-is as it provides us with a guarantee from HW that it is correct. ++ */ ++ + void dml_socbb_set_latencies(soc_bounding_box_st *to_box, soc_bounding_box_st *from_box) + { + to_box->dram_clock_change_latency_us = from_box->dram_clock_change_latency_us; +-- +2.7.4 + |