| Message ID | 20200710160131.7.I1605593efa85e3c9260e64fe51aedd0524544d81@changeid (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/msm: Avoid possible infinite probe deferral and speed booting | expand |
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi index 67cae5f9e47e..491362fe02ac 100644 --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi @@ -1023,7 +1023,7 @@ opp-19200000 { }; mdss: mdss@1a00000 { - compatible = "qcom,mdss"; + compatible = "qcom,mdss", "simple-bus"; reg = <0x1a00000 0x1000>, <0x1ac8000 0x3000>; reg-names = "mdss_phys", "vbif_phys";
As described in the bindings patch, this means that our child nodes are devices in their own right. Signed-off-by: Douglas Anderson <dianders@chromium.org> --- If this patch lands before the patch ("drm/msm: Avoid manually populating our children if "simple-bus" is there") it doesn't seem to be the end of the world. The first time through add_display_components() it'll notice that the children are already populated and it will be a no-op. When it gets a defer it will then depouplate them (even though it didn't populate them) and future calls will just populate them again. NOTE: I have no way to test this patch, but I'm assuming it works OK? arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)