Re: [EXTERNAL] Re: [PATCH v1 7/8] ALSA: hda/tas2781: Add tas2781 SPI-based driver

["Xu, Baojun" <baojun.xu@ti.com>]

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
> 
>