Hans, > -----Original Message----- > From: Hans Verkuil [mailto:hverk...@xs4all.nl] > Sent: Thursday, May 21, 2009 7:45 PM > To: chaithrika > Cc: linux-me...@vger.kernel.org; davinci-linux-open- > sou...@linux.davincidsp.com; 'Manjunath Hadli'; 'Brijesh Jadav' > Subject: Re: [PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display > driver > > On Thursday 21 May 2009 08:12:57 chaithrika wrote: > > Hi All, > > > > Do you have any review comments on this patch set? > > Reviewed-by: Hans Verkuil <hverk...@xs4all.nl> > > I'm happy with these patches! >
Thanks! > There is one thing that can be improved, though. It is really an > enhancement > and does not prevent this from being merged. > > Currently the isr routine refuses to switch to the next frame if the > dma > queue is empty. However, I see no reason for this: it should always go > to > that frame regardless and keep that until new frames are queued. I've > made > this change in the (very old) driver I use at work, but this should > become > standard behavior. > OK. I will look into this. Thanks, Chaithrika > Regards, > > Hans > > > > > Regards, > > Chaithrika > > > > > -----Original Message----- > > > From: Chaithrika U S [mailto:chaithr...@ti.com] > > > Sent: Friday, May 08, 2009 6:55 PM > > > To: linux-me...@vger.kernel.org > > > Cc: davinci-linux-open-source@linux.davincidsp.com; Manjunath > Hadli; > > > Brijesh Jadav; Chaithrika U S > > > Subject: [PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display > > > driver > > > > > > Display driver for TI DM646x EVM > > > > > > Signed-off-by: Manjunath Hadli <m...@ti.com> > > > Signed-off-by: Brijesh Jadav <brijes...@ti.com> > > > Signed-off-by: Chaithrika U S <chaithr...@ti.com> > > > > > > These patches add the display driver support for TI DM646x EVM. > > > This patch set has been tested for basic display functionality for > > > Composite and Component outputs. > > > > > > This patch set consists of the updates based on the review comments > by > > > Hans Verkuil. > > > > > > Patch 1: Display device platform and board setup > > > Patch 2: VPIF driver > > > Patch 3: DM646x display driver > > > Patch 4: Makefile and config files modifications for Display > > > > > > Some of the features like the HBI/VBI support are not yet > implemented. > > > Also there are some known issues in the code implementation like > > > fine tuning to be done to TRY_FMT ioctl.The USERPTR usage has not > been > > > tested extensively. > > > > > > -Chaithrika > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux- > media" in > > the body of a message to majord...@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > -- > Hans Verkuil - video4linux developer - sponsored by TANDBERG _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
