From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934855AbdEAQuL (ORCPT ); Mon, 1 May 2017 12:50:11 -0400 Received: from mout.web.de ([212.227.15.3]:52153 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1765354AbdEAQsr (ORCPT ); Mon, 1 May 2017 12:48:47 -0400 Subject: [PATCH 3/4] drm/dp: Replace six seq_printf() calls by seq_puts() in drm_dp_mst_dump_topology() From: SF Markus Elfring To: dri-devel@lists.freedesktop.org, Daniel Vetter , David Airlie , Jani Nikula , Sean Paul Cc: LKML , kernel-janitors@vger.kernel.org References: <1949e36b-5039-a7b6-5774-6ada7eb05ea6@users.sourceforge.net> Message-ID: <73f05228-be92-c2dd-b121-52bc211f236d@users.sourceforge.net> Date: Mon, 1 May 2017 18:48:32 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.0.1 MIME-Version: 1.0 In-Reply-To: <1949e36b-5039-a7b6-5774-6ada7eb05ea6@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:4vl90iWmk5OegFwfodjY26ZFqZ5+UVHqYNl5A1aZGCD2hZyBTId hgSeeYp0ld59jF3e5om7uqYXzrW4gWNVNsZrE6E98TRJVcKCvz88Qk+LJGnocIfplCm0LjL WCWE2cAXVinUPUxTgM2ScDrTBuRYUV4U2r8bUYQ/qE4w2nzq7I4Dr3JtqADoTRP7ERwPWCb 7jl0sQ+Dt4TVdhy9XNcwA== X-UI-Out-Filterresults: notjunk:1;V01:K0:tE0bTW9k3AM=:5Ff2TMr9qenwn+cYeFsOfX UiHFErHBADB+2tzYmTuTErEBly+em2oa4DJqPvrB0TdKF56kCgnF1DziJtYdtfc/622V74UYp Ld6fc7i60TF8Te3YI0r8Y5mdXWzZYayw6+7YUYE+WUHclVes/PTIxysXS3evvbFG87rij1i/1 dLPZj6yzrzuuRcCCBaYhYKrrW81UYL72bbO0pcm0oqqRCQocULm3amoZk7fb2TmbBGsQqBgvN sdQhkLcNvafc/zdOhlFTGwrrw043cOSQZs6pbBJSVyXq6XxJpZFEKO3uxlWOMG5UiQ302XNeM MOOWeTmlCfGRzP07iZmIwq9HjJSieRmMncdtdijRNKB7ji+/GyvFmxS+k3D9UrjGywVTC+WLU 8HW7gIPITo7x/tJjkXgcREHMBxhbnd456LbC/IuoeAlDo3fDepYlRHDROyd8jZnD3aVFajloP reGz0B4Gh7yegXCUPr98gsYH73kDOB0Pu1X+O1sozS8CW/0zFDWqvDfEsM0LnqFH4/56FXEjD YgLIw+g+jCezRyL2OKuZ4UkcLOOHLL3JafpHAVKqSbtG1hrG+xNXm4VYxwcKcUapFigH1dXkQ ZoK25iSW4pEReTm9aQy+Mfa+ee1KxOeFSwY3QsJ5Sh+/5uMaW1lippKoEg5ET5/IwoSUZihQD rsxRAChPjz2+S5cJDD0dYFGr62kR6ksE9w5GBOEYk/lI16FywnQm0xf8yDTRF7sL2cKMVrTwt A4EMR4ZVK1CuFVtm7ieOJz2eUpWDhKyDXUFL2NLX+6gXaxgTtUX46kFOMKsM+UC013te1ucfP qn0u/No Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Mon, 1 May 2017 17:23:24 +0200 Strings which did not contain data format specifications should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/drm_dp_mst_topology.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index 2ddcb3d2f65a..bcc76f15f1ce 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -2837,27 +2837,27 @@ void drm_dp_mst_dump_topology(struct seq_file *m, bool bret; int ret; ret = drm_dp_dpcd_read(mgr->aux, DP_DPCD_REV, buf, DP_RECEIVER_CAP_SIZE); - seq_printf(m, "dpcd: "); + seq_puts(m, "dpcd: "); for (i = 0; i < DP_RECEIVER_CAP_SIZE; i++) seq_printf(m, "%02x ", buf[i]); seq_putc(m, '\n'); ret = drm_dp_dpcd_read(mgr->aux, DP_FAUX_CAP, buf, 2); - seq_printf(m, "faux/mst: "); + seq_puts(m, "faux/mst: "); for (i = 0; i < 2; i++) seq_printf(m, "%02x ", buf[i]); seq_putc(m, '\n'); ret = drm_dp_dpcd_read(mgr->aux, DP_MSTM_CTRL, buf, 1); - seq_printf(m, "mst ctrl: "); + seq_puts(m, "mst ctrl: "); for (i = 0; i < 1; i++) seq_printf(m, "%02x ", buf[i]); seq_putc(m, '\n'); /* dump the standard OUI branch header */ ret = drm_dp_dpcd_read(mgr->aux, DP_BRANCH_OUI, buf, DP_BRANCH_OUI_HEADER_SIZE); - seq_printf(m, "branch oui: "); + seq_puts(m, "branch oui: "); for (i = 0; i < 0x3; i++) seq_printf(m, "%02x", buf[i]); - seq_printf(m, " devid: "); + seq_puts(m, " devid: "); for (i = 0x3; i < 0x8 && buf[i]; i++) seq_printf(m, "%c", buf[i]); @@ -2865,7 +2865,7 @@ void drm_dp_mst_dump_topology(struct seq_file *m, buf[0x9] >> 4, buf[0x9] & 0xf, buf[0xa], buf[0xb]); bret = dump_dp_payload_table(mgr, buf); if (bret == true) { - seq_printf(m, "payload table: "); + seq_puts(m, "payload table: "); for (i = 0; i < 63; i++) seq_printf(m, "%02x ", buf[i]); seq_putc(m, '\n'); -- 2.12.2