Re: [PATCH RFC 0/1] ASoC: rt1320: LG gram 16Z95U speaker works with the BIOS init values

[fixca <jaehoon1744@dankook.ac.kr>]

From: JaeHoon Kim <jaehoon1744@dankook.ac.kr>

Hi Shuming,

Please disregard this RFC. The BIOS init values are not needed: the
stock rt1320 driver works on this machine.

While narrowing the register groups down (after the Sashiko review,
which correctly pointed out that my table dropped the two PDE23 entries
and that 0x1000db15-0x1000db23 is the same command list in a different
order, not an L/R swap), the baseline with no BIOS values also played
sound. I then retested the unmodified v7.2.15 rt1320 driver, with only
the two ACP patches Mark applied, in three boot scenarios:

  boot                                   amp func_status   speaker
  cold boot (power fully removed)        0x61              works
  warm reboot Linux -> Linux             0x41              works
  Windows (speaker used) -> warm reboot  0x43              works

v7.2.15 has no audio related changes compared to v7.2.14, so I cannot
explain the silence I saw earlier on v7.2.14. It is no longer
reproducible; it may have been a stale amplifier state left over from
my earlier experiments. Once (also no longer reproducible) the RT1320
read back 0x81 for every register after a warm reboot, which only a
full power removal cleared. I will report back if either problem shows
up again.

The remaining open point is the one from before: no DSP firmware / RAE
files are loaded for this machine, so the speaker protection set up on
Windows is not active on Linux. If Realtek can provide those files for
the LG gram 16Z95U (SSID 1854:056c), I am happy to test them.

Sorry for the noise.

Thanks,
JaeHoon Kim