From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [RFC v2 0/5] Common Display Framework Date: Mon, 24 Dec 2012 14:00:52 +0100 Message-ID: <21195616.A64r85Hc5F@avalon> References: <1353620736-6517-1-git-send-email-laurent.pinchart@ideasonboard.com> <3133576.BkqAl7V01U@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-media-owner@vger.kernel.org To: Vikas Sajjan Cc: Jesse Barker , Ragesh Radhakrishnan , Tomi Valkeinen , Thomas Petazzoni , linux-fbdev@vger.kernel.org, Philipp Zabel , Tom Gall , dri-devel@lists.freedesktop.org, Rob Clark , Kyungmin Park , Benjamin Gaignard , Bryan Wu , Maxime Ripard , Sumit Semwal , Sebastien Guiriec , linux-media@vger.kernel.org, sunil joshi List-Id: dri-devel@lists.freedesktop.org Hi Vikas, On Tuesday 18 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 d= elays > > 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 platform, wh= at 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 ba= sed > > > Display controller. > > >=20 > > > So my question is, do we have any generic framework for MIPI DSI = based > > > display controller? basically I wanted to know, how to go about p= orting > > > CDF for such kind of display controller. > >=20 > > MIPI DSI support is not available yet. The only reason for that 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 DSI. I = think > > the existing MIPI DBI code could be used as a base, so the implemen= tation > > shouldn't be too high. > >=20 > > Yeah, i was also thinking in similar lines, below is my though 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-d= t.c > file) > =A7 mipi_dsi_register_driver(mipi_driver, bus ops) (will be called = 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 DSI) wil= l > 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 display > entity with its control and video ops with CDF. After discussing the DBI/DSI busses with Tomi Valkeinen we concluded th= at=20 creating a real bus for DBI and DSI, although possible, wasn't required= =2E DSI=20 operations should thus be provided through display entity video source=20 operations. You can find a proposed implementation in Tomi's patch set. > > I can give this a try. Does the existing Exynos 5250 driver support= MIPI > > DSI ? Is the device documentation publicly available ? Can you poin= t me to > > a MIPI DSI panel with public documentation (preferably with an exis= ting > > mainline driver if possible) ? > > yeah, existing Exynos 5250 driver support MIPI DSI ass well as eDP. >=20 > i think device documentation is NOT available publicly. --=20 Regards, Laurent Pinchart