Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp146768pxa; Tue, 18 Aug 2020 19:08:33 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy1LCA48AsnPpmEvgiOfPqzMaYhdXtmKk3j/eFaBCQryK4NqYh4z/odRCs1tKzDt0g4Cj5h X-Received: by 2002:a05:6402:1457:: with SMTP id d23mr22570675edx.149.1597802913619; Tue, 18 Aug 2020 19:08:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597802913; cv=none; d=google.com; s=arc-20160816; b=p3tSDP8wo1tWGkRDfZP30Zb36CGxYDkSBQUPEDNyps9BL5N1CNezdRan+jleSbsnlQ xcZQDJv8vijrnIN9ltEcqgFt/+d2zRlGeqtetCraczMBd3ZKXD/QGbwehlHJjbZOJ5co J8DDFJFN12rn6N62clkgiHRP0atQYNYjO+pKlG2b5bkCBpRaW12b+Oz4iY58pyJyXSul 4Zo0/aSNmFbKiSkpMI6B32NtBgzl38MmXVqZmedoL/HlxTtpKO2/87BnJ0s3TN0TG/xa voWukGqOM7XJDoUtqapJHOYaSMBGRUBGJ4kAzwEHXsXBpA51mi3xEtJuUwAxEj3OUWY+ tb1g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:ironport-sdr:ironport-sdr; bh=xq+rVfexTw0BOrueTg7vvKgOq2Np4y0Db5CGqbeWcQs=; b=fS6zph++NQqtav83COwlU0BEt/DJ+8As3ynCQLYXGipH2DK72jeUlEaCC14zanp07J z0CPclvoyTEK0PXOqyy0oQYuk5q1osflO7I1lZ5VO+vVgWSjc1qcS2O/zPyAD9myReqQ 10MVJs6hF/75qbnzowWmMtzru2Po5qkmFnUzXS9bFq40PcCXRFq2mUN0WAG0jkjpQrHu T5XLhdQnrPX7mwkDOtZsn7aAw7fTUrIsZxcSBqYXOysOmhwpQY7Cwn9XvcKXvvDOBdqJ kYWmznRaRGvmx0xb7hrbtgdxCrVN1HGvk4MIIetDvHNUJR3F5/JwLzRkto4sroP/3UOA qMKQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id gq9si14496618ejb.442.2020.08.18.19.08.10; Tue, 18 Aug 2020 19:08:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727952AbgHSCBL (ORCPT + 99 others); Tue, 18 Aug 2020 22:01:11 -0400 Received: from mga04.intel.com ([192.55.52.120]:3696 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727926AbgHSCBE (ORCPT ); Tue, 18 Aug 2020 22:01:04 -0400 IronPort-SDR: sbr9IDXaoXfWa/quWwNihgEfSbIpvBKnXRDpkLNkGojmvZlxO0DNq/3fUqEVynA+oqd0cxZyMj o5iIUT0a6EDQ== X-IronPort-AV: E=McAfee;i="6000,8403,9717"; a="152449469" X-IronPort-AV: E=Sophos;i="5.76,329,1592895600"; d="scan'208";a="152449469" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Aug 2020 19:01:03 -0700 IronPort-SDR: jm7ZcW/dZk/waEJZvpQVmAefCr+re9Cl8AOVJzwLIKkUmWPu/P76g1d0J7z39WZ+RX/b8V8ant B6hE+uJ1dxWA== X-IronPort-AV: E=Sophos;i="5.76,329,1592895600"; d="scan'208";a="279565122" Received: from bard-ubuntu.sh.intel.com ([10.239.13.33]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Aug 2020 19:00:59 -0700 From: Bard Liao To: alsa-devel@alsa-project.org, vkoul@kernel.org Cc: vinod.koul@linaro.org, linux-kernel@vger.kernel.org, tiwai@suse.de, broonie@kernel.org, gregkh@linuxfoundation.org, jank@cadence.com, srinivas.kandagatla@linaro.org, rander.wang@linux.intel.com, ranjani.sridharan@linux.intel.com, hui.wang@canonical.com, pierre-louis.bossart@linux.intel.com, sanyog.r.kale@intel.com, mengdong.lin@intel.com, bard.liao@intel.com Subject: [PATCH 5/7] ASoC: codecs: realtek-soundwire: ignore initial PARITY errors Date: Tue, 18 Aug 2020 22:06:54 +0800 Message-Id: <20200818140656.29014-6-yung-chuan.liao@linux.intel.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20200818140656.29014-1-yung-chuan.liao@linux.intel.com> References: <20200818140656.29014-1-yung-chuan.liao@linux.intel.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Pierre-Louis Bossart The parity calculation is not reset on a Severe Reset, which leads to misleading/harmless errors reported on startup. The addition of a quirk helps filter out such errors while leaving the error checks on in steady-state. Signed-off-by: Pierre-Louis Bossart Reviewed-by: Kai Vehmanen Reviewed-by: Guennadi Liakhovetski Signed-off-by: Bard Liao --- sound/soc/codecs/rt1308-sdw.c | 1 + sound/soc/codecs/rt5682-sdw.c | 1 + sound/soc/codecs/rt700-sdw.c | 1 + sound/soc/codecs/rt711-sdw.c | 1 + sound/soc/codecs/rt715-sdw.c | 1 + 5 files changed, 5 insertions(+) diff --git a/sound/soc/codecs/rt1308-sdw.c b/sound/soc/codecs/rt1308-sdw.c index 5cf10fd447eb..81e3e2caeb2f 100644 --- a/sound/soc/codecs/rt1308-sdw.c +++ b/sound/soc/codecs/rt1308-sdw.c @@ -124,6 +124,7 @@ static int rt1308_read_prop(struct sdw_slave *slave) struct sdw_dpn_prop *dpn; prop->scp_int1_mask = SDW_SCP_INT1_BUS_CLASH | SDW_SCP_INT1_PARITY; + prop->quirks = SDW_SLAVE_QUIRKS_INVALID_INITIAL_PARITY; prop->paging_support = true; diff --git a/sound/soc/codecs/rt5682-sdw.c b/sound/soc/codecs/rt5682-sdw.c index 544073975020..148d4cff267b 100644 --- a/sound/soc/codecs/rt5682-sdw.c +++ b/sound/soc/codecs/rt5682-sdw.c @@ -545,6 +545,7 @@ static int rt5682_read_prop(struct sdw_slave *slave) prop->scp_int1_mask = SDW_SCP_INT1_IMPL_DEF | SDW_SCP_INT1_BUS_CLASH | SDW_SCP_INT1_PARITY; + prop->quirks = SDW_SLAVE_QUIRKS_INVALID_INITIAL_PARITY; prop->paging_support = false; diff --git a/sound/soc/codecs/rt700-sdw.c b/sound/soc/codecs/rt700-sdw.c index a46b957a3f1b..2d475405b20d 100644 --- a/sound/soc/codecs/rt700-sdw.c +++ b/sound/soc/codecs/rt700-sdw.c @@ -341,6 +341,7 @@ static int rt700_read_prop(struct sdw_slave *slave) prop->scp_int1_mask = SDW_SCP_INT1_IMPL_DEF | SDW_SCP_INT1_BUS_CLASH | SDW_SCP_INT1_PARITY; + prop->quirks = SDW_SLAVE_QUIRKS_INVALID_INITIAL_PARITY; prop->paging_support = false; diff --git a/sound/soc/codecs/rt711-sdw.c b/sound/soc/codecs/rt711-sdw.c index a877e366fec5..7a1ae7442e75 100644 --- a/sound/soc/codecs/rt711-sdw.c +++ b/sound/soc/codecs/rt711-sdw.c @@ -345,6 +345,7 @@ static int rt711_read_prop(struct sdw_slave *slave) prop->scp_int1_mask = SDW_SCP_INT1_IMPL_DEF | SDW_SCP_INT1_BUS_CLASH | SDW_SCP_INT1_PARITY; + prop->quirks = SDW_SLAVE_QUIRKS_INVALID_INITIAL_PARITY; prop->paging_support = false; diff --git a/sound/soc/codecs/rt715-sdw.c b/sound/soc/codecs/rt715-sdw.c index 0eb8943ed6ff..761d4663e813 100644 --- a/sound/soc/codecs/rt715-sdw.c +++ b/sound/soc/codecs/rt715-sdw.c @@ -439,6 +439,7 @@ static int rt715_read_prop(struct sdw_slave *slave) prop->scp_int1_mask = SDW_SCP_INT1_IMPL_DEF | SDW_SCP_INT1_BUS_CLASH | SDW_SCP_INT1_PARITY; + prop->quirks = SDW_SLAVE_QUIRKS_INVALID_INITIAL_PARITY; prop->paging_support = false; -- 2.17.1