diff options
author | Damian Hobson-Garcia <dhobsong@igel.co.jp> | 2011-06-22 07:46:25 +0000 |
---|---|---|
committer | Paul Mundt <lethal@linux-sh.org> | 2011-06-24 16:40:28 +0900 |
commit | 06c8a6a3e40265ff580428bbf51991617477f65b (patch) | |
tree | 8631d0f0d32a72e7f3cc4a42dad82876189bcda0 /drivers/video | |
parent | bccaeafd7c117acee36e90d37c7e05c19be9e7bf (diff) | |
download | op-kernel-dev-06c8a6a3e40265ff580428bbf51991617477f65b.zip op-kernel-dev-06c8a6a3e40265ff580428bbf51991617477f65b.tar.gz |
fbdev: sh_mobile_meram: Correct pointer check for YCbCr chroma plane
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>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/sh_mobile_meram.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/video/sh_mobile_meram.c b/drivers/video/sh_mobile_meram.c index 9170c82..cc7d732 100644 --- a/drivers/video/sh_mobile_meram.c +++ b/drivers/video/sh_mobile_meram.c @@ -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); } |