* [PATCH 1/1] drm: xlnx: zynqmp_disp: use dmaengine_get_copy_align() accessor
@ 2026-11-11 20:07 Frank.Li
2026-11-11 23:16 ` Laurent Pinchart
0 siblings, 1 reply; 2+ messages in thread
From: Frank.Li @ 2026-11-11 20:07 UTC (permalink / raw)
To: Laurent Pinchart, Tomi Valkeinen, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
Michal Simek, open list:DRM DRIVERS FOR XILINX,
moderated list:ARM/ZYNQ ARCHITECTURE, open list
Cc: imx, Frank Li
From: Frank Li <Frank.Li@nxp.com>
Use the dmaengine_get_copy_align() helper instead of directly accessing
the copy_align field from the DMA device structure.
No functional change and prepare to move struct dma_device to private.
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
drivers/gpu/drm/xlnx/zynqmp_disp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/xlnx/zynqmp_disp.c b/drivers/gpu/drm/xlnx/zynqmp_disp.c
index e998f22686748..0753236b9546a 100644
--- a/drivers/gpu/drm/xlnx/zynqmp_disp.c
+++ b/drivers/gpu/drm/xlnx/zynqmp_disp.c
@@ -1433,7 +1433,7 @@ int zynqmp_disp_probe(struct zynqmp_dpsub *dpsub)
struct zynqmp_disp_layer *layer;
layer = &disp->layers[ZYNQMP_DPSUB_LAYER_VID];
- dpsub->dma_align = 1 << layer->dmas[0].chan->device->copy_align;
+ dpsub->dma_align = 1 << dmaengine_get_copy_align(layer->dmas[0].chan);
}
dpsub->disp = disp;
--
2.43.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/1] drm: xlnx: zynqmp_disp: use dmaengine_get_copy_align() accessor
2026-11-11 20:07 [PATCH 1/1] drm: xlnx: zynqmp_disp: use dmaengine_get_copy_align() accessor Frank.Li
@ 2026-11-11 23:16 ` Laurent Pinchart
0 siblings, 0 replies; 2+ messages in thread
From: Laurent Pinchart @ 2026-11-11 23:16 UTC (permalink / raw)
To: Frank.Li
Cc: Tomi Valkeinen, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Simona Vetter, Michal Simek,
open list:DRM DRIVERS FOR XILINX,
moderated list:ARM/ZYNQ ARCHITECTURE, open list, imx, Frank Li
On Wed, Nov 11, 2026 at 03:07:37PM -0500, Frank.Li@oss.nxp.com wrote:
> From: Frank Li <Frank.Li@nxp.com>
>
> Use the dmaengine_get_copy_align() helper instead of directly accessing
> the copy_align field from the DMA device structure.
>
> No functional change and prepare to move struct dma_device to private.
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Tomi, I assume you'll push this to drm-misc. Please let me know if
that's not correct.
> ---
> drivers/gpu/drm/xlnx/zynqmp_disp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xlnx/zynqmp_disp.c b/drivers/gpu/drm/xlnx/zynqmp_disp.c
> index e998f22686748..0753236b9546a 100644
> --- a/drivers/gpu/drm/xlnx/zynqmp_disp.c
> +++ b/drivers/gpu/drm/xlnx/zynqmp_disp.c
> @@ -1433,7 +1433,7 @@ int zynqmp_disp_probe(struct zynqmp_dpsub *dpsub)
> struct zynqmp_disp_layer *layer;
>
> layer = &disp->layers[ZYNQMP_DPSUB_LAYER_VID];
> - dpsub->dma_align = 1 << layer->dmas[0].chan->device->copy_align;
> + dpsub->dma_align = 1 << dmaengine_get_copy_align(layer->dmas[0].chan);
> }
>
> dpsub->disp = disp;
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-11-11 23:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-11 20:07 [PATCH 1/1] drm: xlnx: zynqmp_disp: use dmaengine_get_copy_align() accessor Frank.Li
2026-11-11 23:16 ` Laurent Pinchart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox