Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936144AbdLRWTc (ORCPT ); Mon, 18 Dec 2017 17:19:32 -0500 Received: from mail-pl0-f66.google.com ([209.85.160.66]:37324 "EHLO mail-pl0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933147AbdLRWT3 (ORCPT ); Mon, 18 Dec 2017 17:19:29 -0500 X-Google-Smtp-Source: ACJfBot+LvXac/3VMLmaA0tXwcsiPIE4FPBtrg/TMmChYsSety+2ZYynVklaHlIdz0cDlfnf8GCRCpwo3Yh5cs96Ci0= MIME-Version: 1.0 In-Reply-To: <4e69ef31-3aa6-66ea-331b-8a7472a66a15@tabi.org> References: <1513565530-33957-1-git-send-email-nicoleotsuka@gmail.com> <4e69ef31-3aa6-66ea-331b-8a7472a66a15@tabi.org> From: Caleb Crome Date: Mon, 18 Dec 2017 14:19:08 -0800 Message-ID: Subject: Re: [PATCH v4 00/11] ASoC: fsl_ssi: Clean up - coding style level To: Timur Tabi Cc: Nicolin Chen , Mark Brown , linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, alsa-devel@alsa-project.org, Liam Girdwood , Fabio Estevam , mail@maciej.szmigiero.name, Arnaud Mouiche , lukma@denx.de, Sascha Hauer Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2502 Lines: 67 On Sun, Dec 17, 2017 at 7:13 PM, Timur Tabi wrote: > > On 12/17/17 8:51 PM, Nicolin Chen wrote: >> >> Nicolin Chen (11): >> ASoC: fsl_ssi: Rename fsl_ssi_private to fsl_ssi >> ASoC: fsl_ssi: Cache pdev->dev pointer >> ASoC: fsl_ssi: Refine all comments >> ASoC: fsl_ssi: Rename registers and fields macros >> ASoC: fsl_ssi: Refine indentations and wrappings >> ASoC: fsl_ssi: Refine printk outputs >> ASoC: fsl_ssi: Rename cpu_dai parameter to dai >> ASoC: fsl_ssi: Rename scr_val to scr >> ASoC: fsl_ssi: Replace fsl_ssi_rxtx_reg_val with fsl_ssi_regvals >> ASoC: fsl_ssi: Rename i2smode to i2s_net >> ASoC: fsl_ssi: Define ternary macros to simplify code > > > Acked-by: Timur Tabi I'm re-setting up my loopback test to try to verify these most recent changes. I'm starting with the for-next branch at git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git (which is a 4.15 numbered kernel). The basic setup is using a wandboard with a TX pin tied directly to an RX pin do test if what is sent is what is received, using Arnaud's tester here: https://github.com/amouiche/atest Unfortunately, it doesn't pass the loopback test, even before these patches. I'm not actively developing on this platform at the moment, and my brain has been out of kernel development for quite a while. I'm getting channel slips again. output from atest: warn: 11a0 11a1 1160 11a3 11a4 11a5 11a6 11a7 warn: Valid frame after 1 invalid frames warn: 11c0 11c1 11c2 11c3 11c4 11c5 11c6 11c7 warn: first invalid frame while expecting frame 0x00a0 warn: 13e7 1400 1401 1402 1403 1404 1405 1404 warn: 1407 1420 1421 1422 1423 1424 1425 1426 warn: 1427 1440 1441 1442 1443 1444 1445 1484 warn: 1447 1460 1461 1462 1463 1464 1465 1466 Those last 4 lines are the channel slips -- the least significant nibble should be the channel number: i.e. should go 0, 1, 2, 3, 4, 5, 6, 7. Ugh, so it's basically quite broken again -- before these patches. I guess I need to go backwards in time and see what rev re-broke it. I don't really have time to dig too deep on this again. I'd be happy to provide the hardware to anybody that can diagnose and debug this more quickly than I can. I'm very inefficient at kernel drivers I think. My day job is acoustical and electrical engineering. Here's what the hardware looks like for anybody that's interested. Just a single wire loopback on the wandboard header. https://ibb.co/d2rfzm -Caleb