linux-kernel - Re: CSC3551 and devices missing related

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <78488c71-eef5-1a44-0b85-73c7ad0848c9@opensource.cirrus.com>
Date:   Thu, 1 Jun 2023 10:42:56 +0100
From:   Stuart Henderson <stuarth@...nsource.cirrus.com>
To:     Armas Spann <armas@...ux.tech>, Luke Jones <luke@...nes.dev>,
        Takashi Iwai <tiwai@...e.de>
CC:     <linux-kernel@...r.kernel.org>, <tiwai@...e.com>,
        <sbinding@...nsource.cirrus.com>, <perex@...ex.cz>,
        <tangmeng@...ontech.com>, <andy.chi@...onical.com>,
        <p.jungkamp@....net>, <kasper93@...il.com>,
        <yangyuchi66@...il.com>, <ealex95@...il.com>,
        <james.schulman@...rus.com>, <david.rhodes@...rus.com>,
        <tanureal@...nsource.cirrus.com>, <rf@...nsource.cirrus.com>,
        <patches@...nsource.cirrus.com>, <alsa-devel@...a-project.org>
Subject: Re: CSC3551 and devices missing related _DSD bits


> Hi Takashi, Hi Stuart (and of course, all others in here),
>
> would you mind to evaluate this small (pseudo-)patch to be harmless?
> (concerning the blow-up theory the first answer in this converstion)
>
> I won't push it upstream right now but I want to know if this patch
> might be harmfull. I'm owning a GA402XY myself and we digged out that
> the initial setting of the cr3551 can be done via:
>
> diff --git a/sound/pci/hda/cs35l41_hda.c b/sound/pci/hda/cs35l41_hda.c
> index 75020edd39e7..eaa06751bd48 100644
> --- a/sound/pci/hda/cs35l41_hda.c
> +++ b/sound/pci/hda/cs35l41_hda.c
> @@ -1243,6 +1243,12 @@ static int cs35l41_no_acpi_dsd(struct
> cs35l41_hda *cs35l41, struct device *physd
>   		hw_cfg->bst_type = CS35L41_EXT_BOOST;
>   		hw_cfg->gpio1.func = CS35l41_VSPK_SWITCH;
>   		hw_cfg->gpio1.valid = true;
> +	} else if (strncmp(hid, "CSC3551", 7) == 0 && strcmp(cs35l41-
>> acpi_subsystem_id, "10431463") == 0) {
> +		// TESTING - (Hook for GA402X)
> +		dev_warn(cs35l41->dev, "Warning: ASUS didn't provide
> the needed ACPI _DSD properties for GA402X series, using defaults..");
> +		hw_cfg->bst_type = CS35L41_EXT_BOOST;
> +		hw_cfg->gpio1.func = CS35l41_VSPK_SWITCH;
> +		hw_cfg->gpio1.valid = true;
>   	} else {
>   		/*
>   		 * Note: CLSA010(0/1) are special cases which use a
> slightly different design.
I wouldn't recommend making speculative changes like the above. There 
are no "default" values as these attributes describe the hardware for 
the particular device and these can be different on each SKU.  In this 
case, it looks like the boost is being set incorrectly.  We're looking 
into how we can support these older devices, so please bear with us 
while we investigate.