[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJZ5v0i4NtjW_6a5d9r7gRrtaUR-OM8U5UKKai8fm5v+PaXW1Q@mail.gmail.com>
Date: Thu, 29 Feb 2024 18:57:11 +0100
From: "Rafael J. Wysocki" <rafael@...nel.org>
To: Saravana Kannan <saravanak@...gle.com>
Cc: Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, Conor Dooley <conor+dt@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>, "Rafael J. Wysocki" <rafael@...nel.org>,
Ard Biesheuvel <ardb@...nel.org>, Frank Rowand <frowand.list@...il.com>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, Daniel Scally <djrscally@...il.com>,
Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
Sakari Ailus <sakari.ailus@...ux.intel.com>, Len Brown <lenb@...nel.org>, kernel-team@...roid.com,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-efi@...r.kernel.org, linux-acpi@...r.kernel.org
Subject: Re: [PATCH v4 0/4] Add post-init-providers binding to improve
suspend/resume stability
On Thu, Feb 22, 2024 at 4:46 AM Saravana Kannan <saravanak@...gle.com> wrote:
>
> This patch series adds a "post-init-providers" device tree binding that
> can be used to break dependency cycles in device tree and enforce a more
> determinstic probe/suspend/resume order. This will also improve the
> stability of global async probing and async suspend/resume and allow us
> to enable them more easily. Yet another step away from playing initcall
> chicken with probing and step towards fully async probing and
> suspend/resume.
>
> Patch 3 (the binding documentation) provides a lot more details and
> examples.
>
> v3->v4:
> - Fixed MAINTAINERS file to go with the file rename.
>
> v2->v3:
> - Changes doc/code from "post-init-supplier" to "post-init-providers"
> - Fixed some wording that was ambiguous for Conor.
> - Fixed indentation, additionalProperies and white space issues in the
> yaml syntax.
> - Fixed syntax errors in the example.
>
> v1->v2:
> - Addressed Documentation/commit text errors pointed out by Rob
> - Reordered MAINTAINERS chunk as pointed out by Krzysztof
>
> Saravana Kannan (4):
> driver core: Adds flags param to fwnode_link_add()
> driver core: Add FWLINK_FLAG_IGNORE to completely ignore a fwnode link
> dt-bindings: Add post-init-providers property
> of: property: fw_devlink: Add support for "post-init-providers"
> property
>
> .../bindings/post-init-providers.yaml | 105 ++++++++++++++++++
> MAINTAINERS | 13 ++-
> drivers/base/core.c | 14 ++-
> drivers/firmware/efi/sysfb_efi.c | 2 +-
> drivers/of/property.c | 17 ++-
> include/linux/fwnode.h | 5 +-
> 6 files changed, 142 insertions(+), 14 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/post-init-providersyaml
>
> --
This looks reasonable to me overall, so
Acked-by: Rafael J. Wysocki <rafael@...nel.org>
Powered by blists - more mailing lists