RE: [PATCH v19 2/4] dmaengine: tegra: Add tegra gpcdma driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 07.02.2022 18:01, Akhil R пишет:
> > Adding GPC DMA controller driver for Tegra. The driver supports dma
> > transfers between memory to memory, IO peripheral to memory and
> > memory to IO peripheral.
> >
> > Co-developed-by: Pavan Kunapuli <pkunapuli@xxxxxxxxxx>
> > Signed-off-by: Pavan Kunapuli <pkunapuli@xxxxxxxxxx>
> > Co-developed-by: Rajesh Gumasta <rgumasta@xxxxxxxxxx>
> > Signed-off-by: Rajesh Gumasta <rgumasta@xxxxxxxxxx>
> > Signed-off-by: Akhil R <akhilrajeev@xxxxxxxxxx>
> > Reviewed-by: Jon Hunter <jonathanh@xxxxxxxxxx>
> > ---
> >  drivers/dma/Kconfig            |   11 +
> >  drivers/dma/Makefile           |    1 +
> >  drivers/dma/tegra186-gpc-dma.c | 1505
> ++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 1517 insertions(+)
> >  create mode 100644 drivers/dma/tegra186-gpc-dma.c
> 
> Looks okay to me. Thank you for the effort!
> 
> Coding style isn't ideal:
> 
>  - inconsistent alignment of the code
>  - unnecessary whitespaces and parens
>  - inconsistent variables type signes
>  - all abbreviation should be capitalized in comments and commit msg
> 
> but this is not critical and could be improved later on.
> 
> Reviewed-by: Dmitry Osipenko <digetx@xxxxxxxxx>

Thanks for the review.

Regards,
Akhil




[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux