On 03/10/2016 12:57 PM, Mauro Carvalho Chehab wrote: > As au0828 assumes that all links to ATV decoder and DTV demod > should be disabled, make sure this is the case. > > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx> > --- > drivers/media/usb/au0828/au0828-core.c | 19 +++++++++++-------- > 1 file changed, 11 insertions(+), 8 deletions(-) > > diff --git a/drivers/media/usb/au0828/au0828-core.c b/drivers/media/usb/au0828/au0828-core.c > index 5dc82e8c8670..af68663915fd 100644 > --- a/drivers/media/usb/au0828/au0828-core.c > +++ b/drivers/media/usb/au0828/au0828-core.c > @@ -456,7 +456,9 @@ static int au0828_media_device_register(struct au0828_dev *dev, > { > #ifdef CONFIG_MEDIA_CONTROLLER > int ret; > - struct media_entity *entity, *demod = NULL, *tuner = NULL; > + struct media_entity *entity; > + struct media_entity *demod = NULL, *tuner = NULL, *decoder = NULL; > + struct media_link *link; > > if (!dev->media_dev) > return 0; > @@ -490,18 +492,19 @@ static int au0828_media_device_register(struct au0828_dev *dev, > media_device_for_each_entity(entity, dev->media_dev) { > if (entity->function == MEDIA_ENT_F_DTV_DEMOD) > demod = entity; > + else if (entity->function == MEDIA_ENT_F_ATV_DECODER) > + decoder = entity; > else if (entity->function == MEDIA_ENT_F_TUNER) > tuner = entity; > } > - /* Disable link between tuner and demod */ > - if (tuner && demod) { > - struct media_link *link; > > - list_for_each_entry(link, &demod->links, list) { > - if (link->sink->entity == demod && > - link->source->entity == tuner) { > + /* Disable link between tuner->demod and/or tuner->decoder */ > + if (tuner) { > + list_for_each_entry(link, &tuner->links, list) { > + if (demod && link->sink->entity == demod) > + media_entity_setup_link(link, 0); > + if (decoder && link->sink->entity == decoder) > media_entity_setup_link(link, 0); > - } > } > } > > You probably don't need this patch if we revert the common graph create patch. We will need this when we get the common routine working. thanks, -- Shuah -- Shuah Khan Sr. Linux Kernel Developer Open Source Innovation Group Samsung Research America (Silicon Valley) shuahkh@xxxxxxxxxxxxxxx | (970) 217-8978 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html