Received-SPF: pass (google.com: domain of linux-kernel+bounces-196334-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: "Xu, Baojun" <baojun.xu@ti.com>
To: "tiwai@suse.de" <tiwai@suse.de>
CC: "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "andriy.shevchenko@linux.intel.com" <andriy.shevchenko@linux.intel.com>,
        "lgirdwood@gmail.com" <lgirdwood@gmail.com>,
        "perex@perex.cz"
	<perex@perex.cz>,
        "pierre-louis.bossart@linux.intel.com"
	<pierre-louis.bossart@linux.intel.com>,
        "Lu, Kevin" <kevin-lu@ti.com>,
        "Ding,
 Shenghao" <shenghao-ding@ti.com>,
        "Navada Kanyana, Mukund" <navada@ti.com>,
        "13916275206@139.com" <13916275206@139.com>,
        "Hampiholi, Vallabha"
	<v-hampiholi@ti.com>,
        "P O, Vijeth" <v-po@ti.com>,
        "Holalu Yogendra,
 Niranjan" <niranjan.hy@ti.com>,
        "alsa-devel@alsa-project.org"
	<alsa-devel@alsa-project.org>,
        "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>,
        "liam.r.girdwood@intel.com"
	<liam.r.girdwood@intel.com>,
        "yung-chuan.liao@linux.intel.com"
	<yung-chuan.liao@linux.intel.com>,
        "broonie@kernel.org" <broonie@kernel.org>,
        "soyer@irl.hu" <soyer@irl.hu>
Subject: Re: [PATCH v6 0/1] Add tas2781 driver for SPI.
Thread-Topic: [PATCH v6 0/1] Add tas2781 driver for SPI.
Thread-Index: AQHaseDnrvmrnJlJZ0CQY5+1p/UdJbGw25Ri
Date: Fri, 31 May 2024 06:10:44 +0000
Message-ID: <62a298a58b944ecdaa1b2842d51f3a38@ti.com>
References: <20240529155653.3037-1-baojun.xu@ti.com>
In-Reply-To: <20240529155653.3037-1-baojun.xu@ti.com>
Accept-Language: en-GB, zh-CN, en-US
Content-Language: en-GB
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
MIME-Version: 1.0

Hi Pierre,

Do you know any different between windows and Ubuntu on SPI inialyze?
We are testing my patch on Laptop (HP-Varcolac, SSID:103C8B92, fly-out SPI =
& I2S cables), no any issue with wondows, but with Linux, second speaker ha=
s no sound, it's initialyze is totally same with first AMP, and relative re=
gisters can be dumped, no problem found.

Best Regards
Jim
________________________________________
From: Xu, Baojun
Sent: 29 May 2024 23:56
To: tiwai@suse.de
Cc: robh+dt@kernel.org; andriy.shevchenko@linux.intel.com; lgirdwood@gmail.=
com; perex@perex.cz; pierre-louis.bossart@linux.intel.com; Lu, Kevin; Ding,=
 Shenghao; Navada Kanyana, Mukund; 13916275206@139.com; Hampiholi, Vallabha=
; P O, Vijeth; Holalu Yogendra, Niranjan; alsa-devel@alsa-project.org; linu=
x-kernel@vger.kernel.org; liam.r.girdwood@intel.com; yung-chuan.liao@linux.=
intel.com; Xu, Baojun; broonie@kernel.org; soyer@irl.hu
Subject: [PATCH v6 0/1] Add tas2781 driver for SPI.

This patch was used to add TAS2781 devices on SPI support in sound/pci/hda.
It use ACPI node descript about parameters of TAS2781 on SPI, it like:
    Scope (_SB.PC00.SPI0)
    {
        Device (GSPK)
        {
            Name (_HID, "TXNW2781")  // _HID: Hardware ID
            Method (_CRS, 0, NotSerialized)
            {
                Name (RBUF, ResourceTemplate ()
                {
                    SpiSerialBusV2 (...)
                    SpiSerialBusV2 (...)
                }
            }
        }
    }

And in platform/x86/serial-multi-instantiate.c, those spi devices will be
added into system as a single SPI device, so TAS2781 SPI driver will
probe twice for every single SPI device. And driver will also parser
mono DSP firmware binary and RCA binary for itself.

Signed-off-by: Baojun Xu <baojun.xu@ti.com>

Baojun Xu (1):
  ALSA: hda/tas2781: Add tas2781 hda driver based on SPI

 drivers/acpi/scan.c                           |    1 +
 .../platform/x86/serial-multi-instantiate.c   |   12 +
 sound/pci/hda/Kconfig                         |   14 +
 sound/pci/hda/Makefile                        |    2 +
 sound/pci/hda/patch_realtek.c                 |   13 +
 sound/pci/hda/tas2781-spi.h                   |  148 ++
 sound/pci/hda/tas2781_hda_spi.c               | 1266 ++++++++++
 sound/pci/hda/tas2781_spi_fwlib.c             | 2101 +++++++++++++++++
 8 files changed, 3557 insertions(+)
 create mode 100644 sound/pci/hda/tas2781-spi.h
 create mode 100644 sound/pci/hda/tas2781_hda_spi.c
 create mode 100644 sound/pci/hda/tas2781_spi_fwlib.c

--
2.40.1