ALSA: hda/tegra: Hardcode GCAP ISS value on T234
authorMohan Kumar <mkumard@nvidia.com>
Wed, 16 Feb 2022 09:22:36 +0000 (14:52 +0530)
committerTakashi Iwai <tiwai@suse.de>
Mon, 21 Feb 2022 10:03:35 +0000 (11:03 +0100)
The GCAP register on Tegra234 implies no Input Streams(ISS)
supported, but the HW output stream descriptor programming
should start with offset 0x20*4 from base stream descriptor
address. This will be a problem while calculating the offset
for output stream descriptor which will be considering input
stream also. So here output stream starts with offset 0 which
is wrong as HW register for output stream offset starts with 4.
So hardcode the input stream numbers to 4 to avoid the issue
in offset calculation.

Signed-off-by: Mohan Kumar <mkumard@nvidia.com>
Link: https://lore.kernel.org/r/20220216092240.26464-3-mkumard@nvidia.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/hda_tegra.c

index 95df52b0505b20a5918d1920ae81e3d51f6da7f7..2347d0304f936665c50ed21518ec32b32561e8ba 100644 (file)
@@ -315,6 +315,18 @@ static int hda_tegra_first_init(struct azx *chip, struct platform_device *pdev)
         * hardcoded value
         */
        chip->capture_streams = (gcap >> 8) & 0x0f;
+
+       /* The GCAP register on Tegra234 implies no Input Streams(ISS) support,
+        * but the HW output stream descriptor programming should start with
+        * offset 0x20*4 from base stream descriptor address. This will be a
+        * problem while calculating the offset for output stream descriptor
+        * which will be considering input stream also. So here output stream
+        * starts with offset 0 which is wrong as HW register for output stream
+        * offset starts with 4.
+        */
+       if (of_device_is_compatible(np, "nvidia,tegra234-hda"))
+               chip->capture_streams = 4;
+
        chip->playback_streams = (gcap >> 12) & 0x0f;
        if (!chip->playback_streams && !chip->capture_streams) {
                /* gcap didn't give any info, switching to old method */