From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [RFC PATCH v3] mipi-dsi-bus: add MIPI DSI bus support Date: Wed, 4 Dec 2013 16:59:00 +0100 Message-ID: <20131204155859.GA21833@ulmo.nvidia.com> References: <1386171312-8173-1-git-send-email-a.hajda@samsung.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0061330885==" Return-path: Received: from mail-bk0-f43.google.com (mail-bk0-f43.google.com [209.85.214.43]) by gabe.freedesktop.org (Postfix) with ESMTP id C64CBFB632 for ; Wed, 4 Dec 2013 07:59:59 -0800 (PST) Received: by mail-bk0-f43.google.com with SMTP id mz12so6664659bkb.16 for ; Wed, 04 Dec 2013 07:59:58 -0800 (PST) In-Reply-To: <1386171312-8173-1-git-send-email-a.hajda@samsung.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org To: Andrzej Hajda Cc: tomi.valkeinen@ti.com, Kyungmin Park , Thierry Reding , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0061330885== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="TB36FDmn/VVEgNH/" Content-Disposition: inline --TB36FDmn/VVEgNH/ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Dec 04, 2013 at 04:35:12PM +0100, Andrzej Hajda wrote: > MIPI DSI bus allows to model DSI hosts and DSI devices using Linux bus. > DSI host should be registered by DSI master driver. During registration > DSI devices will be created according to Device Tree sub-nodes, > registration based on board info will be added later. > DSI host exposes operations which can be used by slave drivers > to access associated devices. >=20 > Signed-off-by: Andrzej Hajda > Signed-off-by: Kyungmin Park > --- > Hi Thierry, >=20 > This is the 3rd version of MIPI DSI bus patch. > Most of changes comes from Thierry comments. > The main difference is complete change of 'philosophy' behind it. > In previous version DSI host could be standalone driver without DRM > dependency. In the current version it should be DRM device. > This change breaks my drivers so I cannot test this version of patch on l= ive system yet. =46rom a brief glance this looks pretty good. I'll start porting the Tegra DSI driver to this and report back. > There are few things which still bother me: > - DSI mode flags - some of them seems to be too specific for my HW, > - other DSI parameters, probably there should be more, like max clock, su= pported pixel modes,.... I think both of those can be extended or cleaned up once we have a few working drivers. No need to try and make it perfect right away, chances are we'll get a few things wrong anyway. Thierry --TB36FDmn/VVEgNH/ Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJSn1FDAAoJEN0jrNd/PrOh8u4QALLQPlwaOoW2J/rwcmutnQ0V D9AT7FYI4xomDw8g+Gc2W3xl5yeYhG0rUO5TL5Gjr0OWJSx4JhDNRHUzzFRp1c+Z p5TxomlbrVU+fQzP2hBz1AoN5a1jpkPQ/IWUOw6yjEC0HWKVcQzBFtJ/cJdOst9Q k4iU135V+RWWy3G1ib6GBzSoRRZMh11s0OFnV/Y5LPeiZTWasLfYn1w3ZZITFw3N PEs00KWRKHtnpzo7JGcodwytfBHidqsxQhtPdHtA0AZ+4XMDNlZe52SAf0SSt1gU 9fI0d3UoAyuhd18h6HdyFORiZnsEaBn+n5yaPGP3EmBpsw5yWuHcVt0oo3TxTUFl SzO84j/nNhl5VD5KSARbzIDJ+DvkRGlkgoTcnQJFWhRDMSL7P+DtL7ulB+jByXPe bVTvozpruRXsaj7pT8R2MhQ4FW3ZUNd4I/ZWpoLSWk/pl0m3ny2t1uUPw8XEIdD7 8m6QBCgb+VBUrNy9313qz8gJiB4mdCKqa3QaSQF6MjzYFO+MrS8rbN7TxrDX/ScJ 0TbpTisvlh457XF0qo8wC53bobpZtFHIolhJtkiA4mRLHvmBuC+VS6St57Ej4X+x G411O5HCLQ0qx31dkktnWYhksLRB2Nqf5IRrTAuVec+W+x/IM5MmhVb0g0wKip3V MSi6EGYe7ti2dbXwV/Fj =4UH5 -----END PGP SIGNATURE----- --TB36FDmn/VVEgNH/-- --===============0061330885== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============0061330885==--