On Mon, Oct 29, 2018 at 07:24:09PM +0100, Marco Felsch wrote: > Replace the vague binding by a more verbose. Remove the remote property > from the example since the driver don't support such a property. Also > remove the bus-width property from the endpoint since the driver don't > take care of it. > > Signed-off-by: Marco Felsch <m.fel...@pengutronix.de> > --- > .../devicetree/bindings/media/i2c/mt9m111.txt | 11 +++++++---- > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/mt9m111.txt > b/Documentation/devicetree/bindings/media/i2c/mt9m111.txt > index 6b910036b57e..921cc48c488b 100644 > --- a/Documentation/devicetree/bindings/media/i2c/mt9m111.txt > +++ b/Documentation/devicetree/bindings/media/i2c/mt9m111.txt > @@ -9,8 +9,13 @@ Required Properties: > - clocks: reference to the master clock. > - clock-names: shall be "mclk". > > -For further reading on port node refer to > -Documentation/devicetree/bindings/media/video-interfaces.txt. > +The device node must contain one 'port' child node with one 'endpoint' child > +sub-node for its digital output video port, in accordance with the video > +interface bindings defined in: > +Documentation/devicetree/bindings/media/video-interfaces.txt > + > +Required endpoint properties: > +- remote-endpoint: For information see ../video-interfaces.txt.
You don't need to explicitly document this. Otherwise, Reviewed-by: Rob Herring <r...@kernel.org>
