* Re: [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero
2014-03-11 21:12 [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero mengdong.lin
@ 2014-03-11 8:16 ` Takashi Iwai
2014-07-22 6:52 ` David Henningsson
1 sibling, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2014-03-11 8:16 UTC (permalink / raw)
To: mengdong.lin; +Cc: alsa-devel
At Tue, 11 Mar 2014 17:12:52 -0400,
mengdong.lin@intel.com wrote:
>
> From: Mengdong Lin <mengdong.lin@intel.com>
>
> This patch initialized the local audio InfoFrame variable 'ai' to be all zero,
> thus the data bytes will indicate "Refer to Stream Header" by default.
>
> Signed-off-by: Mengdong Lin <mengdong.lin@intel.com>
Thanks, applied.
Takashi
>
> diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
> index 5ef9503..3ab7063 100644
> --- a/sound/pci/hda/patch_hdmi.c
> +++ b/sound/pci/hda/patch_hdmi.c
> @@ -1062,6 +1062,7 @@ static void hdmi_pin_setup_infoframe(struct hda_codec *codec,
> {
> union audio_infoframe ai;
>
> + memset(&ai, 0, sizeof(ai));
> if (conn_type == 0) { /* HDMI */
> struct hdmi_audio_infoframe *hdmi_ai = &ai.hdmi;
>
> --
> 1.8.1.2
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero
@ 2014-03-11 21:12 mengdong.lin
2014-03-11 8:16 ` Takashi Iwai
2014-07-22 6:52 ` David Henningsson
0 siblings, 2 replies; 6+ messages in thread
From: mengdong.lin @ 2014-03-11 21:12 UTC (permalink / raw)
To: alsa-devel, tiwai; +Cc: Mengdong Lin
From: Mengdong Lin <mengdong.lin@intel.com>
This patch initialized the local audio InfoFrame variable 'ai' to be all zero,
thus the data bytes will indicate "Refer to Stream Header" by default.
Signed-off-by: Mengdong Lin <mengdong.lin@intel.com>
diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
index 5ef9503..3ab7063 100644
--- a/sound/pci/hda/patch_hdmi.c
+++ b/sound/pci/hda/patch_hdmi.c
@@ -1062,6 +1062,7 @@ static void hdmi_pin_setup_infoframe(struct hda_codec *codec,
{
union audio_infoframe ai;
+ memset(&ai, 0, sizeof(ai));
if (conn_type == 0) { /* HDMI */
struct hdmi_audio_infoframe *hdmi_ai = &ai.hdmi;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero
2014-03-11 21:12 [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero mengdong.lin
2014-03-11 8:16 ` Takashi Iwai
@ 2014-07-22 6:52 ` David Henningsson
2014-07-22 7:01 ` [alsa-devel] " Takashi Iwai
1 sibling, 1 reply; 6+ messages in thread
From: David Henningsson @ 2014-07-22 6:52 UTC (permalink / raw)
To: mengdong.lin, alsa-devel, tiwai
Hi,
It looks like this patch should have been sent to stable, is there any
reason why it wasn't?
On 2014-03-11 22:12, mengdong.lin@intel.com wrote:
> From: Mengdong Lin <mengdong.lin@intel.com>
>
> This patch initialized the local audio InfoFrame variable 'ai' to be all zero,
> thus the data bytes will indicate "Refer to Stream Header" by default.
>
> Signed-off-by: Mengdong Lin <mengdong.lin@intel.com>
>
> diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
> index 5ef9503..3ab7063 100644
> --- a/sound/pci/hda/patch_hdmi.c
> +++ b/sound/pci/hda/patch_hdmi.c
> @@ -1062,6 +1062,7 @@ static void hdmi_pin_setup_infoframe(struct hda_codec *codec,
> {
> union audio_infoframe ai;
>
> + memset(&ai, 0, sizeof(ai));
> if (conn_type == 0) { /* HDMI */
> struct hdmi_audio_infoframe *hdmi_ai = &ai.hdmi;
>
>
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [alsa-devel] [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero
2014-07-22 6:52 ` David Henningsson
@ 2014-07-22 7:01 ` Takashi Iwai
2014-07-23 0:08 ` Greg KH
0 siblings, 1 reply; 6+ messages in thread
From: Takashi Iwai @ 2014-07-22 7:01 UTC (permalink / raw)
To: Greg KH; +Cc: David Henningsson, mengdong.lin, alsa-devel, stable
At Tue, 22 Jul 2014 08:52:31 +0200,
David Henningsson wrote:
>
> Hi,
>
> It looks like this patch should have been sent to stable, is there any
> reason why it wasn't?
Just forgotten, I suppose. This should be included in stable, indeed.
Greg, could you cherry-pick this upstream commit
caaf5ef9493f72390905f1e97b310b8906d32dac
ALSA: hda - initialize audio InfoFrame to be all zero
to stable kernels?
thanks,
Takashi
>
> On 2014-03-11 22:12, mengdong.lin@intel.com wrote:
> > From: Mengdong Lin <mengdong.lin@intel.com>
> >
> > This patch initialized the local audio InfoFrame variable 'ai' to be all zero,
> > thus the data bytes will indicate "Refer to Stream Header" by default.
> >
> > Signed-off-by: Mengdong Lin <mengdong.lin@intel.com>
> >
> > diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
> > index 5ef9503..3ab7063 100644
> > --- a/sound/pci/hda/patch_hdmi.c
> > +++ b/sound/pci/hda/patch_hdmi.c
> > @@ -1062,6 +1062,7 @@ static void hdmi_pin_setup_infoframe(struct hda_codec *codec,
> > {
> > union audio_infoframe ai;
> >
> > + memset(&ai, 0, sizeof(ai));
> > if (conn_type == 0) { /* HDMI */
> > struct hdmi_audio_infoframe *hdmi_ai = &ai.hdmi;
> >
> >
>
> --
> David Henningsson, Canonical Ltd.
> https://launchpad.net/~diwic
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [alsa-devel] [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero
2014-07-22 7:01 ` [alsa-devel] " Takashi Iwai
@ 2014-07-23 0:08 ` Greg KH
2014-07-23 10:19 ` Takashi Iwai
0 siblings, 1 reply; 6+ messages in thread
From: Greg KH @ 2014-07-23 0:08 UTC (permalink / raw)
To: Takashi Iwai; +Cc: David Henningsson, mengdong.lin, alsa-devel, stable
On Tue, Jul 22, 2014 at 09:01:51AM +0200, Takashi Iwai wrote:
> At Tue, 22 Jul 2014 08:52:31 +0200,
> David Henningsson wrote:
> >
> > Hi,
> >
> > It looks like this patch should have been sent to stable, is there any
> > reason why it wasn't?
>
> Just forgotten, I suppose. This should be included in stable, indeed.
>
> Greg, could you cherry-pick this upstream commit
> caaf5ef9493f72390905f1e97b310b8906d32dac
> ALSA: hda - initialize audio InfoFrame to be all zero
>
> to stable kernels?
Only seemd to apply to 3.14-stable, so I put it there.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [alsa-devel] [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero
2014-07-23 0:08 ` Greg KH
@ 2014-07-23 10:19 ` Takashi Iwai
0 siblings, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2014-07-23 10:19 UTC (permalink / raw)
To: Greg KH; +Cc: David Henningsson, mengdong.lin, alsa-devel, stable
At Tue, 22 Jul 2014 17:08:52 -0700,
Greg KH wrote:
>
> On Tue, Jul 22, 2014 at 09:01:51AM +0200, Takashi Iwai wrote:
> > At Tue, 22 Jul 2014 08:52:31 +0200,
> > David Henningsson wrote:
> > >
> > > Hi,
> > >
> > > It looks like this patch should have been sent to stable, is there any
> > > reason why it wasn't?
> >
> > Just forgotten, I suppose. This should be included in stable, indeed.
> >
> > Greg, could you cherry-pick this upstream commit
> > caaf5ef9493f72390905f1e97b310b8906d32dac
> > ALSA: hda - initialize audio InfoFrame to be all zero
> >
> > to stable kernels?
>
> Only seemd to apply to 3.14-stable, so I put it there.
Fair enough. The only missing would be 3.13, so Canonical should take
care of it.
Thanks!
Takashi
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-23 10:19 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-11 21:12 [PATCH] ALSA: hda - initialize audio InfoFrame to be all zero mengdong.lin
2014-03-11 8:16 ` Takashi Iwai
2014-07-22 6:52 ` David Henningsson
2014-07-22 7:01 ` [alsa-devel] " Takashi Iwai
2014-07-23 0:08 ` Greg KH
2014-07-23 10:19 ` Takashi Iwai
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).