On Tue, May 12, 2009 at 4:39 PM, <[email protected]> wrote: > From: Roel Kluin <[email protected]> > > Fix &&/|| typo. `default_norm' can be 0 (PAL), 1 (NTSC) or 2 (SECAM), > the condition tested was impossible. > > Signed-off-by: Roel Kluin <[email protected]> > Cc: Mauro Carvalho Chehab <[email protected]> > Cc: Hans Verkuil <[email protected]> > Signed-off-by: Andrew Morton <[email protected]> > ---
Hello, Was the patch actually tested against the hardware in question? While I agree that it looks ok, it can result in the default logic being inverted in some cases, which could expose other bugs and result in a regression. I just want to be confident that this patch was tested by somebody with the hardware and it isn't going into the codebase because "it obviously cannot be right". Cheers, Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
