On Thu, Aug 22, 2013 at 08:53:13AM +0200, Jean-Francois Moine wrote:
On Wed, 21 Aug 2013 23:36:05 +0100 Russell King - ARM Linux linux@arm.linux.org.uk wrote:
AFAIK, the TI boards have no "pin-swapped", nor has the Cubox (there is no need to set the bit CFG_GRA_SWAPRB of the register LCD_SPU_DMA_CTRL0 of the Dove lcd for RGB or YUV formats).
Which board needs a special VIP configuration?
If you run the NXP driver, and then run this driver, things get messed up - which has already been covered months ago when this patch was first brought up.
It's there to ensure that the TDA998x is correctly configured no matter what it's previous state is, and prevent the thing being fragile as hell.
The NXP driver will never go to the mainline, so, I don't see the problem. If you want to use it to test some other drivers, you should better patch it instead of adding useless code in the TDA998x driver.
Sorry, you're wrong.