From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [RFC v2 0/5] Common Display Framework Date: Tue, 08 Jan 2013 09:18:35 +0100 Message-ID: <3584709.mPLC5exzRY@avalon> References: <1353620736-6517-1-git-send-email-laurent.pinchart@ideasonboard.com> <3445117.L94DmxEvrl@avalon> <2529718.glQX8guWfJ@amdc1227> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <2529718.glQX8guWfJ@amdc1227> Sender: linux-media-owner@vger.kernel.org To: Tomasz Figa Cc: dri-devel@lists.freedesktop.org, Vikas Sajjan , Thomas Petazzoni , linux-fbdev@vger.kernel.org, Benjamin Gaignard , Tom Gall , Kyungmin Park , Rob Clark , Ragesh Radhakrishnan , Tomi Valkeinen , Philipp Zabel , Bryan Wu , Maxime Ripard , sunil joshi , Sumit Semwal , Sebastien Guiriec , linux-media@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org Hi Tomasz, On Thursday 27 December 2012 15:43:34 Tomasz Figa wrote: > On Monday 24 of December 2012 15:12:28 Laurent Pinchart wrote: > > On Friday 21 December 2012 11:00:52 Tomasz Figa wrote: > > > On Tuesday 18 of December 2012 08:31:30 Vikas Sajjan wrote: > > > > On 17 December 2012 20:55, Laurent Pinchart wrote: > > > > > Hi Vikas, > > > > >=20 > > > > > Sorry for the late reply. I now have more time to work on CDF= , so > > > > > delays should be much shorter. > > > > >=20 > > > > > On Thursday 06 December 2012 10:51:15 Vikas Sajjan wrote: > > > > > > Hi Laurent, > > > > > >=20 > > > > > > I was thinking of porting CDF to samsung EXYNOS 5250 platfo= rm, > > > > > > what I found is that, the exynos display controller is MIPI= DSI > > > > > > based controller. > > > > > >=20 > > > > > > But if I look at CDF patches, it has only support for MIPI = DBI > > > > > > based Display controller. > > > > > >=20 > > > > > > So my question is, do we have any generic framework for MIP= I DSI > > > > > > based display controller? basically I wanted to know, how t= o go > > > > > > about porting CDF for such kind of display controller. > > > > >=20 > > > > > MIPI DSI support is not available yet. The only reason for th= at is > > > > > that I don't have any MIPI DSI hardware to write and test the= code > > > > > with :-) > > > > >=20 > > > > > The common display framework should definitely support MIPI D= SI. I > > > > > think the existing MIPI DBI code could be used as a base, so = the > > > > > implementation shouldn't be too high. > > > > >=20 > > > > > Yeah, i was also thinking in similar lines, below is my thoug= h for > > > > > MIPI DSI support in CDF. > > > >=20 > > > > o MIPI DSI support as part of CDF framework will expose > > > > =A7 mipi_dsi_register_device(mpi_device) (will be called mach-= xxx-dt.c > > > > file ) > > > > =A7 mipi_dsi_register_driver(mipi_driver, bus ops) (will be ca= lled > > > > from platform specific init driver call ) > > > > =B7 bus ops will be > > > > o read data > > > > o write data > > > > o write command > > > > =A7 MIPI DSI will be registered as bus_register() > > > >=20 > > > > When MIPI DSI probe is called, it (e.g., Exynos or OMAP MIPI DS= I) > > > > will initialize the MIPI DSI HW IP. > > > >=20 > > > > This probe will also parse the DT file for MIPI DSI based panel= , add > > > > the panel device (device_add() ) to kernel and register the dis= play > > > > entity with its control and video ops with CDF. > > > >=20 > > > > I can give this a try. > > >=20 > > > I am currently in progress of reworking Exynos MIPI DSIM code and > > > s6e8ax0 LCD driver to use the v2 RFC of Common Display Framework.= I > > > have most of the work done, I have just to solve several remainin= g > > > problems. > >=20 > > Do you already have code that you can publish ? I'm particularly > > interested (and I think Tomi Valkeinen would be as well) in looking= at > > the DSI operations you expose to DSI sinks (panels, transceivers, .= =2E.). >=20 > Well, I'm afraid this might be little below your expectations, but he= re's > an initial RFC of the part defining just the DSI bus. I need a bit mo= re > time for patches for Exynos MIPI DSI master and s6e8ax0 LCD. No worries. I was particularly interested in the DSI operations you nee= ded to=20 export, they seem pretty simple. Thank you for sharing the code. > The implementation is very simple and heavily based on your MIPI DBI > support and existing Exynos MIPI DSIM framework. Provided operation s= et is > based on operation set used by Exynos s6e8ax0 LCD driver. Unfortunate= ly > this is my only source of information about MIPI DSI. --=20 Regards, Laurent Pinchart