From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D8F97C433E0 for ; Mon, 10 Aug 2020 23:47:38 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B1186206E9 for ; Mon, 10 Aug 2020 23:47:38 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B1186206E9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=emeril.freedesktop.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 29C546E08E; Mon, 10 Aug 2020 23:47:38 +0000 (UTC) Received: from emeril.freedesktop.org (emeril.freedesktop.org [IPv6:2610:10:20:722:a800:ff:feee:56cf]) by gabe.freedesktop.org (Postfix) with ESMTP id 6679E6E08E; Mon, 10 Aug 2020 23:47:36 +0000 (UTC) Received: from emeril.freedesktop.org (localhost [127.0.0.1]) by emeril.freedesktop.org (Postfix) with ESMTP id 60BDEA0118; Mon, 10 Aug 2020 23:47:36 +0000 (UTC) MIME-Version: 1.0 From: Patchwork To: "Manasi Navare" Date: Mon, 10 Aug 2020 23:47:36 -0000 Message-ID: <159710325636.17063.6980025532100655630@emeril.freedesktop.org> X-Patchwork-Hint: ignore References: <20200715224222.7557-1-manasi.d.navare@intel.com> In-Reply-To: <20200715224222.7557-1-manasi.d.navare@intel.com> Subject: [Intel-gfx] =?utf-8?b?4pyXIEZpLkNJLkNIRUNLUEFUQ0g6IHdhcm5pbmcg?= =?utf-8?q?for_series_starting_with_=5Bv6=2C01/11=5D_HAX_to_make_DSC_work_?= =?utf-8?q?on_the_icelake_test_system_=28rev3=29?= X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: intel-gfx@lists.freedesktop.org Cc: intel-gfx@lists.freedesktop.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" == Series Details == Series: series starting with [v6,01/11] HAX to make DSC work on the icelake test system (rev3) URL : https://patchwork.freedesktop.org/series/79534/ State : warning == Summary == $ dim checkpatch origin/drm-tip 94f4928909c8 HAX to make DSC work on the icelake test system 3b320ffb4bc2 drm/i915: Remove hw.mode 167ba475cb68 drm/i915: Add hw.pipe_mode to allow bigjoiner pipe/transcoder split -:216: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #216: FILE: drivers/gpu/drm/i915/display/intel_display.c:13229: + crtc_state->hw.pipe_mode = crtc_state->hw.adjusted_mode = crtc_state->uapi.adjusted_mode; total: 0 errors, 0 warnings, 1 checks, 445 lines checked 59fbefeedaa6 drm/i915/dp: Allow big joiner modes in intel_dp_mode_valid(), v3. 354e47b3d9eb drm/i915: Try to make bigjoiner work in atomic check -:143: WARNING:LONG_LINE: line length of 101 exceeds 100 columns #143: FILE: drivers/gpu/drm/i915/display/intel_display.c:13242: + crtc_state->bigjoiner_linked_crtc); -:202: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #202: FILE: drivers/gpu/drm/i915/display/intel_display.c:13313: + crtc_state->nv12_planes = crtc_state->c8_planes = crtc_state->update_planes = 0; -:296: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #296: FILE: drivers/gpu/drm/i915/display/intel_display.c:14930: + slave = new_crtc_state->bigjoiner_linked_crtc = -:330: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #330: FILE: drivers/gpu/drm/i915/display/intel_display.c:14964: + slave_crtc_state->bigjoiner = master_crtc_state->bigjoiner = false; -:331: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #331: FILE: drivers/gpu/drm/i915/display/intel_display.c:14965: + slave_crtc_state->bigjoiner_slave = master_crtc_state->bigjoiner_slave = false; -:332: WARNING:LONG_LINE: line length of 106 exceeds 100 columns #332: FILE: drivers/gpu/drm/i915/display/intel_display.c:14966: + slave_crtc_state->bigjoiner_linked_crtc = master_crtc_state->bigjoiner_linked_crtc = NULL; -:332: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #332: FILE: drivers/gpu/drm/i915/display/intel_display.c:14966: + slave_crtc_state->bigjoiner_linked_crtc = master_crtc_state->bigjoiner_linked_crtc = NULL; -:387: WARNING:BRACES: braces {} are not necessary for any arm of this statement #387: FILE: drivers/gpu/drm/i915/display/intel_display.c:15361: + if (new_crtc_state->bigjoiner) { [...] + } else if (INTEL_GEN(dev_priv) >= 9) [...] else [...] total: 0 errors, 3 warnings, 5 checks, 401 lines checked 54000a12eca0 drm/i915: Enable big joiner support in enable and disable sequences. -:178: WARNING:LONG_LINE_COMMENT: line length of 106 exceeds 100 columns #178: FILE: drivers/gpu/drm/i915/display/intel_ddi.c:4427: + /* Our own transcoder needs to be disabled when reading it in intel_ddi_read_func_ctl() */ -:180: WARNING:LONG_LINE: line length of 104 exceeds 100 columns #180: FILE: drivers/gpu/drm/i915/display/intel_ddi.c:4429: + pipe_config->cpu_transcoder = (enum transcoder)pipe_config->bigjoiner_linked_crtc->pipe; -:791: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV) #791: FILE: drivers/gpu/drm/i915/display/intel_display_types.h:829: +#define PIPE_CONFIG_QUIRK_BIGJOINER_SLAVE (1<<1) /* bigjoiner slave, partial readout */ ^ total: 0 errors, 2 warnings, 1 checks, 995 lines checked 6adc9031acd5 drm/i915: Make hardware readout work on i915. -:33: WARNING:TABSTOP: Statements should start on a tabstop #33: FILE: drivers/gpu/drm/i915/display/intel_display.c:3610: + struct intel_crtc_state *crtc_state = -:76: WARNING:LONG_LINE: line length of 111 exceeds 100 columns #76: FILE: drivers/gpu/drm/i915/display/intel_display.c:10695: + (intel_de_read(dev_priv, PLANE_SURF(pipe, plane_id)) & 0xfffff000) == plane_config->base) { total: 0 errors, 2 warnings, 0 checks, 118 lines checked 0aebc7f853d3 drm/i915: Link planes in a bigjoiner configuration, v3. -:203: ERROR:CODE_INDENT: code indent should use tabs where possible #203: FILE: drivers/gpu/drm/i915/display/intel_display.c:12636: + * Setup and teardown the new bigjoiner plane mappings.$ -:204: ERROR:CODE_INDENT: code indent should use tabs where possible #204: FILE: drivers/gpu/drm/i915/display/intel_display.c:12637: + */$ -:289: ERROR:CODE_INDENT: code indent should use tabs where possible #289: FILE: drivers/gpu/drm/i915/display/intel_display.c:12718: + *$ -:305: WARNING:LONG_LINE: line length of 105 exceeds 100 columns #305: FILE: drivers/gpu/drm/i915/display/intel_display.c:12732: + for_each_oldnew_intel_plane_in_state(state, plane, old_plane_state, new_plane_state, i) { -:321: ERROR:CODE_INDENT: code indent should use tabs where possible #321: FILE: drivers/gpu/drm/i915/display/intel_display.c:12748: + * Make sure bigjoiner slave crtc's are also pulled in. This is not done automatically$ -:322: ERROR:CODE_INDENT: code indent should use tabs where possible #322: FILE: drivers/gpu/drm/i915/display/intel_display.c:12749: + * when adding slave planes, because plane_state->crtc is null.$ -:323: ERROR:CODE_INDENT: code indent should use tabs where possible #323: FILE: drivers/gpu/drm/i915/display/intel_display.c:12750: + */$ -:435: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8) #435: FILE: drivers/gpu/drm/i915/display/intel_display.h:472: + for_each_intel_plane_mask(((crtc_state)->uapi.state->dev), (iter), \ [...] + for_each_if ((((plane_state) = \ -:441: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #441: FILE: drivers/gpu/drm/i915/display/intel_display.h:478: + for_each_if ((((plane_state) = \ -:442: WARNING:LONG_LINE: line length of 124 exceeds 100 columns #442: FILE: drivers/gpu/drm/i915/display/intel_display.h:479: + to_intel_plane_state(__drm_atomic_get_current_plane_state((crtc_state)->uapi.state, &iter->base))), \ -:443: WARNING:LONG_LINE: line length of 109 exceeds 100 columns #443: FILE: drivers/gpu/drm/i915/display/intel_display.h:480: + ((plane) = (plane_state)->bigjoiner_slave ? (plane_state)->bigjoiner_plane : (iter)), \ -:445: WARNING:LONG_LINE: line length of 125 exceeds 100 columns #445: FILE: drivers/gpu/drm/i915/display/intel_display.h:482: + to_intel_plane_state(__drm_atomic_get_current_plane_state((crtc_state)->uapi.state, &plane->base)) : \ total: 6 errors, 6 warnings, 0 checks, 481 lines checked 83120ea46d35 drm/i915: Add bigjoiner aware plane clipping checks e0a96e6c643e drm/i915: Add intel_update_bigjoiner handling. d598c90ba017 drm/i915: Add debugfs dumping for bigjoiner, v3. -:46: WARNING:LONG_LINE: line length of 123 exceeds 100 columns #46: FILE: drivers/gpu/drm/i915/display/intel_display_debugfs.c:777: + seq_printf(m, "\t\tuapi: fb=%d,%s,%dx%d, visible=%s, src=" DRM_RECT_FP_FMT ", dst=" DRM_RECT_FMT ", rotation=%s\n", -:56: WARNING:LONG_LINE: line length of 119 exceeds 100 columns #56: FILE: drivers/gpu/drm/i915/display/intel_display_debugfs.c:787: + plane_state->planar_linked_plane->base.base.id, plane_state->planar_linked_plane->base.name, -:60: WARNING:LONG_LINE: line length of 111 exceeds 100 columns #60: FILE: drivers/gpu/drm/i915/display/intel_display_debugfs.c:791: + plane_state->bigjoiner_plane->base.base.id, plane_state->bigjoiner_plane->base.name, total: 0 errors, 3 warnings, 0 checks, 52 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx