From patchwork Thu Apr 17 14:25:02 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 882186 Received: from mail-lf1-f42.google.com (mail-lf1-f42.google.com [209.85.167.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4212F252287 for ; Thu, 17 Apr 2025 14:25:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.42 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744899941; cv=none; b=bSsnNGBrrchG9KDGJlIWDTjxrI+SedkDIzLjQta1OSdvhewGf1giCqFJUBe/OHFmtbAtRlvo2MjvaxoA4LDJAbTD4iQclQ0HyzjnCBsP9Qf5xSjE6g4hhucp5b0UL1HBFSyKbpqxVcBLRC1N0YXkSMurDS3O2n4i5V4SZvjufA0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744899941; c=relaxed/simple; bh=9Au4Fe2OLSeRsUR1xwYHxdSnAkAsfOjjKZwl2VCoWc0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Lz+sXPsOY1Awrj6E2a5yC5k3/VWGGxbDcRvsO9Dz3L7j8DNco9NO9IBYGJUPPEoE3XH0kB3WEaLprkA3RwvTs4JDe2USFCFtB3DmQDMjsOsNlmBbflRym6qrsh/IrcpQ2xRa1A1gzfRpUMsXEJJFzFXgepHTn8RKyiHib+E0QhU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=ijiDNEaB; arc=none smtp.client-ip=209.85.167.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="ijiDNEaB" Received: by mail-lf1-f42.google.com with SMTP id 2adb3069b0e04-54998f865b8so845668e87.3 for ; Thu, 17 Apr 2025 07:25:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1744899937; x=1745504737; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=J79qRcZGJZJmCItGNs9XR09SxwFcm6gRBRDcexcVIfk=; b=ijiDNEaBaxLMYig1oNR9LWDd2a1dIGwM9A8nWnjLQkun68n+s0wh1avqAqo8GHOuO8 T5nsMhnjs0lRWoV7I8ayUpgtmvo+483R0M46LOlAREimkJOi9lnu4lPNgapEnZhn+YWT wT1MucED4DaSYGbhXNna0P1XSv3o+dCOzLaFGmwVzryKNk1Fct3xLxndNP3dsfmtz51e b0JkdyqN6GzRuxD8BtuVc9PdHmQ3c7KNwNvjGwZQvY+rRt4EL9EuHN7GYPF4oOkeH4Lz gn+eQczWUlalS8PbbqQ3v7Os4KJmqNjGdBq89Vv5DgG6bYYz6CjnTuD+JhvOJerDEFgu BZdw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744899937; x=1745504737; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=J79qRcZGJZJmCItGNs9XR09SxwFcm6gRBRDcexcVIfk=; b=OCbSDmuXuYjGuExHbvX4tnWegq6205qyvv8601P8mXJMXO6lZ7Jeom3/bEhzZ0jBZv HtEYQnOWfbZ3fJ0BQLdWFi6MSGcNPlKVWOGo92f05FW+L8VETtuJQbTsb8HHBamUIcUP nrPJASNEUJrnakv5LSqzwkQRBLcaJm/rFYhJsgfvtbbYHi3yxTRvoLJy5FUbYrzFQVzl nFYp0n4s7hA2WDl/H+AcMxVxhKf+D1xvz1r66iVX40Do1/7A4xFiHsfhQGFRrNnaObso Np/mqYCABKkSYofgRUOrN+7oaztJ+nWmkCSe3ZDaw92Sf2dhtth2otmjcAdIwSNH7x4u 6ZpA== X-Forwarded-Encrypted: i=1; AJvYcCWsqmyplibcxrJFNwsNt560GF17O/emxOfNIaKeDD0T5XAAoWxNkTaTdtdaKJXw93Pmewsdm0PUMg==@vger.kernel.org X-Gm-Message-State: AOJu0YxYgjEyvgK2kYelNmnspb6k/mM5IqhG3C/MdZuVrFtZDnfMqsCA 8+io45uage6jPsukZ+2tv/dykROGh32QwqqQdwpNcVFCU8kKgXA19C4ssk7QEkM= X-Gm-Gg: ASbGnctQJ0ZKq4SH6AmFzwsSJpg0/Y56ZspqT/xZmbNARIBYC+9Izuj9ucj9oFqVRzj Ohf/2sKw7bDhxYcL4kCa+pJGsO+F8ap3b+qoDwOFvm1tmkmLk8u3jTI43cpAcxueC9V9tccgXr4 jtsADgN5hOGMUOYa5JY8GxEioUD20K7fkmQN2RrgGJ6hpS8d3jnIvHxNu2+9BqTYdayPycSj2Sd s0eQPGiPEVCaPTs3XAHMANGChEDZy9Zbg2VHB9QcVp6ldWRdks//U7XrAbrHKFHxZOXX1k+SQOo k1rRlFFyxO91O9vGhtj/RPpWDTttXR0n/IfjjHfm1Wny1oKsI0btZ0sO3YWPbmQ6OpWSr7rDK54 6LRLOf1NyLIIUfuYqORzwZFv59g== X-Google-Smtp-Source: AGHT+IFiUhuBtQQApad2nNH4tL0zFojb+VPiB8t6cSOcnI01Uz7KMV7xXLgUmCgNPnGefcFd0iptEA== X-Received: by 2002:ac2:4e0b:0:b0:54a:cc11:b558 with SMTP id 2adb3069b0e04-54d64ab204amr1985603e87.33.1744899937335; Thu, 17 Apr 2025 07:25:37 -0700 (PDT) Received: from uffe-tuxpro14.. (h-178-174-189-39.A498.priv.bahnhof.se. [178.174.189.39]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-54d65e43647sm370096e87.58.2025.04.17.07.25.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 17 Apr 2025 07:25:36 -0700 (PDT) From: Ulf Hansson To: Saravana Kannan , Stephen Boyd , linux-pm@vger.kernel.org Cc: "Rafael J . Wysocki" , Greg Kroah-Hartman , Michael Grzeschik , Bjorn Andersson , Abel Vesa , Devarsh Thakkar , Peng Fan , Tomi Valkeinen , Johan Hovold , Maulik Shah , Ulf Hansson , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 04/11] pmdomain: core: Add a bus and a driver for genpd providers Date: Thu, 17 Apr 2025 16:25:02 +0200 Message-ID: <20250417142513.312939-5-ulf.hansson@linaro.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250417142513.312939-1-ulf.hansson@linaro.org> References: <20250417142513.312939-1-ulf.hansson@linaro.org> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 When we create a genpd via pm_genpd_init() we are initializing a corresponding struct device for it, but we don't add the device to any bus_type. It has not really been needed as the device is used as cookie to help us manage OPP tables. However, to prepare to make better use of the device let's add a new genpd provider bus_type and a corresponding genpd provider driver. Subsequent changes will make use of this. Signed-off-by: Ulf Hansson --- drivers/pmdomain/core.c | 89 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 88 insertions(+), 1 deletion(-) diff --git a/drivers/pmdomain/core.c b/drivers/pmdomain/core.c index 035b65563947..da51a61a974c 100644 --- a/drivers/pmdomain/core.c +++ b/drivers/pmdomain/core.c @@ -27,6 +27,11 @@ /* Provides a unique ID for each genpd device */ static DEFINE_IDA(genpd_ida); +/* The parent for genpd_provider devices. */ +static struct device genpd_provider_bus = { + .init_name = "genpd_provider", +}; + #define GENPD_RETRY_MAX_MS 250 /* Approximate */ #define GENPD_DEV_CALLBACK(genpd, type, callback, dev) \ @@ -44,6 +49,14 @@ static DEFINE_IDA(genpd_ida); static LIST_HEAD(gpd_list); static DEFINE_MUTEX(gpd_list_lock); +#define to_genpd_provider_drv(d) container_of(d, struct genpd_provider_drv, drv) + +struct genpd_provider_drv { + struct device_driver drv; + int (*probe)(struct device *dev); + void (*remove)(struct device *dev); +}; + struct genpd_lock_ops { void (*lock)(struct generic_pm_domain *genpd); void (*lock_nested)(struct generic_pm_domain *genpd, int depth); @@ -2225,6 +2238,26 @@ static int genpd_set_default_power_state(struct generic_pm_domain *genpd) return 0; } +static int genpd_provider_bus_probe(struct device *dev) +{ + struct genpd_provider_drv *drv = to_genpd_provider_drv(dev->driver); + + return drv->probe(dev); +} + +static void genpd_provider_bus_remove(struct device *dev) +{ + struct genpd_provider_drv *drv = to_genpd_provider_drv(dev->driver); + + drv->remove(dev); +} + +static const struct bus_type genpd_provider_bus_type = { + .name = "genpd_provider", + .probe = genpd_provider_bus_probe, + .remove = genpd_provider_bus_remove, +}; + static void genpd_provider_release(struct device *dev) { /* nothing to be done here */ @@ -2262,6 +2295,8 @@ static int genpd_alloc_data(struct generic_pm_domain *genpd) genpd->gd = gd; device_initialize(&genpd->dev); genpd->dev.release = genpd_provider_release; + genpd->dev.bus = &genpd_provider_bus_type; + genpd->dev.parent = &genpd_provider_bus; if (!genpd_is_dev_name_fw(genpd)) { dev_set_name(&genpd->dev, "%s", genpd->name); @@ -3355,9 +3390,61 @@ int of_genpd_parse_idle_states(struct device_node *dn, } EXPORT_SYMBOL_GPL(of_genpd_parse_idle_states); +static int genpd_provider_probe(struct device *dev) +{ + return 0; +} + +static void genpd_provider_remove(struct device *dev) +{ +} + +static void genpd_provider_sync_state(struct device *dev) +{ +} + +static struct genpd_provider_drv genpd_provider_drv = { + .drv = { + .name = "genpd_provider", + .bus = &genpd_provider_bus_type, + .sync_state = genpd_provider_sync_state, + .suppress_bind_attrs = true, + }, + .probe = genpd_provider_probe, + .remove = genpd_provider_remove, +}; + static int __init genpd_bus_init(void) { - return bus_register(&genpd_bus_type); + int ret; + + ret = device_register(&genpd_provider_bus); + if (ret) { + put_device(&genpd_provider_bus); + return ret; + } + + ret = bus_register(&genpd_provider_bus_type); + if (ret) + goto err_dev; + + ret = bus_register(&genpd_bus_type); + if (ret) + goto err_prov_bus; + + ret = driver_register(&genpd_provider_drv.drv); + if (ret) + goto err_bus; + + return 0; + +err_bus: + bus_unregister(&genpd_bus_type); +err_prov_bus: + bus_unregister(&genpd_provider_bus_type); +err_dev: + device_unregister(&genpd_provider_bus); + return ret; } core_initcall(genpd_bus_init);