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 <[email protected]> > Cc: Rob Clark <[email protected]> > Cc: Abhinav Kumar <[email protected]> > Cc: Dmitry Baryshkov <[email protected]> > Cc: Sean Paul <[email protected]> > Cc: David Airlie <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: Rob Herring <[email protected]> > Cc: Krzysztof Kozlowski <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected]
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 <[email protected]> Best regards, Krzysztof
