Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2291399imu; Thu, 17 Jan 2019 11:33:27 -0800 (PST) X-Google-Smtp-Source: ALg8bN5cOEc4iMLSBnJ2yekn/TG1Iv1QsfW4Cm6IlsqPd3o99AAnCGW3IUgBRWLJ5lgMjNcU2EUW X-Received: by 2002:a63:ba4d:: with SMTP id l13mr902944pgu.194.1547753607261; Thu, 17 Jan 2019 11:33:27 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1547753607; cv=none; d=google.com; s=arc-20160816; b=RWq+Z4nKc8ytkUEcfDoS76R3znSOftuYAiXt7VV1DvjaxNW2FUWXLr5vUEtMvtCjWl wAbj+S/lhbSGeVR3h7pQKXWYQ51YVVlQMabzBzWQSni9Xqr9A7E9JClUCwj43HUYg/ou x8w596fy9saTcUu6uNQ+LWQxm7byFPsX0MpGcnaxlkG1iXV1s+GUbnOTEWcWdhVa3pqJ vkcl8M7fow0oR72Zhw/SLC+rYw/yUFx1jhXr6qN78Pbe0eBtjgYWDhsMHJtzCZ8W3n6Y R59kzAEbZ2IDaxPPyKMR/hmu5RYV5u9R3uPCdu6uBwM13h19+0rpHaXT3L7qNRC+NX/k Br8g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :feedback-id:references:in-reply-to:message-id:subject:reply-to:cc :from:to:dkim-signature:date; bh=HJBKZ8PtqxY+VcQp4wvvTxv0AeJJfjPihAJvQvXbFJ4=; b=U3Q4ICAqtL0Q9BQqCYBqkeDrCFwWW0wVr/XSd0CvqEI+/eWuD/ndy5Qhv/BXmi4Xe8 UfEjC/flet7eFllXIKpnKqjPMCPBeyhEAOnWk/2AFReBbfgLKUkkVIMy/Gop8LvhxPq9 WDXzk/4hoqjFxfBnEJfqmlimHzt0rwuMWxWsOr89Jgl9LmcFAh4GuM2Zi0KpRnUCh5PT MavwkUUjADYBEMyNGl7E8EjvEM4m/6V0ywvy46fJBpU5kUtmehR8DxaqonFJvD9b3M7y Cgm0zZ6BHg1frMYmzaUlTSw5KGzawcxq5JxgYTdYoFekPuidPDJaN751mY4t7UCrt6jw jMDQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@protonmail.com header.s=default header.b=QDmSszFI; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=protonmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 30si2087026pgv.191.2019.01.17.11.33.11; Thu, 17 Jan 2019 11:33:27 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@protonmail.com header.s=default header.b=QDmSszFI; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=protonmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728785AbfAQTaw (ORCPT + 99 others); Thu, 17 Jan 2019 14:30:52 -0500 Received: from mail-40130.protonmail.ch ([185.70.40.130]:46731 "EHLO mail-40130.protonmail.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727704AbfAQTaw (ORCPT ); Thu, 17 Jan 2019 14:30:52 -0500 Date: Thu, 17 Jan 2019 19:30:41 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonmail.com; s=default; t=1547753448; bh=HJBKZ8PtqxY+VcQp4wvvTxv0AeJJfjPihAJvQvXbFJ4=; h=Date:To:From:Cc:Reply-To:Subject:In-Reply-To:References: Feedback-ID:From; b=QDmSszFILA5oT5phVVPaNnP7z7h7fMNDjX8+qFGL6fi7LPHlTJ9RaLytNgDAn7CWB 1LgSm7Fyqg8UzrD8f6iLP6omi8lWaJuxeO6HWtdaTLvHesWw5eBW1evoQCKmMHjEL/ er5qwHsSmruJs69y7V+Q4UoAWbLnLThHkinhqQSw= To: Hans de Goede From: Mogens Jensen Cc: Dean Wallace , Pierre-Louis Bossart , Andy Shevchenko , Stephen Boyd , Michael Turquette , linux-clk , Stable , Johannes Stezenbach , Carlo Caione , Andy Shevchenko , Linux Kernel Mailing List Reply-To: Mogens Jensen Subject: Re: Regression found (Stop-marking-clocks-as-CLK_IS_CRITICAL) Message-ID: In-Reply-To: References: <20181025232517.ywnw54qibemosjws@picard> <3iO9ehQbZm_haTV0IuZ0qhsVHR0QLUbTgRJT8ZenGuRsnz2_uBvO93f0bHVYnsApibUT16JsJ0dgphLhUBd-u0t-lDBNsbvvlKWTgq8XOlw=@protonmail.com> <3e12e051-b874-187d-d4f5-e146f59c659b@redhat.com> <20190117091214.7djttnpo4jjzi4su@spock> Feedback-ID: 4dFv7M8pOtP7CMTtfNdOwXARWpF2vcQYB4dRuX1h9mb-6008qkzO2MVweOzrAYHEdkjJ6P5PUdgJEubk4l9Lhg==:Ext:ProtonMail MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-1.2 required=7.0 tests=ALL_TRUSTED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mail.protonmail.ch Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90 Original Me= ssage =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90 On Thursday, January 17, 2019 12:05 PM, Hans de Goede = wrote: > Hi, > > On 17-01-19 10:12, Dean Wallace wrote: > > > Hi Hans, Mogens, > > On 17-01-19, Mogens Jensen wrote: > > > > > Kernel is compiled with SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH and th= e quirk seems to have fixed the problem caused by commit 648e921888ad ("clk= : x86: Stop marking clocks as CLK_IS_CRITICAL"), as sound is now working if= running "speaker-test" on my system which is clean ALSA. > > Note being "clean ALSA" is really not a good thing now a days, > for lots of things we depend on pulseaudio (like setting > up UCM mixer profiles). > I'm using UCM mixer profile from: https://github.com/plbossart/UCM/tree/master/chtmax98090 This is enabled with: alsaucm -c chtmax98090 set _verb HiFi set _enadev Speakers > > > Unfortunately, SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH driver is unusa= ble on Clapper Chromebooks as audio played from everything but "speaker-tes= t" as video players or web browsers is extremly low and sounds like played = at 10x speed. At the same time kernel log is spammed with messages like thi= s: > > > max98090 i2c-193C9890:00: PLL unlocked > > > intel_sst_acpi 80860F28:00: FW Version 01.0c.00.01 > > > writing to lpe: 00000000: 01 01 01 01 00 00 08 00 ff ff ff ff 55 00 0= 0 00 ............U... > > > writing to lpe: 00000000: 01 01 01 01 00 00 1a 00 ff ff ff ff 75 00 1= 2 00 ............u... > > > This is probably not related to the problem discussed in this thread,= but the result is that I have to use the legacy driver SND_SOC_INTEL_BYT_M= AX98090_MACH and therefore still has to revert commit 648e921888ad for soun= d to work. > > > Is it possible to create a fix for SND_SOC_INTEL_BYT_MAX98090_MACH on= kernel 4.19? Kernel 4.19 is a long term release so it would be very nice t= o have fix for this version upstream. > > > > I have been reverting "clk: x86: Stop marking clocks as CLK_IS_CRITICAL= " > > and the patch that initially added the quirk for swanky because of soun= d > > instability issues as you described. I'm compiling vanilla Archlinux > > kernel with SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH, using pulseaudio, > > and have sound in all my apps. > > Baytrail sound has always been a little touchy, especially using headse= t > > with mic, but since the clk patch breaking sound and the quirk patch to > > fix it, there is a lot more instability. Just running pavucontrol, or > > plugging in headset sets it off. It's a head scratcher. > > Mogens, Dean, can you please try the SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MA= CH > driver, without reverting any patches, with the attached patch on top and > see if that helps? > > Thanks & Regards, > > Hans I have applied the patch to kernel 4.19.15 and unfortunately this has not s= olved the problems. Audio generated from "speaker-test" is normal, but from everything else is = very low and played at 10x speed. However, I'm not seeing the "max98090 i2c= -193C9890:00: PLL unlocked" message in kernel log anymore, but it's still s= pammed with "writing to lpe: ...". Regards, Mogens