From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lucas Stach Subject: Re: [PATCH 2/2] clk: tegra: add ac97 controller clock Date: Thu, 21 Mar 2013 15:01:03 +0100 Message-ID: <1363874463.1846.2.camel@antimon> References: <1359325055-5160-1-git-send-email-dev@lynxeye.de> <1359325055-5160-2-git-send-email-dev@lynxeye.de> <5106CE6D.9030008@wwwdotorg.org> <1359401144.1558.11.camel@tellur> <5106EB03.6080606@wwwdotorg.org> <20130130093525.GC2364@tbergstrom-lnx.Nvidia.com> <51096618.3000100@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51096618.3000100-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: Peter De Schrijver , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Prashant Gaikwad List-Id: linux-tegra@vger.kernel.org Am Mittwoch, den 30.01.2013, 11:27 -0700 schrieb Stephen Warren: > On 01/30/2013 02:35 AM, Peter De Schrijver wrote: > >> > >>> and also ac97 clock == pll_a_out0, so you can > >>> just as well use the pll out in that case. > >> > >> I suspect that "ac97 clock == pll_a_out0" isn't categorically true; it's > >> quite possible that the TRM simply doesn't document the mux > >> register/field for the ac97 clock since the ac97 module is considered > >> deprecated. The diagram at the end of section 5.2.3 "Audio Clocks" > >> certainly implies that all of the i2s*, spdifout, and ac97 clocks have > >> the same structure. > > > > It indeed does, but I can't find any reference to a mux control register for > > ac97 in any document... > > For both these two issues, I'll try and track down the relevant HW > engineers or other sources of documentation internally... > Ping. I would really like to see those two patches in 3.9, as otherwise Colibri T20 boots up with kernel warnings and a non-functional audio. Regards, Lucas