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 <[email protected]> I'm happy with these patches! 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. Regards, Hans > > Regards, > Chaithrika > > > -----Original Message----- > > From: Chaithrika U S [mailto:[email protected]] > > Sent: Friday, May 08, 2009 6:55 PM > > To: [email protected] > > Cc: [email protected]; 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 <[email protected]> > > Signed-off-by: Brijesh Jadav <[email protected]> > > Signed-off-by: Chaithrika U S <[email protected]> > > > > 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 [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Hans Verkuil - video4linux developer - sponsored by TANDBERG _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
