Re: [PATCH v3 00/13] ARM/DT: edma: IP configuration from hardware and cleanups

Sekhar Nori
2014-05-19T13:07:44Z

On Friday 16 May 2014 05:47 PM, Peter Ujfalusi wrote:
> Hi,
>
> Changes since v2:
> - Comments from Sekhar and Arnd has been addressed best as I could.
> - Use the CCCFG information in all cases instead of pdata provided information
> - To achieve this I needed to do a bit more cleanup in this series
> - In the documentation patch, retrain the old properties for reference
> - Cleanups in the old davinci board files and removing edma_soc_info members
>
> Changes sicne v1:
> - added missing patch to remove the memset from edma_of_parse_dt()
>
> We are requesting redundant information via DT for the driver since the very same
> data is available in the HW: by reading and decoding the content of CCCFG
> register we can get:
> Number of channels: NUM_DMACH
> Number of regions: NUM_REGN
> Number of slots (PaRAM sets): NUM_PAENTRY
> Number of TC/EQ: NUM_EVQUE
>
> So these does not need to be provided by the DT binding.
>
> The driver will no longer look for these properties from DT and they can be
> removed from the binding documentation and from the dtsi files as well.
> The change will not introduce regression when new kernel is booted using older
> DTB (since we just ignore the mentioned properties).

Applied all patches and pushed to branch v3.16/edma of:

git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git

Since the patches did not apply cleanly, please verify. I tested on
DA850 EVM using MMC/SD as EDMA user.

I will wait to get acks from DT maintainers before sending to ARM-SoC.

Thanks,
Sekhar