Messages in this thread | | | Date | Tue, 8 Nov 2022 22:10:29 +0100 | Subject | Re: [PATCH v2 02/18] dt-bindings: msm: dsi-controller-main: Fix power-domain constraint | From | Krzysztof Kozlowski <> |
| |
On 08/11/2022 00:56, Bryan O'Donoghue wrote: > power-domain is required for the sc7180 dispcc GDSC but not every qcom SoC > has a similar dependency for example the aqp8064. > > Most Qcom SoC's using mdss-dsi-ctrl seem to have the ability to > power-collapse the MDP without collapsing DSI. > > For example the qcom vendor kernel commit for apq8084, msm8226, msm8916, msm8974 > > https://review.carbonrom.org/plugins/gitiles/CarbonROM/android_kernel_oneplus_msm8994/+/7b5c011a770daa2811778937ed646237a28a8694 > > "ARM: dts: msm: add mdss gdsc supply to dsi controller device > > It is possible for the DSI controller to be active when MDP is > power collapsed. DSI controller needs to have it's own vote for > mdss gdsc to ensure that gdsc remains on in such cases." > > This however doesn't appear to be the case for the apq8064 so we shouldn't > be marking power-domain as required in yaml checks. > > Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings") > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > Cc: Rob Clark <robdclark@gmail.com> > Cc: Abhinav Kumar <quic_abhinavk@quicinc.com> > Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > Cc: Sean Paul <sean@poorly.run> > Cc: David Airlie <airlied@gmail.com> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > Cc: linux-arm-msm@vger.kernel.org > Cc: dri-devel@lists.freedesktop.org > Cc: freedreno@lists.freedesktop.org > Cc: devicetree@vger.kernel.org > Cc: linux-kernel@vger.kernel.org
Your Cc list is huge and not necessary to store in git log. For example I am appearing there twice. Please keep it under '---'.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards, Krzysztof
| |