From: Mauro Carvalho Chehab <mchehab@osg.samsung.com> To: Shuah Khan <shuahkh@osg.samsung.com> Cc: tiwai@suse.com, clemens@ladisch.de, hans.verkuil@cisco.com, laurent.pinchart@ideasonboard.com, sakari.ailus@linux.intel.com, javier@osg.samsung.com, pawel@osciak.com, m.szyprowski@samsung.com, kyungmin.park@samsung.com, perex@perex.cz, arnd@arndb.de, dan.carpenter@oracle.com, tvboxspy@gmail.com, crope@iki.fi, ruchandani.tina@gmail.com, corbet@lwn.net, chehabrafael@gmail.com, k.kozlowski@samsung.com, stefanr@s5r6.in-berlin.de, inki.dae@samsung.com, jh1009.sung@samsung.com, elfring@users.sourceforge.net, prabhakar.csengg@gmail.com, sw0312.kim@samsung.com, p.zabel@pengutronix.de, ricardo.ribalda@gmail.com, labbott@fedoraproject.org, pierre-louis.bossart@linux.intel.com, ricard.wanderlof@axis.com, julian@jusst.de, takamichiho@gmail.com, dominic.sacre@gmx.de, misterpib@gmail.com, daniel@zonque.org, gtmkramer@xs4all.nl, normalperson@yhbt.net, joe@oampo.co.uk, linuxbugs@vittgam.net, johan@oljud.se, klock.android@gmail.com, nenggun.kim@samsung.com, j.anaszewski@samsung.com, geliangtang@163.com, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [PATCH v2 20/22] media: au0828 add enable, disable source handlers Date: Tue, 9 Feb 2016 08:51:57 -0200 [thread overview] Message-ID: <20160209085157.06836b58@recife.lan> (raw) In-Reply-To: <56B919C7.80801@osg.samsung.com> Em Mon, 08 Feb 2016 15:42:15 -0700 Shuah Khan <shuahkh@osg.samsung.com> escreveu: > On 02/04/2016 03:26 AM, Mauro Carvalho Chehab wrote: > > Em Wed, 03 Feb 2016 21:03:52 -0700 > > Shuah Khan <shuahkh@osg.samsung.com> escreveu: > > > >> Add enable_source and disable_source handlers. > >> The enable source handler is called from > >> v4l2-core, dvb-core, and ALSA drivers to check > >> if the shared media source is free. The disable > >> source handler is called to release the shared > >> media source. > >> > >> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com> > >> --- > >> drivers/media/usb/au0828/au0828-core.c | 149 +++++++++++++++++++++++++++++++++ > >> drivers/media/usb/au0828/au0828.h | 3 + > >> 2 files changed, 152 insertions(+) > >> > >> diff --git a/drivers/media/usb/au0828/au0828-core.c b/drivers/media/usb/au0828/au0828-core.c > >> index 4c90f28..fd2265c 100644 > >> --- a/drivers/media/usb/au0828/au0828-core.c > >> +++ b/drivers/media/usb/au0828/au0828-core.c > >> @@ -282,6 +282,7 @@ static int au0828_create_media_graph(struct au0828_dev *dev) > >> return -EINVAL; > >> > >> if (tuner) { > >> + dev->tuner = tuner; > >> /* create tuner to decoder link in deactivated state */ > >> ret = media_create_pad_link(tuner, TUNER_PAD_OUTPUT, > >> decoder, 0, 0); > >> @@ -373,6 +374,150 @@ void au0828_media_graph_notify(struct media_entity *new, void *notify_data) > >> #endif > >> } > >> > >> +static int au0828_enable_source(struct media_entity *entity, > >> + struct media_pipeline *pipe) > >> +{ > >> +#ifdef CONFIG_MEDIA_CONTROLLER > >> + struct media_entity *source; > >> + struct media_entity *sink; > >> + struct media_link *link, *found_link = NULL; > >> + int ret = 0; > >> + struct media_device *mdev = entity->graph_obj.mdev; > >> + struct au0828_dev *dev; > >> + > >> + if (!mdev) > >> + return -ENODEV; > >> + > >> + /* for Audio and Video entities, source is the decoder */ > >> + mutex_lock(&mdev->graph_mutex); > >> + > >> + dev = mdev->source_priv; > >> + if (!dev->tuner || !dev->decoder) { > >> + ret = -ENODEV; > >> + goto end; > >> + } > > > > This is wrong. There are devices without tuner (capture devices) and > > without analog decoder (pure DVB devices). > > Removed linux-api from the list. > > Yes this logic is making an assumption that both > decoder and tuner are present. Based on your comment > here, is the following check for decoder in > au0828_create_media_graph() incorrect? When decoder > is null, au0828_usb_probe() bails out. Please see > au0828_create_media_graph() return handling in > au0828_usb_probe()? It seems so. at its current state, au0828 always register a V4L2 node, except if !CONFIG_VIDEO_AU0828_V4L2. So, it is missing a #ifdef CONFIG_VIDEO_AU0828_V4L2 inside it. I'll write such fixup. > > > /* Something bad happened! */ > if (!decoder) > return -EINVAL; > > > > > In the case of pure DVB devices (e. g. no dev->decoder), it should > > just enable the DVB path. > > > > In the case of devices without tuner, it should use the same logic > > needed to handle the S-Video/Composite connector inputs. > > > > Btw, I'm not seeing how this logic would do the right thing if the user > > selects either S-Video or Composite connectors. > > > >> + > >> + /* > >> + * For Audio and V4L2 entity, find the link to which decoder > >> + * is the sink. Look for an active link between decoder and > >> + * tuner, if one exists, nothing to do. If not, look for any > >> + * active links between tuner and any other entity. If one > >> + * exists, tuner is busy. If tuner is free, setup link and > >> + * start pipeline from source (tuner). > >> + * For DVB FE entity, the source for the link is the tuner. > >> + * Check if tuner is available and setup link and start > >> + * pipeline. > >> + */ > >> + if (entity->function != MEDIA_ENT_F_DTV_DEMOD) > >> + sink = dev->decoder; > >> + else > >> + sink = entity; > >> + > >> + /* Is an active link between sink and tuner */ > >> + if (dev->active_link) { > >> + if (dev->active_link->sink->entity == sink && > >> + dev->active_link->source->entity == dev->tuner) { > >> + ret = 0; > >> + goto end; > >> + } else { > >> + ret = -EBUSY; > >> + goto end; > >> + } > >> + } > >> + > >> + list_for_each_entry(link, &sink->links, list) { > >> + /* Check sink, and source */ > >> + if (link->sink->entity == sink && > >> + link->source->entity == dev->tuner) { > >> + found_link = link; > >> + break; > >> + } > >> + } > >> + > >> + if (!found_link) { > >> + ret = -ENODEV; > >> + goto end; > >> + } > >> + > >> + /* activate link between source and sink and start pipeline */ > >> + source = found_link->source->entity; > >> + ret = __media_entity_setup_link(found_link, MEDIA_LNK_FL_ENABLED); > >> + if (ret) { > >> + pr_err( > >> + "Activate tuner link %s->%s. Error %d\n", > >> + source->name, sink->name, ret); > >> + goto end; > >> + } > >> + > >> + ret = __media_entity_pipeline_start(entity, pipe); > >> + if (ret) { > >> + pr_err("Start Pipeline: %s->%s Error %d\n", > >> + source->name, entity->name, ret); > >> + ret = __media_entity_setup_link(found_link, 0); > >> + pr_err("Deactive link Error %d\n", ret); > >> + goto end; > >> + } > > > > Hmm... isn't it to early to activate the pipeline here? My original > > guess is that, on the analog side, this should happen only at the stream > > on code. Wouldn't this break apps like mythTV? > > > >> + /* > >> + * save active link and active link owner to avoid audio > >> + * deactivating video owned link from disable_source and > >> + * vice versa > >> + */ > >> + dev->active_link = found_link; > >> + dev->active_link_owner = entity; > >> +end: > >> + mutex_unlock(&mdev->graph_mutex); > >> + pr_debug("au0828_enable_source() end %s %d %d\n", > >> + entity->name, entity->function, ret); > >> + return ret; > >> +#endif > >> + return 0; > >> +} > >> + > >> +static void au0828_disable_source(struct media_entity *entity) > >> +{ > >> +#ifdef CONFIG_MEDIA_CONTROLLER > >> + struct media_entity *sink; > >> + int ret = 0; > >> + struct media_device *mdev = entity->graph_obj.mdev; > >> + struct au0828_dev *dev; > >> + > >> + if (!mdev) > >> + return; > >> + > >> + mutex_lock(&mdev->graph_mutex); > >> + dev = mdev->source_priv; > >> + if (!dev->tuner || !dev->decoder || !dev->active_link) { > >> + ret = -ENODEV; > >> + goto end; > >> + } > > > > Same note as before. > > Same comment as before here about au0828_create_media_graph() > and au0828_usb_probe() handling. > > > > >> + > >> + if (entity->function != MEDIA_ENT_F_DTV_DEMOD) > >> + sink = dev->decoder; > >> + else > >> + sink = entity; > >> + > >> + /* link is active - stop pipeline from source (tuner) */ > >> + if (dev->active_link && dev->active_link->sink->entity == sink && > >> + dev->active_link->source->entity == dev->tuner) { > >> + /* > >> + * prevent video from deactivating link when audio > >> + * has active pipeline > >> + */ > >> + if (dev->active_link_owner != entity) > >> + goto end; > >> + __media_entity_pipeline_stop(entity); > >> + ret = __media_entity_setup_link(dev->active_link, 0); > >> + if (ret) > >> + pr_err("Deactive link Error %d\n", ret); > >> + dev->active_link = NULL; > >> + dev->active_link_owner = NULL; > >> + } > > > > Most code here looks like the one at au0828_enable_source(). Wouldn't > > be simpler to merge those code and add a "bool enable" to the function > > parameters? > > I would rather keep these separate. A very short > section is common really. > > thanks, > -- Shuah > > > > >> + > >> +end: > >> + mutex_unlock(&mdev->graph_mutex); > >> +#endif > >> +} > >> + > >> static int au0828_media_device_register(struct au0828_dev *dev, > >> struct usb_device *udev) > >> { > >> @@ -403,6 +548,10 @@ static int au0828_media_device_register(struct au0828_dev *dev, > >> ret); > >> return ret; > >> } > >> + /* set enable_source */ > >> + dev->media_dev->source_priv = (void *) dev; > >> + dev->media_dev->enable_source = au0828_enable_source; > >> + dev->media_dev->disable_source = au0828_disable_source; > >> #endif > >> return 0; > >> } > >> diff --git a/drivers/media/usb/au0828/au0828.h b/drivers/media/usb/au0828/au0828.h > >> index 54379ec..a7c88a1 100644 > >> --- a/drivers/media/usb/au0828/au0828.h > >> +++ b/drivers/media/usb/au0828/au0828.h > >> @@ -284,6 +284,9 @@ struct au0828_dev { > >> struct media_entity input_ent[AU0828_MAX_INPUT]; > >> struct media_pad input_pad[AU0828_MAX_INPUT]; > >> struct media_entity_notify entity_notify; > >> + struct media_entity *tuner; > >> + struct media_link *active_link; > >> + struct media_entity *active_link_owner; > >> #endif > >> }; > >> > >
WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <mchehab@osg.samsung.com> To: Shuah Khan <shuahkh@osg.samsung.com> Cc: tiwai@suse.com, clemens@ladisch.de, hans.verkuil@cisco.com, laurent.pinchart@ideasonboard.com, sakari.ailus@linux.intel.com, javier@osg.samsung.com, pawel@osciak.com, m.szyprowski@samsung.com, kyungmin.park@samsung.com, perex@perex.cz, arnd@arndb.de, dan.carpenter@oracle.com, tvboxspy@gmail.com, crope@iki.fi, ruchandani.tina@gmail.com, corbet@lwn.net, chehabrafael@gmail.com, k.kozlowski@samsung.com, stefanr@s5r6.in-berlin.de, inki.dae@samsung.com, jh1009.sung@samsung.com, elfring@users.sourceforge.net, prabhakar.csengg@gmail.com, sw0312.kim@samsung.com, p.zabel@pengutronix.de, ricardo.ribalda@gmail.com, labbott@fedoraproject.org, pierre-louis.bossart@linux.intel.com, ricard.wanderlof@axis.com, julian@jusst.de, takamichiho@gmail.com, dominic.sacre@gmx.de, misterpib@gmail.com, daniel@zonque.org, gtmkramer@xs4all.nl, normalperson@yhbt.net, joe@oampo.co.uk, linuxbugs@vittga Subject: Re: [PATCH v2 20/22] media: au0828 add enable, disable source handlers Date: Tue, 9 Feb 2016 08:51:57 -0200 [thread overview] Message-ID: <20160209085157.06836b58@recife.lan> (raw) In-Reply-To: <56B919C7.80801@osg.samsung.com> Em Mon, 08 Feb 2016 15:42:15 -0700 Shuah Khan <shuahkh@osg.samsung.com> escreveu: > On 02/04/2016 03:26 AM, Mauro Carvalho Chehab wrote: > > Em Wed, 03 Feb 2016 21:03:52 -0700 > > Shuah Khan <shuahkh@osg.samsung.com> escreveu: > > > >> Add enable_source and disable_source handlers. > >> The enable source handler is called from > >> v4l2-core, dvb-core, and ALSA drivers to check > >> if the shared media source is free. The disable > >> source handler is called to release the shared > >> media source. > >> > >> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com> > >> --- > >> drivers/media/usb/au0828/au0828-core.c | 149 +++++++++++++++++++++++++++++++++ > >> drivers/media/usb/au0828/au0828.h | 3 + > >> 2 files changed, 152 insertions(+) > >> > >> diff --git a/drivers/media/usb/au0828/au0828-core.c b/drivers/media/usb/au0828/au0828-core.c > >> index 4c90f28..fd2265c 100644 > >> --- a/drivers/media/usb/au0828/au0828-core.c > >> +++ b/drivers/media/usb/au0828/au0828-core.c > >> @@ -282,6 +282,7 @@ static int au0828_create_media_graph(struct au0828_dev *dev) > >> return -EINVAL; > >> > >> if (tuner) { > >> + dev->tuner = tuner; > >> /* create tuner to decoder link in deactivated state */ > >> ret = media_create_pad_link(tuner, TUNER_PAD_OUTPUT, > >> decoder, 0, 0); > >> @@ -373,6 +374,150 @@ void au0828_media_graph_notify(struct media_entity *new, void *notify_data) > >> #endif > >> } > >> > >> +static int au0828_enable_source(struct media_entity *entity, > >> + struct media_pipeline *pipe) > >> +{ > >> +#ifdef CONFIG_MEDIA_CONTROLLER > >> + struct media_entity *source; > >> + struct media_entity *sink; > >> + struct media_link *link, *found_link = NULL; > >> + int ret = 0; > >> + struct media_device *mdev = entity->graph_obj.mdev; > >> + struct au0828_dev *dev; > >> + > >> + if (!mdev) > >> + return -ENODEV; > >> + > >> + /* for Audio and Video entities, source is the decoder */ > >> + mutex_lock(&mdev->graph_mutex); > >> + > >> + dev = mdev->source_priv; > >> + if (!dev->tuner || !dev->decoder) { > >> + ret = -ENODEV; > >> + goto end; > >> + } > > > > This is wrong. There are devices without tuner (capture devices) and > > without analog decoder (pure DVB devices). > > Removed linux-api from the list. > > Yes this logic is making an assumption that both > decoder and tuner are present. Based on your comment > here, is the following check for decoder in > au0828_create_media_graph() incorrect? When decoder > is null, au0828_usb_probe() bails out. Please see > au0828_create_media_graph() return handling in > au0828_usb_probe()? It seems so. at its current state, au0828 always register a V4L2 node, except if !CONFIG_VIDEO_AU0828_V4L2. So, it is missing a #ifdef CONFIG_VIDEO_AU0828_V4L2 inside it. I'll write such fixup. > > > /* Something bad happened! */ > if (!decoder) > return -EINVAL; > > > > > In the case of pure DVB devices (e. g. no dev->decoder), it should > > just enable the DVB path. > > > > In the case of devices without tuner, it should use the same logic > > needed to handle the S-Video/Composite connector inputs. > > > > Btw, I'm not seeing how this logic would do the right thing if the user > > selects either S-Video or Composite connectors. > > > >> + > >> + /* > >> + * For Audio and V4L2 entity, find the link to which decoder > >> + * is the sink. Look for an active link between decoder and > >> + * tuner, if one exists, nothing to do. If not, look for any > >> + * active links between tuner and any other entity. If one > >> + * exists, tuner is busy. If tuner is free, setup link and > >> + * start pipeline from source (tuner). > >> + * For DVB FE entity, the source for the link is the tuner. > >> + * Check if tuner is available and setup link and start > >> + * pipeline. > >> + */ > >> + if (entity->function != MEDIA_ENT_F_DTV_DEMOD) > >> + sink = dev->decoder; > >> + else > >> + sink = entity; > >> + > >> + /* Is an active link between sink and tuner */ > >> + if (dev->active_link) { > >> + if (dev->active_link->sink->entity == sink && > >> + dev->active_link->source->entity == dev->tuner) { > >> + ret = 0; > >> + goto end; > >> + } else { > >> + ret = -EBUSY; > >> + goto end; > >> + } > >> + } > >> + > >> + list_for_each_entry(link, &sink->links, list) { > >> + /* Check sink, and source */ > >> + if (link->sink->entity == sink && > >> + link->source->entity == dev->tuner) { > >> + found_link = link; > >> + break; > >> + } > >> + } > >> + > >> + if (!found_link) { > >> + ret = -ENODEV; > >> + goto end; > >> + } > >> + > >> + /* activate link between source and sink and start pipeline */ > >> + source = found_link->source->entity; > >> + ret = __media_entity_setup_link(found_link, MEDIA_LNK_FL_ENABLED); > >> + if (ret) { > >> + pr_err( > >> + "Activate tuner link %s->%s. Error %d\n", > >> + source->name, sink->name, ret); > >> + goto end; > >> + } > >> + > >> + ret = __media_entity_pipeline_start(entity, pipe); > >> + if (ret) { > >> + pr_err("Start Pipeline: %s->%s Error %d\n", > >> + source->name, entity->name, ret); > >> + ret = __media_entity_setup_link(found_link, 0); > >> + pr_err("Deactive link Error %d\n", ret); > >> + goto end; > >> + } > > > > Hmm... isn't it to early to activate the pipeline here? My original > > guess is that, on the analog side, this should happen only at the stream > > on code. Wouldn't this break apps like mythTV? > > > >> + /* > >> + * save active link and active link owner to avoid audio > >> + * deactivating video owned link from disable_source and > >> + * vice versa > >> + */ > >> + dev->active_link = found_link; > >> + dev->active_link_owner = entity; > >> +end: > >> + mutex_unlock(&mdev->graph_mutex); > >> + pr_debug("au0828_enable_source() end %s %d %d\n", > >> + entity->name, entity->function, ret); > >> + return ret; > >> +#endif > >> + return 0; > >> +} > >> + > >> +static void au0828_disable_source(struct media_entity *entity) > >> +{ > >> +#ifdef CONFIG_MEDIA_CONTROLLER > >> + struct media_entity *sink; > >> + int ret = 0; > >> + struct media_device *mdev = entity->graph_obj.mdev; > >> + struct au0828_dev *dev; > >> + > >> + if (!mdev) > >> + return; > >> + > >> + mutex_lock(&mdev->graph_mutex); > >> + dev = mdev->source_priv; > >> + if (!dev->tuner || !dev->decoder || !dev->active_link) { > >> + ret = -ENODEV; > >> + goto end; > >> + } > > > > Same note as before. > > Same comment as before here about au0828_create_media_graph() > and au0828_usb_probe() handling. > > > > >> + > >> + if (entity->function != MEDIA_ENT_F_DTV_DEMOD) > >> + sink = dev->decoder; > >> + else > >> + sink = entity; > >> + > >> + /* link is active - stop pipeline from source (tuner) */ > >> + if (dev->active_link && dev->active_link->sink->entity == sink && > >> + dev->active_link->source->entity == dev->tuner) { > >> + /* > >> + * prevent video from deactivating link when audio > >> + * has active pipeline > >> + */ > >> + if (dev->active_link_owner != entity) > >> + goto end; > >> + __media_entity_pipeline_stop(entity); > >> + ret = __media_entity_setup_link(dev->active_link, 0); > >> + if (ret) > >> + pr_err("Deactive link Error %d\n", ret); > >> + dev->active_link = NULL; > >> + dev->active_link_owner = NULL; > >> + } > > > > Most code here looks like the one at au0828_enable_source(). Wouldn't > > be simpler to merge those code and add a "bool enable" to the function > > parameters? > > I would rather keep these separate. A very short > section is common really. > > thanks, > -- Shuah > > > > >> + > >> +end: > >> + mutex_unlock(&mdev->graph_mutex); > >> +#endif > >> +} > >> + > >> static int au0828_media_device_register(struct au0828_dev *dev, > >> struct usb_device *udev) > >> { > >> @@ -403,6 +548,10 @@ static int au0828_media_device_register(struct au0828_dev *dev, > >> ret); > >> return ret; > >> } > >> + /* set enable_source */ > >> + dev->media_dev->source_priv = (void *) dev; > >> + dev->media_dev->enable_source = au0828_enable_source; > >> + dev->media_dev->disable_source = au0828_disable_source; > >> #endif > >> return 0; > >> } > >> diff --git a/drivers/media/usb/au0828/au0828.h b/drivers/media/usb/au0828/au0828.h > >> index 54379ec..a7c88a1 100644 > >> --- a/drivers/media/usb/au0828/au0828.h > >> +++ b/drivers/media/usb/au0828/au0828.h > >> @@ -284,6 +284,9 @@ struct au0828_dev { > >> struct media_entity input_ent[AU0828_MAX_INPUT]; > >> struct media_pad input_pad[AU0828_MAX_INPUT]; > >> struct media_entity_notify entity_notify; > >> + struct media_entity *tuner; > >> + struct media_link *active_link; > >> + struct media_entity *active_link_owner; > >> #endif > >> }; > >> > >
next prev parent reply other threads:[~2016-02-09 10:52 UTC|newest] Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-02-04 4:03 [PATCH v2 00/22] Sharing media resources across ALSA and au0828 drivers Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 01/22] uapi/media.h: Declare interface types for ALSA Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 8:38 ` Mauro Carvalho Chehab 2016-02-04 8:38 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 02/22] media: Add ALSA Media Controller function entities Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 8:40 ` Mauro Carvalho Chehab 2016-02-04 8:40 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 03/22] media: Media Controller register/unregister entity_notify API Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 9:03 ` Mauro Carvalho Chehab 2016-02-04 9:03 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 04/22] media: Media Controller enable/disable source handler API Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 05/22] media: Media Controller export non locking __media_entity_setup_link() Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 06/22] media: Media Controller non-locking __media_entity_pipeline_start/stop() Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 9:18 ` Mauro Carvalho Chehab 2016-02-04 9:18 ` Mauro Carvalho Chehab 2016-02-04 15:00 ` Shuah Khan 2016-02-04 15:00 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 07/22] media: v4l-core add enable/disable source common interfaces Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 6:29 ` kbuild test robot 2016-02-04 6:29 ` kbuild test robot 2016-02-04 9:25 ` Mauro Carvalho Chehab 2016-02-04 9:25 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 08/22] media: Move au8522_media_pads enum to au8522.h from au8522_priv.h Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 09/22] media: au8522 change to create MC pad for ALSA Audio Out Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 10/22] media: Change v4l-core to check if source is free Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 11/22] media: dvb-frontend invoke enable/disable_source handlers Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 9:35 ` Mauro Carvalho Chehab 2016-02-04 9:35 ` Mauro Carvalho Chehab 2016-02-04 14:58 ` Shuah Khan 2016-02-04 14:58 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 12/22] media: au0828 video remove au0828_enable_analog_tuner() Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 9:37 ` Mauro Carvalho Chehab 2016-02-04 9:37 ` Mauro Carvalho Chehab 2016-02-04 14:49 ` Shuah Khan 2016-02-04 14:49 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 13/22] media: au0828 video change to use v4l_enable_media_source() Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 10:00 ` Mauro Carvalho Chehab 2016-02-04 10:00 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 14/22] media: au0828 change to use Managed Media Controller API Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 10:05 ` Mauro Carvalho Chehab 2016-02-04 10:05 ` Mauro Carvalho Chehab 2016-02-04 10:08 ` Mauro Carvalho Chehab 2016-02-04 10:08 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 15/22] media: au0828 handle media_init and media_register window Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 16/22] media: au0828 create tuner to decoder link in disabled state Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 17/22] media: au0828 disable tuner to demod link Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 18/22] media: au0828 Use au8522_media_pads enum for pad defines Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 19/22] media: au0828-core register entity_notify hook Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 4:03 ` [PATCH v2 20/22] media: au0828 add enable, disable source handlers Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 10:26 ` Mauro Carvalho Chehab 2016-02-04 10:26 ` Mauro Carvalho Chehab 2016-02-08 22:42 ` Shuah Khan 2016-02-08 22:42 ` Shuah Khan 2016-02-08 23:00 ` Shuah Khan 2016-02-08 23:00 ` Shuah Khan 2016-02-09 13:42 ` Mauro Carvalho Chehab 2016-02-09 13:42 ` Mauro Carvalho Chehab 2016-02-09 10:51 ` Mauro Carvalho Chehab [this message] 2016-02-09 10:51 ` Mauro Carvalho Chehab 2016-02-04 4:03 ` [PATCH v2 21/22] sound/usb: Use Media Controller API to share media resources Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 10:36 ` Mauro Carvalho Chehab 2016-02-04 10:36 ` Mauro Carvalho Chehab 2016-02-04 10:46 ` Takashi Iwai 2016-02-04 10:46 ` Takashi Iwai 2016-02-07 11:25 ` kbuild test robot 2016-02-07 11:25 ` kbuild test robot 2016-02-04 4:03 ` [PATCH v2 22/22] media: Ensure media device unregister is done only once Shuah Khan 2016-02-04 4:03 ` Shuah Khan 2016-02-04 10:40 ` Mauro Carvalho Chehab 2016-02-04 10:40 ` Mauro Carvalho Chehab 2016-02-04 10:54 ` [PATCH v2 00/22] Sharing media resources across ALSA and au0828 drivers Mauro Carvalho Chehab 2016-02-04 10:54 ` Mauro Carvalho Chehab
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160209085157.06836b58@recife.lan \ --to=mchehab@osg.samsung.com \ --cc=alsa-devel@alsa-project.org \ --cc=arnd@arndb.de \ --cc=chehabrafael@gmail.com \ --cc=clemens@ladisch.de \ --cc=corbet@lwn.net \ --cc=crope@iki.fi \ --cc=dan.carpenter@oracle.com \ --cc=daniel@zonque.org \ --cc=dominic.sacre@gmx.de \ --cc=elfring@users.sourceforge.net \ --cc=geliangtang@163.com \ --cc=gtmkramer@xs4all.nl \ --cc=hans.verkuil@cisco.com \ --cc=inki.dae@samsung.com \ --cc=j.anaszewski@samsung.com \ --cc=javier@osg.samsung.com \ --cc=jh1009.sung@samsung.com \ --cc=joe@oampo.co.uk \ --cc=johan@oljud.se \ --cc=julian@jusst.de \ --cc=k.kozlowski@samsung.com \ --cc=klock.android@gmail.com \ --cc=kyungmin.park@samsung.com \ --cc=labbott@fedoraproject.org \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=linuxbugs@vittgam.net \ --cc=m.szyprowski@samsung.com \ --cc=misterpib@gmail.com \ --cc=nenggun.kim@samsung.com \ --cc=normalperson@yhbt.net \ --cc=p.zabel@pengutronix.de \ --cc=pawel@osciak.com \ --cc=perex@perex.cz \ --cc=pierre-louis.bossart@linux.intel.com \ --cc=prabhakar.csengg@gmail.com \ --cc=ricard.wanderlof@axis.com \ --cc=ricardo.ribalda@gmail.com \ --cc=ruchandani.tina@gmail.com \ --cc=sakari.ailus@linux.intel.com \ --cc=shuahkh@osg.samsung.com \ --cc=stefanr@s5r6.in-berlin.de \ --cc=sw0312.kim@samsung.com \ --cc=takamichiho@gmail.com \ --cc=tiwai@suse.com \ --cc=tvboxspy@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.