* [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support
@ 2021-12-01 0:46 Brent Lu
2021-12-01 2:32 ` Pierre-Louis Bossart
0 siblings, 1 reply; 6+ messages in thread
From: Brent Lu @ 2021-12-01 0:46 UTC (permalink / raw)
To: alsa-devel
Cc: Jaroslav Kysela, Takashi Iwai, Pierre-Louis Bossart,
Kai Vehmanen, Bard Liao, Xiuli Pan, Brent Lu, Hans de Goede,
linux-kernel
Add rules to select SOF driver for Jasper Lake systems if digital
microphone is present or the system is a Chromebook.
Signed-off-by: Brent Lu <brent.lu@intel.com>
---
sound/hda/intel-dsp-config.c | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/sound/hda/intel-dsp-config.c b/sound/hda/intel-dsp-config.c
index b9ac9e9e45a4..be47a1ca12b6 100644
--- a/sound/hda/intel-dsp-config.c
+++ b/sound/hda/intel-dsp-config.c
@@ -299,6 +299,27 @@ static const struct config_entry config_table[] = {
},
#endif
+/* Jasper Lake */
+#if IS_ENABLED(CONFIG_SND_SOC_SOF_JASPERLAKE)
+ {
+ .flags = FLAG_SOF,
+ .device = 0x4dc8,
+ .dmi_table = (const struct dmi_system_id []) {
+ {
+ .ident = "Google Chromebooks",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Google"),
+ }
+ },
+ {}
+ }
+ },
+ {
+ .flags = FLAG_SOF | FLAG_SOF_ONLY_IF_DMIC,
+ .device = 0x4dc8,
+ },
+#endif
+
/* Tigerlake */
#if IS_ENABLED(CONFIG_SND_SOC_SOF_TIGERLAKE)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support
2021-12-01 0:46 [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support Brent Lu
@ 2021-12-01 2:32 ` Pierre-Louis Bossart
2021-12-01 6:44 ` Takashi Iwai
0 siblings, 1 reply; 6+ messages in thread
From: Pierre-Louis Bossart @ 2021-12-01 2:32 UTC (permalink / raw)
To: Brent Lu, alsa-devel
Cc: Kai Vehmanen, linux-kernel, Takashi Iwai, Hans de Goede, Bard Liao
On 11/30/21 6:46 PM, Brent Lu wrote:
> Add rules to select SOF driver for Jasper Lake systems if digital
> microphone is present or the system is a Chromebook.
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
+/* 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.
Thanks!
>
> Signed-off-by: Brent Lu <brent.lu@intel.com>
> ---
> sound/hda/intel-dsp-config.c | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
>
> diff --git a/sound/hda/intel-dsp-config.c b/sound/hda/intel-dsp-config.c
> index b9ac9e9e45a4..be47a1ca12b6 100644
> --- a/sound/hda/intel-dsp-config.c
> +++ b/sound/hda/intel-dsp-config.c
> @@ -299,6 +299,27 @@ static const struct config_entry config_table[] = {
> },
> #endif
>
> +/* Jasper Lake */
> +#if IS_ENABLED(CONFIG_SND_SOC_SOF_JASPERLAKE)
> + {
> + .flags = FLAG_SOF,
> + .device = 0x4dc8,
> + .dmi_table = (const struct dmi_system_id []) {
> + {
> + .ident = "Google Chromebooks",
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Google"),
> + }
> + },
> + {}
> + }
> + },
> + {
> + .flags = FLAG_SOF | FLAG_SOF_ONLY_IF_DMIC,
> + .device = 0x4dc8,
> + },
> +#endif
> +
> /* Tigerlake */
> #if IS_ENABLED(CONFIG_SND_SOC_SOF_TIGERLAKE)
> {
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support
2021-12-01 2:32 ` Pierre-Louis Bossart
@ 2021-12-01 6:44 ` Takashi Iwai
2021-12-02 8:15 ` Lu, Brent
0 siblings, 1 reply; 6+ messages in thread
From: Takashi Iwai @ 2021-12-01 6:44 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: Brent Lu, alsa-devel, Hans de Goede, Bard Liao, linux-kernel,
Kai Vehmanen, Takashi Iwai
On Wed, 01 Dec 2021 03:32:39 +0100,
Pierre-Louis Bossart wrote:
>
>
>
> On 11/30/21 6:46 PM, Brent Lu wrote:
> > Add rules to select SOF driver for Jasper Lake systems if digital
> > microphone is present or the system is a Chromebook.
>
> 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.
thanks,
Takashi
>
> Thanks!
>
> >
> > Signed-off-by: Brent Lu <brent.lu@intel.com>
> > ---
> > sound/hda/intel-dsp-config.c | 21 +++++++++++++++++++++
> > 1 file changed, 21 insertions(+)
> >
> > diff --git a/sound/hda/intel-dsp-config.c b/sound/hda/intel-dsp-config.c
> > index b9ac9e9e45a4..be47a1ca12b6 100644
> > --- a/sound/hda/intel-dsp-config.c
> > +++ b/sound/hda/intel-dsp-config.c
> > @@ -299,6 +299,27 @@ static const struct config_entry config_table[] = {
> > },
> > #endif
> >
> > +/* Jasper Lake */
> > +#if IS_ENABLED(CONFIG_SND_SOC_SOF_JASPERLAKE)
> > + {
> > + .flags = FLAG_SOF,
> > + .device = 0x4dc8,
> > + .dmi_table = (const struct dmi_system_id []) {
> > + {
> > + .ident = "Google Chromebooks",
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Google"),
> > + }
> > + },
> > + {}
> > + }
> > + },
> > + {
> > + .flags = FLAG_SOF | FLAG_SOF_ONLY_IF_DMIC,
> > + .device = 0x4dc8,
> > + },
> > +#endif
> > +
> > /* Tigerlake */
> > #if IS_ENABLED(CONFIG_SND_SOC_SOF_TIGERLAKE)
> > {
> >
>
^ permalink raw reply [flat|nested] 6+ messages in thread* RE: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support
2021-12-01 6:44 ` Takashi Iwai
@ 2021-12-02 8:15 ` Lu, Brent
2021-12-02 8:27 ` Takashi Iwai
0 siblings, 1 reply; 6+ messages in thread
From: Lu, Brent @ 2021-12-02 8:15 UTC (permalink / raw)
To: Takashi Iwai, Pierre-Louis Bossart
Cc: alsa-devel, Hans de Goede, Liao, Bard, linux-kernel,
Kai Vehmanen, Takashi Iwai
> >
> > 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
Regards,
Brent
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support
2021-12-02 8:15 ` Lu, Brent
@ 2021-12-02 8:27 ` Takashi Iwai
2022-01-13 10:57 ` Lu, Brent
0 siblings, 1 reply; 6+ messages in thread
From: Takashi Iwai @ 2021-12-02 8:27 UTC (permalink / raw)
To: Lu, Brent
Cc: Pierre-Louis Bossart, alsa-devel, Hans de Goede, Liao, Bard,
linux-kernel, Kai Vehmanen, Takashi Iwai
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
^ permalink raw reply [flat|nested] 6+ messages in thread* RE: [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support
2021-12-02 8:27 ` Takashi Iwai
@ 2022-01-13 10:57 ` Lu, Brent
0 siblings, 0 replies; 6+ messages in thread
From: Lu, Brent @ 2022-01-13 10:57 UTC (permalink / raw)
To: Takashi Iwai
Cc: Pierre-Louis Bossart, alsa-devel, Hans de Goede, Liao, Bard,
linux-kernel, Kai Vehmanen, Takashi Iwai
> > 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.
Sorry I forgot this mail. A v2 patch is sent to rebased on the right tree and on
top of ES8336 changes.
>
> 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
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-01-13 10:58 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-01 0:46 [PATCH] ALSA: hda: intel-dsp-config: add JasperLake support Brent Lu
2021-12-01 2:32 ` Pierre-Louis Bossart
2021-12-01 6:44 ` Takashi Iwai
2021-12-02 8:15 ` Lu, Brent
2021-12-02 8:27 ` Takashi Iwai
2022-01-13 10:57 ` Lu, Brent
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).