Re: [PATCH v2 04/18] video: add command mode and command mode display timing

[Thierry Reding <thierry.reding@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1532 bytes --]

On Thu, May 22, 2014 at 10:07:50PM +0900, Inki Dae wrote:
> On 2014년 05월 21일 20:41, YoungJun Cho wrote:
> > Hi Therry
> > 
> > On 05/21/2014 08:02 PM, Thierry Reding wrote:
> >> On Wed, May 21, 2014 at 01:42:56PM +0900, YoungJun Cho wrote:
> >>> This patch is based on videomode and display_timing relevant codes.
> >>> To support command mode panel, it does not need to guide its timing
> >>> information to the display controller like video mode panel,
> >>> but it requires signal timings to transfer video data.
> >>> So this patch adds cmdmode struct, cmdmode_display_timing struct and
> >>> the according helper functions to convert cmdmode_display_timing
> >>> to a generic cmdmode.
> >>
> >> Can you point me to relevant documentation? I wasn't able to find it by
> >> a quick scan through the DSI specification.
> >>
> > 
> > I'm sorry to say that there is no specific one document.
> > 
> > I referred to several ones, CPU interface, I80 interface and
> > DBI document(last time you said it).
> > I think this is good to check it.
> > [ http://cache.freescale.com/files/dsp/doc/app_note/AN4180.pdf ]
> > 
> 
> Ping~. Is there other comment? If so then I'd like to pick them up. We
> would be happy for you leave reviewed-by or acked-by.

I'd rather not see this merged yet. Can somebody from the i.MX crowd
perhaps look at this, to see if they've had to solve a similar problem
(since the above document refers to i.MX)?

Adding Russell and Philipp for a second opinion.

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]