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 providethe 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;
- 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.