aboutsummaryrefslogtreecommitdiffstats
path: root/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2687-amdgpu-dc-fix-indentation-warning-from-smatch.patch
diff options
context:
space:
mode:
Diffstat (limited to 'meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2687-amdgpu-dc-fix-indentation-warning-from-smatch.patch')
-rw-r--r--meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2687-amdgpu-dc-fix-indentation-warning-from-smatch.patch304
1 files changed, 304 insertions, 0 deletions
diff --git a/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2687-amdgpu-dc-fix-indentation-warning-from-smatch.patch b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2687-amdgpu-dc-fix-indentation-warning-from-smatch.patch
new file mode 100644
index 00000000..ff9ea9d2
--- /dev/null
+++ b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2687-amdgpu-dc-fix-indentation-warning-from-smatch.patch
@@ -0,0 +1,304 @@
+From 9852aaa478cd8af4246c3f417f026af8427c7bbd Mon Sep 17 00:00:00 2001
+From: Dave Airlie <airlied@redhat.com>
+Date: Tue, 7 Nov 2017 05:30:47 +1000
+Subject: [PATCH 2687/4131] amdgpu/dc: fix indentation warning from smatch.
+
+This fixes all the current smatch:
+warn: inconsistent indenting
+
+Reviewed-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
+Reviewed-by: Harry Wentland <harry.wentland@amd.com>
+Signed-off-by: Dave Airlie <airlied@redhat.com>
+Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
+---
+ drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c | 2 +-
+ .../gpu/drm/amd/display/dc/bios/command_table2.c | 18 ++++++-------
+ drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c | 2 +-
+ drivers/gpu/drm/amd/display/dc/core/dc_stream.c | 4 +--
+ drivers/gpu/drm/amd/display/dc/dce/dce_audio.c | 26 +++++++++----------
+ drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.c | 16 ++++++------
+ .../amd/display/dc/dce110/dce110_hw_sequencer.c | 30 +++++++++++-----------
+ .../display/dc/dce120/dce120_timing_generator.c | 7 +++--
+ .../gpu/drm/amd/display/dc/dcn10/dcn10_dpp_cm.c | 2 +-
+ .../dc/i2caux/dce110/i2c_hw_engine_dce110.c | 30 +++++++++++-----------
+ 10 files changed, 68 insertions(+), 69 deletions(-)
+
+diff --git a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
+index 43e9a99..1ee1717 100644
+--- a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
++++ b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
+@@ -1373,7 +1373,7 @@ static enum bp_result get_firmware_info_v3_1(
+ bp->cmd_tbl.get_smu_clock_info(bp) * 10;
+ }
+
+- return BP_RESULT_OK;
++ return BP_RESULT_OK;
+ }
+
+ static enum bp_result bios_parser_get_encoder_cap_info(
+diff --git a/drivers/gpu/drm/amd/display/dc/bios/command_table2.c b/drivers/gpu/drm/amd/display/dc/bios/command_table2.c
+index 64eab35..ba68693 100644
+--- a/drivers/gpu/drm/amd/display/dc/bios/command_table2.c
++++ b/drivers/gpu/drm/amd/display/dc/bios/command_table2.c
+@@ -373,15 +373,15 @@ static void init_set_crtc_timing(struct bios_parser *bp)
+ uint32_t dtd_version =
+ BIOS_CMD_TABLE_PARA_REVISION(setcrtc_usingdtdtiming);
+
+- switch (dtd_version) {
+- case 3:
+- bp->cmd_tbl.set_crtc_timing =
+- set_crtc_using_dtd_timing_v3;
+- break;
+- default:
+- bp->cmd_tbl.set_crtc_timing = NULL;
+- break;
+- }
++ switch (dtd_version) {
++ case 3:
++ bp->cmd_tbl.set_crtc_timing =
++ set_crtc_using_dtd_timing_v3;
++ break;
++ default:
++ bp->cmd_tbl.set_crtc_timing = NULL;
++ break;
++ }
+ }
+
+ static enum bp_result set_crtc_using_dtd_timing_v3(
+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 01f92f8..6d64a06 100644
+--- a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c
++++ b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c
+@@ -1155,7 +1155,7 @@ static unsigned int dcn_find_normalized_clock_vdd_Level(
+ unsigned factor = (ddr4_dram_factor_single_Channel * dc->dcn_soc->number_of_channels);
+
+ if (clocks_in_khz > dc->dcn_soc->fabric_and_dram_bandwidth_vmax0p9*1000000/factor) {
+- vdd_level = dcn_bw_v_max0p91;
++ vdd_level = dcn_bw_v_max0p91;
+ BREAK_TO_DEBUGGER();
+ } else if (clocks_in_khz > dc->dcn_soc->fabric_and_dram_bandwidth_vnom0p8*1000000/factor) {
+ vdd_level = dcn_bw_v_max0p9;
+diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c
+index 572b885..de04b95 100644
+--- a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c
++++ b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c
+@@ -182,11 +182,11 @@ bool dc_stream_set_cursor_attributes(
+
+ if (NULL == stream) {
+ dm_error("DC: dc_stream is NULL!\n");
+- return false;
++ return false;
+ }
+ if (NULL == attributes) {
+ dm_error("DC: attributes is NULL!\n");
+- return false;
++ return false;
+ }
+
+ if (attributes->address.quad_part == 0) {
+diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_audio.c b/drivers/gpu/drm/amd/display/dc/dce/dce_audio.c
+index 526ec5c..d882adf 100644
+--- a/drivers/gpu/drm/amd/display/dc/dce/dce_audio.c
++++ b/drivers/gpu/drm/amd/display/dc/dce/dce_audio.c
+@@ -179,19 +179,19 @@ static void check_audio_bandwidth_hdmi(
+ /* Number of Audio Packets (multiplied by 10) per Line (for 8 ch number
+ * of Audio samples per line multiplied by 10 - Layout 1)
+ */
+- samples /= 32;
+- samples *= crtc_info->v_active;
+- /*Number of samples multiplied by 10, per second */
+- samples *= crtc_info->refresh_rate;
+- /*Number of Audio samples per second */
+- samples /= 10;
+-
+- /* @todo do it after deep color is implemented
+- * 8xx - deep color bandwidth scaling
+- * Extra bandwidth is avaliable in deep color b/c link runs faster than
+- * pixel rate. This has the effect of allowing more tmds characters to
+- * be transmitted during blank
+- */
++ samples /= 32;
++ samples *= crtc_info->v_active;
++ /*Number of samples multiplied by 10, per second */
++ samples *= crtc_info->refresh_rate;
++ /*Number of Audio samples per second */
++ samples /= 10;
++
++ /* @todo do it after deep color is implemented
++ * 8xx - deep color bandwidth scaling
++ * Extra bandwidth is avaliable in deep color b/c link runs faster than
++ * pixel rate. This has the effect of allowing more tmds characters to
++ * be transmitted during blank
++ */
+
+ switch (crtc_info->color_depth) {
+ case COLOR_DEPTH_888:
+diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.c b/drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.c
+index b60524a..508c1aa 100644
+--- a/drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.c
++++ b/drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.c
+@@ -139,14 +139,14 @@ static void dce_dmcu_set_psr_enable(struct dmcu *dmcu, bool enable, bool wait)
+ if (wait == true) {
+ for (retryCount = 0; retryCount <= 100; retryCount++) {
+ dce_get_dmcu_psr_state(dmcu, &psr_state);
+- if (enable) {
+- if (psr_state != 0)
+- break;
+- } else {
+- if (psr_state == 0)
+- break;
+- }
+- udelay(10);
++ if (enable) {
++ if (psr_state != 0)
++ break;
++ } else {
++ if (psr_state == 0)
++ break;
++ }
++ udelay(10);
+ }
+ }
+ }
+diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
+index a93513f..6e0ea54 100644
+--- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
++++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
+@@ -775,8 +775,8 @@ void dce110_enable_stream(struct pipe_ctx *pipe_ctx)
+ * connect DIG back_end to front_end while enable_stream and
+ * disconnect them during disable_stream
+ * BY this, it is logic clean to separate stream and link */
+- link->link_enc->funcs->connect_dig_be_to_fe(link->link_enc,
+- pipe_ctx->stream_res.stream_enc->id, true);
++ link->link_enc->funcs->connect_dig_be_to_fe(link->link_enc,
++ pipe_ctx->stream_res.stream_enc->id, true);
+
+ }
+
+@@ -1306,7 +1306,7 @@ static enum dc_status apply_single_controller_ctx_to_hw(
+ stream->timing.display_color_depth,
+ pipe_ctx->stream->signal);
+
+- pipe_ctx->stream_res.opp->funcs->opp_program_fmt(
++ pipe_ctx->stream_res.opp->funcs->opp_program_fmt(
+ pipe_ctx->stream_res.opp,
+ &stream->bit_depth_params,
+ &stream->clamping);
+@@ -2595,21 +2595,21 @@ uint32_t dce110_get_min_vblank_time_us(const struct dc_state *context)
+ uint8_t j;
+ uint32_t min_vertical_blank_time = -1;
+
+- for (j = 0; j < context->stream_count; j++) {
+- struct dc_stream_state *stream = context->streams[j];
+- uint32_t vertical_blank_in_pixels = 0;
+- uint32_t vertical_blank_time = 0;
++ for (j = 0; j < context->stream_count; j++) {
++ struct dc_stream_state *stream = context->streams[j];
++ uint32_t vertical_blank_in_pixels = 0;
++ uint32_t vertical_blank_time = 0;
+
+- vertical_blank_in_pixels = stream->timing.h_total *
+- (stream->timing.v_total
+- - stream->timing.v_addressable);
++ vertical_blank_in_pixels = stream->timing.h_total *
++ (stream->timing.v_total
++ - stream->timing.v_addressable);
+
+- vertical_blank_time = vertical_blank_in_pixels
+- * 1000 / stream->timing.pix_clk_khz;
++ vertical_blank_time = vertical_blank_in_pixels
++ * 1000 / stream->timing.pix_clk_khz;
+
+- if (min_vertical_blank_time > vertical_blank_time)
+- min_vertical_blank_time = vertical_blank_time;
+- }
++ if (min_vertical_blank_time > vertical_blank_time)
++ min_vertical_blank_time = vertical_blank_time;
++ }
+
+ return min_vertical_blank_time;
+ }
+diff --git a/drivers/gpu/drm/amd/display/dc/dce120/dce120_timing_generator.c b/drivers/gpu/drm/amd/display/dc/dce120/dce120_timing_generator.c
+index 1d4efb0..0aa60e5 100644
+--- a/drivers/gpu/drm/amd/display/dc/dce120/dce120_timing_generator.c
++++ b/drivers/gpu/drm/amd/display/dc/dce120/dce120_timing_generator.c
+@@ -293,10 +293,9 @@ void dce120_timing_generator_tear_down_global_swap_lock(
+ FD(DCP0_DCP_GSL_CONTROL__DCP_GSL_SYNC_SOURCE), 0,
+ FD(DCP0_DCP_GSL_CONTROL__DCP_GSL_DELAY_SURFACE_UPDATE_PENDING), 0);
+
+- CRTC_REG_SET_2(
+- CRTC0_CRTC_GSL_CONTROL,
+- CRTC_GSL_CHECK_LINE_NUM, 0,
+- CRTC_GSL_FORCE_DELAY, 0x2); /*TODO Why this value here ?*/
++ CRTC_REG_SET_2(CRTC0_CRTC_GSL_CONTROL,
++ CRTC_GSL_CHECK_LINE_NUM, 0,
++ CRTC_GSL_FORCE_DELAY, 0x2); /*TODO Why this value here ?*/
+ }
+
+ /* Reset slave controllers on master VSync */
+diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp_cm.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp_cm.c
+index c6d2fd7..9cb44c9 100644
+--- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp_cm.c
++++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp_cm.c
+@@ -124,7 +124,7 @@ static void program_gamut_remap(
+ const uint16_t *regval,
+ enum gamut_remap_select select)
+ {
+- uint16_t selection = 0;
++ uint16_t selection = 0;
+ struct color_matrices_reg gam_regs;
+
+ if (regval == NULL || select == GAMUT_REMAP_BYPASS) {
+diff --git a/drivers/gpu/drm/amd/display/dc/i2caux/dce110/i2c_hw_engine_dce110.c b/drivers/gpu/drm/amd/display/dc/i2caux/dce110/i2c_hw_engine_dce110.c
+index 1a5b3f7..56e25b3 100644
+--- a/drivers/gpu/drm/amd/display/dc/i2caux/dce110/i2c_hw_engine_dce110.c
++++ b/drivers/gpu/drm/amd/display/dc/i2caux/dce110/i2c_hw_engine_dce110.c
+@@ -133,13 +133,13 @@ static void release_engine(
+ safe_to_reset = (i2c_sw_status == 1);
+ }
+
+- if (safe_to_reset)
+- REG_UPDATE_2(
+- DC_I2C_CONTROL,
+- DC_I2C_SOFT_RESET, 1,
+- DC_I2C_SW_STATUS_RESET, 1);
+- else
+- REG_UPDATE(DC_I2C_CONTROL, DC_I2C_SW_STATUS_RESET, 1);
++ if (safe_to_reset)
++ REG_UPDATE_2(
++ DC_I2C_CONTROL,
++ DC_I2C_SOFT_RESET, 1,
++ DC_I2C_SW_STATUS_RESET, 1);
++ else
++ REG_UPDATE(DC_I2C_CONTROL, DC_I2C_SW_STATUS_RESET, 1);
+
+ /* HW I2c engine - clock gating feature */
+ if (!hw_engine->engine_keep_power_up_count)
+@@ -301,16 +301,16 @@ static bool process_transaction(
+ * For an I2C send operation, the LSB must be programmed to 0;
+ * for I2C receive operation, the LSB must be programmed to 1. */
+ if (hw_engine->transaction_count == 0) {
+- value = REG_SET_4(DC_I2C_DATA, 0,
+- DC_I2C_DATA_RW, false,
+- DC_I2C_DATA, request->address,
+- DC_I2C_INDEX, 0,
+- DC_I2C_INDEX_WRITE, 1);
++ value = REG_SET_4(DC_I2C_DATA, 0,
++ DC_I2C_DATA_RW, false,
++ DC_I2C_DATA, request->address,
++ DC_I2C_INDEX, 0,
++ DC_I2C_INDEX_WRITE, 1);
+ hw_engine->buffer_used_write = 0;
+ } else
+- value = REG_SET_2(DC_I2C_DATA, 0,
+- DC_I2C_DATA_RW, false,
+- DC_I2C_DATA, request->address);
++ value = REG_SET_2(DC_I2C_DATA, 0,
++ DC_I2C_DATA_RW, false,
++ DC_I2C_DATA, request->address);
+
+ hw_engine->buffer_used_write++;
+
+--
+2.7.4
+