Re: [PATCH v3 36/48] adv7604: Make output format configurable through pad format operations

2014-03-18 Thread Hans Verkuil
Hi Laurent, I've tested it and I thought I was going crazy. Everything was fine after applying this patch, but as soon as I applied the next patch (37/48) the colors were wrong. But that patch had nothing whatsoever to do with the bus ordering. You managed to make a small but crucial bug and it

Re: [PATCH v3 36/48] adv7604: Make output format configurable through pad format operations

2014-03-18 Thread Laurent Pinchart
Hi Hans, On Tuesday 18 March 2014 10:32:32 Hans Verkuil wrote: Hi Laurent, I've tested it and I thought I was going crazy. Everything was fine after applying this patch, but as soon as I applied the next patch (37/48) the colors were wrong. But that patch had nothing whatsoever to do with

Re: [PATCH v3 36/48] adv7604: Make output format configurable through pad format operations

2014-03-18 Thread Hans Verkuil
On 03/18/2014 02:02 PM, Laurent Pinchart wrote: Hi Hans, On Tuesday 18 March 2014 10:32:32 Hans Verkuil wrote: Hi Laurent, I've tested it and I thought I was going crazy. Everything was fine after applying this patch, but as soon as I applied the next patch (37/48) the colors were wrong.

Re: [PATCH v3 36/48] adv7604: Make output format configurable through pad format operations

2014-03-13 Thread Hans Verkuil
On 03/11/2014 04:10 PM, Laurent Pinchart wrote: Replace the dummy video format operations by pad format operations that configure the output format. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com My apologies, I completely forgot to test this today. It's going to be

[PATCH v3 36/48] adv7604: Make output format configurable through pad format operations

2014-03-11 Thread Laurent Pinchart
Replace the dummy video format operations by pad format operations that configure the output format. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- drivers/media/i2c/adv7604.c | 280 include/media/adv7604.h | 56 -