| Message ID | 20240212213147.489377-1-saravanak@google.com |
|---|---|
| Headers | show |
| Series | Add post-init-supplier binding to improve suspend/resume stability | expand |
On Mon, 12 Feb 2024 13:31:44 -0800, Saravana Kannan wrote: > The post-init-supplier property can be used to break a dependency cycle by > marking some supplier(s) as a post device initialization supplier(s). This > allows an OS to do a better job at ordering initialization and > suspend/resume of the devices in a dependency cycle. > > Signed-off-by: Saravana Kannan <saravanak@google.com> > --- > .../bindings/post-init-supplier.yaml | 101 ++++++++++++++++++ > MAINTAINERS | 13 +-- > 2 files changed, 108 insertions(+), 6 deletions(-) > create mode 100644 Documentation/devicetree/bindings/post-init-supplier.yaml > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: ./Documentation/devicetree/bindings/post-init-supplier.yaml:84:12: [error] syntax error: mapping values are not allowed here (syntax) dtschema/dtc warnings/errors: make[2]: *** Deleting file 'Documentation/devicetree/bindings/post-init-supplier.example.dts' Documentation/devicetree/bindings/post-init-supplier.yaml:84:12: mapping values are not allowed in this context make[2]: *** [Documentation/devicetree/bindings/Makefile:26: Documentation/devicetree/bindings/post-init-supplier.example.dts] Error 1 make[2]: *** Waiting for unfinished jobs.... ./Documentation/devicetree/bindings/post-init-supplier.yaml:84:12: mapping values are not allowed in this context /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/post-init-supplier.yaml: ignoring, error parsing file make[1]: *** [/builds/robherring/dt-review-ci/linux/Makefile:1428: dt_binding_check] Error 2 make: *** [Makefile:240: __sub-make] Error 2 doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240212213147.489377-4-saravanak@google.com The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
This patch series adds a "post-init-supplier" 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 docunentation) provide a lot more details and examples. 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-supplier property of: property: fw_devlink: Add support for "post-init-supplier" property .../bindings/post-init-supplier.yaml | 101 ++++++++++++++++++ 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, 138 insertions(+), 14 deletions(-) create mode 100644 Documentation/devicetree/bindings/post-init-supplier.yaml