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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <a64f246f48f447bc959f9d7591fbc477@ti.com>
Date: Fri, 5 Apr 2024 15:51:36 +0000
From: "Xu, Baojun" <baojun.xu@...com>
To: Gergo Koteles <soyer@....hu>, "tiwai@...e.de" <tiwai@...e.de>
CC: "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "andriy.shevchenko@...ux.intel.com" <andriy.shevchenko@...ux.intel.com>,
        "lgirdwood@...il.com" <lgirdwood@...il.com>,
        "perex@...ex.cz"
	<perex@...ex.cz>,
        "pierre-louis.bossart@...ux.intel.com"
	<pierre-louis.bossart@...ux.intel.com>,
        "Lu, Kevin" <kevin-lu@...com>,
        "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "liam.r.girdwood@...el.com" <liam.r.girdwood@...el.com>,
        "yung-chuan.liao@...ux.intel.com" <yung-chuan.liao@...ux.intel.com>,
        "broonie@...nel.org" <broonie@...nel.org>,
        "Ding, Shenghao"
	<shenghao-ding@...com>
Subject: Re: [EXTERNAL] Re: [PATCH v1 7/8] ALSA: hda/tas2781: Add tas2781
 SPI-based driver

Hi Gergo,

Answer in line:

> From: Gergo Koteles <soyer@....hu>
> Sent: 27 March 2024 18:02
> To: Xu, Baojun; tiwai@...e.de
> Cc: robh+dt@...nel.org; andriy.shevchenko@...ux.intel.com; lgirdwood@...il.com; perex@...ex.cz; pierre-louis.bossart@...ux.intel.com; Lu, Kevin; 13916275206@....com; alsa-devel@...a-project.org; linux-kernel@...r.kernel.org; liam.r.girdwood@...el.com; yung-chuan.liao@...ux.intel.com; broonie@...nel.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@...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
> 
>