Hi Sakari,

Thanks for your work.

On 2019-03-19 15:02:00 +0200, Sakari Ailus wrote:
[snip]
> diff --git a/drivers/media/platform/rcar_drif.c 
> b/drivers/media/platform/rcar_drif.c
> index c417ff8f6fe5..39da118f882a 100644
> --- a/drivers/media/platform/rcar_drif.c
> +++ b/drivers/media/platform/rcar_drif.c
> @@ -1222,7 +1222,8 @@ static int rcar_drif_parse_subdevs(struct rcar_drif_sdr 
> *sdr)
>       if (!ep)
>               return 0;
>  
> -     fwnode = fwnode_graph_get_remote_port_parent(ep);
> +     notifier->subdevs[notifier->num_subdevs] = &sdr->ep.asd;
> +     fwnode = fwnode_graph_get_remote_endpoint(ep);
>       if (!fwnode) {
>               dev_warn(sdr->dev, "bad remote port parent\n");
>               fwnode_handle_put(ep);

I tried the branch you referred to in your cover letter but this driver 
fails to compile for me. Looking at struct v4l2_async_notifier it has 
neither a 'subdevs' nor 'num_subdevs' members so I might be missing a 
dependency. However looking at the change it seems different from what 
you do for other drivers, is this a typo?

-- 
Regards,
Niklas Söderlund

Reply via email to