From: Tony Lindgren Subject: Re: [PATCH] ARM: dts: Add missing ranges for dra7 mcasp l3 ports Date: Wed, 12 Dec 2018 07:28:29 -0800 Message-ID: <20181212152829.GF6707@atomide.com> References: <20181208005403.26330-1-tony@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Peter Ujfalusi Cc: devicetree@vger.kernel.org, linux-omap@vger.kernel.org, =?utf-8?Q?Beno=C3=AEt?= Cousson , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org * Peter Ujfalusi [181212 08:00]: > > > On 08/12/2018 2.54, Tony Lindgren wrote: > > We need to add mcasp l3 port ranges for mcasp to use a correct l3 > > data port address for dma. And we're also missing the optional clocks > > that we have tagged with HWMOD_OPT_CLKS_NEEDED in omap_hwmod_7xx_data.c. > > > > Note that for reading the module revision register HWMOD_OPT_CLKS_NEEDED > > do not seem to be needed. So they could be probably directly managed > > only by the mcasp driver, and then we could leave them out for the > > interconnect target module. > > Tested on dra7-evm and dra72-evm. Works nicely, thanks! > > Tested-by: Peter Ujfalusi OK good to hear and thanks for testing. I already pushed this one out yesterday as I was able to test it myself on beagle-x15. Regards, Tony