Re: [PATCH 7/7] marvell-cam: Basic working MMP camera driver

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

 



On Tue, 7 Jun 2011, Jonathan Corbet wrote:

> On Tue, 7 Jun 2011 09:44:45 +0200 (CEST)
> Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote:
> 
> > > +obj-$(CONFIG_VIDEO_MMP_CAMERA) += marvell-ccic/  
> > 
> > Wouldn't it be better to have only one symbol, selecting the marvell-ccic 
> > directory in the Makefile and have all CAFE implementations select that 
> > symbol?
> 
> Except there's two drivers in that directory and you'll almost never want
> to build them both.  I guess I could replace one Makefile line with two
> Kconfig lines, but I'm not sure it would improve things.

Yes, indeed I meant 3 Kconfig symbols: 1 common and 1 per driver, each of 
which selects the common one. Sorry for being unclear:)

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
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