From: Thomas Zimmermann <tzimmermann@suse.de> To: Sam Ravnborg <sam@ravnborg.org> Cc: alexandre.belloni@bootlin.com, linux-aspeed@lists.ozlabs.org, narmstrong@baylibre.com, airlied@linux.ie, linus.walleij@linaro.org, liviu.dudau@arm.com, stefan@agner.ch, philippe.cornu@st.com, paul@crapouillou.net, laurent.pinchart@ideasonboard.com, benjamin.gaignard@linaro.org, mihail.atanassov@arm.com, festevam@gmail.com, alexandre.torgue@st.com, marex@denx.de, khilman@baylibre.com, abrodkin@synopsys.com, ludovic.desroches@microchip.com, xinliang.liu@linaro.org, kong.kongxinwei@hisilicon.com, tomi.valkeinen@ti.com, james.qian.wang@arm.com, joel@jms.id.au, linux-imx@nxp.com, p.zabel@pengutronix.de, puck.chen@hisilicon.com, s.hauer@pengutronix.de, alison.wang@nxp.com, maarten.lankhorst@linux.intel.com, mripard@kernel.org, john.stultz@linaro.org, jsarha@ti.com, wens@csie.org, vincent.abriou@st.com, kernel@pengutronix.de, linux-arm-kernel@lists.infradead.org, mcoquelin.stm32@gmail.com, noralf@tronnes.org, bbrezillon@kernel.org, andrew@aj.id.au, dri-devel@lists.freedesktop.org, yannick.fertre@st.com, kieran.bingham+renesas@ideasonboard.com, daniel@ffwll.ch, zourongrong@gmail.com, shawnguo@kernel.org, brian.starkey@arm.com Subject: Re: [PATCH 05/21] drm/atmel-hlcdc: Use GEM CMA object functions Date: Mon, 25 May 2020 14:37:33 +0200 [thread overview] Message-ID: <2c220ab0-e96b-2f5a-0d7f-8e770a13ef56@suse.de> (raw) In-Reply-To: <20200522192546.GA1516695@ravnborg.org> [-- Attachment #1.1.1: Type: text/plain, Size: 2513 bytes --] Hi Am 22.05.20 um 21:25 schrieb Sam Ravnborg: > Hi Thomas. > > On Fri, May 22, 2020 at 03:52:30PM +0200, Thomas Zimmermann wrote: >> The atmel-hlcdc driver uses the default implementation for CMA functions. The >> DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. >> All remaining operations are provided by CMA GEM object functions. >> >> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de> >> --- >> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 11 +---------- >> 1 file changed, 1 insertion(+), 10 deletions(-) >> >> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c >> index 112aa5066ceed..871293d1aeeba 100644 >> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c >> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c >> @@ -821,16 +821,7 @@ static struct drm_driver atmel_hlcdc_dc_driver = { >> .irq_preinstall = atmel_hlcdc_dc_irq_uninstall, >> .irq_postinstall = atmel_hlcdc_dc_irq_postinstall, >> .irq_uninstall = atmel_hlcdc_dc_irq_uninstall, >> - .gem_free_object_unlocked = drm_gem_cma_free_object, >> - .gem_vm_ops = &drm_gem_cma_vm_ops, >> - .prime_handle_to_fd = drm_gem_prime_handle_to_fd, >> - .prime_fd_to_handle = drm_gem_prime_fd_to_handle, >> - .gem_prime_get_sg_table = drm_gem_cma_prime_get_sg_table, >> - .gem_prime_import_sg_table = drm_gem_cma_prime_import_sg_table, >> - .gem_prime_vmap = drm_gem_cma_prime_vmap, >> - .gem_prime_vunmap = drm_gem_cma_prime_vunmap, > >> - .gem_prime_mmap = drm_gem_cma_prime_mmap, > When using DRM_GEM_CMA_DRIVER_OPS gem_prime_mmap is set to > drm_gem_prime_mmap. > Why is this the same as drm_gem_cma_prime_mmap? > > Maybe this is all obvious when you know all the CMA stuff, > but this puzzeled me. Following through the calls is far from easy. I took the macro from the aspeed driver. I had some doubts about the mmap code, but expected the driver to be working correctly. Maybe we should set that field to drm_gem_cma_prime_mmap or implement the mmap object function. Best regards Thomas > > Sam > > >> - .dumb_create = drm_gem_cma_dumb_create, >> + DRM_GEM_CMA_DRIVER_OPS, >> .fops = &fops, >> .name = "atmel-hlcdc", >> .desc = "Atmel HLCD Controller DRM", >> -- >> 2.26.2 -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Felix Imendörffer [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Thomas Zimmermann <tzimmermann@suse.de> To: Sam Ravnborg <sam@ravnborg.org> Cc: alexandre.belloni@bootlin.com, linux-aspeed@lists.ozlabs.org, narmstrong@baylibre.com, airlied@linux.ie, liviu.dudau@arm.com, philippe.cornu@st.com, paul@crapouillou.net, laurent.pinchart@ideasonboard.com, mihail.atanassov@arm.com, alexandre.torgue@st.com, marex@denx.de, khilman@baylibre.com, abrodkin@synopsys.com, ludovic.desroches@microchip.com, xinliang.liu@linaro.org, kong.kongxinwei@hisilicon.com, tomi.valkeinen@ti.com, james.qian.wang@arm.com, joel@jms.id.au, linux-imx@nxp.com, puck.chen@hisilicon.com, s.hauer@pengutronix.de, alison.wang@nxp.com, jsarha@ti.com, wens@csie.org, vincent.abriou@st.com, kernel@pengutronix.de, linux-arm-kernel@lists.infradead.org, mcoquelin.stm32@gmail.com, bbrezillon@kernel.org, andrew@aj.id.au, dri-devel@lists.freedesktop.org, nicolas.ferre@microchip.com, yannick.fertre@st.com, kieran.bingham+renesas@ideasonboard.com, zourongrong@gmail.com, shawnguo@kernel.org Subject: Re: [PATCH 05/21] drm/atmel-hlcdc: Use GEM CMA object functions Date: Mon, 25 May 2020 14:37:33 +0200 [thread overview] Message-ID: <2c220ab0-e96b-2f5a-0d7f-8e770a13ef56@suse.de> (raw) In-Reply-To: <20200522192546.GA1516695@ravnborg.org> [-- Attachment #1.1.1: Type: text/plain, Size: 2513 bytes --] Hi Am 22.05.20 um 21:25 schrieb Sam Ravnborg: > Hi Thomas. > > On Fri, May 22, 2020 at 03:52:30PM +0200, Thomas Zimmermann wrote: >> The atmel-hlcdc driver uses the default implementation for CMA functions. The >> DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. >> All remaining operations are provided by CMA GEM object functions. >> >> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de> >> --- >> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 11 +---------- >> 1 file changed, 1 insertion(+), 10 deletions(-) >> >> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c >> index 112aa5066ceed..871293d1aeeba 100644 >> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c >> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c >> @@ -821,16 +821,7 @@ static struct drm_driver atmel_hlcdc_dc_driver = { >> .irq_preinstall = atmel_hlcdc_dc_irq_uninstall, >> .irq_postinstall = atmel_hlcdc_dc_irq_postinstall, >> .irq_uninstall = atmel_hlcdc_dc_irq_uninstall, >> - .gem_free_object_unlocked = drm_gem_cma_free_object, >> - .gem_vm_ops = &drm_gem_cma_vm_ops, >> - .prime_handle_to_fd = drm_gem_prime_handle_to_fd, >> - .prime_fd_to_handle = drm_gem_prime_fd_to_handle, >> - .gem_prime_get_sg_table = drm_gem_cma_prime_get_sg_table, >> - .gem_prime_import_sg_table = drm_gem_cma_prime_import_sg_table, >> - .gem_prime_vmap = drm_gem_cma_prime_vmap, >> - .gem_prime_vunmap = drm_gem_cma_prime_vunmap, > >> - .gem_prime_mmap = drm_gem_cma_prime_mmap, > When using DRM_GEM_CMA_DRIVER_OPS gem_prime_mmap is set to > drm_gem_prime_mmap. > Why is this the same as drm_gem_cma_prime_mmap? > > Maybe this is all obvious when you know all the CMA stuff, > but this puzzeled me. Following through the calls is far from easy. I took the macro from the aspeed driver. I had some doubts about the mmap code, but expected the driver to be working correctly. Maybe we should set that field to drm_gem_cma_prime_mmap or implement the mmap object function. Best regards Thomas > > Sam > > >> - .dumb_create = drm_gem_cma_dumb_create, >> + DRM_GEM_CMA_DRIVER_OPS, >> .fops = &fops, >> .name = "atmel-hlcdc", >> .desc = "Atmel HLCD Controller DRM", >> -- >> 2.26.2 -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Felix Imendörffer [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-05-25 12:37 UTC|newest] Thread overview: 92+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-05-22 13:52 [PATCH 00/21] drm: Convert most CMA-based drivers to GEM object functions Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 01/21] drm/cma-helper: Rework DRM_GEM_CMA_VMAP_DRIVER_OPS macro Thomas Zimmermann 2020-05-22 17:48 ` Sam Ravnborg 2020-05-22 17:48 ` Sam Ravnborg 2020-05-22 18:15 ` Emil Velikov 2020-05-22 18:15 ` Emil Velikov 2020-05-22 18:45 ` Sam Ravnborg 2020-05-22 18:45 ` Sam Ravnborg 2020-05-22 19:48 ` Laurent Pinchart 2020-05-22 19:48 ` Laurent Pinchart 2020-05-25 12:03 ` Thomas Zimmermann 2020-05-25 12:03 ` Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 02/21] drm/arc: Use GEM CMA object functions Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 03/21] drm/arm: " Thomas Zimmermann 2020-05-26 23:30 ` Liviu Dudau 2020-05-26 23:30 ` Liviu Dudau 2020-05-22 13:52 ` [PATCH 04/21] drm/aspeed: Set driver CMA functions with DRM_GEM_CMA_DRIVER_OPS Thomas Zimmermann 2020-10-09 7:54 ` Joel Stanley 2020-10-09 7:54 ` Joel Stanley 2020-10-09 8:01 ` Thomas Zimmermann 2020-10-09 8:01 ` Thomas Zimmermann 2020-10-09 8:06 ` Joel Stanley 2020-10-09 8:06 ` Joel Stanley 2020-10-09 8:26 ` Thomas Zimmermann 2020-10-09 8:26 ` Thomas Zimmermann 2020-10-09 10:45 ` Joel Stanley 2020-10-09 10:45 ` Joel Stanley 2020-05-22 13:52 ` [PATCH 05/21] drm/atmel-hlcdc: Use GEM CMA object functions Thomas Zimmermann 2020-05-22 18:08 ` Sam Ravnborg 2020-05-22 18:08 ` Sam Ravnborg 2020-05-25 12:10 ` Thomas Zimmermann 2020-05-25 12:10 ` Thomas Zimmermann 2020-05-22 19:25 ` Sam Ravnborg 2020-05-22 19:25 ` Sam Ravnborg 2020-05-25 12:37 ` Thomas Zimmermann [this message] 2020-05-25 12:37 ` Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 06/21] drm/fsl-dcu: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 07/21] drm/hisilicon/kirin: " Thomas Zimmermann 2020-05-22 18:11 ` Emil Velikov 2020-05-25 12:41 ` Thomas Zimmermann 2020-05-25 12:41 ` Thomas Zimmermann 2020-05-28 14:34 ` Emil Velikov 2020-05-28 14:34 ` Emil Velikov 2020-05-22 13:52 ` [PATCH 08/21] drm/imx: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 09/21] drm/ingenic: " Thomas Zimmermann 2020-05-27 0:28 ` Paul Cercueil 2020-05-27 0:28 ` Paul Cercueil 2020-05-22 13:52 ` [PATCH 10/21] drm/komeda: " Thomas Zimmermann 2020-05-26 23:33 ` Liviu Dudau 2020-05-26 23:33 ` Liviu Dudau 2020-05-22 13:52 ` [PATCH 11/21] drm/malidp: " Thomas Zimmermann 2020-05-26 23:33 ` Liviu Dudau 2020-05-26 23:33 ` Liviu Dudau 2020-05-22 13:52 ` [PATCH 12/21] drm/mcde: " Thomas Zimmermann 2020-05-25 11:36 ` Linus Walleij 2020-05-25 11:36 ` Linus Walleij 2020-05-25 12:51 ` Thomas Zimmermann 2020-05-25 12:51 ` Thomas Zimmermann 2020-05-25 13:08 ` Linus Walleij 2020-05-25 13:08 ` Linus Walleij 2020-05-25 13:27 ` Thomas Zimmermann 2020-05-25 13:27 ` Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 13/21] drm/meson: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 14/21] drm/mxsfb: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 15/21] drm/rcar-du: " Thomas Zimmermann 2020-05-22 20:12 ` Laurent Pinchart 2020-05-22 20:12 ` Laurent Pinchart 2020-05-25 12:49 ` Thomas Zimmermann 2020-05-25 12:49 ` Thomas Zimmermann 2020-05-25 15:38 ` Kieran Bingham 2020-05-25 15:38 ` Kieran Bingham 2020-06-03 7:48 ` Thomas Zimmermann 2020-06-03 7:48 ` Thomas Zimmermann 2020-05-26 1:50 ` Laurent Pinchart 2020-05-26 1:50 ` Laurent Pinchart 2020-05-22 13:52 ` [PATCH 16/21] drm/shmobile: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 17/21] drm/stm: " Thomas Zimmermann 2020-05-26 13:22 ` Philippe CORNU 2020-05-26 13:22 ` Philippe CORNU 2020-05-22 13:52 ` [PATCH 18/21] drm/sti: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 19/21] drm/tilcdc: " Thomas Zimmermann 2020-05-22 13:52 ` [PATCH 20/21] drm/tv200: " Thomas Zimmermann 2020-05-25 11:36 ` Linus Walleij 2020-05-25 11:36 ` Linus Walleij 2020-05-22 13:52 ` [PATCH 21/21] drm/zte: " Thomas Zimmermann 2020-05-22 19:03 ` Sam Ravnborg 2020-05-22 19:03 ` Sam Ravnborg 2020-05-24 14:50 ` kbuild test robot 2020-05-24 14:50 ` kbuild test robot 2020-05-24 20:11 ` kbuild test robot 2020-05-24 20:11 ` kbuild test robot 2020-05-22 18:19 ` [PATCH 00/21] drm: Convert most CMA-based drivers to GEM " Emil Velikov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=2c220ab0-e96b-2f5a-0d7f-8e770a13ef56@suse.de \ --to=tzimmermann@suse.de \ --cc=abrodkin@synopsys.com \ --cc=airlied@linux.ie \ --cc=alexandre.belloni@bootlin.com \ --cc=alexandre.torgue@st.com \ --cc=alison.wang@nxp.com \ --cc=andrew@aj.id.au \ --cc=bbrezillon@kernel.org \ --cc=benjamin.gaignard@linaro.org \ --cc=brian.starkey@arm.com \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=festevam@gmail.com \ --cc=james.qian.wang@arm.com \ --cc=joel@jms.id.au \ --cc=john.stultz@linaro.org \ --cc=jsarha@ti.com \ --cc=kernel@pengutronix.de \ --cc=khilman@baylibre.com \ --cc=kieran.bingham+renesas@ideasonboard.com \ --cc=kong.kongxinwei@hisilicon.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-aspeed@lists.ozlabs.org \ --cc=linux-imx@nxp.com \ --cc=liviu.dudau@arm.com \ --cc=ludovic.desroches@microchip.com \ --cc=maarten.lankhorst@linux.intel.com \ --cc=marex@denx.de \ --cc=mcoquelin.stm32@gmail.com \ --cc=mihail.atanassov@arm.com \ --cc=mripard@kernel.org \ --cc=narmstrong@baylibre.com \ --cc=noralf@tronnes.org \ --cc=p.zabel@pengutronix.de \ --cc=paul@crapouillou.net \ --cc=philippe.cornu@st.com \ --cc=puck.chen@hisilicon.com \ --cc=s.hauer@pengutronix.de \ --cc=sam@ravnborg.org \ --cc=shawnguo@kernel.org \ --cc=stefan@agner.ch \ --cc=tomi.valkeinen@ti.com \ --cc=vincent.abriou@st.com \ --cc=wens@csie.org \ --cc=xinliang.liu@linaro.org \ --cc=yannick.fertre@st.com \ --cc=zourongrong@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.