On 3/25/2021 11:32 PM, Vamshi Krishna wrote:
> From: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
> Sent: Thursday, March 25, 2021 12:04 AM
> To: Gopal, Vamshi Krishna <vamshi.krishna.gopal@xxxxxxxxx>; alsa-
> devel@xxxxxxxxxxxxxxxx
> Cc: N, Harshapriya <harshapriya.n@xxxxxxxxx>; broonie@xxxxxxxxxx; M R,
> Sathya Prakash <sathya.prakash.m.r@xxxxxxxxx>; biernacki@xxxxxxxxxx;
> Bossart, Pierre-louis <pierre-louis.bossart@xxxxxxxxx>
> Subject: Re: [PATCH 1/2] ASoC: Intel: kbl: Add MST route change to kbl
> machine drivers
>
>
> > diff --git a/sound/soc/intel/boards/kbl_da7219_max98357a.c
> > b/sound/soc/intel/boards/kbl_da7219_max98357a.c
> > index dc3d897ad280..1d6b2855874d 100644
> > --- a/sound/soc/intel/boards/kbl_da7219_max98357a.c
> > +++ b/sound/soc/intel/boards/kbl_da7219_max98357a.c
> > @@ -91,7 +91,9 @@ static const struct snd_soc_dapm_widget
> kabylake_widgets[] = {
> > SND_SOC_DAPM_SPK("Spk", NULL),
> > SND_SOC_DAPM_MIC("SoC DMIC", NULL),
> > SND_SOC_DAPM_SPK("DP", NULL),
> > - SND_SOC_DAPM_SPK("HDMI", NULL),
> > + SND_SOC_DAPM_SPK("HDMI1", NULL),
> > + SND_SOC_DAPM_SPK("HDMI2", NULL),
> > + SND_SOC_DAPM_SPK("HDMI3", NULL),
>
> that seems consistent with other BXT/KBL machine drivers, but...
>
> > SND_SOC_DAPM_SUPPLY("Platform Clock", SND_SOC_NOPM, 0, 0,
> > platform_clock_control, SND_SOC_DAPM_PRE_PMU
> |
> > SND_SOC_DAPM_POST_PMD),
> > @@ -108,9 +110,6 @@ static const struct snd_soc_dapm_route
> kabylake_map[] = {
> > { "MIC", NULL, "Headset Mic" },
> > { "DMic", NULL, "SoC DMIC" },
> >
> > - { "HDMI", NULL, "hif5 Output" },
> > - { "DP", NULL, "hif6 Output" },
> > -
>
> ... this doesn't:
>
> other machine drivers use this:
>
> {"HDMI1", NULL, "hif5-0 Output"},
> {"HDMI2", NULL, "hif6-0 Output"},
> {"HDMI2", NULL, "hif7-0 Output"},
>
Hello Pierre,
Thanks for reviewing the patch.
I looked through the change you suggested in bxt_da7219_max98357a.c machine, but I noticed hif6-0 Output and hif7-0 Output are having same port HDMI2, This looks not correct.
> And if you start changing HDMI support, you should also fix the other
> machine drivers that used the same pattern, e.g.
>
> kbl_da7219_max98927.c\0129: { "HDMI", NULL, "hif5 Output" },
> kbl_rt5663_max98927.c\0214: { "HDMI", NULL, "hif5 Output" },
>
Submitted a v2 patch which follows same pattern across KBL machine drivers.
> > /* CODEC BE connections */
> > { "HiFi Playback", NULL, "ssp0 Tx" },
> > { "ssp0 Tx", NULL, "codec0_out" },
> >
