From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932150AbaESNHo (ORCPT ); Mon, 19 May 2014 09:07:44 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:44880 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754303AbaESNHl (ORCPT ); Mon, 19 May 2014 09:07:41 -0400 Message-ID: <537A01F2.2090107@ti.com> Date: Mon, 19 May 2014 18:36:58 +0530 From: Sekhar Nori User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Peter Ujfalusi , , CC: , , , , , , , , Subject: Re: [PATCH v3 00/13] ARM/DT: edma: IP configuration from hardware and cleanups References: <1400242640-9902-1-git-send-email-peter.ujfalusi@ti.com> In-Reply-To: <1400242640-9902-1-git-send-email-peter.ujfalusi@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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