Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752914AbcDVWYW (ORCPT ); Fri, 22 Apr 2016 18:24:22 -0400 Received: from muru.com ([72.249.23.125]:51702 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751995AbcDVWYU (ORCPT ); Fri, 22 Apr 2016 18:24:20 -0400 Date: Fri, 22 Apr 2016 15:24:15 -0700 From: Tony Lindgren To: Peter Ujfalusi Cc: Paul Walmsley , jarkko.nikula@bitmer.com, t-kristo@ti.com, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v2 0/3] ARM: OMAP3: Fix McBSP2/3 hwmod setup for sidetone Message-ID: <20160422222415.GN5995@atomide.com> References: <20160413152829.GQ5995@atomide.com> <570F4804.4050006@ti.com> <20160414165514.GL5995@atomide.com> <570FF163.1050603@ti.com> <20160414203457.GM5995@atomide.com> <5710C10A.6040908@ti.com> <20160415151651.GP5995@atomide.com> <571145F6.2040508@ti.com> <20160418235152.GZ5995@atomide.com> <571A2357.3060006@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <571A2357.3060006@ti.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 955 Lines: 21 * Peter Ujfalusi [160422 06:15]: > > From the documents it is also clear that McBSPLP.sidetone is using the > McBSPLP's ICLK, but what is not explained in the TRM is that there are > internal clocks going from McBSP to sidetone for the data bus between them. > The iclk is needed so the core can kind of run independently from the clocks > coming from McBSPLP (for data exchange between the two modules). > If McBSP is not configured these clocks are not running which renders the > sidetone non operational. THe McBSP ick is not coming from McBSP, it's coming from the L4 interconnect. Both McBSP are just consumers for that same clock. AFAIK there is no clock line going from McBSP to the sidetone. > I can send a cut down series to fix the current sidetone hwmod (main_clk and > prevent it to look at the PRCM bit) plus reworking the pdata callback so we > can support both legacy and DT boot. OK sounds good to me :) Tony