提交 3495dcef 编写于 作者: S Sylwester Nawrocki 提交者: Mauro Carvalho Chehab

[media] s5p-fimc: Fix data structures documentation and cleanup debug trace

Correct inconsistencies in data structures' documentation.
Remove meaningless debug traces.
Signed-off-by: NSylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: NKyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
上级 6ba8d13b
...@@ -262,12 +262,7 @@ static unsigned int get_plane_size(struct fimc_frame *fr, unsigned int plane) ...@@ -262,12 +262,7 @@ static unsigned int get_plane_size(struct fimc_frame *fr, unsigned int plane)
{ {
if (!fr || plane >= fr->fmt->memplanes) if (!fr || plane >= fr->fmt->memplanes)
return 0; return 0;
dbg("%s: w: %d. h: %d. depth[%d]: %d",
__func__, fr->width, fr->height, plane, fr->fmt->depth[plane]);
return fr->f_width * fr->f_height * fr->fmt->depth[plane] / 8; return fr->f_width * fr->f_height * fr->fmt->depth[plane] / 8;
} }
static int queue_setup(struct vb2_queue *vq, unsigned int *num_buffers, static int queue_setup(struct vb2_queue *vq, unsigned int *num_buffers,
...@@ -283,12 +278,8 @@ static int queue_setup(struct vb2_queue *vq, unsigned int *num_buffers, ...@@ -283,12 +278,8 @@ static int queue_setup(struct vb2_queue *vq, unsigned int *num_buffers,
*num_planes = fmt->memplanes; *num_planes = fmt->memplanes;
dbg("%s, buffer count=%d, plane count=%d",
__func__, *num_buffers, *num_planes);
for (i = 0; i < fmt->memplanes; i++) { for (i = 0; i < fmt->memplanes; i++) {
sizes[i] = get_plane_size(&ctx->d_frame, i); sizes[i] = get_plane_size(&ctx->d_frame, i);
dbg("plane: %u, plane_size: %lu", i, sizes[i]);
allocators[i] = ctx->fimc_dev->alloc_ctx; allocators[i] = ctx->fimc_dev->alloc_ctx;
} }
......
...@@ -231,11 +231,7 @@ static int fimc_get_scaler_factor(u32 src, u32 tar, u32 *ratio, u32 *shift) ...@@ -231,11 +231,7 @@ static int fimc_get_scaler_factor(u32 src, u32 tar, u32 *ratio, u32 *shift)
return 0; return 0;
} }
} }
*shift = 0, *ratio = 1; *shift = 0, *ratio = 1;
dbg("s: %d, t: %d, shift: %d, ratio: %d",
src, tar, *shift, *ratio);
return 0; return 0;
} }
...@@ -267,10 +263,8 @@ int fimc_set_scaler_info(struct fimc_ctx *ctx) ...@@ -267,10 +263,8 @@ int fimc_set_scaler_info(struct fimc_ctx *ctx)
err("invalid source size: %d x %d", sx, sy); err("invalid source size: %d x %d", sx, sy);
return -EINVAL; return -EINVAL;
} }
sc->real_width = sx; sc->real_width = sx;
sc->real_height = sy; sc->real_height = sy;
dbg("sx= %d, sy= %d, tx= %d, ty= %d", sx, sy, tx, ty);
ret = fimc_get_scaler_factor(sx, tx, &sc->pre_hratio, &sc->hfactor); ret = fimc_get_scaler_factor(sx, tx, &sc->pre_hratio, &sc->hfactor);
if (ret) if (ret)
......
...@@ -135,9 +135,10 @@ enum fimc_color_fmt { ...@@ -135,9 +135,10 @@ enum fimc_color_fmt {
* @name: format description * @name: format description
* @fourcc: the fourcc code for this format, 0 if not applicable * @fourcc: the fourcc code for this format, 0 if not applicable
* @color: the corresponding fimc_color_fmt * @color: the corresponding fimc_color_fmt
* @depth: per plane driver's private 'number of bits per pixel'
* @memplanes: number of physically non-contiguous data planes * @memplanes: number of physically non-contiguous data planes
* @colplanes: number of physically contiguous data planes * @colplanes: number of physically contiguous data planes
* @depth: per plane driver's private 'number of bits per pixel'
* @flags: flags indicating which operation mode format applies to
*/ */
struct fimc_fmt { struct fimc_fmt {
enum v4l2_mbus_pixelcode mbus_code; enum v4l2_mbus_pixelcode mbus_code;
...@@ -171,7 +172,7 @@ struct fimc_dma_offset { ...@@ -171,7 +172,7 @@ struct fimc_dma_offset {
}; };
/** /**
* struct fimc_effect - the configuration data for the "Arbitrary" image effect * struct fimc_effect - color effect information
* @type: effect type * @type: effect type
* @pat_cb: cr value when type is "arbitrary" * @pat_cb: cr value when type is "arbitrary"
* @pat_cr: cr value when type is "arbitrary" * @pat_cr: cr value when type is "arbitrary"
...@@ -184,7 +185,6 @@ struct fimc_effect { ...@@ -184,7 +185,6 @@ struct fimc_effect {
/** /**
* struct fimc_scaler - the configuration data for FIMC inetrnal scaler * struct fimc_scaler - the configuration data for FIMC inetrnal scaler
*
* @scaleup_h: flag indicating scaling up horizontally * @scaleup_h: flag indicating scaling up horizontally
* @scaleup_v: flag indicating scaling up vertically * @scaleup_v: flag indicating scaling up vertically
* @copy_mode: flag indicating transparent DMA transfer (no scaling * @copy_mode: flag indicating transparent DMA transfer (no scaling
...@@ -220,7 +220,6 @@ struct fimc_scaler { ...@@ -220,7 +220,6 @@ struct fimc_scaler {
/** /**
* struct fimc_addr - the FIMC physical address set for DMA * struct fimc_addr - the FIMC physical address set for DMA
*
* @y: luminance plane physical address * @y: luminance plane physical address
* @cb: Cb plane physical address * @cb: Cb plane physical address
* @cr: Cr plane physical address * @cr: Cr plane physical address
...@@ -234,6 +233,7 @@ struct fimc_addr { ...@@ -234,6 +233,7 @@ struct fimc_addr {
/** /**
* struct fimc_vid_buffer - the driver's video buffer * struct fimc_vid_buffer - the driver's video buffer
* @vb: v4l videobuf buffer * @vb: v4l videobuf buffer
* @list: linked list structure for buffer queue
* @paddr: precalculated physical address set * @paddr: precalculated physical address set
* @index: buffer index for the output DMA engine * @index: buffer index for the output DMA engine
*/ */
...@@ -254,11 +254,10 @@ struct fimc_vid_buffer { ...@@ -254,11 +254,10 @@ struct fimc_vid_buffer {
* @offs_v: image vertical pixel offset * @offs_v: image vertical pixel offset
* @width: image pixel width * @width: image pixel width
* @height: image pixel weight * @height: image pixel weight
* @paddr: image frame buffer physical addresses
* @buf_cnt: number of buffers depending on a color format
* @payload: image size in bytes (w x h x bpp) * @payload: image size in bytes (w x h x bpp)
* @color: color format * @paddr: image frame buffer physical addresses
* @dma_offset: DMA offset in bytes * @dma_offset: DMA offset in bytes
* @fmt: fimc color format pointer
*/ */
struct fimc_frame { struct fimc_frame {
u32 f_width; u32 f_width;
...@@ -390,21 +389,22 @@ struct fimc_ctx; ...@@ -390,21 +389,22 @@ struct fimc_ctx;
/** /**
* struct fimc_dev - abstraction for FIMC entity * struct fimc_dev - abstraction for FIMC entity
*
* @slock: the spinlock protecting this data structure * @slock: the spinlock protecting this data structure
* @lock: the mutex protecting this data structure * @lock: the mutex protecting this data structure
* @pdev: pointer to the FIMC platform device * @pdev: pointer to the FIMC platform device
* @pdata: pointer to the device platform data * @pdata: pointer to the device platform data
* @variant: the IP variant information
* @id: FIMC device index (0..FIMC_MAX_DEVS) * @id: FIMC device index (0..FIMC_MAX_DEVS)
* @num_clocks: the number of clocks managed by this device instance * @num_clocks: the number of clocks managed by this device instance
* @clock[]: the clocks required for FIMC operation * @clock: clocks required for FIMC operation
* @regs: the mapped hardware registers * @regs: the mapped hardware registers
* @regs_res: the resource claimed for IO registers * @regs_res: the resource claimed for IO registers
* @irq: interrupt number of the FIMC subdevice * @irq: FIMC interrupt number
* @irq_queue: * @irq_queue: interrupt handler waitqueue
* @m2m: memory-to-memory V4L2 device information * @m2m: memory-to-memory V4L2 device information
* @vid_cap: camera capture device information * @vid_cap: camera capture device information
* @state: flags used to synchronize m2m and capture mode operation * @state: flags used to synchronize m2m and capture mode operation
* @alloc_ctx: videobuf2 memory allocator context
*/ */
struct fimc_dev { struct fimc_dev {
spinlock_t slock; spinlock_t slock;
...@@ -427,8 +427,7 @@ struct fimc_dev { ...@@ -427,8 +427,7 @@ struct fimc_dev {
/** /**
* fimc_ctx - the device context data * fimc_ctx - the device context data
* * @slock: spinlock protecting this data structure
* @lock: mutex protecting this data structure
* @s_frame: source frame properties * @s_frame: source frame properties
* @d_frame: destination frame properties * @d_frame: destination frame properties
* @out_order_1p: output 1-plane YCBCR order * @out_order_1p: output 1-plane YCBCR order
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册