Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 3 Apr 2019 21:54:13 +0200
From:   Sebastian Reichel <sre@kernel.org>
To:     Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc:     kernel@collabora.com, Tony Lindgren <tony@atomide.com>,
        "H. Nikolaus Schaller" <hns@goldelico.com>,
        dri-devel@lists.freedesktop.org, Pavel Machek <pavel@ucw.cz>,
        linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCHv5 0/6] omapdrm: DSI command mode panel support
Message-ID: <20190403195413.djfrgzuj7povdksi@earth.universe>
References: <20181121160916.22017-1-sebastian.reichel@collabora.com>
 <3fed7238-776e-9ee7-9c54-b7c44a5e3a2e@ti.com>
 <20190402155508.GJ4805@pendragon.ideasonboard.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="oz6n6y7l5eyf43yk"
Content-Disposition: inline
In-Reply-To: <20190402155508.GJ4805@pendragon.ideasonboard.com>
User-Agent: NeoMutt/20180716
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--oz6n6y7l5eyf43yk
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Tomi & Laurent,

On Tue, Apr 02, 2019 at 06:55:08PM +0300, Laurent Pinchart wrote:
> On Tue, Apr 02, 2019 at 06:36:21PM +0300, Tomi Valkeinen wrote:
> > On 21/11/2018 18:09, Sebastian Reichel wrote:
> > > Hi,
> > >=20
> > > Here is another round of the DSI command mode panel patchset
> > > integrating the feedback from PATCHv4. The patches are based
> > > on 4.20-rc1 + fixes from Laurent and Tony. I dropped the patches
> > > for OMAP3 support (it needs a workaround for a hardware bug) and
> > > for automatic display rotation. They should get their own series,
> > > once this patchset has landed.
> >=20
> > The big omapdrm bridge/panel series from Laurent has been merged to
> > drm-next. I think this series can now be rebased and merged.
> >=20
> > However, all this needs to be changed to DRM bridge/panel model sooner
> > or later (sooner, please!), but perhaps it's best to first go with the
> > legacy way.
>=20
> I would be OK with that, but I'd like to understand what we still need
> to do to convert DSI support to DRM bridge and panel. Sebastian, if
> you've looked into it, could you provide some insight ?

I have a rebased & tested version of the patches getting the Droid 4
panel working in this branch (together with 2 patches adding
backlight support for testing purposes). I will send it as a patch
series in a few minutes.

https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-n900.git/log/?h=
=3Domapdrm-5.2-with-dsi

I also had a look at converting to DRM bridge/panel model and
started working on it in the following branch. It's completley
untested and probably not working at all. Also the most recent
patches are not yet properly splitted / formated. But they show
my idea of how this can be achieved:

1. Add DSI transfer() function
2. Convert custom functions to use transfer() instead
3. Convert to use mipi_dsi_host
4. Get rid of custom DSI related omapdss functions
5. at this point panel conversion can hopefully happen the
   same way as for the other modules

https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-n900.git/log/?h=
=3Domapdrm-5.2-with-dsi-untested-work-branch

P.S.: I will continue to work on this, but probably not in
the next 4 weeks. I'm currently very busy with non-kernel
tasks and still have to review > 200 power-supply patches.

-- Sebastian

--oz6n6y7l5eyf43yk
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAlylD2IACgkQ2O7X88g7
+pqmNg//X3X81hK2Jt2e4MuElTUGUr/XiB5k83gP9XsCZ+OHr8q5ue9joV5PV7oi
Yfg3XsfqCiZDvzwb3CRVYXgOZOf2MV4w5SnunA/Ca6mXNkiB2j/SF2K65TqBEtZN
GwthUu0aaOJ/uT+JOLcdQGfAkkypOxKLjgAOqbOSx7fOwm4Pe0XygC1TsFUjYNkv
tUvC9TzsLS5AWkwvWHjpyXtOU316QY2rVr32C/QlGEKOdXgzY3Gw7D06+w6UnAe0
teIfTbym0SaEGhI7VlwbrfgiCXDSTXxC3+8z8NTPR2t6eVVsRsZedYhQcxE7wtnf
JIONd8dA5s9zSGQsGTU2ydgIiP/qW9zV4I3a5J5P0dxUGyuXOLLPl1klX09vJXCc
7EHHIfxB29TOj4ee2KEdrjpFTAgn1p9rxoEYWuiluXXlzQvasLSGkOcD6oYz81/h
0jMmmLaTJZvvCY3Yhm961pU2S116Vg1HZYo/QJSmfKSUolMiMcDmdTZR8Yofc7vx
K0uur0cjI1pbSG8Yw1ISrcgbRMSeGjBBOX8TDSPhfrfzfoeZm8O6gBONp88I8emr
ZOP1cGPZ3GM6pQFaq6LjdrqYjICAG8c3D6kbJRBqvwafkIq3Q6fXEnpkXTA+TpwB
KRGAeYy/vVHPen5qoq7nPamjxzPjfv3XdjBaI4lWgn+8s8rBvGc=
=KUNl
-----END PGP SIGNATURE-----

--oz6n6y7l5eyf43yk--