RE: [PATCH 1/2] ASoC: Intel: kbl: Add MST route change to kbl machine drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> >>>
> >>> -	{ "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.
> 
> D'oh! You're right, this makes no sense to me either. I see 4 occurrences in
> the code.
> 
[Gopal, Vamshi Krishna]  Hello Pierre,
I will send the patches for bxt and GLK drivers separately after doing the validation.
I have submitted the v2 patch with fix for KBL drivers, can we merge the KBL patches first ?
 
> bxt_da7219_max98357a.c:	{"HDMI1", NULL, "hif5-0 Output"},
> bxt_da7219_max98357a.c:	{"HDMI2", NULL, "hif6-0 Output"},
> bxt_da7219_max98357a.c:	{"HDMI2", NULL, "hif7-0 Output"},
> 
> bxt_rt298.c:	{"HDMI1", NULL, "hif5-0 Output"},
> bxt_rt298.c:	{"HDMI2", NULL, "hif6-0 Output"},
> bxt_rt298.c:	{"HDMI2", NULL, "hif7-0 Output"},
> 
> bxt_rt298.c:	{"HDMI1", NULL, "hif5-0 Output"},
> bxt_rt298.c:	{"HDMI2", NULL, "hif6-0 Output"},
> bxt_rt298.c:	{"HDMI2", NULL, "hif7-0 Output"},
> 
> glk_rt5682_max98357a.c:	{ "HDMI1", NULL, "hif5-0 Output" },
> glk_rt5682_max98357a.c:	{ "HDMI2", NULL, "hif6-0 Output" },
> glk_rt5682_max98357a.c:	{ "HDMI2", NULL, "hif7-0 Output" },
> 
> Harsha and team, the HDMI2 duplicates seem like recurring copy/paste
> mistakes, can you double check what the intent was? If this is indeed
> unintentional, we probably need a patch per file with a Fixes tag to have this
> applied to the stable kernel.
> 
> Thanks!




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux