[PATCH] drivers/media: nuvoton: fix chip id probe v2

2010-11-19 Thread Nicolas Kaiser
Make sure we have a matching chip id high and one or the other of the chip id low values. Print the values if the probe fails. Signed-off-by: Nicolas Kaiser ni...@nikai.net --- drivers/media/IR/nuvoton-cir.c |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH] drivers/media: nuvoton: fix chip id probe

2010-11-17 Thread Nicolas Kaiser
Make sure we have a matching chip id high and one or the other of the chip id low values. Print the values if the probe fails. Signed-off-by: Nicolas Kaiser ni...@nikai.net --- Like this? Supersedes patch drivers/media: nuvoton: always true expression. drivers/media/IR/nuvoton-cir.c |7

[PATCH] drivers/media: nuvoton: always true expression

2010-11-16 Thread Nicolas Kaiser
I noticed that the second part of this conditional is always true. Would the intention be to strictly check on both chip_major and chip_minor? Signed-off-by: Nicolas Kaiser ni...@nikai.net --- drivers/media/IR/nuvoton-cir.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff

[PATCH] media/video/gspca: fix error check

2010-10-24 Thread Nicolas Kaiser
It looks to me like it was intended to check the return value at this point. Signed-off-by: Nicolas Kaiser ni...@nikai.net --- drivers/media/video/gspca/cpia1.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/media/video/gspca/cpia1.c b/drivers/media/video

[PATCH] staging/cx25821: fix messages that line wrap within quotes

2010-10-17 Thread Nicolas Kaiser
Fix messages that line wrap within quotes. Signed-off-by: Nicolas Kaiser ni...@nikai.net --- drivers/staging/cx25821/cx25821-core.c |4 +- drivers/staging/cx25821/cx25821-video-upstream.c | 36 +++--- drivers/staging/cx25821/cx25821-video.c |8 ++-- 3