Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751630AbbEZXA7 (ORCPT ); Tue, 26 May 2015 19:00:59 -0400 Received: from mail-pd0-f174.google.com ([209.85.192.174]:36551 "EHLO mail-pd0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751207AbbEZXA4 (ORCPT ); Tue, 26 May 2015 19:00:56 -0400 Date: Tue, 26 May 2015 16:00:49 -0700 From: Nicolin Chen To: Zidan Wang Cc: Fabio Estevam , Mark Brown , Fabio Estevam , "alsa-devel@alsa-project.org" , linux-kernel , Liam Girdwood , "linuxppc-dev@lists.ozlabs.org" Subject: Re: [alsa-devel] [PATCH] ASoC: fsl_spdif: Don't try to round-up for clock divisor calculation Message-ID: <20150526230048.GA2752@Asurada-GSX> References: <1432455161-4924-1-git-send-email-nicoleotsuka@gmail.com> <20150525115812.GG21391@sirena.org.uk> <20150525151127.GA8184@Asurada> <20150525152424.GA8164@Asurada> <20150526110246.GA13725@shlinux2> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150526110246.GA13725@shlinux2> User-Agent: Mutt/1.5.22 (2013-10-16) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1963 Lines: 44 On Tue, May 26, 2015 at 07:02:48PM +0800, Zidan Wang wrote: > On Mon, May 25, 2015 at 08:24:25AM -0700, Nicolin Chen wrote: > > On Mon, May 25, 2015 at 12:13:45PM -0300, Fabio Estevam wrote: > > > Hi Nicolin, > > > > > > On Mon, May 25, 2015 at 12:11 PM, Nicolin Chen wrote: > > > > > > > Hi Mark, > > > > > > > > Is that possible for you to provisionally revert this patch? > > > > I wanted to wait for the test result from Fabio or Zidan in > > > > the Cc list because I don't have a test environment for SPDIF > > > > even though this change seems to make sense. > > > > > > I currently don't have access to a SPDIF receiver to test it. > > > > Okay, let's wait for Zidan then. We only need to test the > > playback route of supporting sample rates. > I don't have the board which supported by community to test spdif out. So i used the imx7 board i.MX6 ARM2, the one with the square green base board, should work, although you may add some device nodes in dts to make it work. > and test it with internal branch. > I found that (txclk_df + 1) is better than txclk_df. > I suspect the patch for clk_round_rate() is not in our branch. Could you please tell me which > patch is for clk_round_rate? I want to cherry-pick it to our branch and test it. https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/log/drivers/clk/clk-divider.c Here is commit logs for the clock driver. You can check git-log of your local one and cherry-pick those incremental patches. Another clue is that after updating the clock driver to the latest version, SSI master mode in your local branch would become abnormal since it's still using pm + 2 to round the calculation. Thanks a lot Nicolin -- 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/