Hi Robin,
Thank you for the patch.
On Thu, Sep 03, 2020 at 10:12:27PM +0100, Robin Murphy wrote:
> Since commit 9495b7e92f71 ("driver core: platform: Initialize dma_parms
> for platform devices"), struct platform_device already provides a
> dma_parms structure, so we can save allocating another one.
>
> Also the DMA segment size is simply a size, not a bitmask.
>
> Signed-off-by: Robin Murphy <robin.murphy@xxxxxxx>
Looks good to me, I like the simplification.
Should I get this upstream, or do you have other similar patches
tree-wide that you plan to get merged in one go ?
> ---
> drivers/media/platform/rcar-fcp.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/media/platform/rcar-fcp.c b/drivers/media/platform/rcar-fcp.c
> index 5c6b00737fe7..13a84c7e3586 100644
> --- a/drivers/media/platform/rcar-fcp.c
> +++ b/drivers/media/platform/rcar-fcp.c
> @@ -22,7 +22,6 @@
> struct rcar_fcp_device {
> struct list_head list;
> struct device *dev;
> - struct device_dma_parameters dma_parms;
> };
>
> static LIST_HEAD(fcp_devices);
> @@ -138,8 +137,7 @@ static int rcar_fcp_probe(struct platform_device *pdev)
>
> fcp->dev = &pdev->dev;
>
> - fcp->dev->dma_parms = &fcp->dma_parms;
> - dma_set_max_seg_size(fcp->dev, DMA_BIT_MASK(32));
> + dma_set_max_seg_size(fcp->dev, UINT_MAX);
>
> pm_runtime_enable(&pdev->dev);
>
--
Regards,
Laurent Pinchart
