RE: [PATCH 3/3] OMAP2/3 V4L2 Display Driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Shah, Hardik
> Sent: Thursday, April 30, 2009 11:51 AM
> To: InKi Dae
> Cc: tomi.valkeinen@xxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux-
> omap@xxxxxxxxxxxxxxx; Jadav, Brijesh R; Hiremath, Vaibhav
> Subject: RE: [PATCH 3/3] OMAP2/3 V4L2 Display Driver
> 
> 
> 
> > -----Original Message-----
> > From: InKi Dae [mailto:daeinki@xxxxxxxxx]
> > Sent: Thursday, April 30, 2009 11:48 AM
> > To: Shah, Hardik
> > Cc: tomi.valkeinen@xxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux-
> > omap@xxxxxxxxxxxxxxx; Jadav, Brijesh R; Hiremath, Vaibhav
> > Subject: Re: [PATCH 3/3] OMAP2/3 V4L2 Display Driver
> >
> > hello Shah, Hardik..
> >
> > your omap_vout.c has the problem that it disables video1 or fb1.
> > so I have modified your code.
> >
> > I defined and set platform_data for DSS2 in machine code.(or board file)
> >
> > static struct omapfb_platform_data xxx_dss_platform_data = {
> >     .mem_desc.region[0].format = OMAPFB_COLOR_ARGB32,
> >     .mem_desc.region[0].format_used=1,
> >
> >     .mem_desc.region[1].format = OMAPFB_COLOR_RGB24U,
> >     .mem_desc.region[1].format_used=1,
> >
> >     .mem_desc.region[2].format = OMAPFB_COLOR_ARGB32,
> >     .mem_desc.region[2].format_used=1,
> > };
> >
> > omapfb_set_platform_data(&xxx_dss_platform_data);
> >
> > after that, omap_vout has resource count got referring to framebuffer count,
> > registers overlay as vout's one and would decide to use which overlay.
> >
> > at that time, your code would face with impact on some overlay(fb or video).
> >
> > this patch would solve that problem.
> > when it sets overlay to vout, vout would get overlay array index to
> > avoid overlapping with other overlay.
> >
> >
> > sighed-off-by: InKi Dae. <inki.dae@xxxxxxxxxxx>
> > ---
> > diff --git a/drivers/media/video/omap/omap_vout.c
> > b/drivers/media/video/omap/omap_vout.c
> > index 9b4a0d7..051298a 100644
> > --- a/drivers/media/video/omap/omap_vout.c
> > +++ b/drivers/media/video/omap/omap_vout.c
> > @@ -2246,11 +2246,13 @@ free_buffers:
> >  /* Create video out devices */
> >  static int __init omap_vout_create_video_devices(struct platform_device
> > *pdev)
> >  {
> > -	int r = 0, k;
> > +	int r = 0, k, vout_count;
> >  	struct omap_vout_device *vout;
> >  	struct video_device *vfd = NULL;
> >  	struct omap2video_device *vid_dev = platform_get_drvdata(pdev);
> >
> > +	vout_count = 3 - pdev->num_resources;
> > +
> >  	for (k = 0; k < pdev->num_resources; k++) {
> >
> >  		vout = kmalloc(sizeof(struct omap_vout_device), GFP_KERNEL);
> > @@ -2266,9 +2268,9 @@ static int __init
> > omap_vout_create_video_devices(struct platform_device *pdev)
> >  		vout->vid = k;
> >  		vid_dev->vouts[k] = vout;
> >  		vout->vid_info.vid_dev = vid_dev;
> > -		vout->vid_info.overlays[0] = vid_dev->overlays[k + 1];
> > +		vout->vid_info.overlays[0] = vid_dev->overlays[k + vout_count];
> >  		vout->vid_info.num_overlays = 1;
> > -		vout->vid_info.id = k + 1;
> > +		vout->vid_info.id = k + vout_count;
> >  		vid_dev->num_videos++;
> >
> >  		/* Setup the default configuration for the video devices
> > @@ -2289,7 +2291,7 @@ static int __init
> > omap_vout_create_video_devices(struct platform_device *pdev)
> >  		/* Register the Video device with V4L2
> >  		 */
> >  		vfd = vout->vfd;
> > -		if (video_register_device(vfd, VFL_TYPE_GRABBER, k + 1) < 0) {
> > +		if (video_register_device(vfd, VFL_TYPE_GRABBER, k + vout_count) <
> > 0) {
> >  			printk(KERN_ERR VOUT_NAME ": could not register \
> >  					Video for Linux device\n");
> >  			vfd->minor = -1;
> >
> >
> > 2009/4/22 Shah, Hardik <hardik.shah@xxxxxx>:
> [Shah, Hardik] Yes this is correct,
> I will apply this patch.  I already found it and fixed it in different way but
> any way I will apply your patch.
[Shah, Hardik] Further on this inki.  Solving this bug will give rise to couple of more bugs related to global_alpha and pixel formats. That also is fixed. You can refer http://arago-project.org/git/people/vaibhav/ti-psp-omap-video.git?p=people/vaibhav/ti-psp-omap-video.git;a=summary
> > >
> > >
> > >> -----Original Message-----
> > >> From: Tomi Valkeinen [mailto:tomi.valkeinen@xxxxxxxxx]
> > >> Sent: Wednesday, April 22, 2009 1:53 PM
> > >> To: Shah, Hardik
> > >> Cc: linux-media@xxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx; Jadav,
> Brijesh
> > R;
> > >> Hiremath, Vaibhav
> > >> Subject: Re: [PATCH 3/3] OMAP2/3 V4L2 Display Driver
> > >>
> > >> Hi,
> > >>
> > >> On Wed, 2009-04-22 at 08:25 +0200, ext Hardik Shah wrote:
> > >> > This is the version 5th of the Driver.
> > >> >
> > >> > Following are the features supported.
> > >> > 1. Provides V4L2 user interface for the video pipelines of DSS
> > >> > 2. Basic streaming working on LCD and TV.
> > >> > 3. Support for various pixel formats like YUV, UYVY, RGB32, RGB24,
> RGB565
> > >> > 4. Supports Alpha blending.
> > >> > 5. Supports Color keying both source and destination.
> > >> > 6. Supports rotation with RGB565 and RGB32 pixels formats.
> > >> > 7. Supports cropping.
> > >> > 8. Supports Background color setting.
> > >> > 9. Works on latest DSS2 library from Tomi
> > >> > http://www.bat.org/~tomba/git/linux-omap-dss.git/
> > >> > 10. 1/4x scaling added.  Detail testing left
> > >> >
> > >> > TODOS
> > >> > 1. Ioctls needs to be added for color space conversion matrix
> > >> > coefficient programming.
> > >> > 2. To be tested on DVI resolutions.
> > >> >
> > >> > Comments fixed from community.
> > >> > 1. V4L2 Driver for OMAP3/3 DSS.
> > >> > 2.  Conversion of the custom ioctls to standard V4L2 ioctls like alpha
> > >> blending,
> > >> > color keying, rotation and back ground color setting
> > >> > 3.  Re-organised the code as per community comments.
> > >> > 4.  Added proper copyright year.
> > >> > 5.  Added module name in printk
> > >> > 6.  Kconfig option copy/paste error
> > >> > 7.  Module param desc addded.
> > >> > 8.  Query control implemented using standard query_fill
> > >> > 9.  Re-arranged if-else constructs.
> > >> > 10. Changed to use mutex instead of semaphore.
> > >> > 11. Removed dual usage of rotation angles.
> > >> > 12. Implemented function to convert the V4L2 angle to DSS angle.
> > >> > 13. Y-position was set half by video driver for TV output
> > >> > Now its done by DSS so removed that.
> > >> > 14. Minor cleanup
> > >> > 15. Added support to pass the page offset to application.
> > >> > 14. Minor cleanup
> > >> > 15. Added support to pass the page offset to application.
> > >> > 16. Renamed V4L2_CID_ROTATION to V4L2_CID_ROTATE
> > >> > 17. Major comment from Hans fixed.
> > >> > 18. Copy right year changed.
> > >> > 19. Added module name for each error/warning print message.
> > >> >
> > >> > Changes from Previous Version.
> > >> > 1. Supported YUV rotation.
> > >> > 2. Supported Flipping.
> > >> > 3. Rebased line with Tomi's latest DSS2 master branch with commit  id
> > >> > f575a02edf2218a18d6f2ced308b4f3e26b44ce2.
> > >> > 4. Kconfig option removed to select between the TV and LCD.
> > >> > Now supported dynamically by DSS2 library.
> > >> > 5. Kconfig option for the NTSC_M and PAL_BDGHI mode but not
> > >> > supported by DSS2.  so it will not work now.
> > >>
> > >> There is basic support for this. See the DSS doc:
> > >>
> > >> /sys/devices/platform/omapdss/display? directory:
> > >> ...
> > >> timings         Display timings
> > (pixclock,xres/hfp/hbp/hsw,yres/vfp/vbp/vsw)
> > >>                 When writing, two special timings are accepted for tv-
> out:
> > >>                 "pal" and "ntsc"
> > > [Shah, Hardik] I was not aware of it will remove the compile time option
> and
> > for now let the sysfs entry change the standard.  In future I will try to do
> > it with the S_STD and G_STD ioctls of the V4L2 framework.
> > >>
> > >>  Tomi
> > >>
> > >>
> > >
> > > --
> > > To unsubscribe from this list: send the line "unsubscribe linux-media" in
> > > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > >
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux