diff options
| author | Liu Ying <b17645@freescale.com> | 2010-01-12 13:00:02 -0500 |
|---|---|---|
| committer | Alejandro Gonzalez <alex.gonzalez@digi.com> | 2010-05-24 11:50:03 +0200 |
| commit | 80c16707393747ecb1539185823fdd222c562959 (patch) | |
| tree | 31e03677cf06c0da93f8967ab37c62276e51993a /drivers/mxc | |
| parent | 50ece9945c968bcd39162c79b8bda056907f545e (diff) | |
ENGR00119975 IPUv3:Inform user when IDMAC UV-offset overflows
UBO and VBO are 22-bit in CPMEM. When processing a high resolution
frame in YUV format, the value of UV-offset may overflow, but the
driver keeps silent.
Signed-off-by: Liu Ying <b17645@freescale.com>
Signed-off-by: Alejandro Gonzalez <alex.gonzalez@digi.com>
Diffstat (limited to 'drivers/mxc')
| -rw-r--r-- | drivers/mxc/ipu3/ipu_param_mem.h | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/drivers/mxc/ipu3/ipu_param_mem.h b/drivers/mxc/ipu3/ipu_param_mem.h index 28358fd2f3b6..74bb9d9eb31f 100644 --- a/drivers/mxc/ipu3/ipu_param_mem.h +++ b/drivers/mxc/ipu3/ipu_param_mem.h @@ -284,6 +284,14 @@ static inline void _ipu_ch_param_init(int ch, v_offset = v; } + /* UBO and VBO are 22-bit */ + if (u_offset/8 > 0x3fffff) + dev_err(g_ipu_dev, + "The value of U offset exceeds IPU limitation\n"); + if (v_offset/8 > 0x3fffff) + dev_err(g_ipu_dev, + "The value of V offset exceeds IPU limitation\n"); + ipu_ch_param_set_field(¶ms, 0, 46, 22, u_offset / 8); ipu_ch_param_set_field(¶ms, 0, 68, 22, v_offset / 8); @@ -500,6 +508,15 @@ static inline void _ipu_ch_offset_update(int ch, if (v_fix > v_offset) v_offset = v_fix; + + /* UBO and VBO are 22-bit */ + if (u_offset/8 > 0x3fffff) + dev_err(g_ipu_dev, + "The value of U offset exceeds IPU limitation\n"); + if (v_offset/8 > 0x3fffff) + dev_err(g_ipu_dev, + "The value of V offset exceeds IPU limitation\n"); + ipu_ch_param_mod_field(ipu_ch_param_addr(ch), 0, 46, 22, u_offset / 8); ipu_ch_param_mod_field(ipu_ch_param_addr(ch), 0, 68, 22, v_offset / 8); |