[PATCH 2/5] dt-bindings: display/msm: Document the DPU for QCS8300

[Dmitry Baryshkov]

On Thu, 5 Dec 2024 at 04:46, Yongxing Mou <quic_yongmou at quicinc.com> wrote:
>
>
>
> On 2024/11/29 21:44, Dmitry Baryshkov wrote:
> > On Fri, 29 Nov 2024 at 11:51, Yongxing Mou <quic_yongmou at quicinc.com> wrote:
> >>
> >>
> >>
> >> On 2024/11/27 21:56, Dmitry Baryshkov wrote:
> >>> On Wed, Nov 27, 2024 at 03:05:02PM +0800, Yongxing Mou wrote:
> >>>> Document the DPU for Qualcomm QCS8300 platform.
> >>>>
> >>>> Signed-off-by: Yongxing Mou <quic_yongmou at quicinc.com>
> >>>> ---
> >>>>    Documentation/devicetree/bindings/display/msm/qcom,sm8650-dpu.yaml | 1 +
> >>>>    1 file changed, 1 insertion(+)
> >>>>
> >>>> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8650-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8650-dpu.yaml
> >>>> index 01cf79bd754b491349c52c5aef49ba06e835d0bf..631181df2bcf2752679be4de0dee74e15e2c66c2 100644
> >>>> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm8650-dpu.yaml
> >>>> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8650-dpu.yaml
> >>>> @@ -14,6 +14,7 @@ $ref: /schemas/display/msm/dpu-common.yaml#
> >>>>    properties:
> >>>>      compatible:
> >>>>        enum:
> >>>> +      - qcom,qcs8300-dpu
> >>>
> >>> The DPU is the same as the one on SA8775P. Drop it completely.
> >> Thanks,got it.The DPU of qcs8300 and sa8775p are exactly the same, but
> >> sa8775p has two DPUs, while qcs8300 has only one DPU and has removed
> >> intf2, intf4, and intf8. That's their difference.Can qcs8300 reuse
> >> sa8775p's dpu driver? if we can reuse it,will drop this patch.
> >
> > Are INTF_n blocks implemented in silicon or not? What happens if one
> > reads one of INTF_2 or INTF_4 registers?
> > If they are actually RAZ or cause data aborts, you can not reuse
> > SA8775P catalog entry. Please add corresponding data structures to
> > dpu_8_4_sa8775p.h.
> Sorry for later reponse,i took some time to confirm the hardware
> details.For QCS8300,DP1 CTRL/INTF_n is present in silicon.Only DP1 phy
> is removed in silicon,it not available.so i think we can reuse the
> sa8775p dpu.

Sounds good, thanks for the confirmation

> >
> >>
> >>>
> >>>>          - qcom,sa8775p-dpu
> >>>>          - qcom,sm8650-dpu
> >>>>          - qcom,x1e80100-dpu
> >>>>
> >>>> --
> >>>> 2.34.1
> >>>>
> >>>
> >>
> >
> >
>


-- 
With best wishes
Dmitry