linux-kernel - Re: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <s5h4k7rwgvk.wl-tiwai@suse.de>
Date:   Thu, 02 Dec 2021 09:27:59 +0100
From:   Takashi Iwai <tiwai@...e.de>
To:     "Lu, Brent" <brent.lu@...el.com>
Cc:     Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>,
        "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
        Hans de Goede <hdegoede@...hat.com>,
        "Liao, Bard" <bard.liao@...el.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Kai Vehmanen <kai.vehmanen@...ux.intel.com>,
        Takashi Iwai <tiwai@...e.com>
Subject: Re: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support

On Thu, 02 Dec 2021 09:15:37 +0100,
Lu, Brent wrote:
> 
> > >
> > > Thanks Brent, yes this is an oversight, we should have added this a
> > > long time ago.
> > >
> > > I don't think however that this will apply on Takashi's for-next
> > > branch, it'll conflict with the ALSA: intel-dsp-config: add quirk for
> > > JSL devices based on ES8336 codec
> > 
> > Actually it's for-linus branch, destined to 5.16-rc4.
> > 
> > > +/* JasperLake */
> > > +#if IS_ENABLED(CONFIG_SND_SOC_SOF_JASPERLAKE)
> > > +	{
> > > +		.flags = FLAG_SOF,
> > > +		.device = 0x4dc8,
> > > +		.codec_hid = "ESSX8336",
> > > +	},
> > > +#endif
> > >
> > > Do you mind rebasing and resending the update? Let's keep the ESS
> > > stuff last as done in other platforms, and add your changes on top of this.
> > 
> > Yes, please resubmit.  I can merge and resolve the conflict in my side, but it's
> > better to get a cleaner patch from the beginning.
> > 
> Hi gentlement,
> 
> I found following two patches in sof github do not exist in both broonie:for-next
> and broonie:for-linus. Maybe it would be easier if I resubmit my patch to sof
> github instead of alsa-dev?
> 
> sof: topic/sof-dev
> 986f58741d0d ALSA: intel-dsp-config: add quirk for CML devices based on ES8336 codec
> 2cccdf5b4414 ALSA: intel-dsp-config: add quirk for JSL devices based on ES8336 codec

You'll need to rebase to the target tree, i.e. my sound.git tree in
this case, as it's not directly about ASoC.

Mark will need to sync his tree eventually later, but for this
particular patch, just use my tree (for-linus branch) as the base or
use linux-next tree.


thanks,

Takashi