See comments in line

thanks
JimQu

获取 Outlook for Android<https://aka.ms/ghei36>



发件人: Takashi Iwai
发送时间: 7月13日星期五 16:33
主题: Re: [alsa-devel] [PATCH 1/2] ALSA: HDA: use PCI_BASE_CLASS_DISPLAY tocover 
more display adapters
收件人: Qu, Jim
抄送: alsa-de...@alsa-project.org, dri-devel@lists.freedesktop.org, 
amd-...@lists.freedesktop.org


On Fri, 13 Jul 2018 10:06:01 +0200, Jim Qu wrote: > > Signed-off-by: Jim Qu 
More explanations please. e.g. why is this needed? The code change itself looks 
good, but ... > @@ -31,7 +31,7 @@ > * CHANGES: > * > * 2004.12.01 Major rewrite 
by tiwai, merged the work of pshou > - * > + * > */ > > #include > @@ -390,8 
+390,8 @@ static char *driver_short_names[] = { > [AZX_DRIVER_SIS] = "HDA 
SIS966", > [AZX_DRIVER_ULI] = "HDA ULI M5461", > [AZX_DRIVER_NVIDIA] = "HDA 
NVidia", > - [AZX_DRIVER_TERA] = "HDA Teradici", > - [AZX_DRIVER_CTX] = "HDA 
Creative", > + [AZX_DRIVER_TERA] = "HDA Teradici", > + [AZX_DRIVER_CTX] = "HDA 
Creative", > [AZX_DRIVER_CTHDA] = "HDA Creative", > [AZX_DRIVER_CMEDIA] = "HDA 
C-Media", > [AZX_DRIVER_GENERIC] = "HD-Audio Generic", These two hunks are 
utterly irrelevant with the fix. Please drop them.

Jim: Yeah, I forget to generate a new one. I will delete them when push the 
patch.

thanks, Takashi

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to