| Message ID | 20200710160131.2.Id4b5c92d28b7350d047218774afaf69b683651e0@changeid |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | drm/msm: Avoid possible infinite probe deferral and speed booting | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success |
diff --git a/Documentation/devicetree/bindings/display/msm/dpu.txt b/Documentation/devicetree/bindings/display/msm/dpu.txt index 551ae26f60da..b88269524365 100644 --- a/Documentation/devicetree/bindings/display/msm/dpu.txt +++ b/Documentation/devicetree/bindings/display/msm/dpu.txt @@ -8,7 +8,9 @@ The DPU display controller is found in SDM845 SoC. MDSS: Required properties: -- compatible: "qcom,sdm845-mdss", "qcom,sc7180-mdss" +- compatible: One of: + - "qcom,sdm845-mdss", "simple-bus" + - "qcom,sc7180-mdss", "simple-bus" - reg: physical base address and length of contoller's registers. - reg-names: register region names. The following region is required: * "mdss"
We have a whole bunch of child devices that we really just want treated as other devices to instantiate. Add the "simple-bus" property for this. Commit-notes This bindings file really needs to move over to yaml. Hopefully someone at Qualcomm who's working on display code can help with that. Right now on Linux we have a manual call to of_platform_populate() to populate our children. That's pretty non-ideal as described in another patch in this series and I'm trying to remove it. I'm not sure how much of a hack to consider "simple-bus". I've seen it used like this before, but if folks tell me that it's terrible then I guess we'll have to figure some other way out of our crazy -EPROBE_DEFER loop in Linux. END Signed-off-by: Douglas Anderson <dianders@chromium.org> --- Documentation/devicetree/bindings/display/msm/dpu.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)