Re: [RFC PATCH 0/2] hdmi audio dynamically bind PCM to pin

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

 



> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai@xxxxxxx]
> Sent: Wednesday, November 18, 2015 7:24 PM
> To: libin.yang@xxxxxxxxxxxxxxx
> Cc: alsa-devel@xxxxxxxxxxxxxxxx; mengdong.lin@xxxxxxxxxxxxxxx; Yang,
> Libin
> Subject: Re:  [RFC PATCH 0/2] hdmi audio dynamically bind
> PCM to pin
> 
> On Wed, 18 Nov 2015 09:46:57 +0100,
> libin.yang@xxxxxxxxxxxxxxx wrote:
> >
> > From: Libin Yang <libin.yang@xxxxxxxxxxxxxxx>
> >
> > The patches are trying to support dynamically binding PCM to pin in
> HDMI audio.
> >
> > The first patch:
> > When PCM is open when there is no pin (port) bound to the PCM,
> > the driver try to open the PCM successfully. It will try to find
> > an available converter to assign to the PCM.
> > In PCM prepare, if there is no pin bound to the PCM, the driver
> > will do the simple configuration and return true.
> > In PCM close, if there is no pin bound to the PCM, the driver will
> > simply close it.
> > All the pin configuration will be setup in the hotplug which is done
> > in patch 2.
> > As the PCM is dynamically bound to the pin, the pcm_lock is necessary
> > for the whole pcm open/prepare/close process.
> 
> Such a detailed explanation should be put into the patch itself, not
> in the cover letter.  The current description is way too sparse.

OK. I see.

Regards,
Libin
> 
> 
> Takashi
> 
> > The second patch:
> > The patch mainly handle the hotplug event.
> > Each time monitor is connected, a proper PCM will be bound to the pin.
> > If the PCM is in use, the pin must be configured.
> > Each time monitor is disconnect, the PCM will be unbound from the pin.
> > The pin will be reset.
> > Libin Yang (2):
> >   ALSA: hda - hdmi playback without monitor in dynamic pcm bind
> mode
> >   ALSA: hda - hdmi dynamically bind PCM to pin when monitor hotplug
> >
> >  sound/pci/hda/hda_codec.h  |   1 +
> >  sound/pci/hda/patch_hdmi.c | 354
> ++++++++++++++++++++++++++++++++++++++++++---
> >  2 files changed, 337 insertions(+), 18 deletions(-)
> >
> > --
> > 1.9.1
> >
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



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

  Powered by Linux