Hi Mauro,

Thanks for the patch.

> -----Original Message-----
> From: Mauro Carvalho Chehab [mailto:mche...@smtp.s-opensource.com]
> On Behalf Of Mauro Carvalho Chehab
> Sent: Tuesday, December 19, 2017 3:18 AM
> To: Linux Media Mailing List <linux-me...@vger.kernel.org>
> Cc: Mauro Carvalho Chehab <mche...@s-opensource.com>; Mauro
> Carvalho Chehab <mche...@infradead.org>; Lad, Prabhakar
> <prabhakar.cse...@gmail.com>; Songjun Wu
> <songjun...@microchip.com>; Nicolas Ferre
> <nicolas.fe...@microchip.com>; Alexandre Belloni <alexandre.belloni@free-
> electrons.com>; Ludovic Desroches <ludovic.desroc...@microchip.com>;
> Kyungmin Park <kyungmin.p...@samsung.com>; Sylwester Nawrocki
> <s.nawro...@samsung.com>; Kukjin Kim <kg...@kernel.org>; Krzysztof
> Kozlowski <k...@kernel.org>; Todor Tomov <todor.to...@linaro.org>;
> Niklas Söderlund <niklas.soderl...@ragnatech.se>; Ramesh
> Shanmugasundaram <ramesh.shanmugasunda...@bp.renesas.com>;
> Guennadi Liakhovetski <g.liakhovet...@gmx.de>; Maxime Coquelin
> <mcoquelin.st...@gmail.com>; Alexandre Torgue
> <alexandre.tor...@st.com>; Benoit Parrot <bpar...@ti.com>; Hyun Kwon
> <hy...@xilinx.com>; Laurent Pinchart
> <laurent.pinch...@ideasonboard.com>; Michal Simek
> <michal.si...@xilinx.com>; Steve Longerbeam <slongerb...@gmail.com>;
> Philipp Zabel <p.za...@pengutronix.de>; Greg Kroah-Hartman
> <gre...@linuxfoundation.org>; Hans Verkuil <hverk...@xs4all.nl>; Petr
> Cvek <petr.c...@tul.cz>; Sakari Ailus <sakari.ai...@iki.fi>; Julia Lawall
> <julia.law...@lip6.fr>; Arnd Bergmann <a...@arndb.de>; Hugues Fruchet
> <hugues.fruc...@st.com>; Gustavo A. R. Silva
> <garsi...@embeddedor.com>; Sebastian Reichel <s...@kernel.org>; Tomasz
> Figa <tf...@chromium.org>; linux-arm-ker...@lists.infradead.org; linux-
> samsung-...@vger.kernel.org; linux-renesas-...@vger.kernel.org;
> de...@driverdev.osuosl.org
> Subject: [PATCH v2 3/8] media: v4l2-async: simplify v4l2_async_subdev
> structure
> 
> The V4L2_ASYNC_MATCH_FWNODE match criteria requires just one
> struct to be filled (struct fwnode_handle). The
> V4L2_ASYNC_MATCH_DEVNAME
> match criteria requires just a device name.
> 
> So, it doesn't make sense to enclose those into structs,
> as the criteria can go directly into the union.
> 
> That makes easier to document it, as we don't need to document
> weird senseless structs.
> 
> At drivers, this makes even clearer about the match criteria.
> 
> Acked-by: Sylwester Nawrocki <s.nawro...@samsung.com>
> Acked-by: Benoit Parrot <bpar...@ti.com>
> Acked-by: Alexandre Belloni <alexandre.bell...@free-electrons.com>
> Acked-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Acked-by: Philipp Zabel <p.za...@pengutronix.de>
> Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

For xilinx-vipp.c, please add,

        Acked-by: Hyun Kwon <hyun.k...@xilinx.com>

Thanks,
-hyun

> ---
>  drivers/media/platform/am437x/am437x-vpfe.c    |  6 +++---
>  drivers/media/platform/atmel/atmel-isc.c       |  2 +-
>  drivers/media/platform/atmel/atmel-isi.c       |  2 +-
>  drivers/media/platform/davinci/vpif_capture.c  |  4 ++--
>  drivers/media/platform/exynos4-is/media-dev.c  |  4 ++--
>  drivers/media/platform/pxa_camera.c            |  2 +-
>  drivers/media/platform/qcom/camss-8x16/camss.c |  2 +-
>  drivers/media/platform/rcar-vin/rcar-core.c    |  2 +-
>  drivers/media/platform/rcar_drif.c             |  4 ++--
>  drivers/media/platform/soc_camera/soc_camera.c |  2 +-
>  drivers/media/platform/stm32/stm32-dcmi.c      |  2 +-
>  drivers/media/platform/ti-vpe/cal.c            |  2 +-
>  drivers/media/platform/xilinx/xilinx-vipp.c    |  2 +-
>  drivers/media/v4l2-core/v4l2-async.c           | 16 ++++++++--------
>  drivers/media/v4l2-core/v4l2-fwnode.c          | 10 +++++-----
>  drivers/staging/media/imx/imx-media-dev.c      |  4 ++--
>  include/media/v4l2-async.h                     |  8 ++------
>  17 files changed, 35 insertions(+), 39 deletions(-)
> 
> diff --git a/drivers/media/platform/am437x/am437x-vpfe.c
> b/drivers/media/platform/am437x/am437x-vpfe.c
> index 0997c640191d..601ae6487617 100644
> --- a/drivers/media/platform/am437x/am437x-vpfe.c
> +++ b/drivers/media/platform/am437x/am437x-vpfe.c
> @@ -2304,8 +2304,8 @@ vpfe_async_bound(struct v4l2_async_notifier
> *notifier,
>       vpfe_dbg(1, vpfe, "vpfe_async_bound\n");
> 
>       for (i = 0; i < ARRAY_SIZE(vpfe->cfg->asd); i++) {
> -             if (vpfe->cfg->asd[i]->match.fwnode.fwnode ==
> -                 asd[i].match.fwnode.fwnode) {
> +             if (vpfe->cfg->asd[i]->match.fwnode ==
> +                 asd[i].match.fwnode) {
>                       sdinfo = &vpfe->cfg->sub_devs[i];
>                       vpfe->sd[i] = subdev;
>                       vpfe->sd[i]->grp_id = sdinfo->grp_id;
> @@ -2510,7 +2510,7 @@ vpfe_get_pdata(struct platform_device *pdev)
>               }
> 
>               pdata->asd[i]->match_type =
> V4L2_ASYNC_MATCH_FWNODE;
> -             pdata->asd[i]->match.fwnode.fwnode =
> of_fwnode_handle(rem);
> +             pdata->asd[i]->match.fwnode = of_fwnode_handle(rem);
>               of_node_put(rem);
>       }
> 
> diff --git a/drivers/media/platform/atmel/atmel-isc.c
> b/drivers/media/platform/atmel/atmel-isc.c
> index 0c2635647f69..34676409ca08 100644
> --- a/drivers/media/platform/atmel/atmel-isc.c
> +++ b/drivers/media/platform/atmel/atmel-isc.c
> @@ -2088,7 +2088,7 @@ static int isc_parse_dt(struct device *dev, struct
> isc_device *isc)
>                       subdev_entity->pfe_cfg0 |=
> ISC_PFE_CFG0_PPOL_LOW;
> 
>               subdev_entity->asd->match_type =
> V4L2_ASYNC_MATCH_FWNODE;
> -             subdev_entity->asd->match.fwnode.fwnode =
> +             subdev_entity->asd->match.fwnode =
>                       of_fwnode_handle(rem);
>               list_add_tail(&subdev_entity->list, &isc->subdev_entities);
>       }
> diff --git a/drivers/media/platform/atmel/atmel-isi.c
> b/drivers/media/platform/atmel/atmel-isi.c
> index e900995143a3..9958918e2449 100644
> --- a/drivers/media/platform/atmel/atmel-isi.c
> +++ b/drivers/media/platform/atmel/atmel-isi.c
> @@ -1128,7 +1128,7 @@ static int isi_graph_parse(struct atmel_isi *isi,
> struct device_node *node)
>               /* Remote node to connect */
>               isi->entity.node = remote;
>               isi->entity.asd.match_type =
> V4L2_ASYNC_MATCH_FWNODE;
> -             isi->entity.asd.match.fwnode.fwnode =
> of_fwnode_handle(remote);
> +             isi->entity.asd.match.fwnode = of_fwnode_handle(remote);
>               return 0;
>       }
>  }
> diff --git a/drivers/media/platform/davinci/vpif_capture.c
> b/drivers/media/platform/davinci/vpif_capture.c
> index e45916f69def..e1c273c8b9a6 100644
> --- a/drivers/media/platform/davinci/vpif_capture.c
> +++ b/drivers/media/platform/davinci/vpif_capture.c
> @@ -1390,7 +1390,7 @@ static int vpif_async_bound(struct
> v4l2_async_notifier *notifier,
> 
>       for (i = 0; i < vpif_obj.config->asd_sizes[0]; i++) {
>               struct v4l2_async_subdev *_asd = vpif_obj.config->asd[i];
> -             const struct fwnode_handle *fwnode = _asd-
> >match.fwnode.fwnode;
> +             const struct fwnode_handle *fwnode = _asd-
> >match.fwnode;
> 
>               if (fwnode == subdev->fwnode) {
>                       vpif_obj.sd[i] = subdev;
> @@ -1595,7 +1595,7 @@ vpif_capture_get_pdata(struct platform_device
> *pdev)
>               }
> 
>               pdata->asd[i]->match_type =
> V4L2_ASYNC_MATCH_FWNODE;
> -             pdata->asd[i]->match.fwnode.fwnode =
> of_fwnode_handle(rem);
> +             pdata->asd[i]->match.fwnode = of_fwnode_handle(rem);
>               of_node_put(rem);
>       }
> 
> diff --git a/drivers/media/platform/exynos4-is/media-dev.c
> b/drivers/media/platform/exynos4-is/media-dev.c
> index 0ef583cfc424..78b48a1fa26c 100644
> --- a/drivers/media/platform/exynos4-is/media-dev.c
> +++ b/drivers/media/platform/exynos4-is/media-dev.c
> @@ -456,7 +456,7 @@ static int fimc_md_parse_port_node(struct
> fimc_md *fmd,
>       }
> 
>       fmd->sensor[index].asd.match_type =
> V4L2_ASYNC_MATCH_FWNODE;
> -     fmd->sensor[index].asd.match.fwnode.fwnode =
> of_fwnode_handle(rem);
> +     fmd->sensor[index].asd.match.fwnode = of_fwnode_handle(rem);
>       fmd->async_subdevs[index] = &fmd->sensor[index].asd;
> 
>       fmd->num_sensors++;
> @@ -1364,7 +1364,7 @@ static int subdev_notifier_bound(struct
> v4l2_async_notifier *notifier,
> 
>       /* Find platform data for this sensor subdev */
>       for (i = 0; i < ARRAY_SIZE(fmd->sensor); i++)
> -             if (fmd->sensor[i].asd.match.fwnode.fwnode ==
> +             if (fmd->sensor[i].asd.match.fwnode ==
>                   of_fwnode_handle(subdev->dev->of_node))
>                       si = &fmd->sensor[i];
> 
> diff --git a/drivers/media/platform/pxa_camera.c
> b/drivers/media/platform/pxa_camera.c
> index 305cf1cac210..f028084f0775 100644
> --- a/drivers/media/platform/pxa_camera.c
> +++ b/drivers/media/platform/pxa_camera.c
> @@ -2335,7 +2335,7 @@ static int pxa_camera_pdata_from_dt(struct
> device *dev,
>       asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
>       remote = of_graph_get_remote_port(np);
>       if (remote) {
> -             asd->match.fwnode.fwnode = of_fwnode_handle(remote);
> +             asd->match.fwnode = of_fwnode_handle(remote);
>               of_node_put(remote);
>       } else {
>               dev_notice(dev, "no remote for %pOF\n", np);
> diff --git a/drivers/media/platform/qcom/camss-8x16/camss.c
> b/drivers/media/platform/qcom/camss-8x16/camss.c
> index 390a42c17b66..05f06c98aa64 100644
> --- a/drivers/media/platform/qcom/camss-8x16/camss.c
> +++ b/drivers/media/platform/qcom/camss-8x16/camss.c
> @@ -341,7 +341,7 @@ static int camss_of_parse_ports(struct device *dev,
>               }
> 
>               csd->asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
> -             csd->asd.match.fwnode.fwnode =
> of_fwnode_handle(remote);
> +             csd->asd.match.fwnode = of_fwnode_handle(remote);
>       }
> 
>       return notifier->num_subdevs;
> diff --git a/drivers/media/platform/rcar-vin/rcar-core.c
> b/drivers/media/platform/rcar-vin/rcar-core.c
> index 108d776f3265..f1fc7978d6d1 100644
> --- a/drivers/media/platform/rcar-vin/rcar-core.c
> +++ b/drivers/media/platform/rcar-vin/rcar-core.c
> @@ -187,7 +187,7 @@ static int rvin_digital_graph_init(struct rvin_dev
> *vin)
>               return -ENODEV;
> 
>       vin_dbg(vin, "Found digital subdevice %pOF\n",
> -             to_of_node(vin->digital->asd.match.fwnode.fwnode));
> +             to_of_node(vin->digital->asd.match.fwnode));
> 
>       vin->notifier.ops = &rvin_digital_notify_ops;
>       ret = v4l2_async_notifier_register(&vin->v4l2_dev, &vin->notifier);
> diff --git a/drivers/media/platform/rcar_drif.c
> b/drivers/media/platform/rcar_drif.c
> index 63c94f4028a7..b2e080ef5391 100644
> --- a/drivers/media/platform/rcar_drif.c
> +++ b/drivers/media/platform/rcar_drif.c
> @@ -1107,7 +1107,7 @@ static int rcar_drif_notify_bound(struct
> v4l2_async_notifier *notifier,
>       struct rcar_drif_sdr *sdr =
>               container_of(notifier, struct rcar_drif_sdr, notifier);
> 
> -     if (sdr->ep.asd.match.fwnode.fwnode !=
> +     if (sdr->ep.asd.match.fwnode !=
>           of_fwnode_handle(subdev->dev->of_node)) {
>               rdrif_err(sdr, "subdev %s cannot bind\n", subdev->name);
>               return -EINVAL;
> @@ -1235,7 +1235,7 @@ static int rcar_drif_parse_subdevs(struct
> rcar_drif_sdr *sdr)
>               return -EINVAL;
>       }
> 
> -     sdr->ep.asd.match.fwnode.fwnode = fwnode;
> +     sdr->ep.asd.match.fwnode = fwnode;
>       sdr->ep.asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
>       notifier->num_subdevs++;
> 
> diff --git a/drivers/media/platform/soc_camera/soc_camera.c
> b/drivers/media/platform/soc_camera/soc_camera.c
> index 916ff68b73d4..d13e2c5fb06f 100644
> --- a/drivers/media/platform/soc_camera/soc_camera.c
> +++ b/drivers/media/platform/soc_camera/soc_camera.c
> @@ -1517,7 +1517,7 @@ static int soc_of_bind(struct soc_camera_host
> *ici,
>       if (!info)
>               return -ENOMEM;
> 
> -     info->sasd.asd.match.fwnode.fwnode = of_fwnode_handle(remote);
> +     info->sasd.asd.match.fwnode = of_fwnode_handle(remote);
>       info->sasd.asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
>       info->subdev = &info->sasd.asd;
> 
> diff --git a/drivers/media/platform/stm32/stm32-dcmi.c
> b/drivers/media/platform/stm32/stm32-dcmi.c
> index ac4c450a6c7d..9460b3080dca 100644
> --- a/drivers/media/platform/stm32/stm32-dcmi.c
> +++ b/drivers/media/platform/stm32/stm32-dcmi.c
> @@ -1520,7 +1520,7 @@ static int dcmi_graph_parse(struct stm32_dcmi
> *dcmi, struct device_node *node)
>               /* Remote node to connect */
>               dcmi->entity.node = remote;
>               dcmi->entity.asd.match_type =
> V4L2_ASYNC_MATCH_FWNODE;
> -             dcmi->entity.asd.match.fwnode.fwnode =
> of_fwnode_handle(remote);
> +             dcmi->entity.asd.match.fwnode =
> of_fwnode_handle(remote);
>               return 0;
>       }
>  }
> diff --git a/drivers/media/platform/ti-vpe/cal.c
> b/drivers/media/platform/ti-vpe/cal.c
> index 719ed1d79957..d1febe5baa6d 100644
> --- a/drivers/media/platform/ti-vpe/cal.c
> +++ b/drivers/media/platform/ti-vpe/cal.c
> @@ -1702,7 +1702,7 @@ static int of_cal_create_instance(struct cal_ctx
> *ctx, int inst)
>               goto cleanup_exit;
>       }
>       asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
> -     asd->match.fwnode.fwnode = of_fwnode_handle(sensor_node);
> +     asd->match.fwnode = of_fwnode_handle(sensor_node);
> 
>       remote_ep = of_graph_get_remote_endpoint(ep_node);
>       if (!remote_ep) {
> diff --git a/drivers/media/platform/xilinx/xilinx-vipp.c
> b/drivers/media/platform/xilinx/xilinx-vipp.c
> index f4c3e48ed2c0..6bb28cd49dae 100644
> --- a/drivers/media/platform/xilinx/xilinx-vipp.c
> +++ b/drivers/media/platform/xilinx/xilinx-vipp.c
> @@ -387,7 +387,7 @@ static int xvip_graph_parse_one(struct
> xvip_composite_device *xdev,
> 
>               entity->node = remote;
>               entity->asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
> -             entity->asd.match.fwnode.fwnode =
> of_fwnode_handle(remote);
> +             entity->asd.match.fwnode = of_fwnode_handle(remote);
>               list_add_tail(&entity->list, &xdev->entities);
>               xdev->num_subdevs++;
>       }
> diff --git a/drivers/media/v4l2-core/v4l2-async.c b/drivers/media/v4l2-
> core/v4l2-async.c
> index e5acfab470a5..2b08d03b251d 100644
> --- a/drivers/media/v4l2-core/v4l2-async.c
> +++ b/drivers/media/v4l2-core/v4l2-async.c
> @@ -68,12 +68,12 @@ static bool match_i2c(struct v4l2_subdev *sd,
> struct v4l2_async_subdev *asd)
>  static bool match_devname(struct v4l2_subdev *sd,
>                         struct v4l2_async_subdev *asd)
>  {
> -     return !strcmp(asd->match.device_name.name, dev_name(sd-
> >dev));
> +     return !strcmp(asd->match.device_name, dev_name(sd->dev));
>  }
> 
>  static bool match_fwnode(struct v4l2_subdev *sd, struct
> v4l2_async_subdev *asd)
>  {
> -     return sd->fwnode == asd->match.fwnode.fwnode;
> +     return sd->fwnode == asd->match.fwnode;
>  }
> 
>  static bool match_custom(struct v4l2_subdev *sd, struct
> v4l2_async_subdev *asd)
> @@ -319,7 +319,7 @@ static bool
> __v4l2_async_notifier_fwnode_has_async_subdev(
>               if (asd->match_type != V4L2_ASYNC_MATCH_FWNODE)
>                       continue;
> 
> -             if (asd->match.fwnode.fwnode == fwnode)
> +             if (asd->match.fwnode == fwnode)
>                       return true;
>       }
> 
> @@ -330,7 +330,7 @@ static bool
> __v4l2_async_notifier_fwnode_has_async_subdev(
>               if (sd->asd->match_type != V4L2_ASYNC_MATCH_FWNODE)
>                       continue;
> 
> -             if (sd->asd->match.fwnode.fwnode == fwnode)
> +             if (sd->asd->match.fwnode == fwnode)
>                       return true;
>       }
> 
> @@ -355,8 +355,8 @@ static bool
> v4l2_async_notifier_fwnode_has_async_subdev(
>               struct v4l2_async_subdev *other_asd = notifier->subdevs[j];
> 
>               if (other_asd->match_type ==
> V4L2_ASYNC_MATCH_FWNODE &&
> -                 asd->match.fwnode.fwnode ==
> -                 other_asd->match.fwnode.fwnode)
> +                 asd->match.fwnode ==
> +                 other_asd->match.fwnode)
>                       return true;
>       }
> 
> @@ -395,7 +395,7 @@ static int __v4l2_async_notifier_register(struct
> v4l2_async_notifier *notifier)
>                       break;
>               case V4L2_ASYNC_MATCH_FWNODE:
>                       if (v4l2_async_notifier_fwnode_has_async_subdev(
> -                                 notifier, asd->match.fwnode.fwnode, i)) {
> +                                 notifier, asd->match.fwnode, i)) {
>                               dev_err(dev,
>                                       "fwnode has already been registered
> or in notifier's subdev list\n");
>                               ret = -EEXIST;
> @@ -510,7 +510,7 @@ void v4l2_async_notifier_cleanup(struct
> v4l2_async_notifier *notifier)
> 
>               switch (asd->match_type) {
>               case V4L2_ASYNC_MATCH_FWNODE:
> -                     fwnode_handle_put(asd->match.fwnode.fwnode);
> +                     fwnode_handle_put(asd->match.fwnode);
>                       break;
>               default:
>                       WARN_ON_ONCE(true);
> diff --git a/drivers/media/v4l2-core/v4l2-fwnode.c b/drivers/media/v4l2-
> core/v4l2-fwnode.c
> index fb72c7ac04d4..d630640642ee 100644
> --- a/drivers/media/v4l2-core/v4l2-fwnode.c
> +++ b/drivers/media/v4l2-core/v4l2-fwnode.c
> @@ -359,9 +359,9 @@ static int
> v4l2_async_notifier_fwnode_parse_endpoint(
>               return -ENOMEM;
> 
>       asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
> -     asd->match.fwnode.fwnode =
> +     asd->match.fwnode =
>               fwnode_graph_get_remote_port_parent(endpoint);
> -     if (!asd->match.fwnode.fwnode) {
> +     if (!asd->match.fwnode) {
>               dev_warn(dev, "bad remote port parent\n");
>               ret = -EINVAL;
>               goto out_err;
> @@ -393,7 +393,7 @@ static int
> v4l2_async_notifier_fwnode_parse_endpoint(
>       return 0;
> 
>  out_err:
> -     fwnode_handle_put(asd->match.fwnode.fwnode);
> +     fwnode_handle_put(asd->match.fwnode);
>       kfree(asd);
> 
>       return ret == -ENOTCONN ? 0 : ret;
> @@ -566,7 +566,7 @@ static int v4l2_fwnode_reference_parse(
>               }
> 
>               notifier->subdevs[notifier->num_subdevs] = asd;
> -             asd->match.fwnode.fwnode = args.fwnode;
> +             asd->match.fwnode = args.fwnode;
>               asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
>               notifier->num_subdevs++;
>       }
> @@ -853,7 +853,7 @@ static int v4l2_fwnode_reference_parse_int_props(
>               }
> 
>               notifier->subdevs[notifier->num_subdevs] = asd;
> -             asd->match.fwnode.fwnode = fwnode;
> +             asd->match.fwnode = fwnode;
>               asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
>               notifier->num_subdevs++;
>       }
> diff --git a/drivers/staging/media/imx/imx-media-dev.c
> b/drivers/staging/media/imx/imx-media-dev.c
> index 2800700482d6..f7ed5f506fa9 100644
> --- a/drivers/staging/media/imx/imx-media-dev.c
> +++ b/drivers/staging/media/imx/imx-media-dev.c
> @@ -48,7 +48,7 @@ find_async_subdev(struct imx_media_dev *imxmd,
>               asd = &imxasd->asd;
>               switch (asd->match_type) {
>               case V4L2_ASYNC_MATCH_FWNODE:
> -                     if (fwnode && asd->match.fwnode.fwnode ==
> fwnode)
> +                     if (fwnode && asd->match.fwnode == fwnode)
>                               return asd;
>                       break;
>               case V4L2_ASYNC_MATCH_DEVNAME:
> @@ -104,7 +104,7 @@ int imx_media_add_async_subdev(struct
> imx_media_dev *imxmd,
> 
>       if (fwnode) {
>               asd->match_type = V4L2_ASYNC_MATCH_FWNODE;
> -             asd->match.fwnode.fwnode = fwnode;
> +             asd->match.fwnode = fwnode;
>       } else {
>               asd->match_type = V4L2_ASYNC_MATCH_DEVNAME;
>               asd->match.device_name.name = devname;
> diff --git a/include/media/v4l2-async.h b/include/media/v4l2-async.h
> index 6152434cbe82..a010af5134b2 100644
> --- a/include/media/v4l2-async.h
> +++ b/include/media/v4l2-async.h
> @@ -58,12 +58,8 @@ enum v4l2_async_match_type {
>  struct v4l2_async_subdev {
>       enum v4l2_async_match_type match_type;
>       union {
> -             struct {
> -                     struct fwnode_handle *fwnode;
> -             } fwnode;
> -             struct {
> -                     const char *name;
> -             } device_name;
> +             struct fwnode_handle *fwnode;
> +             const char *device_name;
>               struct {
>                       int adapter_id;
>                       unsigned short address;
> --
> 2.14.3

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to