Re: [PATCH 0/2] Add MSM8996/MSM8953 dpu catalog

[barnabas.czeman@mainlining.org]

On 2024-09-27 18:02, Dmitry Baryshkov wrote:
> On Fri, 27 Sept 2024 at 17:39, <barnabas.czeman@mainlining.org> wrote:
>> 
>> On 2024-08-01 21:25, Dmitry Baryshkov wrote:
>> > On Fri, Jun 28, 2024 at 04:39:38PM GMT, Barnabás Czémán wrote:
>> >> This patch series add dpu support for MSM8996/MSM8953 devices.
>> >>
>> >> Note, by default these platforms are still handled by the MDP5 driver
>> >> unless the `msm.prefer_mdp5=false' parameter is provided.
>> >
>> > Could you please provide a summary of features actually tested with the
>> > DPU driver? Have you tested YUV output? Have you tested RGB planes?
>> I have checked all planes they are working fine.
>> 
>> > Which LMs have you tested?
>> I have done some more testing and msm8953 LMs are fine but i have 
>> found
>> out
>> on msmm8996 LM_3 and LM_4 is not working as i see in downstream sde 
>> code
>> they are not exists.
>> This kind of messages i got for LM_3 and LM_4
>> [   34.751091] [drm:_dpu_rm_make_reservation] [dpu error]unable to 
>> find
>> appropriate mixers
>> [   34.751112] [drm:dpu_rm_reserve] [dpu error]failed to reserve hw
>> resources: -119
> 
> I think LM_3 / LM_4 are WB-only, but I didn't have time to check that.
> Maybe it's easier to omit them for now.
> 
> I hope to get back to those platforms during the forthcoming or the
> next development cycle, after (hopefully) finishing the HDMI
> conversion story.
I have tested MSM8917 and MSM8937 patches they are also working fine
at least with video mode panels, maybe they can be added for next 
iteration
intr_start also should be removed from CTLs.