From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934717AbdEAQuI (ORCPT ); Mon, 1 May 2017 12:50:08 -0400 Received: from mout.web.de ([212.227.15.4]:61781 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1765338AbdEAQrg (ORCPT ); Mon, 1 May 2017 12:47:36 -0400 Subject: [PATCH 2/4] drm/dp: Combine two seq_printf() calls into one call 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: <0632f7bd-e58c-0b7d-163d-7e1a6e97ff25@users.sourceforge.net> Date: Mon, 1 May 2017 18:47:21 +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:mAS3+HG+cOwnNvlEWk/B3VDVv6CerYrEQ8QLxE98XKYksDBa6kY Ifz+fQDAziEUlKFARQZRvNXQT8hvjaSGOHHel4LuA3F+tyKd88G7W3a3POkD/qDc4JnMMQE tWUDydOD9CeScCi44Ii9hs7rPv0IVW1iBrc6Kdw/sACeH7IoIA7kk7kior+96uDlTvpVitY nqF1foC6VxAFn5/fWwVyQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:cfQjeSWLnpM=:uXJ6d/dDVoPECvxmr/fSas /gbOsA4RVkmKxTj8hQ2ExVlvH/wNhl656g78irGcUr+B+um6XocVPd2GJW3glztoiZiBuuRgq zBOqYin9NoVvV2tlUlWBP7CW+am9cO9TfAmGC8vhVao2VSqNcMBUVuEg8kpKHIs6A5WzjB0WB drQh3YmNc+e7Jypw2yII9GQV/nRbVOFUXD27PdDlLKr1rMs2Zh6777t/OTZ2Hk0Ztaa/7AC// IHgCKTKXTcp4r6bZBwQrVEN3o+ckEDmK3Yp6HOTiYE+2HnPExamDsucljiYgSWA1IyE/cfkFg KJj4Uwhe8KkMVcOCTHWIG5QpJw+S5LJxxG68AbKhIp8Mq0ETqHYCCaux/M5hf4NukYty5foXp WT8+KLBOMggAw21jYeapt287jrmWSmOO6x9aOWapvijiDjdW8wNY2fJVH2IbV1Mq5JkK7DMxS iiPmirY/SeFWPi/64cGN/xchuBMV3+CjOB724hOqKcpo+s/KrErGh8g2IMWDROKRLFL1LWiO2 B9/puDx2x1Cfey8EtDIHNU+xLttVLLPyPGCMEfglTXIEYD2Qt37g/IqZU9FbXg/LohPwRxl8T scfBR/SMZd26iS0JZLom2PquA2sn+bh4LgDV1xksgCgS9W6aMgtM3ZhRld5gaNW9mf8pSXsGL dIuVHTDFwmrXSYEi7tN8y+zGKFobaAKGqUEsgDMGIlx0QaJQz+dOqT+G/roAtdoQ3REGg2JKH a7J1awqjEGrINYMfG46GpADe7Gm27ZIfLApHInQoN78UanLfkUIZa8WuoUYNxSKuH+V0jnH5F FUF6vnv 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:15:43 +0200 Some data were put into a sequence by two separate function calls. Print the same data by a single function call instead. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/drm_dp_mst_topology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index 89fc05fa6a74..2ddcb3d2f65a 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -2861,8 +2861,8 @@ void drm_dp_mst_dump_topology(struct seq_file *m, for (i = 0x3; i < 0x8 && buf[i]; i++) seq_printf(m, "%c", buf[i]); - seq_printf(m, " revision: hw: %x.%x sw: %x.%x", buf[0x9] >> 4, buf[0x9] & 0xf, buf[0xa], buf[0xb]); - seq_printf(m, "\n"); + seq_printf(m, " revision: hw: %x.%x sw: %x.%x\n", + 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: "); -- 2.12.2