Message ID | 1527606359-19261-3-git-send-email-jacopo+renesas@jmondi.org |
---|---|
State | Not Applicable, archived |
Headers | show |
Series | media: rcar-vin: Brush endpoint properties | expand |
On Tue, May 29, 2018 at 05:05:53PM +0200, Jacopo Mondi wrote: > Add 'data-enable-active' property to endpoint node properties list. > > The property allows to specify the polarity of the data-enable signal, which > when in active state determinates when data lanes have to sampled for valid > pixel data. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > --- > v3: > - new patch > --- > Documentation/devicetree/bindings/media/video-interfaces.txt | 2 ++ > 1 file changed, 2 insertions(+) Reviewed-by: Rob Herring <robh@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Jacopo, Thanks for the patch. On Tue, May 29, 2018 at 05:05:53PM +0200, Jacopo Mondi wrote: > Add 'data-enable-active' property to endpoint node properties list. > > The property allows to specify the polarity of the data-enable signal, which > when in active state determinates when data lanes have to sampled for valid > pixel data. Lanes or lines? I understand this is forthe parallel interface. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > --- > v3: > - new patch > --- > Documentation/devicetree/bindings/media/video-interfaces.txt | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt b/Documentation/devicetree/bindings/media/video-interfaces.txt > index 258b8df..9839d26 100644 > --- a/Documentation/devicetree/bindings/media/video-interfaces.txt > +++ b/Documentation/devicetree/bindings/media/video-interfaces.txt > @@ -109,6 +109,8 @@ Optional endpoint properties > Note, that if HSYNC and VSYNC polarities are not specified, embedded > synchronization may be required, where supported. > - data-active: similar to HSYNC and VSYNC, specifies data line polarity. > +- data-enable-active: similar to HSYNC and VSYNC, specifies the data enable > + signal polarity. > - field-even-active: field signal level during the even field data transmission. > - pclk-sample: sample data on rising (1) or falling (0) edge of the pixel clock > signal.
Hi Sakari, On Fri, Jun 01, 2018 at 01:29:10PM +0300, Sakari Ailus wrote: > Hi Jacopo, > > Thanks for the patch. > > On Tue, May 29, 2018 at 05:05:53PM +0200, Jacopo Mondi wrote: > > Add 'data-enable-active' property to endpoint node properties list. > > > > The property allows to specify the polarity of the data-enable signal, which > > when in active state determinates when data lanes have to sampled for valid > > pixel data. > > Lanes or lines? I understand this is forthe parallel interface. > Now I'm confused. Are the parallel data 'lines' and the CSI-2 one 'lanes'? I thought 'lanes' applies to both.. am I wrong? Thanks j > > > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > > --- > > v3: > > - new patch > > --- > > Documentation/devicetree/bindings/media/video-interfaces.txt | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt b/Documentation/devicetree/bindings/media/video-interfaces.txt > > index 258b8df..9839d26 100644 > > --- a/Documentation/devicetree/bindings/media/video-interfaces.txt > > +++ b/Documentation/devicetree/bindings/media/video-interfaces.txt > > @@ -109,6 +109,8 @@ Optional endpoint properties > > Note, that if HSYNC and VSYNC polarities are not specified, embedded > > synchronization may be required, where supported. > > - data-active: similar to HSYNC and VSYNC, specifies data line polarity. > > +- data-enable-active: similar to HSYNC and VSYNC, specifies the data enable > > + signal polarity. > > - field-even-active: field signal level during the even field data transmission. > > - pclk-sample: sample data on rising (1) or falling (0) edge of the pixel clock > > signal. > > -- > Sakari Ailus > sakari.ailus@linux.intel.com
On Fri, Jun 01, 2018 at 04:58:27PM +0200, jacopo mondi wrote: > Hi Sakari, > > On Fri, Jun 01, 2018 at 01:29:10PM +0300, Sakari Ailus wrote: > > Hi Jacopo, > > > > Thanks for the patch. > > > > On Tue, May 29, 2018 at 05:05:53PM +0200, Jacopo Mondi wrote: > > > Add 'data-enable-active' property to endpoint node properties list. > > > > > > The property allows to specify the polarity of the data-enable signal, which > > > when in active state determinates when data lanes have to sampled for valid > > > pixel data. > > > > Lanes or lines? I understand this is forthe parallel interface. > > > > Now I'm confused. Are the parallel data 'lines' and the CSI-2 one > 'lanes'? I thought 'lanes' applies to both.. am I wrong? "Lane" is conventionally refer to a differential pair of wires on a serial bus such as CSI-2. "Line" is used of a wire on a parallel bus.
diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt b/Documentation/devicetree/bindings/media/video-interfaces.txt index 258b8df..9839d26 100644 --- a/Documentation/devicetree/bindings/media/video-interfaces.txt +++ b/Documentation/devicetree/bindings/media/video-interfaces.txt @@ -109,6 +109,8 @@ Optional endpoint properties Note, that if HSYNC and VSYNC polarities are not specified, embedded synchronization may be required, where supported. - data-active: similar to HSYNC and VSYNC, specifies data line polarity. +- data-enable-active: similar to HSYNC and VSYNC, specifies the data enable + signal polarity. - field-even-active: field signal level during the even field data transmission. - pclk-sample: sample data on rising (1) or falling (0) edge of the pixel clock signal.
Add 'data-enable-active' property to endpoint node properties list. The property allows to specify the polarity of the data-enable signal, which when in active state determinates when data lanes have to sampled for valid pixel data. Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> --- v3: - new patch --- Documentation/devicetree/bindings/media/video-interfaces.txt | 2 ++ 1 file changed, 2 insertions(+) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html