[alsa-devel] [PATCH] add the nvidia HDMI codec driverfor MCP77/79

[Wei Ni]

Hi, Takashi
I use git to generate patches. (I run git-fetch origin, git-fetch-rebase
origin, and git-format-patch origin)

There are two patch, one for the addition of HDMI, and one for the fix
for our aza controller.

Please check it.

Thanks

-----Original Message-----
From: Takashi Iwai [mailto:tiwai at suse.de] 
Sent: Monday, September 22, 2008 5:58 PM
To: Wei Ni
Cc: peerchen; Pavel Hofman; Peer Chen; alsa-devel; linux-kernel; akpm
Subject: Re: [alsa-devel] [PATCH] add the nvidia HDMI codec driverfor
MCP77/79

At Mon, 22 Sep 2008 14:33:58 +0800,
Wei Ni wrote:
> 
> Hi, Takashi
> 
> About the "long delay", it seemed that we need to add this "delay" to
> fix some issue on our aza controller.
> Because it related with the controller, not just codec, so I think
this
> flag should not be added in codec-initialization part.
> I think it's better to add the flag in that place which I submitted.

OK, then could you split the patch to separate ones, i.e. one for the
fix for this problem regarding Nvidia controller chip, and one for the
addition of HDMI?

Don't forget to add a proper description for each patch, please ;)

As mentioned, that flag is a last resort, and I think it can be fixed
in a saner way.  But, the flag is surely safe, and we can track down
the problem later on.


thanks,

Takashi


> 
> Thanks
> 
> -----Original Message-----
> From: Wei Ni 
> Sent: Friday, September 19, 2008 12:43 PM
> To: 'Takashi Iwai'; peerchen
> Cc: Pavel Hofman; Peer Chen; alsa-devel; linux-kernel; akpm
> Subject: RE: [alsa-devel] [PATCH] add the nvidia HDMI codec driverfor
> MCP77/79
> 
> Thanks for your reply.
> 
> About the "long delay", we had some problems on reading RIRB buffer
when
> testing this driver, so we add this "delay"
> We will try to find a best way to fix this issue.
> 
> And we didn't familiar with git, we will research it more such as
> "rebase" and then submit the patch file again.
> 
> Thanks
> 
> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai at suse.de] 
> Sent: Wednesday, September 17, 2008 10:05 PM
> To: peerchen
> Cc: Wei Ni; Pavel Hofman; Peer Chen; alsa-devel; linux-kernel; akpm
> Subject: Re: [alsa-devel] [PATCH] add the nvidia HDMI codec driverfor
> MCP77/79
> 
> At Wed, 17 Sep 2008 17:00:04 +0800,
> peerchen wrote:
> > 
> > new patch base on git tree.
> > 
> > Signed-off-by: Wei Ni <wni at nvidia.com>
> > Signed-off-by: Peer Chen <peerchen at gmail.com>
> 
> Thanks!
> 
> But, the patch looks like no "rebase".  You secretly added a new
> function call that wasn't in your previous patch, namely...
> 
> > diff -uprN -X linux-2.6-tiwai-sound/Documentation/dontdiff
> linux-2.6-tiwai-sound/sound/pci/hda/hda_intel.c
> linux-2.6-tiwai-sound-niwei/sound/pci/hda/hda_intel.c
> > --- linux-2.6-tiwai-sound/sound/pci/hda/hda_intel.c	2008-09-10
> 17:40:52.000000000 +0800
> > +++ linux-2.6-tiwai-sound-niwei/sound/pci/hda/hda_intel.c
> 2008-09-10 17:49:31.000000000 +0800
> > @@ -1220,6 +1220,9 @@ static int __devinit azx_codec_create(st
> >  	if (err < 0)
> >  		return err;
> >  
> > +	if (chip->driver_type == AZX_DRIVER_NVIDIA)
> > +		chip->bus->needs_damn_long_delay = 1;
> > +
> 
> Do we really need this inevitably?  I don't think so -- otherwise I
> would have far more bug reports.
> 
> This flag is the last resort and should be avoided as much as
> possible.  This results in a significant slow down of suspend/resume
> speed, for example.
> 
> If it's really needed with some devices and there is no other way to
> fix it, add this flag rather in the codec-initialization part.
> 
> Also, if you resend a patch, please add the original patch description

> again.  This helps my patch work a lot indeed.
> 
> 
> thanks,
> 
> Takashi
>
------------------------------------------------------------------------
-----------
> This email message is for the sole use of the intended recipient(s)
and may contain
> confidential information.  Any unauthorized review, use, disclosure or
distribution
> is prohibited.  If you are not the intended recipient, please contact
the sender by
> reply email and destroy all copies of the original message.
>
------------------------------------------------------------------------
-----------
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Support-NVIDIA-MCP78-HDMI-audio.patch
Type: application/octet-stream
Size: 7215 bytes
Desc: 0001-Support-NVIDIA-MCP78-HDMI-audio.patch
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20080925/571be7cf/attachment.obj 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Fix-for-reading-RIRB-buffer-on-NVIDIA-aza-controller.patch
Type: application/octet-stream
Size: 790 bytes
Desc: 0002-Fix-for-reading-RIRB-buffer-on-NVIDIA-aza-controller.patch
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20080925/571be7cf/attachment.dll