On Mon 2017-01-02 09:00:10, Sakari Ailus wrote: > Hi Pavel, > > On Wed, Dec 28, 2016 at 07:30:36PM +0100, Pavel Machek wrote: > > From: Sakari Ailus <sakari.ailus@xxxxxx> > > > > In the vast majority of cases the bus type is known to the driver(s) > > since a receiver or transmitter can only support a single one. There > > are cases however where different options are possible. > > > > Document the CSI1/CCP2 properties strobe_clk_inv and strobe_clock > > properties. The former tells whether the strobe/clock signal is > > inverted, while the latter signifies the clock or strobe mode. > > > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx> > > Signed-off-by: Ivaylo Dimitrov <ivo.g.dimitrov.75@xxxxxxxxx> > > Signed-off-by: Pavel Machek <pavel@xxxxxx> > > > > diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt b/Documentation/devicetree/bindings/media/video-interfaces.txt > > index 9cd2a36..f0523f7 100644 > > --- a/Documentation/devicetree/bindings/media/video-interfaces.txt > > +++ b/Documentation/devicetree/bindings/media/video-interfaces.txt > > @@ -76,6 +76,10 @@ Optional endpoint properties > > mode horizontal and vertical synchronization signals are provided to the > > slave device (data source) by the master device (data sink). In the master > > mode the data source device is also the source of the synchronization signals. > > +- bus-type: data bus type. Possible values are: > > + 0 - CSI2 > > + 1 - parallel / Bt656 > > + 2 - CCP2 > > I wonder if we should make a difference between CCP2 and CSI-1 here, as it > may make a difference in hardware configuration. The next patch does > recognise that difference, so it should be present here as well. > > Perhaps 2 - CSI1; 3 - CCP2. What do you think? Me, think..? Nah. :-). Well, you have way more experience here, and yes, based on future patches, this makes sense. Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Attachment:
signature.asc
Description: Digital signature
