Skip to content

Commit

Permalink
fbdev: sh_mobile_meram: Correct pointer check for YCbCr chroma plane
Browse files Browse the repository at this point in the history
The check was intended to test if we have a valid pointer to write into,
but it mistakenly checks the pointer contents instead.

Since a valid pointer is mandatory for the chroma data if a YCbCr format
is used, the pointer check has been removed.

Signed-off-by: Damian Hobson-Garcia <dhobsong@igel.co.jp>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
  • Loading branch information
Damian Hobson-Garcia authored and Paul Mundt committed Jun 24, 2011
1 parent bccaeaf commit 06c8a6a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/video/sh_mobile_meram.c
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ static inline void meram_get_next_icb_addr(struct sh_mobile_meram_info *pdata,
icb_offset = 0xc0000000 | (cfg->current_reg << 23);

*icb_addr_y = icb_offset | (cfg->icb[0].marker_icb << 24);
if ((*icb_addr_c) && is_nvcolor(cfg->pixelformat))
if (is_nvcolor(cfg->pixelformat))
*icb_addr_c = icb_offset | (cfg->icb[1].marker_icb << 24);
}

Expand Down

0 comments on commit 06c8a6a

Please sign in to comment.