diff mbox

ALSA: hda - Add missing terminating entry to SND_HDA_PIN_QUIRK macro

Message ID 1414137638-8678-1-git-send-email-david.henningsson@canonical.com (mailing list archive)
State Accepted
Commit fb54a645b2739fb196446ffbbbe3f3589d117b55
Delegated to: Takashi Iwai
Headers show

Commit Message

David Henningsson Oct. 24, 2014, 8 a.m. UTC
Without this terminating entry, the pin matching would continue
across random memory until a zero or a non-matching entry was found.

The result being that in some cases, the pin quirk would not be
applied correctly.

Cc: stable@vger.kernel.org
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
---
 sound/pci/hda/hda_local.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Takashi Iwai Oct. 24, 2014, 8:06 a.m. UTC | #1
At Fri, 24 Oct 2014 10:00:38 +0200,
David Henningsson wrote:
> 
> Without this terminating entry, the pin matching would continue
> across random memory until a zero or a non-matching entry was found.
> 
> The result being that in some cases, the pin quirk would not be
> applied correctly.
> 
> Cc: stable@vger.kernel.org
> Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Thanks, applied.


Takashi

> ---
>  sound/pci/hda/hda_local.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/sound/pci/hda/hda_local.h b/sound/pci/hda/hda_local.h
> index 7eb44e7..62658f2 100644
> --- a/sound/pci/hda/hda_local.h
> +++ b/sound/pci/hda/hda_local.h
> @@ -419,7 +419,7 @@ struct snd_hda_pin_quirk {
>  	  .subvendor = _subvendor,\
>  	  .name = _name,\
>  	  .value = _value,\
> -	  .pins = (const struct hda_pintbl[]) { _pins } \
> +	  .pins = (const struct hda_pintbl[]) { _pins, {0, 0}} \
>  	}
>  #else
>  
> @@ -427,7 +427,7 @@ struct snd_hda_pin_quirk {
>  	{ .codec = _codec,\
>  	  .subvendor = _subvendor,\
>  	  .value = _value,\
> -	  .pins = (const struct hda_pintbl[]) { _pins } \
> +	  .pins = (const struct hda_pintbl[]) { _pins, {0, 0}} \
>  	}
>  
>  #endif
> -- 
> 1.9.1
>
diff mbox

Patch

diff --git a/sound/pci/hda/hda_local.h b/sound/pci/hda/hda_local.h
index 7eb44e7..62658f2 100644
--- a/sound/pci/hda/hda_local.h
+++ b/sound/pci/hda/hda_local.h
@@ -419,7 +419,7 @@  struct snd_hda_pin_quirk {
 	  .subvendor = _subvendor,\
 	  .name = _name,\
 	  .value = _value,\
-	  .pins = (const struct hda_pintbl[]) { _pins } \
+	  .pins = (const struct hda_pintbl[]) { _pins, {0, 0}} \
 	}
 #else
 
@@ -427,7 +427,7 @@  struct snd_hda_pin_quirk {
 	{ .codec = _codec,\
 	  .subvendor = _subvendor,\
 	  .value = _value,\
-	  .pins = (const struct hda_pintbl[]) { _pins } \
+	  .pins = (const struct hda_pintbl[]) { _pins, {0, 0}} \
 	}
 
 #endif