Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752100AbbKKICG (ORCPT ); Wed, 11 Nov 2015 03:02:06 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:59040 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751205AbbKKICE (ORCPT ); Wed, 11 Nov 2015 03:02:04 -0500 Subject: Re: [PATCH v3 0/3] ARM: OMAP2+ McASP(3) support for DRA7xx family To: Paul Walmsley References: <1446191700-11057-1-git-send-email-peter.ujfalusi@ti.com> CC: , , , , , From: Peter Ujfalusi Message-ID: <5642F5DC.8030902@ti.com> Date: Wed, 11 Nov 2015 10:01:32 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2974 Lines: 64 Tony, On 10/30/2015 06:11 PM, Paul Walmsley wrote: > Hi P?ter > > On Fri, 30 Oct 2015, Peter Ujfalusi wrote: > >> Changes since v2: >> - DTS patch added which is needed because of the clock handling changes >> >> Felip Balbi reported that linux-next is broken right now since the DTS part of >> the earlier series has been applied, but we do not have the mcasp hwmod in the >> kernel: >> ... >> [ 0.181029] platform 48468000.mcasp: Cannot lookup hwmod 'mcasp3' >> ... >> [ 6.121072] davinci-mcasp 48468000.mcasp: _od_fail_runtime_resume: FIXME: missing hwmod/omap_dev info >> [ 6.130790] ------------[ cut here ]------------ >> [ 6.135643] WARNING: CPU: 0 PID: 244 at drivers/bus/omap_l3_noc.c:147 l3_interrupt_handler+0x220/0x34c() >> [ 6.145576] 44000000.ocp:L3 Custom Error: MASTER MPU TARGET L4_PER2_P3 (Read): Data Access in User mode during Functional access >> ... >> >> This is the followup series for the hwmod changes needed to get audio working >> on DRA7xx family based boards. >> The DTS patches has been applied by Tony from the original series: >> http://www.spinics.net/lists/linux-omap/msg121473.html >> >> I have addressed your comments in the hwmod data and did some research also >> regarding to the use of ahclkx as fclk in the original submission. >> It turned out that McASP _needs_ all clocks to be enabled (fclk, iclk and >> ahclkx/r) to be able to access registers. The original patch where we handled >> the ahclkx as fclk worked, because the fclk clock got enabled in the HW w/o >> any SW interaction. >> All in all, the McASP found in DRA7 needs all clocks to be enabled. >> To satisfy this I have introduced a new flag to hwmod, which means that the >> listed optional clocks need to be handled alongside with the fclk clock. > > Thanks. I'm happy with your series and appreciate the indepth > investigation. As you probably saw last week, we've hit the limit for > v4.4-rc1: > > http://marc.info/?l=linux-omap&m=144564929721826&w=2 > > This is why I haven't done anything with this series at this time. > Unfortunately I don't have a DRA7xx board, so I can't do any testing. > But if this series fixes a problem with DRA7xx in linux-next, we should > definitely merge it. > > Tony, if you want to take this now, you can either take it with my ack, or > I can send a pull request. Or, if you'd prefer to take it for v4.4-rc2, > I can send a pull request after v4.4-rc1. The ARM soc changes for 4.4 are now pulled in to mainline, including the DRA7/72/X15 audio related DTS changes, but this three patch is not. This means that dra7-evm, dra72-evm and bealge-x15 will not boot from mainline. Is there a way to push these patches as soon as possible? Thanks, P?ter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/