Hi Jacek,
On Wed, May 20, 2015 at 04:10:12PM +0200, Jacek Anaszewski wrote:
> This patch adds support for external v4l2-flash devices.
> The support includes parsing "samsung,flash-led" DT property
> and asynchronous subdevice registration.
>
> Signed-off-by: Jacek Anaszewski <j.anaszew...@samsung.com>
> Acked-by: Kyungmin Park <kyungmin.p...@samsung.com>
> Cc: Sylwester Nawrocki <s.nawro...@samsung.com>
> ---
> drivers/media/platform/exynos4-is/media-dev.c | 39
> +++++++++++++++++++++++--
> drivers/media/platform/exynos4-is/media-dev.h | 13 ++++++++-
> 2 files changed, 49 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/platform/exynos4-is/media-dev.c
> b/drivers/media/platform/exynos4-is/media-dev.c
> index f315ef9..80cecd8 100644
> --- a/drivers/media/platform/exynos4-is/media-dev.c
> +++ b/drivers/media/platform/exynos4-is/media-dev.c
> @@ -451,6 +451,28 @@ rpm_put:
> return ret;
> }
>
> +static void fimc_md_register_flash_entities(struct fimc_md *fmd)
> +{
> + int i;
> +
> + fmd->num_flashes = 0;
> +
> + for (i = 0; i < fmd->num_sensors; i++) {
> + const struct device_node *np =
> + fmd->sensor[i].asd.match.of.node;
> + const int nf = fmd->num_flashes;
> +
> + np = of_parse_phandle(np, "samsung,flash-led", 0);
> + if (!np)
> + continue;
> +
> + fmd->flash[nf].asd.match_type = V4L2_ASYNC_MATCH_OF;
> + fmd->flash[nf].asd.match.of.node = np;
> + fmd->async_subdevs[fmd->num_sensors + nf] = &fmd->flash[nf].asd;
> + fmd->num_flashes++;
> + }
> +}
> +
> static int __of_get_csis_id(struct device_node *np)
> {
> u32 reg = 0;
> @@ -1275,6 +1297,15 @@ static int subdev_notifier_bound(struct
> v4l2_async_notifier *notifier,
> struct fimc_sensor_info *si = NULL;
> int i;
>
> + /* Register flash subdev if detected any */
> + for (i = 0; i < ARRAY_SIZE(fmd->flash); i++) {
> + if (fmd->flash[i].asd.match.of.node == subdev->of_node) {
Does the index of a particular sub-device index matter? Could you just use
the next available one?
There would be no need to for the check anything here, please see how the
omap3isp driver does it --- it's in isp_subdev_notifier_bound()
drivers/media/platform/omap3isp/isp.c .
> + fmd->flash[i].subdev = subdev;
> + fmd->num_flashes++;
> + return 0;
> + }
> + }
> +
> /* Find platform data for this sensor subdev */
> for (i = 0; i < ARRAY_SIZE(fmd->sensor); i++)
> if (fmd->sensor[i].asd.match.of.node == subdev->dev->of_node)
--
Kind regards,
Sakari Ailus
e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
