From patchwork Tue Nov 22 14:34:19 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 101468 Delivered-To: patch@linaro.org Received: by 10.140.97.165 with SMTP id m34csp2127328qge; Tue, 22 Nov 2016 06:35:14 -0800 (PST) X-Received: by 10.98.153.20 with SMTP id d20mr26670244pfe.44.1479825314141; Tue, 22 Nov 2016 06:35:14 -0800 (PST) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h91si122845pld.258.2016.11.22.06.35.13; Tue, 22 Nov 2016 06:35:14 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933633AbcKVOfL (ORCPT + 26 others); Tue, 22 Nov 2016 09:35:11 -0500 Received: from mout.kundenserver.de ([212.227.17.24]:52594 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933080AbcKVOfJ (ORCPT ); Tue, 22 Nov 2016 09:35:09 -0500 Received: from wuerfel.lan ([78.43.21.235]) by mrelayeu.kundenserver.de (mreue103 [212.227.15.145]) with ESMTPA (Nemesis) id 0Lnlkd-1cg2TO3075-00hrve; Tue, 22 Nov 2016 15:34:49 +0100 From: Arnd Bergmann To: Daniel Vetter Cc: Arnd Bergmann , Jani Nikula , Sean Paul , David Airlie , Bjorn Helgaas , Dave Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH] vgaarb: use valid dev pointer in vgaarb_info() Date: Tue, 22 Nov 2016 15:34:19 +0100 Message-Id: <20161122143445.1896558-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:/stbG3cnBxr2qt4rB/cYkS1KdMMif4wCwHoM/xBeMkHsadjwH59 qoPWAcuz2+V36r9Xz4wHdQPBn2O+oIyfgtHKWgJ5yWAksD6Edb2Vc3Li0OYcHPzPFLVVx4D PjaHtcnVb0jyfKrlBrxoGR7bzKijdJ0WVDRYM3OkAAGv9R2qnB21xAnhjJLKJKiCIFencrH Jt0Rcy2o8nlyeQ6grGEAA== X-UI-Out-Filterresults: notjunk:1; V01:K0:avXz1FrybFU=:Y7zcJ7W4YToY2QROZPOXrT du18G/dWzYH/BG7QjgVO0mndW33zt+rev3eA8cQJikAPSwY3BWGKwVDsc6n3cY8+o1hxylTZI q8MUBvnELR0IqzOMHoW6AdipvJqvr8LNI9utBm4eemCMWp7xAkJDRiRzgP/wEsANpquyaA6u+ iMjZCuMbnI1Rzz1q3wCx2wY1DINHuyN2+7NN9Mf4gdfEM+0VAhCVEg01r5GDN7uJToqQ1lXvp VAZsUR54+ugCCDGM0lXjTCiT1o1+PIiR/4rHKuQhCXEOiDe+IwVAFts+bHm+lwrYAd1Ms65f9 O+Rgy1qSMEaTwHve8T12l9vAMwBX0hDmwGxSSBAc04mllmpi3R64f25AbLd0X+yxhXz/7rPGg SV4dZeR6Xr9ZTNsN05blAI4W0+stA32B2PXm5JHb2hiYz6RwhtgJhIX/u2F3L3EpWPiH6jXY3 DwdsEtAVDIKfUZJ9f26hTGFHVZuyHM+DgWcSTNlavVvWCoCBOGod0wrZChY74mxq54H02bfe8 q8Ylv32Mcqfk3ejWWGlN6wVoJd1QpJMKbdU3JM7F799uzrEQlEE9t3RcFoNVwf+ew8iF6bKro 4F1Fe/p4zAn6E2UtR8SmJclecbR3F20jrGjur/W49goq7FdpLx1JlE59oljtoOya3f0hgrsIG slwLBN+bneQnTLJjbwWZnx0+7EkuDmu00AaO5ef43s/x1CNI8o4fhUGmNpGFHlHewX5o= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org We now pass the device to the debug messages, but on non-x86, this is an invalid pointer in vga_arb_device_init: drivers/gpu/vga/vgaarb.c: In function 'vga_arb_device_init': drivers/gpu/vga/vgaarb.c:1467:4: error: 'dev' may be used uninitialized in this function [-Werror=maybe-uninitialized] This moves the initialization of the dev pointer outside of the architecture #ifdef. Fixes: a75d68f62106 ("vgaarb: Use dev_printk() when possible") Signed-off-by: Arnd Bergmann --- drivers/gpu/vga/vgaarb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) -- 2.9.0 diff --git a/drivers/gpu/vga/vgaarb.c b/drivers/gpu/vga/vgaarb.c index b3d27182edd9..0f5b2dd24507 100644 --- a/drivers/gpu/vga/vgaarb.c +++ b/drivers/gpu/vga/vgaarb.c @@ -1407,7 +1407,6 @@ static int __init vga_arb_device_init(void) int rc; struct pci_dev *pdev; struct vga_device *vgadev; - struct device *dev; rc = misc_register(&vga_arb_device); if (rc < 0) @@ -1424,6 +1423,7 @@ static int __init vga_arb_device_init(void) vga_arbiter_add_pci_device(pdev); list_for_each_entry(vgadev, &vga_list, list) { + struct device *dev = &vgadev->pdev->dev; #if defined(CONFIG_X86) || defined(CONFIG_IA64) /* * Override vga_arbiter_add_pci_device()'s I/O based detection @@ -1438,7 +1438,6 @@ static int __init vga_arb_device_init(void) int i; limit = screen_info.lfb_base + screen_info.lfb_size; - dev = &vgadev->pdev->dev; /* Does firmware framebuffer belong to us? */ for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) {