Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1536959

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

Show all headers | View raw

Hi Shuah,

On Monday 05 Dec 2016 17:38:23 Shuah Khan wrote:
> On 12/05/2016 04:21 PM, Laurent Pinchart wrote:
> > On Monday 05 Dec 2016 15:44:30 Shuah Khan wrote:
> >> On 11/30/2016 03:01 PM, Shuah Khan wrote:
> >>> Change ALSA driver to use Media Controller API to share media resources
> >>> with DVB, and V4L2 drivers on a AU0828 media device.
> >>> 
> >>> Media Controller specific initialization is done after sound card is
> >>> registered. ALSA creates Media interface and entity function graph
> >>> nodes for Control, Mixer, PCM Playback, and PCM Capture devices.
> >>> 
> >>> snd_usb_hw_params() will call Media Controller enable source handler
> >>> interface to request the media resource. If resource request is granted,
> >>> it will release it from snd_usb_hw_free(). If resource is busy, -EBUSY
> >>> is returned.
> >>> 
> >>> Media specific cleanup is done in usb_audio_disconnect().
> >>> 
> >>> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
> >> 
> >> Hi Takashi,
> >> 
> >> If you are good with this patch, could you please Ack it, so Mauro
> >> can pull it into media tree with the other two patches in this series,
> >> when he is ready to do so.
> > 
> > I *really* want to address the concerns raised by Sakari before pulling
> > more code that makes fixing the race conditions more difficult. Please,
> > let's all work on fixing the core code to build a stable base on which we
> > can build additional features. V4L2 and MC need teamwork, it's time to
> > give the subsystem the love it deserves.
> 
> Hi Laurent,
> 
> The issue Sakari brought up is specific to using devm for video_device in
> omap3 and vsp1. I tried reproducing the problem on two different drivers
> and couldn't on Linux 4.9-rc7.
> 
> After sharing that with Sakari, I suggested to Sakari to pull up his patch
> that removes the devm usage and see if he still needs all the patches in his
> patch series. He didn't back to me on that. I also requested him to rebase
> on top of media dev allocator because the allocator routines he has don't
> address the shared media device need.
> 
> He also didn't respond to my response regarding the reasons for choosing
> graph_mutex to protect enable_source and disable_source handlers.
> 
> So I am not sure how to move forward at the moment without a concrete plan
> for Sakari's RFC series. Sakari's patch series is still RFC and doesn't
> address shared media_device and requires all drivers to change.

Today is a public holiday in Finland, I don't expect Sakari to be available. 
Let's check this with him tomorrow.

-- 
Regards,

Laurent Pinchart

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v6 0/3] Media Device Allocator API Shuah Khan <shuahkh@osg.samsung.com> - 2016-11-30 23:10 +0100
  [PATCH v6 2/3] media: change au0828 to use Media Device Allocator API Shuah Khan <shuahkh@osg.samsung.com> - 2016-11-30 23:10 +0100
  [PATCH v6 1/3] media: Media Device Allocator API Shuah Khan <shuahkh@osg.samsung.com> - 2016-11-30 23:10 +0100
  [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-11-30 23:10 +0100
    Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media  resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-12-05 23:50 +0100
      Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2016-12-06 00:30 +0100
        Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media  resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-12-06 01:40 +0100
          Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2016-12-06 14:20 +0100
          Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share  media resources Sakari Ailus <sakari.ailus@iki.fi> - 2016-12-07 12:00 +0100
            Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media  resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-12-07 21:10 +0100
              Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share  media resources Sakari Ailus <sakari.ailus@iki.fi> - 2016-12-07 23:30 +0100
                Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media  resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-12-08 15:50 +0100
                Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share  media resources Sakari Ailus <sakari.ailus@iki.fi> - 2016-12-09 00:50 +0100
                Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share  media resources Sakari Ailus <sakari.ailus@iki.fi> - 2016-12-09 14:20 +0100
                Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media  resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-12-09 17:50 +0100
      Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources Takashi Iwai <tiwai@suse.de> - 2016-12-06 08:20 +0100
    Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources Takashi Iwai <tiwai@suse.de> - 2016-12-06 08:00 +0100
      Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media  resources Shuah Khan <shuahkh@osg.samsung.com> - 2016-12-06 19:50 +0100
        Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources Takashi Iwai <tiwai@suse.de> - 2016-12-06 20:10 +0100

csiph-web