Re: [PATH 0/3] ARM: OMAP1: Camera: Updates API usage and removes unused files

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

 



Hi Tony,

Sorry for the delayed answer.

On Nov 27, 2007 10:25 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> * Eduardo Valentin <edubezval@xxxxxxxxx> [071121 14:24]:
> >        This patch set updates v4l2 API usage and removes unused
> > files on camera code for omap1. Here is an overall description:
> >
> >        There are two updates on v4l2 API usage: on the use of videobuf and
> > v4l2-int-device deattach.
> >
> >        As the ov9640 is already moved to the v4l2-int-device API and
> > the camera code for omap1 too, this patch set also removes unused and
> > duplicated files. The info from this removed files already moved to board
> > specific files and to other files (see previous related patches).
> >
> > Signed-off-by: Eduardo Valentin <eduardo.valentin@xxxxxxxxxxx>
> >
> >        Here is its  overall diffstat:
> >  b/drivers/media/video/omap/Kconfig         |    3
> >  b/drivers/media/video/omap/camera_core.c   |   11
> >  b/drivers/media/video/omap/camera_core.h   |    3
> >  drivers/media/video/omap/camera_core.c     |   13
> >  drivers/media/video/omap/h3_sensor_power.c |   63 -
> >  drivers/media/video/omap/h3sensorpower.h   |   17
> >  drivers/media/video/omap/h4_sensor_power.c |   76 -
> >  drivers/media/video/omap/h4sensorpower.h   |   17
> >  drivers/media/video/omap/ov9640.h          |  179 ----
> >  drivers/media/video/omap/sensor_if.h       |   50 -
> >  drivers/media/video/omap/sensor_ov9640.c   | 1221 -----------------------------
> >  11 files changed, 14 insertions(+), 1639 deletions(-)
> >
>
> Pushing thise series today. Is there anything else that can be removed
> now regarding the camera stuff?

The code for the omap1 is clean. All unused files were removed.
However, I'd ask what you think about merging the code of

drivers/media/video/omap/camera_core.[c,h]
drivers/media/video/omap/omap16xxcam.[c,h]

and removing the drivers/media/video/omap/ directory. I'm suggesting
this change 'cause the code for omap2 is outside this directory.

>
> Tony
>



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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux