On Mon, 11 Aug 2014 13:19:02 +0100
Mark Rutland <mark.rutl...@arm.com> wrote:

> > -  - pclk-sample: Pixel clock polarity. Defaults to output on the falling 
> > edge.
> > +  - pclk-sample:  Pixel clock polarity. Defaults to output on the falling 
> > edge.
> 
> Unrelated whitespace change?

Is there a sensible way to get miniscule whitespace changes in?

> >    If none of hsync-active, vsync-active and pclk-sample is specified the
> >    endpoint will use embedded BT.656 synchronization.
> >  
> > +  - default-input: Select which input is selected after reset.
> 
> Valid values are?

Chip dependent. 0 for 7611, 0-1 for 7612, I expect there are other chips in the 
family with differing numbers of inputs.

> > +    if (!of_property_read_u32(endpoint, "default_input", &v))
> 
> This doesn't match the binding ('_' vs '-').

Good catch!

-- 
Ian Molton <ian.mol...@codethink.co.uk>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to