| Message ID | 20161119003208.10550-4-khilman@baylibre.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Friday, November 18, 2016 4:32:08 PM CET Kevin Hilman wrote: > + > +Required properties: > +- compatible: must be "ti,vpif-capture" > +- reg: physical base address and length of the registers set for the device; > +- interrupts: should contain IRQ line for the VPIF > + > Shouldn't this have a SoC specific identifier or a version number in the compatible string? "vpif" seems rather generic, so it's likely that TI made more than one variant of it. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Arnd Bergmann <arnd@arndb.de> writes: > On Friday, November 18, 2016 4:32:08 PM CET Kevin Hilman wrote: >> + >> +Required properties: >> +- compatible: must be "ti,vpif-capture" >> +- reg: physical base address and length of the registers set for the device; >> +- interrupts: should contain IRQ line for the VPIF >> + >> > > Shouldn't this have a SoC specific identifier or a version number > in the compatible string? "vpif" seems rather generic, so it's > likely that TI made more than one variant of it. AFAICT, they used this for a single generation of davinci SoCs (dm6467, da850) and then moved on to using something completely different. But, that still proves your point because it's very SoC specific, so I'll make the compatible specific. Thanks for the review, Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Nov 18, 2016 at 4:32 PM, Kevin Hilman <khilman@baylibre.com> wrote: > Cc: Rob Herring <robh@kernel.org> > Signed-off-by: Kevin Hilman <khilman@baylibre.com> > --- > .../devicetree/bindings/media/ti,vpif-capture.txt | 65 ++++++++++++++++++++++ > .../devicetree/bindings/media/ti,vpif.txt | 8 +++ > 2 files changed, 73 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/ti,vpif-capture.txt > create mode 100644 Documentation/devicetree/bindings/media/ti,vpif.txt @DT maintainers: this can be ignored, I'm reworking this after some discussion with Laurent. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/media/ti,vpif-capture.txt b/Documentation/devicetree/bindings/media/ti,vpif-capture.txt new file mode 100644 index 000000000000..eaaaa46d3a5e --- /dev/null +++ b/Documentation/devicetree/bindings/media/ti,vpif-capture.txt @@ -0,0 +1,65 @@ +Texas Instruments VPIF Capture +------------------------------ + +The TI Video Port InterFace (VPIF) capture component is the primary +component for video capture on some TI DaVinci family SoCs. + +TI Document number reference: SPRUH82C + +Required properties: +- compatible: must be "ti,vpif-capture" +- reg: physical base address and length of the registers set for the device; +- interrupts: should contain IRQ line for the VPIF + +VPIF capture has a 16-bit parallel bus input, supporting 2 8-bit +channels or a single 16-bit channel. It should contain at least one +port child node with child 'endpoint' node. Please refer to the +bindings defined in +Documentation/devicetree/bindings/media/video-interfaces.txt. + +Example using 2 8-bit input channels, one of which is connected to an +I2C-connected TVP5147 decoder: + + vpif_capture: video-capture@0x00217000 { + compatible = "ti,vpif-capture"; + reg = <0x00217000 0x1000>; + interrupts = <92>; + + port { + vpif_ch0: endpoint@0 { + reg = <0>; + bus-width = <8>; + remote-endpoint = <&composite>; + }; + + vpif_ch1: endpoint@1 { + reg = <1>; + bus-width = <8>; + data-shift = <8>; + }; + }; + }; + +[ ... ] + +&i2c0 { + + tvp5147@5d { + compatible = "ti,tvp5147"; + reg = <0x5d>; + status = "okay"; + + port { + composite: endpoint { + hsync-active = <1>; + vsync-active = <1>; + pclk-sample = <0>; + + /* VPIF channel 0 (lower 8-bits) */ + remote-endpoint = <&vpif_ch0>; + bus-width = <8>; + }; + }; + }; + +}; diff --git a/Documentation/devicetree/bindings/media/ti,vpif.txt b/Documentation/devicetree/bindings/media/ti,vpif.txt new file mode 100644 index 000000000000..0d5c16531c0e --- /dev/null +++ b/Documentation/devicetree/bindings/media/ti,vpif.txt @@ -0,0 +1,8 @@ +Texas Instruments VPIF +---------------------- + +The Video Port InterFace (VPIF) is the core component for video output +and capture on TI Davinci family SoCs. + +- compatible: must be "ti,vpif" +- reg: physical base address and length of the registers set for the device;
Cc: Rob Herring <robh@kernel.org> Signed-off-by: Kevin Hilman <khilman@baylibre.com> --- .../devicetree/bindings/media/ti,vpif-capture.txt | 65 ++++++++++++++++++++++ .../devicetree/bindings/media/ti,vpif.txt | 8 +++ 2 files changed, 73 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/ti,vpif-capture.txt create mode 100644 Documentation/devicetree/bindings/media/ti,vpif.txt