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=-5.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 0174CC433E6 for ; Mon, 11 Jan 2021 20:30:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id AA1E722CF6 for ; Mon, 11 Jan 2021 20:30:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388047AbhAKUaZ (ORCPT ); Mon, 11 Jan 2021 15:30:25 -0500 Received: from mga12.intel.com ([192.55.52.136]:38642 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728701AbhAKUaW (ORCPT ); Mon, 11 Jan 2021 15:30:22 -0500 IronPort-SDR: bkZ0VmR8qNu7HP2Xk+P0Z1rXzJ3zCU6EXl1YswDvgEfRPnrnOu1jzm11Xy06Gr9qBCDOf5B4JD 3bfdr5h///GA== X-IronPort-AV: E=McAfee;i="6000,8403,9861"; a="157110616" X-IronPort-AV: E=Sophos;i="5.79,339,1602572400"; d="scan'208";a="157110616" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jan 2021 12:28:37 -0800 IronPort-SDR: Wk15VZMYLhTSRJe+wM8eN+4NsVkhDFIAzmHLW+rsMFHB/6IN88TOVK44+axPb81kMjUhRzbz9A VbzTZmXIqtAg== X-IronPort-AV: E=Sophos;i="5.79,339,1602572400"; d="scan'208";a="344995380" Received: from gkbech1x-mobl.amr.corp.intel.com (HELO [10.212.87.85]) ([10.212.87.85]) by fmsmga007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jan 2021 12:28:35 -0800 Subject: Re: [PATCH 13/14] ASoC: Intel: bytcr_wm5102: Add machine driver for BYT/WM5102 To: Charles Keepax , Hans de Goede Cc: Cezary Rojewski , alsa-devel@alsa-project.org, patches@opensource.cirrus.com, Mark Brown , Jie Yang , linux-kernel@vger.kernel.org, Liam Girdwood , Chanwoo Choi , MyungJoo Ham , Lee Jones References: <20201227211232.117801-1-hdegoede@redhat.com> <20201227211232.117801-14-hdegoede@redhat.com> <20201229135836.GO9673@ediswmail.ad.cirrus.com> From: Pierre-Louis Bossart Message-ID: Date: Mon, 11 Jan 2021 11:54:20 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20201229135836.GO9673@ediswmail.ad.cirrus.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >> + return ret; >> + } >> + >> + ret = snd_soc_component_set_sysclk(codec_component, ARIZONA_CLK_OPCLK, 0, >> + rate * sr_mult, SND_SOC_CLOCK_OUT); >> + if (ret) { >> + dev_err(codec_component->dev, "Error setting OPCLK: %d\n", ret); >> + return ret; >> + } > > OPCLK is a clock that can be outputted on the CODECs GPIOs. Is > that being used to clock some external component? If so it should > be added to the DAPM graph, if not you might as well remove this > call. Thanks Charles for the feedback. I have a vague recollection that the routing was supposed to be something like: AP-----V WM5102----> BT HFP Modem--^ with processing on WM5102. But when I look at the initial Android code there are references to the HFP path being handled by the AP. Probably better to remove this clock configuration if the need for it is unclear, the goal is only to enable speaker/headset for now. 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=-5.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 F24C3C433E6 for ; Mon, 11 Jan 2021 20:30:49 +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 08E4822CF6 for ; Mon, 11 Jan 2021 20:30:49 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 08E4822CF6 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com 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 652DC174E; Mon, 11 Jan 2021 21:29:57 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 652DC174E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1610397047; bh=RD+EP1iuw2MN4/WH+x0qA/FEncmO8Axa3NeAZqFHpVM=; h=Subject:To:References:From:Date:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=l6/OqLe9cQBe5QMBm691tV3t+w0dzLYndJ9wL3eWkAcMra2sn13x2tcqM+oLnOEi8 Dc4S3nd9sYrhO0RKSVrxWi/tvAlZzMhAHnjhY19R5GvoIvq+B4wfU1wJ3mRohxLKDm bR/nbFNeonE2+etf8jAiP2dkseiYwfeceX7yeCCg= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id EBFA8F8026D; Mon, 11 Jan 2021 21:29:01 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 46976F80475; Mon, 11 Jan 2021 21:28:52 +0100 (CET) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) (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 48399F8016C for ; Mon, 11 Jan 2021 21:28:41 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 48399F8016C IronPort-SDR: +IgRyG12q+yqr7BRKnkqG7daxlUPMHpCSu8dSRD0bFBb3gwz+OSvgINWYt7gDxZMQYz3nzSKdy O01J+dAJY5IQ== X-IronPort-AV: E=McAfee;i="6000,8403,9861"; a="262712509" X-IronPort-AV: E=Sophos;i="5.79,339,1602572400"; d="scan'208";a="262712509" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jan 2021 12:28:37 -0800 IronPort-SDR: Wk15VZMYLhTSRJe+wM8eN+4NsVkhDFIAzmHLW+rsMFHB/6IN88TOVK44+axPb81kMjUhRzbz9A VbzTZmXIqtAg== X-IronPort-AV: E=Sophos;i="5.79,339,1602572400"; d="scan'208";a="344995380" Received: from gkbech1x-mobl.amr.corp.intel.com (HELO [10.212.87.85]) ([10.212.87.85]) by fmsmga007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jan 2021 12:28:35 -0800 Subject: Re: [PATCH 13/14] ASoC: Intel: bytcr_wm5102: Add machine driver for BYT/WM5102 To: Charles Keepax , Hans de Goede References: <20201227211232.117801-1-hdegoede@redhat.com> <20201227211232.117801-14-hdegoede@redhat.com> <20201229135836.GO9673@ediswmail.ad.cirrus.com> From: Pierre-Louis Bossart Message-ID: Date: Mon, 11 Jan 2021 11:54:20 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20201229135836.GO9673@ediswmail.ad.cirrus.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Cc: Cezary Rojewski , patches@opensource.cirrus.com, Jie Yang , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Liam Girdwood , Chanwoo Choi , Mark Brown , MyungJoo Ham , Lee Jones 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" >> + return ret; >> + } >> + >> + ret = snd_soc_component_set_sysclk(codec_component, ARIZONA_CLK_OPCLK, 0, >> + rate * sr_mult, SND_SOC_CLOCK_OUT); >> + if (ret) { >> + dev_err(codec_component->dev, "Error setting OPCLK: %d\n", ret); >> + return ret; >> + } > > OPCLK is a clock that can be outputted on the CODECs GPIOs. Is > that being used to clock some external component? If so it should > be added to the DAPM graph, if not you might as well remove this > call. Thanks Charles for the feedback. I have a vague recollection that the routing was supposed to be something like: AP-----V WM5102----> BT HFP Modem--^ with processing on WM5102. But when I look at the initial Android code there are references to the HFP path being handled by the AP. Probably better to remove this clock configuration if the need for it is unclear, the goal is only to enable speaker/headset for now.