Hi Thomas, Can you please hold off on applying the kmb patch, I am seeing some issues while testing. Modetest works, but video playback only plays once, and it fails the second time with this patch.
Thanks, Anitha > -----Original Message----- > From: Sam Ravnborg <s...@ravnborg.org> > Sent: Tuesday, August 3, 2021 8:05 AM > To: Thomas Zimmermann <tzimmerm...@suse.de> > Cc: dan...@ffwll.ch; airl...@linux.ie; alexander.deuc...@amd.com; > christian.koe...@amd.com; liviu.du...@arm.com; brian.star...@arm.com; > bbrezil...@kernel.org; nicolas.fe...@microchip.com; > maarten.lankho...@linux.intel.com; mrip...@kernel.org; ste...@agner.ch; > alison.w...@nxp.com; patrik.r.jakobs...@gmail.com; Chrisanthus, Anitha > <anitha.chrisant...@intel.com>; robdcl...@gmail.com; Dea, Edmund J > <edmund.j....@intel.com>; s...@poorly.run; shawn...@kernel.org; > s.ha...@pengutronix.de; ker...@pengutronix.de; jyri.sa...@iki.fi; > to...@kernel.org; dan.sned...@microchip.com; > tomi.valkei...@ideasonboard.com; amd-gfx@lists.freedesktop.org; dri- > de...@lists.freedesktop.org; linux-arm-ker...@lists.infradead.org; linux-arm- > m...@vger.kernel.org; freedr...@lists.freedesktop.org > Subject: Re: [PATCH v2 00/14] drm: Make DRM's IRQ helpers legacy > > Hi Thomas, > > On Tue, Aug 03, 2021 at 11:06:50AM +0200, Thomas Zimmermann wrote: > > DRM's IRQ helpers are only helpful for old, non-KMS drivers. Move > > the code behind CONFIG_DRM_LEGACY. Convert KMS drivers to Linux > > IRQ interfaces. > > > > DRM provides IRQ helpers for setting up, receiving and removing IRQ > > handlers. It's an abstraction over plain Linux functions. The code > > is mid-layerish with several callbacks to hook into the rsp drivers. > > Old UMS driver have their interrupts enabled via ioctl, so these > > abstractions makes some sense. Modern KMS manage all their interrupts > > internally. Using the DRM helpers adds indirection without benefits. > > > > Most KMS drivers already use Linux IRQ functions instead of DRM's > > abstraction layer. Patches 1 to 12 convert the remaining ones. > > The patches also resolve a bug for devices without assigned interrupt > > number. DRM helpers don't test for IRQ_NOTCONNECTED, so drivers do > > not detect if the device has no interrupt assigned. > > > > Patch 13 removes an unused function. > > > > Patch 14 moves the DRM IRQ helpers behind CONFIG_DRM_LEGACY. Only > > the old non-KMS drivers still use the functionality. > > > > v2: > > * drop IRQ_NOTCONNECTED test from atmel-hlcdc (Sam) > > * use devm_request_irq() in atmel-hlcdc (Sam) > > * unify variable names in arm/hlcdc (Sam) > > > > Thomas Zimmermann (14): > > The following patches are all: > Acked-by: Sam Ravnborg <s...@ravnborg.org> > > > drm/fsl-dcu: Convert to Linux IRQ interfaces > > drm/gma500: Convert to Linux IRQ interfaces > > drm/kmb: Convert to Linux IRQ interfaces > > drm/msm: Convert to Linux IRQ interfaces > > drm/mxsfb: Convert to Linux IRQ interfaces > > drm/tidss: Convert to Linux IRQ interfaces > > drm/vc4: Convert to Linux IRQ interfaces > > drm: Remove unused devm_drm_irq_install() > > The remaining patches I either skipped or already had a feedback from > me or I asked a question. > > Sam