* [PATCH] drm/komeda: Enable/Disable vblank interrupts @ 2019-06-07 15:03 ` Ayan Halder 0 siblings, 0 replies; 6+ messages in thread From: Ayan Halder @ 2019-06-07 15:03 UTC (permalink / raw) To: Ayan Halder, james qian wang (Arm Technology China), Liviu Dudau, Brian Starkey, daniel@ffwll.ch, airlied@linux.ie, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Cc: nd One needs to set "(struct drm_device *)->irq_enabled = true" to signal drm core to enable vblank interrupts after the hardware has been initialized. Correspondingly, one needs to disable vblank interrupts by setting "(struct drm_device *)->irq_enabled = false" at the beginning of the de-initializing routine. Signed-off-by: Ayan Kumar halder <ayan.halder@arm.com> --- drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c index 7b5cde14e3ba..b4fd8ee0d05f 100644 --- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c +++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c @@ -204,6 +204,8 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) if (err) goto uninstall_irq; + drm->irq_enabled = true; + err = drm_dev_register(drm, 0); if (err) goto uninstall_irq; @@ -211,6 +213,7 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) return kms; uninstall_irq: + drm->irq_enabled = false; drm_irq_uninstall(drm); cleanup_mode_config: drm_mode_config_cleanup(drm); @@ -225,6 +228,7 @@ void komeda_kms_detach(struct komeda_kms_dev *kms) struct drm_device *drm = &kms->base; struct komeda_dev *mdev = drm->dev_private; + drm->irq_enabled = false; mdev->funcs->disable_irq(mdev); drm_dev_unregister(drm); drm_irq_uninstall(drm); -- 2.21.0 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] drm/komeda: Enable/Disable vblank interrupts @ 2019-06-07 15:03 ` Ayan Halder 0 siblings, 0 replies; 6+ messages in thread From: Ayan Halder @ 2019-06-07 15:03 UTC (permalink / raw) To: Ayan Halder, james qian wang (Arm Technology China), Liviu Dudau, Brian Starkey, daniel@ffwll.ch, airlied@linux.ie, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Cc: nd One needs to set "(struct drm_device *)->irq_enabled = true" to signal drm core to enable vblank interrupts after the hardware has been initialized. Correspondingly, one needs to disable vblank interrupts by setting "(struct drm_device *)->irq_enabled = false" at the beginning of the de-initializing routine. Signed-off-by: Ayan Kumar halder <ayan.halder@arm.com> --- drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c index 7b5cde14e3ba..b4fd8ee0d05f 100644 --- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c +++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c @@ -204,6 +204,8 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) if (err) goto uninstall_irq; + drm->irq_enabled = true; + err = drm_dev_register(drm, 0); if (err) goto uninstall_irq; @@ -211,6 +213,7 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) return kms; uninstall_irq: + drm->irq_enabled = false; drm_irq_uninstall(drm); cleanup_mode_config: drm_mode_config_cleanup(drm); @@ -225,6 +228,7 @@ void komeda_kms_detach(struct komeda_kms_dev *kms) struct drm_device *drm = &kms->base; struct komeda_dev *mdev = drm->dev_private; + drm->irq_enabled = false; mdev->funcs->disable_irq(mdev); drm_dev_unregister(drm); drm_irq_uninstall(drm); -- 2.21.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/komeda: Enable/Disable vblank interrupts 2019-06-07 15:03 ` Ayan Halder @ 2019-06-07 18:18 ` Daniel Vetter -1 siblings, 0 replies; 6+ messages in thread From: Daniel Vetter @ 2019-06-07 18:18 UTC (permalink / raw) To: Ayan Halder Cc: airlied@linux.ie, Liviu Dudau, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, james qian wang (Arm Technology China), nd On Fri, Jun 07, 2019 at 03:03:39PM +0000, Ayan Halder wrote: > One needs to set "(struct drm_device *)->irq_enabled = true" to signal drm core > to enable vblank interrupts after the hardware has been initialized. > Correspondingly, one needs to disable vblank interrupts by setting > "(struct drm_device *)->irq_enabled = false" at the beginning of the > de-initializing routine. Only if you don't use the drm_irq_install helper. Quoting the kerneldoc in full: /** * @irq_enabled: * * Indicates that interrupt handling is enabled, specifically vblank * handling. Drivers which don't use drm_irq_install() need to set this * to true manually. */ bool irq_enabled; Not entirely sure where you've found your quote, but it's not complete. Cheers, Daniel > > Signed-off-by: Ayan Kumar halder <ayan.halder@arm.com> > --- > drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > index 7b5cde14e3ba..b4fd8ee0d05f 100644 > --- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > @@ -204,6 +204,8 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > if (err) > goto uninstall_irq; > > + drm->irq_enabled = true; > + > err = drm_dev_register(drm, 0); > if (err) > goto uninstall_irq; > @@ -211,6 +213,7 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > return kms; > > uninstall_irq: > + drm->irq_enabled = false; > drm_irq_uninstall(drm); > cleanup_mode_config: > drm_mode_config_cleanup(drm); > @@ -225,6 +228,7 @@ void komeda_kms_detach(struct komeda_kms_dev *kms) > struct drm_device *drm = &kms->base; > struct komeda_dev *mdev = drm->dev_private; > > + drm->irq_enabled = false; > mdev->funcs->disable_irq(mdev); > drm_dev_unregister(drm); > drm_irq_uninstall(drm); > -- > 2.21.0 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/komeda: Enable/Disable vblank interrupts @ 2019-06-07 18:18 ` Daniel Vetter 0 siblings, 0 replies; 6+ messages in thread From: Daniel Vetter @ 2019-06-07 18:18 UTC (permalink / raw) To: Ayan Halder Cc: james qian wang (Arm Technology China), Liviu Dudau, Brian Starkey, daniel@ffwll.ch, airlied@linux.ie, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, nd On Fri, Jun 07, 2019 at 03:03:39PM +0000, Ayan Halder wrote: > One needs to set "(struct drm_device *)->irq_enabled = true" to signal drm core > to enable vblank interrupts after the hardware has been initialized. > Correspondingly, one needs to disable vblank interrupts by setting > "(struct drm_device *)->irq_enabled = false" at the beginning of the > de-initializing routine. Only if you don't use the drm_irq_install helper. Quoting the kerneldoc in full: /** * @irq_enabled: * * Indicates that interrupt handling is enabled, specifically vblank * handling. Drivers which don't use drm_irq_install() need to set this * to true manually. */ bool irq_enabled; Not entirely sure where you've found your quote, but it's not complete. Cheers, Daniel > > Signed-off-by: Ayan Kumar halder <ayan.halder@arm.com> > --- > drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > index 7b5cde14e3ba..b4fd8ee0d05f 100644 > --- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > @@ -204,6 +204,8 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > if (err) > goto uninstall_irq; > > + drm->irq_enabled = true; > + > err = drm_dev_register(drm, 0); > if (err) > goto uninstall_irq; > @@ -211,6 +213,7 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > return kms; > > uninstall_irq: > + drm->irq_enabled = false; > drm_irq_uninstall(drm); > cleanup_mode_config: > drm_mode_config_cleanup(drm); > @@ -225,6 +228,7 @@ void komeda_kms_detach(struct komeda_kms_dev *kms) > struct drm_device *drm = &kms->base; > struct komeda_dev *mdev = drm->dev_private; > > + drm->irq_enabled = false; > mdev->funcs->disable_irq(mdev); > drm_dev_unregister(drm); > drm_irq_uninstall(drm); > -- > 2.21.0 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/komeda: Enable/Disable vblank interrupts 2019-06-07 18:18 ` Daniel Vetter @ 2019-06-13 14:27 ` Ayan Halder -1 siblings, 0 replies; 6+ messages in thread From: Ayan Halder @ 2019-06-13 14:27 UTC (permalink / raw) To: james qian wang (Arm Technology China), Liviu Dudau, Brian Starkey, airlied@linux.ie, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, nd On Fri, Jun 07, 2019 at 08:18:56PM +0200, Daniel Vetter wrote: Hi Daniel, > On Fri, Jun 07, 2019 at 03:03:39PM +0000, Ayan Halder wrote: > > One needs to set "(struct drm_device *)->irq_enabled = true" to signal drm core > > to enable vblank interrupts after the hardware has been initialized. > > Correspondingly, one needs to disable vblank interrupts by setting > > "(struct drm_device *)->irq_enabled = false" at the beginning of the > > de-initializing routine. > > Only if you don't use the drm_irq_install helper. Quoting the kerneldoc in > full: > > /** > * @irq_enabled: > * > * Indicates that interrupt handling is enabled, specifically vblank > * handling. Drivers which don't use drm_irq_install() need to set this > * to true manually. > */ > bool irq_enabled; > > Not entirely sure where you've found your quote, but it's not complete. > > Cheers, Daniel Thanks for your review. That was my quote which reflects my half-baked understanding of the issue :(. I had missed reading the header files. That said, I will squash my previous patch "drm/komeda: Avoid using DRIVER_IRQ_SHARED" into this one as the current patch is a consequence of the changes made in the previous patch. Regards, Ayan Kumar Halder > > > > > Signed-off-by: Ayan Kumar halder <ayan.halder@arm.com> > > --- > > drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > > index 7b5cde14e3ba..b4fd8ee0d05f 100644 > > --- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > > @@ -204,6 +204,8 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > > if (err) > > goto uninstall_irq; > > > > +drm->irq_enabled = true; > > + > > err = drm_dev_register(drm, 0); > > if (err) > > goto uninstall_irq; > > @@ -211,6 +213,7 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > > return kms; > > > > uninstall_irq: > > +drm->irq_enabled = false; > > drm_irq_uninstall(drm); > > cleanup_mode_config: > > drm_mode_config_cleanup(drm); > > @@ -225,6 +228,7 @@ void komeda_kms_detach(struct komeda_kms_dev *kms) > > struct drm_device *drm = &kms->base; > > struct komeda_dev *mdev = drm->dev_private; > > > > +drm->irq_enabled = false; > > mdev->funcs->disable_irq(mdev); > > drm_dev_unregister(drm); > > drm_irq_uninstall(drm); > > -- > > 2.21.0 > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/komeda: Enable/Disable vblank interrupts @ 2019-06-13 14:27 ` Ayan Halder 0 siblings, 0 replies; 6+ messages in thread From: Ayan Halder @ 2019-06-13 14:27 UTC (permalink / raw) To: james qian wang (Arm Technology China), Liviu Dudau, Brian Starkey, airlied@linux.ie, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, nd On Fri, Jun 07, 2019 at 08:18:56PM +0200, Daniel Vetter wrote: Hi Daniel, > On Fri, Jun 07, 2019 at 03:03:39PM +0000, Ayan Halder wrote: > > One needs to set "(struct drm_device *)->irq_enabled = true" to signal drm core > > to enable vblank interrupts after the hardware has been initialized. > > Correspondingly, one needs to disable vblank interrupts by setting > > "(struct drm_device *)->irq_enabled = false" at the beginning of the > > de-initializing routine. > > Only if you don't use the drm_irq_install helper. Quoting the kerneldoc in > full: > > /** > * @irq_enabled: > * > * Indicates that interrupt handling is enabled, specifically vblank > * handling. Drivers which don't use drm_irq_install() need to set this > * to true manually. > */ > bool irq_enabled; > > Not entirely sure where you've found your quote, but it's not complete. > > Cheers, Daniel Thanks for your review. That was my quote which reflects my half-baked understanding of the issue :(. I had missed reading the header files. That said, I will squash my previous patch "drm/komeda: Avoid using DRIVER_IRQ_SHARED" into this one as the current patch is a consequence of the changes made in the previous patch. Regards, Ayan Kumar Halder > > > > > Signed-off-by: Ayan Kumar halder <ayan.halder@arm.com> > > --- > > drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > > index 7b5cde14e3ba..b4fd8ee0d05f 100644 > > --- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c > > @@ -204,6 +204,8 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > > if (err) > > goto uninstall_irq; > > > > +drm->irq_enabled = true; > > + > > err = drm_dev_register(drm, 0); > > if (err) > > goto uninstall_irq; > > @@ -211,6 +213,7 @@ struct komeda_kms_dev *komeda_kms_attach(struct komeda_dev *mdev) > > return kms; > > > > uninstall_irq: > > +drm->irq_enabled = false; > > drm_irq_uninstall(drm); > > cleanup_mode_config: > > drm_mode_config_cleanup(drm); > > @@ -225,6 +228,7 @@ void komeda_kms_detach(struct komeda_kms_dev *kms) > > struct drm_device *drm = &kms->base; > > struct komeda_dev *mdev = drm->dev_private; > > > > +drm->irq_enabled = false; > > mdev->funcs->disable_irq(mdev); > > drm_dev_unregister(drm); > > drm_irq_uninstall(drm); > > -- > > 2.21.0 > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-06-13 15:02 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-06-07 15:03 [PATCH] drm/komeda: Enable/Disable vblank interrupts Ayan Halder 2019-06-07 15:03 ` Ayan Halder 2019-06-07 18:18 ` Daniel Vetter 2019-06-07 18:18 ` Daniel Vetter 2019-06-13 14:27 ` Ayan Halder 2019-06-13 14:27 ` Ayan Halder
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.