From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0258DC433DB for ; Wed, 13 Jan 2021 20:10:06 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id EDA63214D8 for ; Wed, 13 Jan 2021 20:10:03 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EDA63214D8 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=gerhold.net Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id E5B97168E; Wed, 13 Jan 2021 21:09:09 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz E5B97168E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1610568600; bh=NiYec8oNBq//wqq6A7UGDY46hdo4d2NZpDHM2yH8VSo=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=mW9kWpqqbuIx8bs5+VLG+je3GpNjienwAbyyEohuKjNgszYK1kDs5BKLbpEeadeul DcvJaPJ7U6NDy6B4DQ1BHJ64EE8PQUBqv7E1g0hZbEAxKx+sdwodsu+H2fx+3P9b0w r+65FvwvNiuLUmKgPX3mftddh+g0eSVD8TzGR9ks= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id D2100F8016D; Wed, 13 Jan 2021 21:09:08 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id A5609F801ED; Wed, 13 Jan 2021 21:09:05 +0100 (CET) Received: from mo4-p01-ob.smtp.rzone.de (mo4-p01-ob.smtp.rzone.de [81.169.146.164]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id A6BAFF80113 for ; Wed, 13 Jan 2021 21:08:57 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz A6BAFF80113 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=gerhold.net header.i=@gerhold.net header.b="Cl+uBYNB" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1610568532; s=strato-dkim-0002; d=gerhold.net; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:From: Subject:Sender; bh=fOkJtKd3YiGSYVYtFQm27IMvVftEvpoE5Xx8IcVzKjs=; b=Cl+uBYNBNgN53mTBY/emqlpEkxiLmQ6YgyGuwgTyZ0Vn7GTbsJ0kD0zYKgAssRRfZu GPxSdp2lmk67lgsYcvUX6sEJi083IzykkNhJpouKuyKbA35IajCQXkLTJBVjQTJW+rWa pJL9mP+2o3afKSq2qwIhBCvPZr5xZuFZN46rpFm2hE7bJBPwpY/HeMmKW5sXiP0+//ts 5xJg9l6P8iP1uqRNxLkn16zg1x9HzQsHDVx+TCwEptFjzwypEraWEMC4pb52+sXMQWA1 JmX7R3OVvyDP3C7VDpID688HW9UfpuZ642WuuFhX9KjNNZmJX3UymmgXsXxWcxHj92Vm ut+A== X-RZG-AUTH: ":P3gBZUipdd93FF5ZZvYFPugejmSTVR2nRPhVOQ/OcYgojyw4j34+u26zEodhPgRDZ8j4IcrMBg==" X-RZG-CLASS-ID: mo00 Received: from gerhold.net by smtp.strato.de (RZmta 47.12.1 DYNA|AUTH) with ESMTPSA id R0a218x0DK8pazp (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Wed, 13 Jan 2021 21:08:51 +0100 (CET) Date: Wed, 13 Jan 2021 21:08:43 +0100 From: Stephan Gerhold To: Srinivas Kandagatla Subject: Re: [PATCH v11 0/7] Qualcomm's lpass-hdmi ASoC driver to support audio over dp port Message-ID: References: <1602134223-2562-1-git-send-email-srivasam@codeaurora.org> <244e74f6-fa8c-692a-aa84-4677a0fd815b@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <244e74f6-fa8c-692a-aa84-4677a0fd815b@linaro.org> Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, bgoswami@codeaurora.org, Srinivasa Rao Mandadapu , linux-arm-msm@vger.kernel.org, plai@codeaurora.org, tiwai@suse.com, agross@kernel.org, robh+dt@kernel.org, lgirdwood@gmail.com, broonie@kernel.org, rohitkr@codeaurora.org, bjorn.andersson@linaro.org, linux-kernel@vger.kernel.org X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" Hi Srinivas, On Thu, Oct 08, 2020 at 06:37:40AM +0100, Srinivas Kandagatla wrote: > On 08/10/2020 06:16, Srinivasa Rao Mandadapu wrote: > > These patches are to support audio over DP port on Qualcomm's SC7180 LPASS > > Asoc. It includes machine driver, cpu driver, platform driver updates for > > HDMI path support, device tree documention, lpass variant structure > > optimization and configuration changes. > > These patches depends on the DP patch series > > https://patchwork.kernel.org/project/dri-devel/list/?series=332029 > > https://lore.kernel.org/patchwork/project/lkml/list/?series=464856 > > > > changes since V10: > > -- Moved hdmi regmap functions from lpass-hdmi.c to lpass-cpu.c > > -- Moved QCOM_REGMAP_FIELD_ALLOC macro from lpass-hdmi.c to lpass.h > > changes since V9: > > -- Removed unused structures lpass_hdmi.h > > changes since V8: > > -- Removed redundant structure wrapper for reg map field memebrs > > -- Updated lpass_hdmi_regmap_volatile API with appropriate registers as true > > and others as false. > > changes since V7: > > -- Fixed typo errors > > -- Created Separate patch for buffer size change > > changes since V6: > > -- Removed compile time define flag, which used for enabling > > HDMI code, based on corresponding config param is included. > > -- Updated reg map alloc API with reg map bulk API. > > -- Removed unnecessary line splits > > changes since V5: > > -- Removed unused struct regmap *map in lpass_platform_alloc_hdmidmactl_fields. > > -- DMA alloc and free API signature change in lpass-apq8016.c, lpass-ipq806x.c > > -- Keeping API "irqreturn_t lpass_platform_hdmiif_irq" under ifdef macro > > Changes Since v4: > > -- Updated with single compatible node for both I2S and HDMI. > > Changes Since v3: > > -- Removed id in lpass variant structure and used snd_soc_dai_driver id. > > Changes Since v2: > > -- Audio buffer size(i.e. LPASS_PLATFORM_BUFFER_SIZE) in lpass-platform.c increased. > > Changes Since v1: > > -- Commit messages are updated > > -- Addressed Rob Herring review comments > > > > V Sujith Kumar Reddy (7): > > ASoC: Add sc7180-lpass binding header hdmi define > > ASoC: dt-bindings: Add dt binding for lpass hdmi > > Asoc:qcom:lpass-cpu:Update dts property read API > > Asoc: qcom: lpass:Update lpaif_dmactl members order > > ASoC: qcom: Add support for lpass hdmi driver > > Asoc: qcom: lpass-platform : Increase buffer size > > ASoC: qcom: sc7180: Add support for audio over DP > > > > .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 74 ++-- > > include/dt-bindings/sound/sc7180-lpass.h | 1 + > > sound/soc/qcom/Kconfig | 5 + > > sound/soc/qcom/Makefile | 2 + > > sound/soc/qcom/lpass-apq8016.c | 4 +- > > sound/soc/qcom/lpass-cpu.c | 249 ++++++++++++- > > sound/soc/qcom/lpass-hdmi.c | 258 ++++++++++++++ > > sound/soc/qcom/lpass-hdmi.h | 102 ++++++ > > sound/soc/qcom/lpass-ipq806x.c | 4 +- > > sound/soc/qcom/lpass-lpaif-reg.h | 49 ++- > > sound/soc/qcom/lpass-platform.c | 395 +++++++++++++++++---- > > sound/soc/qcom/lpass-sc7180.c | 116 +++++- > > sound/soc/qcom/lpass.h | 124 ++++++- > > 13 files changed, 1240 insertions(+), 143 deletions(-) > > create mode 100644 sound/soc/qcom/lpass-hdmi.c > > create mode 100644 sound/soc/qcom/lpass-hdmi.h > > > > Tested this series on DragonBoard 410c > > Tested-by: Srinivas Kandagatla > Reviewed-by: Srinivas Kandagatla I spent quite some time today trying to track down another regression for MSM8916/DragonBoard 410c audio in 5.10 and identified this patch series as the cause. So I'm very surprised that you successfully tested this on DB410c. Attempting to play HDMI audio results in: ADV7533: lpass_platform_pcmops_hw_params: invalid interface: 3 ADV7533: lpass_platform_pcmops_trigger: invalid 3 interface apq8016-lpass-cpu 7708000.audio-controller: ASoC: error at soc_component_trigger on 7708000.audio-controller: -22 Attempting to record analog audio results in: Unable to handle kernel NULL pointer dereference at virtual address 00000000000001e4 Internal error: Oops: 96000004 [#1] PREEMPT SMP CPU: 1 PID: 1568 Comm: arecord Not tainted 5.11.0-rc3 #20 Hardware name: Qualcomm Technologies, Inc. APQ 8016 SBC (DT) pc : regmap_write+0x14/0x80 lr : lpass_platform_pcmops_open+0xd8/0x210 [snd_soc_lpass_platform] Call trace: regmap_write+0x14/0x80 lpass_platform_pcmops_open+0xd8/0x210 [snd_soc_lpass_platform] snd_soc_component_open+0x2c/0x94 ... Looking at the changes in "ASoC: qcom: Add support for lpass hdmi driver" there is a quite obvious mistake there. lpass.h now contains #include and then the SC7810 DAI IDs are hardcoded all over lpass-cpu.c and lpass-platform.c. But apq8016 and ipq806x have completely different DAI IDs so now MI2S_QUATERNARY (HDMI) is invalid and MI2S_TERTIARY (= LPASS_DP_RX in sc7180-lpass.h) is treated as HDMI port. Effectively LPASS is broken on all platforms except SC7810. I have a patch prepared to fix this (will send it tomorrow), but I wonder how you have tested this successfully on DB410c. :) Stephan