Re: [PATCHv1 08/14] drm/omap: panel-dsi-cm: fix driver

[Tomi Valkeinen]

Hi,

On 24/07/17 20:33, Sebastian Reichel wrote:
> From: Tony Lindgren <tony@xxxxxxxxxxx>
> 
> This adds support for get_timings() and check_timings()
> to get the driver working and properly initializes the
> timing information from DT.

I don't know if it matters much, but the timings added to dts in the
following patches are mostly not used, and do not make sense for DSI
command mode panel. Only hactive and vactive are used. But if full video
timings are present in the dts, then I think it would be good to have
the timings correct, so that also pixel clock is defined, and the
timings would produce 60Hz refresh rate.

 Tomi

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki