Hi Gergo,
Answer in line:
From: Gergo Koteles soyer@irl.hu Sent: 27 March 2024 18:02 To: Xu, Baojun; 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; 13916275206@139.com; alsa-devel@alsa-project.org; linux-kernel@vger.kernel.org; liam.r.girdwood@intel.com; yung-chuan.liao@linux.intel.com; broonie@kernel.org Subject: [EXTERNAL] Re: [PATCH v1 7/8] ALSA: hda/tas2781: Add tas2781 SPI-based driver
Hi Baojun, On Tue, 2024-03-26 at 09: 09 +0800, Baojun Xu wrote: > Firmware binary load lib code for tas2781 spi driver. > > Signed-off-by: Baojun Xu <baojun. xu@ ti. com> > --- > sound/pci/hda/tas2781_spi_fwlib. c | 2374 +++++++++++++++++++++++++++++ ZjQcmQRYFpfptBannerStart This message was sent from outside of Texas Instruments. Do not click links or open attachments unless you recognize the source of this email and know the content is safe.
ZjQcmQRYFpfptBannerEnd
Hi Baojun,
On Tue, 2024-03-26 at 09:09 +0800, Baojun Xu wrote:
Firmware binary load lib code for tas2781 spi driver.
Signed-off-by: Baojun Xu baojun.xu@ti.com
sound/pci/hda/tas2781_spi_fwlib.c | 2374 +++++++++++++++++++++++++++++ 1 file changed, 2374 insertions(+) create mode 100644 sound/pci/hda/tas2781_spi_fwlib.c
Is this firmware loading logic the same as the already merged tas2781- fmwlib? If yes, why don't you use that?
It's almost same, but one firmware was loaded in I2C mode as one I2C device was probed, and operate slavers by address. And with SPI mode, we probe two SPI devices in current case, so will load two firmware binary, and some operation need to be processed as special case, for example, under SPI, read need one byte shifted if page number is large than 1.
Thanks, Gergo