On Fri, Mar 02, 2018 at 12:59:00PM -0600, Rob Herring wrote: > On Wed, Feb 28, 2018 at 03:31:26PM +0200, Sakari Ailus wrote: > > Hi Rob, > > > > Thanks for the review. > > > > On Tue, Feb 27, 2018 at 04:10:31PM -0600, Rob Herring wrote: > > > On Fri, Feb 23, 2018 at 10:13 AM, Andy Yeh <andy....@intel.com> wrote: > > > > From: Alan Chiang <alanx.chi...@intel.com> > > > > > > > > Dongwoon DW9807 is a voice coil lens driver. > > > > > > > > Also add a vendor prefix for Dongwoon for one did not exist previously. > > > > > > Where's that? > > > > Added by aece98a912d92444ea9da03b04269407d1308f1f . So that line isn't > > relevant indeed and should be removed. > > > > > > > > > > > > > Signed-off-by: Andy Yeh <andy....@intel.com> > > > > --- > > > > Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807.txt | 9 > > > > +++++++++ > > > > > > DACs generally go in bindings/iio/dac/ > > > > We have quite a few lens voice coil drivers under bindings/media/i2c now. I > > don't really object to putting this one to bindings/iio/dac but then the > > rest should be moved as well. > > > > The camera LED flash drivers are under bindings/leds so this would actually > > be analoguous to that. The lens voice coil drivers are perhaps still a bit > > more bound to the domain (camera) than the LED flash drivers. > > The h/w is bound to that function or just the s/w?
The hardware. I guess in principle you could use them for other purposes --- most devices seem to be current sinks with configurable current --- but I've never seen that. The datasheet (dw9714) is here: <URL:http://www.datasheetspdf.com/datasheet/download.php?id=840322> > > > I can send a patch if you think the existing bindings should be moved; let > > me know. > > I'm okay if they are separate as long as we're not going to see the > same device show up in both places. However, "i2c" is not the best Ack. I wouldn't expect that. The datasheets of such devices clearly label the devices voice coil module drivers. > directory choice. It should be by function, so we can find common > properties. I2c devices in the media subsystem tend to be peripherals that are always used with another device with access to some system bus. Camera sensors, lens devices and tuners can be found there currently. I don't know the original reasoning but it most likely is related to that. In terms of different kinds of devices we have currently at least the following: Camera ISPs and CSI-2 receivers Video muxes Video codecs Camera sensors Camera lens drivers (right now only voice coil modules?) Tuners (DVB, radio, analogue TV, whatever) Radio transmitters HDMI CEC Remote controllers JPEG codecs Cc Hans, too. -- Sakari Ailus sakari.ai...@linux.intel.com