Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp3785834ybi; Mon, 29 Jul 2019 12:35:18 -0700 (PDT) X-Google-Smtp-Source: APXvYqy3u+D7dRB5FIzDtJdraEU8V72QdqqDlOZqokO1WczXecIWPZeo0dXUSPdn49VWrJFvSk0n X-Received: by 2002:a63:a302:: with SMTP id s2mr114028pge.125.1564428917898; Mon, 29 Jul 2019 12:35:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1564428917; cv=none; d=google.com; s=arc-20160816; b=FYcX14U2iON9XWcfcsi0l/4/zAmNDDdWfUlb2KTDipDbQSyb2JSXbUtlIjDehz98XN uILgGKxj6v08nfutGD1iv7dW+yUmUJANpP6EHBqiLjwuirJzjG0BRRdLzCxpgrhqSO+6 nuHGcz8xiCnx4f3YoLsDnzHjBPJJGdlTpwttdDc03O8A0MX66XQDjmC6oN9yarGc23Fk eh21okq9KIpvbnY5RYw5f/Qz9egq1TEEA2zs/BlFkowMyXQKTEeDWEWhljJzIm030LG9 dIfyKWQ4qwLHZwMdL9YLTVQxoFyIDZoTaM+mgTfpmoiZRd1Gwva0eqM1pCSFgpbbxi6z inpQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=ZJ0pF7Ram2rwxdecb3SWfR8O31u1o5s7qRLe0cpVobA=; b=ZugCTN1KUso0clYZ5pTcuLp7V0GQ3auYxJpCUAFYNvoImkxayK+D5a0mDqLYky7w9y pcKAV851QjrtCQ1GsR+TfGNRif6oUcI3rkHmO/vuYgyvWB6DXavA9mnjZXNqdqd57Bzc P6g7TPn7UpdFDxk22x89tLN57eVSDCHOX9J7ViFmaY+quqc+hP9wC883qUQJdlrTDppB 6fkjpmXfb7Dc+EBmXfjCybUlbIXRuVlWzDKr8LDvFjbByOHZs2hsf7b2S2wPr+j+L0px qJvWNceHi0mlua6x3h8iYKQmc3QVwfFcOHhMuFPn4HPROo2UIR1Oyr0iJplBwduVKsLd LT8g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=oYEDWaOe; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id o1si25363563pld.278.2019.07.29.12.35.03; Mon, 29 Jul 2019 12:35:17 -0700 (PDT) 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; dkim=pass header.i=@kernel.org header.s=default header.b=oYEDWaOe; 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 S1730296AbfG2TdC (ORCPT + 99 others); Mon, 29 Jul 2019 15:33:02 -0400 Received: from mail.kernel.org ([198.145.29.99]:46812 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730290AbfG2Tc7 (ORCPT ); Mon, 29 Jul 2019 15:32:59 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 10B2E21655; Mon, 29 Jul 2019 19:32:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1564428778; bh=C3mhXCl2ZWYBLwyb38j1K9FbAQ3C3952L957kgeENwA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=oYEDWaOeGNXbfRZIjscG73c/vbjEayg1rFwlAh1IhAJq0WEm6VS0cuWVlcP8jjQx/ Yb0qlWJwa8sdg4p7Gs3PYnc34JGs/jYYQwDvv4mei94IH3VRKlebEtJkc9I/wg1KRY /lT8cIbBPElmdhQqbY/edtK1UUAZsb3u7OhPJrhI= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Lyude Paul , Marc Meledandri , Ben Skeggs Subject: [PATCH 4.14 146/293] drm/nouveau/i2c: Enable i2c pads & busses during preinit Date: Mon, 29 Jul 2019 21:20:37 +0200 Message-Id: <20190729190835.725603223@linuxfoundation.org> X-Mailer: git-send-email 2.22.0 In-Reply-To: <20190729190820.321094988@linuxfoundation.org> References: <20190729190820.321094988@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Lyude Paul commit 7cb95eeea6706c790571042a06782e378b2561ea upstream. It turns out that while disabling i2c bus access from software when the GPU is suspended was a step in the right direction with: commit 342406e4fbba ("drm/nouveau/i2c: Disable i2c bus access after ->fini()") We also ended up accidentally breaking the vbios init scripts on some older Tesla GPUs, as apparently said scripts can actually use the i2c bus. Since these scripts are executed before initializing any subdevices, we end up failing to acquire access to the i2c bus which has left a number of cards with their fan controllers uninitialized. Luckily this doesn't break hardware - it just means the fan gets stuck at 100%. This also means that we've always been using our i2c busses before initializing them during the init scripts for older GPUs, we just didn't notice it until we started preventing them from being used until init. It's pretty impressive this never caused us any issues before! So, fix this by initializing our i2c pad and busses during subdev pre-init. We skip initializing aux busses during pre-init, as those are guaranteed to only ever be used by nouveau for DP aux transactions. Signed-off-by: Lyude Paul Tested-by: Marc Meledandri Fixes: 342406e4fbba ("drm/nouveau/i2c: Disable i2c bus access after ->fini()") Cc: stable@vger.kernel.org Signed-off-by: Ben Skeggs Signed-off-by: Greg Kroah-Hartman --- drivers/gpu/drm/nouveau/nvkm/subdev/i2c/base.c | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) --- a/drivers/gpu/drm/nouveau/nvkm/subdev/i2c/base.c +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/i2c/base.c @@ -185,6 +185,25 @@ nvkm_i2c_fini(struct nvkm_subdev *subdev } static int +nvkm_i2c_preinit(struct nvkm_subdev *subdev) +{ + struct nvkm_i2c *i2c = nvkm_i2c(subdev); + struct nvkm_i2c_bus *bus; + struct nvkm_i2c_pad *pad; + + /* + * We init our i2c busses as early as possible, since they may be + * needed by the vbios init scripts on some cards + */ + list_for_each_entry(pad, &i2c->pad, head) + nvkm_i2c_pad_init(pad); + list_for_each_entry(bus, &i2c->bus, head) + nvkm_i2c_bus_init(bus); + + return 0; +} + +static int nvkm_i2c_init(struct nvkm_subdev *subdev) { struct nvkm_i2c *i2c = nvkm_i2c(subdev); @@ -238,6 +257,7 @@ nvkm_i2c_dtor(struct nvkm_subdev *subdev static const struct nvkm_subdev_func nvkm_i2c = { .dtor = nvkm_i2c_dtor, + .preinit = nvkm_i2c_preinit, .init = nvkm_i2c_init, .fini = nvkm_i2c_fini, .intr = nvkm_i2c_intr,