From patchwork Thu Oct 3 15:54:06 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg Kroah-Hartman X-Patchwork-Id: 175123 Delivered-To: patch@linaro.org Received: by 2002:a92:7e96:0:0:0:0:0 with SMTP id q22csp549205ill; Thu, 3 Oct 2019 09:24:04 -0700 (PDT) X-Google-Smtp-Source: APXvYqwPvkfzogK9Aex1WSBznJtUxF+6ffIM0iuAkq6+gXQUc0rxlYD0dfYvgcW+81TASqIV0/s4 X-Received: by 2002:a17:906:c79a:: with SMTP id cw26mr8337174ejb.265.1570119762524; Thu, 03 Oct 2019 09:22:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1570119762; cv=none; d=google.com; s=arc-20160816; b=dNcJzeFZfQjiu+63pUudx/Vaxo0Rfk+F4Q7Fnr71PW8afzYvAFfAhxHQgy58bhwuh2 2Lcl6eyjtB9iSw+LOA1IXa5zyayviYmPOv7IvxbIK/bzat3dVu1fCLgJQLIH5IZOtrhA fy2JN8vqYHWjKSQLx+glJgeh5RBk8v1r+Ykoou9aTtkKUPN3zYjoGuT+KGFp81pa/N40 hmuoPCogZdKUfOWkBNNQYGJEnCVLSjDHVD9gHKjBv+aordUvymrShxno7RCXysefPY0H Qcf3x/DBn3xf/sqYZn/zFlvEf4wTahQ0j3yla1lJ4Znci2DBa620vn8Dxig8hWbXhm9I f0Fw== 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=0GKoO9jmZPRYpMRPoO82KM5BDBmvU+BmaUdYvygjXdQ=; b=yYHGzIdZ/rcjpGIcakb36Hgwx9/9NRH4GsdneJA1zmUk/Vcx/qOLGUI6iKKhVIANvP heXTVV1sYffh3jGB6Xn5piXeLf98+2ACUZS8yr8kL+SaK/ebcFXVqyeLW1ET253t9m6O cldU4hcKow5xhdXFzdlZXf/Ze9xCdGKZU2/ddhQaV7udhd2eT9zya5mxxU4NVQDZlk+9 nw+bKa1Jss7dhDXR8x00csGvvIq18pixRa0Gp4b3p50Qs1GgOjhQiEcWwOEGD4q6WxiP EobCBTVFkc7nswbJtVPws+lnGVBdc6GKLWdgkkLL2NUOlrQRm/3YNhGWIoWGwKNxpgcG BI7g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=bHqH9a7H; spf=pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=stable-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 r9si1466232eju.10.2019.10.03.09.22.42; Thu, 03 Oct 2019 09:22:42 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of stable-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=bHqH9a7H; spf=pass (google.com: best guess record for domain of stable-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2390338AbfJCQWf (ORCPT + 14 others); Thu, 3 Oct 2019 12:22:35 -0400 Received: from mail.kernel.org ([198.145.29.99]:51252 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2390333AbfJCQWe (ORCPT ); Thu, 3 Oct 2019 12:22:34 -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 D1CB7215EA; Thu, 3 Oct 2019 16:22:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1570119753; bh=iJES15h1t2wVL0pPglc2PLGWkPNnaLGEqbTRypL0zkQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=bHqH9a7HriKzrgqun8Or1/jeF90ZOg2iTv6XZSoZx9DwIjQVWucuOOOy0RoNCrOWw CGv4dY+LBOESuNDDLcAC28Bu/zZd+KEmMf7aNaBRrBHMGXCpfIQecZ21h8wronAgw4 RxFdbI754tsM/M3RaBUhD9bq3xjoKjKopxEEkY4w= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Mark Brown , Lee Jones Subject: [PATCH 4.19 180/211] regulator: Defer init completion for a while after late_initcall Date: Thu, 3 Oct 2019 17:54:06 +0200 Message-Id: <20191003154527.299754434@linuxfoundation.org> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20191003154447.010950442@linuxfoundation.org> References: <20191003154447.010950442@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Mark Brown commit 55576cf1853798e86f620766e23b604c9224c19c upstream. The kernel has no way of knowing when we have finished instantiating drivers, between deferred probe and systems that build key drivers as modules we might be doing this long after userspace has booted. This has always been a bit of an issue with regulator_init_complete since it can power off hardware that's not had it's driver loaded which can result in user visible effects, the main case is powering off displays. Practically speaking it's not been an issue in real systems since most systems that use the regulator API are embedded and build in key drivers anyway but with Arm laptops coming on the market it's becoming more of an issue so let's do something about it. In the absence of any better idea just defer the powering off for 30s after late_initcall(), this is obviously a hack but it should mask the issue for now and it's no more arbitrary than late_initcall() itself. Ideally we'd have some heuristics to detect if we're on an affected system and tune or skip the delay appropriately, and there may be some need for a command line option to be added. Link: https://lore.kernel.org/r/20190904124250.25844-1-broonie@kernel.org Signed-off-by: Mark Brown Tested-by: Lee Jones Cc: stable@vger.kernel.org Signed-off-by: Greg Kroah-Hartman --- drivers/regulator/core.c | 42 +++++++++++++++++++++++++++++++----------- 1 file changed, 31 insertions(+), 11 deletions(-) --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -4789,7 +4789,7 @@ static int __init regulator_init(void) /* init early to allow our consumers to complete system booting */ core_initcall(regulator_init); -static int __init regulator_late_cleanup(struct device *dev, void *data) +static int regulator_late_cleanup(struct device *dev, void *data) { struct regulator_dev *rdev = dev_to_rdev(dev); const struct regulator_ops *ops = rdev->desc->ops; @@ -4838,18 +4838,9 @@ unlock: return 0; } -static int __init regulator_init_complete(void) +static void regulator_init_complete_work_function(struct work_struct *work) { /* - * Since DT doesn't provide an idiomatic mechanism for - * enabling full constraints and since it's much more natural - * with DT to provide them just assume that a DT enabled - * system has full constraints. - */ - if (of_have_populated_dt()) - has_full_constraints = true; - - /* * Regulators may had failed to resolve their input supplies * when were registered, either because the input supply was * not registered yet or because its parent device was not @@ -4866,6 +4857,35 @@ static int __init regulator_init_complet */ class_for_each_device(®ulator_class, NULL, NULL, regulator_late_cleanup); +} + +static DECLARE_DELAYED_WORK(regulator_init_complete_work, + regulator_init_complete_work_function); + +static int __init regulator_init_complete(void) +{ + /* + * Since DT doesn't provide an idiomatic mechanism for + * enabling full constraints and since it's much more natural + * with DT to provide them just assume that a DT enabled + * system has full constraints. + */ + if (of_have_populated_dt()) + has_full_constraints = true; + + /* + * We punt completion for an arbitrary amount of time since + * systems like distros will load many drivers from userspace + * so consumers might not always be ready yet, this is + * particularly an issue with laptops where this might bounce + * the display off then on. Ideally we'd get a notification + * from userspace when this happens but we don't so just wait + * a bit and hope we waited long enough. It'd be better if + * we'd only do this on systems that need it, and a kernel + * command line option might be useful. + */ + schedule_delayed_work(®ulator_init_complete_work, + msecs_to_jiffies(30000)); class_for_each_device(®ulator_class, NULL, NULL, regulator_register_fill_coupling_array);