Re: [Freedreno] [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-26 Thread Abhinav Kumar
On 5/26/2023 1:43 AM, Dmitry Baryshkov wrote: On Fri, 26 May 2023 at 01:42, Abhinav Kumar wrote: On 5/25/2023 3:30 PM, Dmitry Baryshkov wrote: On Fri, 26 May 2023 at 00:40, Jeykumar Sankaran wrote: On 5/22/2023 2:45 PM, Dmitry Baryshkov wrote: There is no point in having a single enum (a

Re: [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-26 Thread Neil Armstrong
On 22/05/2023 23:45, Dmitry Baryshkov wrote: There is no point in having a single enum (and a single array) for both DPU < 7.0 and DPU >= 7.0 interrupt registers. Instead define a single enum and two IRQ address arrays. Signed-off-by: Dmitry Baryshkov --- .../msm/disp/dpu1/catalog/dpu_7_0_sm8

Re: [Freedreno] [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-26 Thread Dmitry Baryshkov
On Fri, 26 May 2023 at 01:42, Abhinav Kumar wrote: > On 5/25/2023 3:30 PM, Dmitry Baryshkov wrote: > > On Fri, 26 May 2023 at 00:40, Jeykumar Sankaran > > wrote: > >> On 5/22/2023 2:45 PM, Dmitry Baryshkov wrote: > >>> There is no point in having a single enum (and a single array) for both > >>>

Re: [Freedreno] [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-26 Thread Jeykumar Sankaran
On 5/22/2023 2:45 PM, Dmitry Baryshkov wrote: There is no point in having a single enum (and a single array) for both DPU < 7.0 and DPU >= 7.0 interrupt registers. Instead define a single enum and two IRQ address arrays. Signed-off-by: Dmitry Baryshkov --- .../msm/disp/dpu1/catalog/dpu_7_0

Re: [Freedreno] [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-25 Thread Abhinav Kumar
On 5/25/2023 3:30 PM, Dmitry Baryshkov wrote: On Fri, 26 May 2023 at 00:40, Jeykumar Sankaran wrote: On 5/22/2023 2:45 PM, Dmitry Baryshkov wrote: There is no point in having a single enum (and a single array) for both DPU < 7.0 and DPU >= 7.0 interrupt registers. Instead define a single

Re: [Freedreno] [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-25 Thread Dmitry Baryshkov
On Fri, 26 May 2023 at 00:40, Jeykumar Sankaran wrote: > > > > On 5/22/2023 2:45 PM, Dmitry Baryshkov wrote: > > There is no point in having a single enum (and a single array) for both > > DPU < 7.0 and DPU >= 7.0 interrupt registers. Instead define a single > > enum and two IRQ address arrays. >

Re: [PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-22 Thread Marijn Suijten
... for 7xxx? On 2023-05-23 00:45:24, Dmitry Baryshkov wrote: > There is no point in having a single enum (and a single array) for both > DPU < 7.0 and DPU >= 7.0 interrupt registers. Instead define a single > enum and two IRQ address arrays. > > Signed-off-by: Dmitry Baryshkov Really like this

[PATCH v2 3/6] drm/msm/dpu: split interrupt address arrays

2023-05-22 Thread Dmitry Baryshkov
There is no point in having a single enum (and a single array) for both DPU < 7.0 and DPU >= 7.0 interrupt registers. Instead define a single enum and two IRQ address arrays. Signed-off-by: Dmitry Baryshkov --- .../msm/disp/dpu1/catalog/dpu_7_0_sm8350.h| 1 + .../msm/disp/dpu1/catalog/dpu_7