[PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

[PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

[Chaithrika U S]

Display driver for TI DM646x EVM

Signed-off-by: Manjunath Hadli <mrh@ti.com>
Signed-off-by: Brijesh Jadav <brijesh.j@ti.com>
Signed-off-by: Chaithrika U S <chaithrika@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

RE: [PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

[chaithrika]

Hi All,

Do you have any review comments on this patch set?

Regards,
Chaithrika

> -----Original Message-----
> From: Chaithrika U S [mailto:chaithrika@ti.com]
> Sent: Friday, May 08, 2009 6:55 PM
> To: linux-media@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 <mrh@ti.com>
> Signed-off-by: Brijesh Jadav <brijesh.j@ti.com>
> Signed-off-by: Chaithrika U S <chaithrika@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

Re: [PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

[Hans Verkuil]

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 <hverkuil@xs4all.nl>

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:chaithrika@ti.com]
> > Sent: Friday, May 08, 2009 6:55 PM
> > To: linux-media@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 <mrh@ti.com>
> > Signed-off-by: Brijesh Jadav <brijesh.j@ti.com>
> > Signed-off-by: Chaithrika U S <chaithrika@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 majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG

RE: [PATCH v3 0/4] ARM: DaVinci: DM646x Video: DM646x display driver

[chaithrika]

Hans,

> -----Original Message-----
> From: Hans Verkuil [mailto:hverkuil@xs4all.nl]
> Sent: Thursday, May 21, 2009 7:45 PM
> To: chaithrika
> Cc: linux-media@vger.kernel.org; davinci-linux-open-
> source@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 <hverkuil@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:chaithrika@ti.com]
> > > Sent: Friday, May 08, 2009 6:55 PM
> > > To: linux-media@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 <mrh@ti.com>
> > > Signed-off-by: Brijesh Jadav <brijesh.j@ti.com>
> > > Signed-off-by: Chaithrika U S <chaithrika@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 majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> 
> --
> Hans Verkuil - video4linux developer - sponsored by TANDBERG