The HGT is a Histogram Generator Two-Dimensions. It computes a weighted
frequency histograms for hue and saturation areas over a configurable
region of the image with optional subsampling.

Signed-off-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se>
---
 drivers/media/platform/vsp1/Makefile      |   2 +-
 drivers/media/platform/vsp1/vsp1.h        |   3 +
 drivers/media/platform/vsp1/vsp1_drv.c    |  32 +-
 drivers/media/platform/vsp1/vsp1_entity.c |  33 +-
 drivers/media/platform/vsp1/vsp1_entity.h |   1 +
 drivers/media/platform/vsp1/vsp1_hgt.c    | 495 ++++++++++++++++++++++++++++++
 drivers/media/platform/vsp1/vsp1_hgt.h    |  51 +++
 drivers/media/platform/vsp1/vsp1_pipe.c   |  16 +
 drivers/media/platform/vsp1/vsp1_pipe.h   |   2 +
 drivers/media/platform/vsp1/vsp1_regs.h   |   9 +
 drivers/media/platform/vsp1/vsp1_video.c  |  10 +-
 11 files changed, 638 insertions(+), 16 deletions(-)
 create mode 100644 drivers/media/platform/vsp1/vsp1_hgt.c
 create mode 100644 drivers/media/platform/vsp1/vsp1_hgt.h

diff --git a/drivers/media/platform/vsp1/Makefile 
b/drivers/media/platform/vsp1/Makefile
index 8ab6a06..a33afc3 100644
--- a/drivers/media/platform/vsp1/Makefile
+++ b/drivers/media/platform/vsp1/Makefile
@@ -3,7 +3,7 @@ vsp1-y                                  += vsp1_dl.o vsp1_drm.o 
vsp1_video.o
 vsp1-y                                 += vsp1_rpf.o vsp1_rwpf.o vsp1_wpf.o
 vsp1-y                                 += vsp1_clu.o vsp1_hsit.o vsp1_lut.o
 vsp1-y                                 += vsp1_bru.o vsp1_sru.o vsp1_uds.o
-vsp1-y                                 += vsp1_hgo.o vsp1_histo.o
+vsp1-y                                 += vsp1_hgo.o vsp1_hgt.o vsp1_histo.o
 vsp1-y                                 += vsp1_lif.o
 
 obj-$(CONFIG_VIDEO_RENESAS_VSP1)       += vsp1.o
diff --git a/drivers/media/platform/vsp1/vsp1.h 
b/drivers/media/platform/vsp1/vsp1.h
index 9dce3ea..012ce40 100644
--- a/drivers/media/platform/vsp1/vsp1.h
+++ b/drivers/media/platform/vsp1/vsp1.h
@@ -33,6 +33,7 @@ struct vsp1_platform_data;
 struct vsp1_bru;
 struct vsp1_clu;
 struct vsp1_hgo;
+struct vsp1_hgt;
 struct vsp1_hsit;
 struct vsp1_lif;
 struct vsp1_lut;
@@ -52,6 +53,7 @@ struct vsp1_uds;
 #define VSP1_HAS_WPF_VFLIP     (1 << 5)
 #define VSP1_HAS_WPF_HFLIP     (1 << 6)
 #define VSP1_HAS_HGO           (1 << 7)
+#define VSP1_HAS_HGT           (1 << 8)
 
 struct vsp1_device_info {
        u32 version;
@@ -74,6 +76,7 @@ struct vsp1_device {
        struct vsp1_bru *bru;
        struct vsp1_clu *clu;
        struct vsp1_hgo *hgo;
+       struct vsp1_hgt *hgt;
        struct vsp1_hsit *hsi;
        struct vsp1_hsit *hst;
        struct vsp1_lif *lif;
diff --git a/drivers/media/platform/vsp1/vsp1_drv.c 
b/drivers/media/platform/vsp1/vsp1_drv.c
index 9684abf..828584f 100644
--- a/drivers/media/platform/vsp1/vsp1_drv.c
+++ b/drivers/media/platform/vsp1/vsp1_drv.c
@@ -31,6 +31,7 @@
 #include "vsp1_dl.h"
 #include "vsp1_drm.h"
 #include "vsp1_hgo.h"
+#include "vsp1_hgt.h"
 #include "vsp1_hsit.h"
 #include "vsp1_lif.h"
 #include "vsp1_lut.h"
@@ -107,6 +108,7 @@ static int vsp1_create_sink_links(struct vsp1_device *vsp1,
                        continue;
 
                if (source->type == VSP1_ENTITY_HGO ||
+                   source->type == VSP1_ENTITY_HGT ||
                    source->type == VSP1_ENTITY_LIF ||
                    source->type == VSP1_ENTITY_WPF)
                        continue;
@@ -160,6 +162,16 @@ static int vsp1_uapi_create_links(struct vsp1_device *vsp1)
                        return ret;
        }
 
+       if (vsp1->hgt) {
+               ret = media_create_pad_link(&vsp1->hgt->entity.subdev.entity,
+                                           HGT_PAD_SOURCE,
+                                           &vsp1->hgt->histo.video.entity, 0,
+                                           MEDIA_LNK_FL_ENABLED |
+                                           MEDIA_LNK_FL_IMMUTABLE);
+               if (ret < 0)
+                       return ret;
+       }
+
        if (vsp1->lif) {
                ret = media_create_pad_link(&vsp1->wpf[0]->entity.subdev.entity,
                                            RWPF_PAD_SOURCE,
@@ -300,6 +312,16 @@ static int vsp1_create_entities(struct vsp1_device *vsp1)
                list_add_tail(&vsp1->hgo->entity.list_dev, &vsp1->entities);
        }
 
+       if (vsp1->info->features & VSP1_HAS_HGT && vsp1->info->uapi) {
+               vsp1->hgt = vsp1_hgt_create(vsp1);
+               if (IS_ERR(vsp1->hgt)) {
+                       ret = PTR_ERR(vsp1->hgt);
+                       goto done;
+               }
+
+               list_add_tail(&vsp1->hgt->entity.list_dev, &vsp1->entities);
+       }
+
        /* The LIF is only supported when used in conjunction with the DU, in
         * which case the userspace API is disabled. If the userspace API is
         * enabled skip the LIF, even when present.
@@ -583,7 +605,8 @@ static const struct vsp1_device_info vsp1_device_infos[] = {
                .version = VI6_IP_VERSION_MODEL_VSPS_H2,
                .gen = 2,
                .features = VSP1_HAS_BRU | VSP1_HAS_CLU | VSP1_HAS_HGO
-                         | VSP1_HAS_LUT | VSP1_HAS_SRU | VSP1_HAS_WPF_VFLIP,
+                         | VSP1_HAS_HGT | VSP1_HAS_LUT | VSP1_HAS_SRU
+                         | VSP1_HAS_WPF_VFLIP,
                .rpf_count = 5,
                .uds_count = 3,
                .wpf_count = 4,
@@ -612,7 +635,8 @@ static const struct vsp1_device_info vsp1_device_infos[] = {
                .version = VI6_IP_VERSION_MODEL_VSPS_M2,
                .gen = 2,
                .features = VSP1_HAS_BRU | VSP1_HAS_CLU | VSP1_HAS_HGO
-                         | VSP1_HAS_LUT | VSP1_HAS_SRU | VSP1_HAS_WPF_VFLIP,
+                         | VSP1_HAS_HGT | VSP1_HAS_LUT | VSP1_HAS_SRU
+                         | VSP1_HAS_WPF_VFLIP,
                .rpf_count = 5,
                .uds_count = 1,
                .wpf_count = 4,
@@ -621,8 +645,8 @@ static const struct vsp1_device_info vsp1_device_infos[] = {
        }, {
                .version = VI6_IP_VERSION_MODEL_VSPI_GEN3,
                .gen = 3,
-               .features = VSP1_HAS_CLU | VSP1_HAS_HGO | VSP1_HAS_LUT
-                         | VSP1_HAS_SRU | VSP1_HAS_WPF_HFLIP
+               .features = VSP1_HAS_CLU | VSP1_HAS_HGO | VSP1_HAS_HGT
+                         | VSP1_HAS_LUT | VSP1_HAS_SRU | VSP1_HAS_WPF_HFLIP
                          | VSP1_HAS_WPF_VFLIP,
                .rpf_count = 1,
                .uds_count = 1,
diff --git a/drivers/media/platform/vsp1/vsp1_entity.c 
b/drivers/media/platform/vsp1/vsp1_entity.c
index a6b7d87..0a0c46e 100644
--- a/drivers/media/platform/vsp1/vsp1_entity.c
+++ b/drivers/media/platform/vsp1/vsp1_entity.c
@@ -49,6 +49,18 @@ void vsp1_entity_route_setup(struct vsp1_entity *entity,
 
                vsp1_dl_list_write(dl, VI6_DPR_HGO_SMPPT, smppt);
                return;
+       } else if (entity->type == VSP1_ENTITY_HGT) {
+               u32 smppt;
+
+               /* The HGT is a special case, its routing is configured on the
+                * sink pad.
+                */
+               source = media_entity_to_vsp1_entity(entity->sources[0]);
+               smppt = (pipe->output->entity.index << VI6_DPR_SMPPT_TGW_SHIFT)
+                     | (source->route->output << VI6_DPR_SMPPT_PT_SHIFT);
+
+               vsp1_dl_list_write(dl, VI6_DPR_HGT_SMPPT, smppt);
+               return;
        }
 
        source = entity;
@@ -302,9 +314,10 @@ static int vsp1_entity_link_setup_source(const struct 
media_pad *source_pad,
                        = media_entity_to_vsp1_entity(sink_pad->entity);
 
                /* Fan-out is limited to one for the normal data path plus an
-                * optional HGO. We ignore the HGO here.
+                * optional HGO or HGT. We ignore the HGO and HGT here.
                 */
-               if (sink->type != VSP1_ENTITY_HGO) {
+               if (sink->type != VSP1_ENTITY_HGO &&
+                   sink->type != VSP1_ENTITY_HGT) {
                        if (source->sink)
                                return -EBUSY;
                        source->sink = sink_pad->entity;
@@ -357,10 +370,10 @@ int vsp1_entity_link_setup(struct media_entity *entity,
  * links originating or terminating at that pad until an enabled link is found.
  *
  * Our link setup implementation guarantees that the output fan-out will not be
- * higher than one for the data pipelines, except for the link to the HGO that
- * can be enabled in addition to a regular data link. When traversing outgoing
- * links this function ignores HGO entities and should thus be used in place of
- * the generic media_entity_remote_pad() function when traversing data
+ * higher than one for the data pipelines, except for the link to the HGO or 
HGT
+ * that can be enabled in addition to a regular data link. When traversing
+ * outgoing links this function ignores HGO entities and should thus be used in
+ * place of the generic media_entity_remote_pad() function when traversing data
  * pipelines.
  *
  * Return a pointer to the pad at the remote end of the first found enabled
@@ -376,19 +389,20 @@ struct media_pad *vsp1_entity_remote_pad(struct media_pad 
*pad)
                if (!(link->flags & MEDIA_LNK_FL_ENABLED))
                        continue;
 
-               /* If we're the sink the source will never be an HGO. */
+               /* If we're the sink the source will never be an HGO or HGT. */
                if (link->sink == pad)
                        return link->source;
 
                if (link->source != pad)
                        continue;
 
-               /* If the sink isn't a subdevice it can't be an HGO. */
+               /* If the sink isn't a subdevice it can't be an HGO or HGT. */
                if (!is_media_entity_v4l2_subdev(link->sink->entity))
                        return link->sink;
 
                entity = media_entity_to_vsp1_entity(link->sink->entity);
-               if (entity->type != VSP1_ENTITY_HGO)
+               if (entity->type != VSP1_ENTITY_HGO &&
+                   entity->type != VSP1_ENTITY_HGT)
                        return link->sink;
        }
 
@@ -423,6 +437,7 @@ static const struct vsp1_route vsp1_routes[] = {
            VI6_DPR_NODE_BRU_IN(4) }, VI6_DPR_NODE_BRU_OUT },
        VSP1_ENTITY_ROUTE(CLU),
        { VSP1_ENTITY_HGO, 0, 0, { 0, }, 0 },
+       { VSP1_ENTITY_HGT, 0, 0, { 0, }, 0 },
        VSP1_ENTITY_ROUTE(HSI),
        VSP1_ENTITY_ROUTE(HST),
        { VSP1_ENTITY_LIF, 0, 0, { VI6_DPR_NODE_LIF, }, VI6_DPR_NODE_LIF },
diff --git a/drivers/media/platform/vsp1/vsp1_entity.h 
b/drivers/media/platform/vsp1/vsp1_entity.h
index 4fc2cd1..0682400 100644
--- a/drivers/media/platform/vsp1/vsp1_entity.h
+++ b/drivers/media/platform/vsp1/vsp1_entity.h
@@ -26,6 +26,7 @@ enum vsp1_entity_type {
        VSP1_ENTITY_BRU,
        VSP1_ENTITY_CLU,
        VSP1_ENTITY_HGO,
+       VSP1_ENTITY_HGT,
        VSP1_ENTITY_HSI,
        VSP1_ENTITY_HST,
        VSP1_ENTITY_LIF,
diff --git a/drivers/media/platform/vsp1/vsp1_hgt.c 
b/drivers/media/platform/vsp1/vsp1_hgt.c
new file mode 100644
index 0000000..c43373d
--- /dev/null
+++ b/drivers/media/platform/vsp1/vsp1_hgt.c
@@ -0,0 +1,495 @@
+/*
+ * vsp1_hgt.c  --  R-Car VSP1 Histogram Generator 2D
+ *
+ * Copyright (C) 2016 Renesas Electronics Corporation
+ *
+ * Contact: Niklas Söderlund (niklas.soderl...@ragnatech.se)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#include <linux/device.h>
+#include <linux/gfp.h>
+
+#include <media/v4l2-subdev.h>
+#include <media/videobuf2-vmalloc.h>
+
+#include "vsp1.h"
+#include "vsp1_dl.h"
+#include "vsp1_hgt.h"
+
+#define HGT_MIN_SIZE                           4U
+#define HGT_MAX_SIZE                           8192U
+#define HGT_DATA_SIZE                          ((2 + 6 + 6 * 32) * 4)
+
+/* 
-----------------------------------------------------------------------------
+ * Device Access
+ */
+
+static inline u32 vsp1_hgt_read(struct vsp1_hgt *hgt, u32 reg)
+{
+       return vsp1_read(hgt->entity.vsp1, reg);
+}
+
+static inline void vsp1_hgt_write(struct vsp1_hgt *hgt, struct vsp1_dl_list 
*dl,
+                                 u32 reg, u32 data)
+{
+       vsp1_dl_list_write(dl, reg, data);
+}
+
+/* 
-----------------------------------------------------------------------------
+ * Frame End Handler
+ */
+
+void vsp1_hgt_frame_end(struct vsp1_entity *entity)
+{
+       struct vsp1_hgt *hgt = to_hgt(&entity->subdev);
+       struct vsp1_histogram_buffer *buf;
+       unsigned int m, n;
+       u32 *data;
+
+       buf = vsp1_histogram_buffer_get(&hgt->histo);
+       if (!buf)
+               return;
+
+       data = buf->addr;
+
+       *data++ = vsp1_hgt_read(hgt, VI6_HGT_MAXMIN);
+       *data++ = vsp1_hgt_read(hgt, VI6_HGT_SUM);
+
+       for (n = 0; n < 6; n++)
+               *data++ = vsp1_hgt_read(hgt, VI6_HGT_HUE_AREA(n));
+
+       for (m = 0; m < 6; m++)
+               for (n = 0; n < 32; n++)
+                       *data++ = vsp1_hgt_read(hgt, VI6_HGT_HISTO(m, n));
+
+       vsp1_histogram_buffer_complete(&hgt->histo, buf, HGT_DATA_SIZE);
+}
+
+/* 
-----------------------------------------------------------------------------
+ * Controls
+ */
+
+#define V4L2_CID_VSP1_HGT_HUE_AREAS    (V4L2_CID_USER_BASE | 0x1001)
+
+static int hgt_hue_areas_s_ctrl(struct v4l2_ctrl *ctrl)
+{
+       struct vsp1_hgt *hgt = container_of(ctrl->handler, struct vsp1_hgt,
+                                           ctrls.handler);
+       int m, n;
+       bool ok;
+
+       /*
+        * Make sure values meet one of two possible HW constrains
+        * 0L <= 0U <= 1L <= 1U <= 2L <= 2U <= 3L <= 3U <= 4L <= 4U <= 5L <= 5U
+        * 0U <= 1L <= 1U <= 2L <= 2U <= 3L <= 3U <= 4L <= 4U <= 5L <= 5U <= 0L
+        */
+       for (m = 0; m <= 1; m++) {
+               ok = true;
+               for (n = 0; n < HGT_NUM_HUE_AREAS - 1; n++) {
+                       if (ctrl->p_new.p_u8[(m + n + 0) % HGT_NUM_HUE_AREAS] >
+                           ctrl->p_new.p_u8[(m + n + 1) % HGT_NUM_HUE_AREAS])
+                               ok = false;
+               }
+               if (ok)
+                       break;
+       }
+
+       /* Values do not match HW, adjust to a valid setting */
+       if (!ok) {
+               for (n = 0; n < HGT_NUM_HUE_AREAS - 1; n++) {
+                       if (ctrl->p_new.p_u8[n] > ctrl->p_new.p_u8[n+1])
+                               ctrl->p_new.p_u8[n] = ctrl->p_new.p_u8[n+1];
+               }
+       }
+
+       for (n = 0; n < HGT_NUM_HUE_AREAS; n++)
+               hgt->hue_area[n] = ctrl->p_new.p_u8[n];
+
+       return 0;
+}
+
+static const struct v4l2_ctrl_ops hgt_hue_areas_ctrl_ops = {
+       .s_ctrl = hgt_hue_areas_s_ctrl,
+};
+
+static const struct v4l2_ctrl_config hgt_hue_areas = {
+       .ops = &hgt_hue_areas_ctrl_ops,
+       .id = V4L2_CID_VSP1_HGT_HUE_AREAS,
+       .name = "Boundary Values for Hue Area",
+       .type = V4L2_CTRL_TYPE_U8,
+       .min = 0,
+       .max = 255,
+       .def = 0,
+       .step = 1,
+       .dims = { 12 },
+};
+
+
+/* 
-----------------------------------------------------------------------------
+ * V4L2 Subdevice Operations
+ */
+
+static int hgt_enum_mbus_code(struct v4l2_subdev *subdev,
+                              struct v4l2_subdev_pad_config *cfg,
+                              struct v4l2_subdev_mbus_code_enum *code)
+{
+       static const unsigned int codes[] = {
+               MEDIA_BUS_FMT_AHSV8888_1X32,
+       };
+
+       if (code->pad == HGT_PAD_SOURCE) {
+               code->code = MEDIA_BUS_FMT_FIXED;
+               return 0;
+       }
+
+       return vsp1_subdev_enum_mbus_code(subdev, cfg, code, codes,
+                                         ARRAY_SIZE(codes));
+}
+
+static int hgt_enum_frame_size(struct v4l2_subdev *subdev,
+                               struct v4l2_subdev_pad_config *cfg,
+                               struct v4l2_subdev_frame_size_enum *fse)
+{
+       if (fse->pad != HGT_PAD_SINK)
+               return -EINVAL;
+
+       return vsp1_subdev_enum_frame_size(subdev, cfg, fse, HGT_MIN_SIZE,
+                                          HGT_MIN_SIZE, HGT_MAX_SIZE,
+                                          HGT_MAX_SIZE);
+}
+
+static int hgt_get_selection(struct v4l2_subdev *subdev,
+                            struct v4l2_subdev_pad_config *cfg,
+                            struct v4l2_subdev_selection *sel)
+{
+       struct vsp1_hgt *hgt = to_hgt(subdev);
+       struct v4l2_subdev_pad_config *config;
+       struct v4l2_mbus_framefmt *format;
+       struct v4l2_rect *crop;
+
+       if (sel->pad != HGT_PAD_SINK)
+               return -EINVAL;
+
+       config = vsp1_entity_get_pad_config(&hgt->entity, cfg, sel->which);
+       if (!config)
+               return -EINVAL;
+
+       switch (sel->target) {
+       case V4L2_SEL_TGT_COMPOSE_BOUNDS:
+       case V4L2_SEL_TGT_COMPOSE_DEFAULT:
+               crop = vsp1_entity_get_pad_selection(&hgt->entity, config,
+                                                    HGT_PAD_SINK,
+                                                    V4L2_SEL_TGT_CROP);
+               sel->r.left = 0;
+               sel->r.top = 0;
+               sel->r.width = crop->width;
+               sel->r.height = crop->height;
+               return 0;
+
+       case V4L2_SEL_TGT_CROP_BOUNDS:
+       case V4L2_SEL_TGT_CROP_DEFAULT:
+               format = vsp1_entity_get_pad_format(&hgt->entity, config,
+                                                   HGT_PAD_SINK);
+               sel->r.left = 0;
+               sel->r.top = 0;
+               sel->r.width = format->width;
+               sel->r.height = format->height;
+               return 0;
+
+       case V4L2_SEL_TGT_COMPOSE:
+       case V4L2_SEL_TGT_CROP:
+               sel->r = *vsp1_entity_get_pad_selection(&hgt->entity, config,
+                                                       sel->pad, sel->target);
+               return 0;
+
+       default:
+               return -EINVAL;
+       }
+}
+
+static int hgt_set_crop(struct v4l2_subdev *subdev,
+                       struct v4l2_subdev_pad_config *config,
+                       struct v4l2_subdev_selection *sel)
+{
+       struct vsp1_hgt *hgt = to_hgt(subdev);
+       struct v4l2_mbus_framefmt *format;
+       struct v4l2_rect *selection;
+
+       /* The crop rectangle must be inside the input frame. */
+       format = vsp1_entity_get_pad_format(&hgt->entity, config, HGT_PAD_SINK);
+       sel->r.left = clamp_t(unsigned int, sel->r.left, 0, format->width - 1);
+       sel->r.top = clamp_t(unsigned int, sel->r.top, 0, format->height - 1);
+       sel->r.width = clamp_t(unsigned int, sel->r.width, HGT_MIN_SIZE,
+                              format->width - sel->r.left);
+       sel->r.height = clamp_t(unsigned int, sel->r.height, HGT_MIN_SIZE,
+                               format->height - sel->r.top);
+
+       /* Set the crop rectangle and reset the compose rectangle. */
+       selection = vsp1_entity_get_pad_selection(&hgt->entity, config,
+                                                 sel->pad, V4L2_SEL_TGT_CROP);
+       *selection = sel->r;
+
+       selection = vsp1_entity_get_pad_selection(&hgt->entity, config,
+                                                 sel->pad,
+                                                 V4L2_SEL_TGT_COMPOSE);
+       *selection = sel->r;
+
+       return 0;
+}
+
+static int hgt_set_compose(struct v4l2_subdev *subdev,
+                          struct v4l2_subdev_pad_config *config,
+                          struct v4l2_subdev_selection *sel)
+{
+       struct vsp1_hgt *hgt = to_hgt(subdev);
+       struct v4l2_rect *compose;
+       struct v4l2_rect *crop;
+       unsigned int ratio;
+
+       /* The compose rectangle is used to configure downscaling, the top left
+        * corner is fixed to (0,0) and the size to 1/2 or 1/4 of the crop
+        * rectangle.
+        */
+       sel->r.left = 0;
+       sel->r.top = 0;
+
+       crop = vsp1_entity_get_pad_selection(&hgt->entity, config, sel->pad,
+                                            V4L2_SEL_TGT_CROP);
+
+       /* Clamp the width and height to acceptable values first and then
+        * compute the closest rounded dividing ratio.
+        *
+        * Ratio        Rounded ratio
+        * --------------------------
+        * [1.0 1.5[    1
+        * [1.5 3.0[    2
+        * [3.0 4.0]    4
+        *
+        * The rounded ratio can be computed using
+        *
+        * 1 << (ceil(ratio * 2) / 3)
+        */
+       sel->r.width = clamp(sel->r.width, crop->width / 4, crop->width);
+       ratio = 1 << (crop->width * 2 / sel->r.width / 3);
+       sel->r.width = crop->width / ratio;
+
+
+       sel->r.height = clamp(sel->r.height, crop->height / 4, crop->height);
+       ratio = 1 << (crop->height * 2 / sel->r.height / 3);
+       sel->r.height = crop->height / ratio;
+
+       compose = vsp1_entity_get_pad_selection(&hgt->entity, config, sel->pad,
+                                               V4L2_SEL_TGT_COMPOSE);
+       *compose = sel->r;
+
+       return 0;
+}
+
+static int hgt_set_selection(struct v4l2_subdev *subdev,
+                            struct v4l2_subdev_pad_config *cfg,
+                            struct v4l2_subdev_selection *sel)
+{
+       struct vsp1_hgt *hgt = to_hgt(subdev);
+       struct v4l2_subdev_pad_config *config;
+
+       if (sel->pad != HGT_PAD_SINK)
+               return -EINVAL;
+
+       config = vsp1_entity_get_pad_config(&hgt->entity, cfg, sel->which);
+       if (!config)
+               return -EINVAL;
+
+       if (sel->target == V4L2_SEL_TGT_CROP)
+               return hgt_set_crop(subdev, config, sel);
+       else if (sel->target == V4L2_SEL_TGT_COMPOSE)
+               return hgt_set_compose(subdev, config, sel);
+       else
+               return -EINVAL;
+}
+
+static int hgt_get_format(struct v4l2_subdev *subdev,
+                          struct v4l2_subdev_pad_config *cfg,
+                          struct v4l2_subdev_format *fmt)
+{
+       if (fmt->pad == HGT_PAD_SOURCE) {
+               fmt->format.code = MEDIA_BUS_FMT_FIXED;
+               fmt->format.width = 0;
+               fmt->format.height = 0;
+               fmt->format.field = V4L2_FIELD_NONE;
+               fmt->format.colorspace = V4L2_COLORSPACE_RAW;
+               return 0;
+       }
+
+       return vsp1_subdev_get_pad_format(subdev, cfg, fmt);
+}
+
+static int hgt_set_format(struct v4l2_subdev *subdev,
+                          struct v4l2_subdev_pad_config *cfg,
+                          struct v4l2_subdev_format *fmt)
+{
+       struct vsp1_hgt *hgt = to_hgt(subdev);
+       struct v4l2_subdev_pad_config *config;
+       struct v4l2_mbus_framefmt *format;
+       struct v4l2_rect *selection;
+
+       if (fmt->pad != HGT_PAD_SINK)
+               return hgt_get_format(subdev, cfg, fmt);
+
+       config = vsp1_entity_get_pad_config(&hgt->entity, cfg, fmt->which);
+       if (!config)
+               return -EINVAL;
+
+       /* HSV is the only supported format */
+       fmt->format.code = MEDIA_BUS_FMT_AHSV8888_1X32;
+
+       format = vsp1_entity_get_pad_format(&hgt->entity, config, fmt->pad);
+
+       format->code = fmt->format.code;
+       format->width = clamp_t(unsigned int, fmt->format.width,
+                               HGT_MIN_SIZE, HGT_MAX_SIZE);
+       format->height = clamp_t(unsigned int, fmt->format.height,
+                                HGT_MIN_SIZE, HGT_MAX_SIZE);
+       format->field = V4L2_FIELD_NONE;
+       format->colorspace = V4L2_COLORSPACE_SRGB;
+
+       fmt->format = *format;
+
+       /* Reset the crop and compose rectangles */
+       selection = vsp1_entity_get_pad_selection(&hgt->entity, config,
+                                                 fmt->pad, V4L2_SEL_TGT_CROP);
+       selection->left = 0;
+       selection->top = 0;
+       selection->width = format->width;
+       selection->height = format->height;
+
+       selection = vsp1_entity_get_pad_selection(&hgt->entity, config,
+                                                 fmt->pad,
+                                                 V4L2_SEL_TGT_COMPOSE);
+       selection->left = 0;
+       selection->top = 0;
+       selection->width = format->width;
+       selection->height = format->height;
+
+       return 0;
+}
+
+static const struct v4l2_subdev_pad_ops hgt_pad_ops = {
+       .enum_mbus_code = hgt_enum_mbus_code,
+       .enum_frame_size = hgt_enum_frame_size,
+       .get_fmt = hgt_get_format,
+       .set_fmt = hgt_set_format,
+       .get_selection = hgt_get_selection,
+       .set_selection = hgt_set_selection,
+};
+
+static const struct v4l2_subdev_ops hgt_ops = {
+       .pad    = &hgt_pad_ops,
+};
+
+/* 
-----------------------------------------------------------------------------
+ * VSP1 Entity Operations
+ */
+
+static void hgt_configure(struct vsp1_entity *entity,
+                         struct vsp1_pipeline *pipe,
+                         struct vsp1_dl_list *dl, bool full)
+{
+       struct vsp1_hgt *hgt = to_hgt(&entity->subdev);
+       struct v4l2_rect *compose;
+       struct v4l2_rect *crop;
+       unsigned int hratio;
+       unsigned int vratio;
+       uint8_t lower, upper;
+       int i;
+
+       if (!full)
+               return;
+
+       crop = vsp1_entity_get_pad_selection(entity, entity->config,
+                                            HGT_PAD_SINK, V4L2_SEL_TGT_CROP);
+       compose = vsp1_entity_get_pad_selection(entity, entity->config,
+                                               HGT_PAD_SINK,
+                                               V4L2_SEL_TGT_COMPOSE);
+
+       vsp1_hgt_write(hgt, dl, VI6_HGT_REGRST, VI6_HGT_REGRST_RCLEA);
+
+       vsp1_hgt_write(hgt, dl, VI6_HGT_OFFSET,
+                      (crop->left << VI6_HGT_OFFSET_HOFFSET_SHIFT) |
+                      (crop->top << VI6_HGT_OFFSET_VOFFSET_SHIFT));
+       vsp1_hgt_write(hgt, dl, VI6_HGT_SIZE,
+                      (crop->width << VI6_HGT_SIZE_HSIZE_SHIFT) |
+                      (crop->height << VI6_HGT_SIZE_VSIZE_SHIFT));
+
+       mutex_lock(hgt->ctrls.handler.lock);
+       for (i = 0; i < 6; i++) {
+               lower = hgt->hue_area[i*2 + 0];
+               upper = hgt->hue_area[i*2 + 1];
+               vsp1_hgt_write(hgt, dl, VI6_HGT_HUE_AREA(i),
+                              (lower << VI6_HGT_HUE_AREA_LOWER_SHIFT) |
+                              (upper << VI6_HGT_HUE_AREA_UPPER_SHIFT));
+       }
+       mutex_unlock(hgt->ctrls.handler.lock);
+
+       hratio = crop->width * 2 / compose->width / 3;
+       vratio = crop->height * 2 / compose->height / 3;
+       vsp1_hgt_write(hgt, dl, VI6_HGT_MODE,
+                      (hratio << VI6_HGT_MODE_HRATIO_SHIFT) |
+                      (vratio << VI6_HGT_MODE_VRATIO_SHIFT));
+}
+
+static void hgt_destroy(struct vsp1_entity *entity)
+{
+       struct vsp1_hgt *hgt = to_hgt(&entity->subdev);
+
+       vsp1_histogram_cleanup(&hgt->histo);
+}
+
+static const struct vsp1_entity_operations hgt_entity_ops = {
+       .configure = hgt_configure,
+       .destroy = hgt_destroy,
+};
+
+/* 
-----------------------------------------------------------------------------
+ * Initialization and Cleanup
+ */
+
+struct vsp1_hgt *vsp1_hgt_create(struct vsp1_device *vsp1)
+{
+       struct vsp1_hgt *hgt;
+       int i, ret;
+
+       hgt = devm_kzalloc(vsp1->dev, sizeof(*hgt), GFP_KERNEL);
+       if (hgt == NULL)
+               return ERR_PTR(-ENOMEM);
+
+       hgt->entity.ops = &hgt_entity_ops;
+       hgt->entity.type = VSP1_ENTITY_HGT;
+
+       ret = vsp1_entity_init(vsp1, &hgt->entity, "hgt", 2, &hgt_ops,
+                              MEDIA_ENT_F_PROC_VIDEO_STATISTICS);
+       if (ret < 0)
+               return ERR_PTR(ret);
+
+       /* Initialize the control handler. */
+       for (i = 0; i < HGT_NUM_HUE_AREAS; i++)
+               hgt->hue_area[i] = hgt_hue_areas.def;
+       v4l2_ctrl_handler_init(&hgt->ctrls.handler, 12);
+       hgt->ctrls.hue_areas = v4l2_ctrl_new_custom(&hgt->ctrls.handler,
+                                                   &hgt_hue_areas, NULL);
+       hgt->entity.subdev.ctrl_handler = &hgt->ctrls.handler;
+
+       /* Initialize the video device and queue for statistics data. */
+       ret = vsp1_histogram_init(vsp1, &hgt->histo, hgt->entity.subdev.name,
+                                 HGT_DATA_SIZE, V4L2_META_FMT_VSP1_HGT);
+       if (ret < 0) {
+               vsp1_entity_destroy(&hgt->entity);
+               return ERR_PTR(ret);
+       }
+
+       return hgt;
+}
diff --git a/drivers/media/platform/vsp1/vsp1_hgt.h 
b/drivers/media/platform/vsp1/vsp1_hgt.h
new file mode 100644
index 0000000..a2f1eae
--- /dev/null
+++ b/drivers/media/platform/vsp1/vsp1_hgt.h
@@ -0,0 +1,51 @@
+/*
+ * vsp1_hgt.h  --  R-Car VSP1 Histogram Generator 2D
+ *
+ * Copyright (C) 2016 Renesas Electronics Corporation
+ *
+ * Contact: Niklas Söderlund (niklas.soderl...@ragnatech.se)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+#ifndef __VSP1_HGT_H__
+#define __VSP1_HGT_H__
+
+#include <media/media-entity.h>
+#include <media/v4l2-ctrls.h>
+#include <media/v4l2-subdev.h>
+
+#include "vsp1_entity.h"
+#include "vsp1_histo.h"
+
+struct vsp1_device;
+
+#define HGT_PAD_SINK                           0
+#define HGT_PAD_SOURCE                         1
+
+#define HGT_NUM_HUE_AREAS                      12
+
+struct vsp1_hgt {
+       struct vsp1_entity entity;
+       struct vsp1_histogram histo;
+
+       struct {
+               struct v4l2_ctrl_handler handler;
+               struct v4l2_ctrl *hue_areas;
+       } ctrls;
+
+       unsigned int hue_area[HGT_NUM_HUE_AREAS];
+};
+
+
+static inline struct vsp1_hgt *to_hgt(struct v4l2_subdev *subdev)
+{
+       return container_of(subdev, struct vsp1_hgt, entity.subdev);
+}
+
+struct vsp1_hgt *vsp1_hgt_create(struct vsp1_device *vsp1);
+void vsp1_hgt_frame_end(struct vsp1_entity *hgt);
+
+#endif /* __VSP1_HGT_H__ */
diff --git a/drivers/media/platform/vsp1/vsp1_pipe.c 
b/drivers/media/platform/vsp1/vsp1_pipe.c
index 0dd7c16..052a603 100644
--- a/drivers/media/platform/vsp1/vsp1_pipe.c
+++ b/drivers/media/platform/vsp1/vsp1_pipe.c
@@ -24,6 +24,7 @@
 #include "vsp1_dl.h"
 #include "vsp1_entity.h"
 #include "vsp1_hgo.h"
+#include "vsp1_hgt.h"
 #include "vsp1_pipe.h"
 #include "vsp1_rwpf.h"
 #include "vsp1_uds.h"
@@ -191,12 +192,19 @@ void vsp1_pipeline_reset(struct vsp1_pipeline *pipe)
                hgo->histo.pipe = NULL;
        }
 
+       if (pipe->hgt) {
+               struct vsp1_hgt *hgt = to_hgt(&pipe->hgt->subdev);
+
+               hgt->histo.pipe = NULL;
+       }
+
        INIT_LIST_HEAD(&pipe->entities);
        pipe->state = VSP1_PIPELINE_STOPPED;
        pipe->buffers_ready = 0;
        pipe->num_inputs = 0;
        pipe->bru = NULL;
        pipe->hgo = NULL;
+       pipe->hgt = NULL;
        pipe->lif = NULL;
        pipe->uds = NULL;
 }
@@ -278,6 +286,11 @@ int vsp1_pipeline_stop(struct vsp1_pipeline *pipe)
                           (7 << VI6_DPR_SMPPT_TGW_SHIFT) |
                           (VI6_DPR_NODE_UNUSED << VI6_DPR_SMPPT_PT_SHIFT));
 
+       if (pipe->hgt)
+               vsp1_write(vsp1, VI6_DPR_HGT_SMPPT,
+                          (7 << VI6_DPR_SMPPT_TGW_SHIFT) |
+                          (VI6_DPR_NODE_UNUSED << VI6_DPR_SMPPT_PT_SHIFT));
+
        v4l2_subdev_call(&pipe->output->entity.subdev, video, s_stream, 0);
 
        return ret;
@@ -304,6 +317,9 @@ void vsp1_pipeline_frame_end(struct vsp1_pipeline *pipe)
        if (pipe->hgo)
                vsp1_hgo_frame_end(pipe->hgo);
 
+       if (pipe->hgt)
+               vsp1_hgt_frame_end(pipe->hgt);
+
        if (pipe->frame_end)
                pipe->frame_end(pipe);
 
diff --git a/drivers/media/platform/vsp1/vsp1_pipe.h 
b/drivers/media/platform/vsp1/vsp1_pipe.h
index bd42eff..740bde2 100644
--- a/drivers/media/platform/vsp1/vsp1_pipe.h
+++ b/drivers/media/platform/vsp1/vsp1_pipe.h
@@ -73,6 +73,7 @@ enum vsp1_pipeline_state {
  * @output: WPF at the output of the pipeline
  * @bru: BRU entity, if present
  * @hgo: HGO entity, if present
+ * @hgt: HGT entity, if present
  * @lif: LIF entity, if present
  * @uds: UDS entity, if present
  * @uds_input: entity at the input of the UDS, if the UDS is present
@@ -99,6 +100,7 @@ struct vsp1_pipeline {
        struct vsp1_rwpf *output;
        struct vsp1_entity *bru;
        struct vsp1_entity *hgo;
+       struct vsp1_entity *hgt;
        struct vsp1_entity *lif;
        struct vsp1_entity *uds;
        struct vsp1_entity *uds_input;
diff --git a/drivers/media/platform/vsp1/vsp1_regs.h 
b/drivers/media/platform/vsp1/vsp1_regs.h
index d821348..d1d9af9 100644
--- a/drivers/media/platform/vsp1/vsp1_regs.h
+++ b/drivers/media/platform/vsp1/vsp1_regs.h
@@ -628,9 +628,17 @@
  */
 
 #define VI6_HGT_OFFSET                 0x3400
+#define VI6_HGT_OFFSET_HOFFSET_SHIFT   16
+#define VI6_HGT_OFFSET_VOFFSET_SHIFT   0
 #define VI6_HGT_SIZE                   0x3404
+#define VI6_HGT_SIZE_HSIZE_SHIFT       16
+#define VI6_HGT_SIZE_VSIZE_SHIFT       0
 #define VI6_HGT_MODE                   0x3408
+#define VI6_HGT_MODE_HRATIO_SHIFT      2
+#define VI6_HGT_MODE_VRATIO_SHIFT      0
 #define VI6_HGT_HUE_AREA(n)            (0x340c + (n) * 4)
+#define VI6_HGT_HUE_AREA_LOWER_SHIFT   16
+#define VI6_HGT_HUE_AREA_UPPER_SHIFT   0
 #define VI6_HGT_LB_TH                  0x3424
 #define VI6_HGT_LBn_H(n)               (0x3438 + (n) * 8)
 #define VI6_HGT_LBn_V(n)               (0x342c + (n) * 8)
@@ -639,6 +647,7 @@
 #define VI6_HGT_SUM                    0x3754
 #define VI6_HGT_LB_DET                 0x3758
 #define VI6_HGT_REGRST                 0x37fc
+#define VI6_HGT_REGRST_RCLEA           (1 << 0)
 
 /* 
-----------------------------------------------------------------------------
  * LIF Control Registers
diff --git a/drivers/media/platform/vsp1/vsp1_video.c 
b/drivers/media/platform/vsp1/vsp1_video.c
index 75e6e6c..7215e08 100644
--- a/drivers/media/platform/vsp1/vsp1_video.c
+++ b/drivers/media/platform/vsp1/vsp1_video.c
@@ -32,6 +32,7 @@
 #include "vsp1_dl.h"
 #include "vsp1_entity.h"
 #include "vsp1_hgo.h"
+#include "vsp1_hgt.h"
 #include "vsp1_pipe.h"
 #include "vsp1_rwpf.h"
 #include "vsp1_uds.h"
@@ -326,7 +327,7 @@ static int vsp1_video_pipeline_build_branch(struct 
vsp1_pipeline *pipe,
        if (ret < 0)
                return ret;
 
-       /* The main data path doesn't include the HGO, use
+       /* The main data path doesn't include the HGO or HGT, use
         * vsp1_entity_remote_pad() to traverse the graph.
         */
 
@@ -382,7 +383,7 @@ static int vsp1_video_pipeline_build_branch(struct 
vsp1_pipeline *pipe,
                                        : &input->entity;
                }
 
-               /* Follow the source link, ignoring any HGO. */
+               /* Follow the source link, ignoring any HGO or HGT. */
                pad = &entity->pads[entity->source_pad];
                pad = vsp1_entity_remote_pad(pad);
        }
@@ -444,6 +445,11 @@ static int vsp1_video_pipeline_build(struct vsp1_pipeline 
*pipe,
 
                        pipe->hgo = e;
                        hgo->histo.pipe = pipe;
+               } else if (e->type == VSP1_ENTITY_HGT) {
+                       struct vsp1_hgt *hgt = to_hgt(subdev);
+
+                       pipe->hgt = e;
+                       hgt->histo.pipe = pipe;
                }
        }
 
-- 
2.9.3

--
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

Reply via email to