Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <91e34de4-1c85-073f-9c24-ce3133394fe6@neo-zeon.de>
Date:   Tue, 26 Jul 2022 08:35:49 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH 0/2] Add support for CLSA0101
Content-Language: en-US
To:     Lucas Tanure <tanureal@opensource.cirrus.com>,
        Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
Cc:     alsa-devel@alsa-project.org, patches@opensource.cirrus.com,
        linux-kernel@vger.kernel.org
References: <20220726101131.68058-1-tanureal@opensource.cirrus.com>
From:   Cameron Berkenpas <cam@neo-zeon.de>
In-Reply-To: <20220726101131.68058-1-tanureal@opensource.cirrus.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

Hello,

I can confirm this does _not_ work.

More specifically, with these patches, the original behavior still occurs:
1. You get left channel audio from the right speaker only.
2. No sound at all after resuming from sleep.
3. You get sound after resuming from hibernate, but only as described in #1.

If you apply a patch like this: https://paste-bin.xyz/71355

You  get properly working audio until you put the laptop to sleep or 
hibernate. Upon resuming from either state, you get no audio at all.

On 7/26/22 03:11, Lucas Tanure wrote:
> Add Support for the CLSA0101 laptop, an Intel version of CLSA0100.
> This patch has been tested using the CLSA0100, ensuring it
> doesn't break the sound for it.
> We appreciate it if someone with CLSA0101 could verify that this
> the patch works for them.
>
> Lucas Tanure (2):
>    ALSA: hda: cs35l41: Use the CS35L41 HDA internal define
>    ALSA: hda: cs35l41: Support CLSA0101
>
>   sound/pci/hda/cs35l41_hda.c   | 67 +++++++++++++++++++++--------------
>   sound/pci/hda/patch_realtek.c | 12 +++++++
>   2 files changed, 53 insertions(+), 26 deletions(-)
>