Re: [PATCH v2] Revert "drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set"

2024-06-05 Thread Abhinav Kumar
On Wed, 22 May 2024 13:24:28 +0300, Dmitry Baryshkov wrote: > In the DPU driver blank IRQ handling is called from a vblank worker and > can happen outside of the irq_enable / irq_disable pair. Using the > worker makes that completely asynchronous with the rest of the code. > Revert commit

Re: [PATCH v2] Revert "drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set"

2024-05-30 Thread Dmitry Baryshkov
On Wed, May 29, 2024 at 04:44:59PM -0700, Abhinav Kumar wrote: > > > On 5/24/2024 1:22 PM, Dmitry Baryshkov wrote: > > On Fri, May 24, 2024 at 12:58:53PM -0700, Abhinav Kumar wrote: > > > > > > > > > On 5/22/2024 3:24 AM, Dmitry Baryshkov wrote: > > > > In the DPU driver blank IRQ handling is

Re: [PATCH v2] Revert "drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set"

2024-05-29 Thread Abhinav Kumar
On 5/24/2024 1:22 PM, Dmitry Baryshkov wrote: On Fri, May 24, 2024 at 12:58:53PM -0700, Abhinav Kumar wrote: On 5/22/2024 3:24 AM, Dmitry Baryshkov wrote: In the DPU driver blank IRQ handling is called from a vblank worker and can happen outside of the irq_enable / irq_disable pair. Using

Re: [PATCH v2] Revert "drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set"

2024-05-24 Thread Dmitry Baryshkov
On Fri, May 24, 2024 at 12:58:53PM -0700, Abhinav Kumar wrote: > > > On 5/22/2024 3:24 AM, Dmitry Baryshkov wrote: > > In the DPU driver blank IRQ handling is called from a vblank worker and > > can happen outside of the irq_enable / irq_disable pair. Using the > > worker makes that completely

Re: [PATCH v2] Revert "drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set"

2024-05-24 Thread Abhinav Kumar
On 5/22/2024 3:24 AM, Dmitry Baryshkov wrote: In the DPU driver blank IRQ handling is called from a vblank worker and can happen outside of the irq_enable / irq_disable pair. Using the worker makes that completely asynchronous with the rest of the code. Revert commit d13f638c9b88

[PATCH v2] Revert "drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set"

2024-05-22 Thread Dmitry Baryshkov
In the DPU driver blank IRQ handling is called from a vblank worker and can happen outside of the irq_enable / irq_disable pair. Using the worker makes that completely asynchronous with the rest of the code. Revert commit d13f638c9b88 ("drm/msm/dpu: drop dpu_encoder_phys_ops.atomic_mode_set") to