Hi Laurent, On Thu, Dec 22, 2016 at 6:05 PM, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > On Wednesday 21 Dec 2016 08:10:37 Ramesh Shanmugasundaram wrote: >> Add binding documentation for Renesas R-Car Digital Radio Interface >> (DRIF) controller. >> >> Signed-off-by: Ramesh Shanmugasundaram >> <ramesh.shanmugasundaram@xxxxxxxxxxxxxx> --- >> .../devicetree/bindings/media/renesas,drif.txt | 202 ++++++++++++++++++ >> 1 file changed, 202 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/media/renesas,drif.txt >> >> diff --git a/Documentation/devicetree/bindings/media/renesas,drif.txt >> b/Documentation/devicetree/bindings/media/renesas,drif.txt new file mode >> 100644 >> index 0000000..1f3feaf >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/renesas,drif.txt >> +Optional properties of an internal channel when: >> + - It is the only enabled channel of the bond (or) >> + - If it acts as primary among enabled bonds >> +-------------------------------------------------------- >> +- renesas,syncmd : sync mode >> + 0 (Frame start sync pulse mode. 1-bit width pulse >> + indicates start of a frame) >> + 1 (L/R sync or I2S mode) (default) >> +- renesas,lsb-first : empty property indicates lsb bit is received >> first. >> + When not defined msb bit is received first (default) >> +- renesas,syncac-active: Indicates sync signal polarity, 0/1 for low/high >> + respectively. The default is 1 (active high) >> +- renesas,dtdl : delay between sync signal and start of reception. >> + The possible values are represented in 0.5 clock >> + cycle units and the range is 0 to 4. The default >> + value is 2 (i.e.) 1 clock cycle delay. >> +- renesas,syncdl : delay between end of reception and sync signal >> edge. >> + The possible values are represented in 0.5 clock >> + cycle units and the range is 0 to 4 & 6. The default >> + value is 0 (i.e.) no delay. > > Most of these properties are pretty similar to the video bus properties > defined at the endpoint level in > Documentation/devicetree/bindings/media/video-interfaces.txt. I believe it > would make sense to use OF graph and try to standardize these properties > similarly. Note that the last two properties match the those in Documentation/devicetree/bindings/spi/sh-msiof.txt. We may want to use one DRIF channel as a plain SPI slave with the (modified) MSIOF driver in the future. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds