Re: [PATCH v6 04/10] omap2: Fix camera resources for multiomap

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

 



Hi Felipe,

Thanks for the review.

On Monday 14 February 2011 13:35:59 Felipe Balbi wrote:
> On Mon, Feb 14, 2011 at 01:21:31PM +0100, Laurent Pinchart wrote:
> > diff --git a/arch/arm/mach-omap2/devices.c
> > b/arch/arm/mach-omap2/devices.c index 4cf48ea..5d844bd 100644
> > --- a/arch/arm/mach-omap2/devices.c
> > +++ b/arch/arm/mach-omap2/devices.c
> > @@ -38,7 +38,7 @@
> > 
> >  #if defined(CONFIG_VIDEO_OMAP2) || defined(CONFIG_VIDEO_OMAP2_MODULE)
> > 
> > -static struct resource cam_resources[] = {
> > +static struct resource omap2cam_resources[] = {
> 
> should this be __initdata ??

The resources will be used when the OMAP3 ISP module is loaded. Won't they be 
discared if marked as __initdata ?

> > @@ -158,6 +149,14 @@ int omap3_init_camera(void *pdata)
> > 
> >  }
> >  EXPORT_SYMBOL_GPL(omap3_init_camera);
> > 
> > +static inline void omap_init_camera(void)
> 
> why inline ? also, should this be marked __init ?

I suppose because it was inline, so it has been kept as inline. The function 
is used in a single place, so the compiler will probably auto-inline it. Is it 
an issue to keep it as inline ?

-- 
Regards,

Laurent Pinchart
--
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