From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933636AbcIEUhV (ORCPT ); Mon, 5 Sep 2016 16:37:21 -0400 Received: from down.free-electrons.com ([37.187.137.238]:55334 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932266AbcIEUhU (ORCPT ); Mon, 5 Sep 2016 16:37:20 -0400 Date: Mon, 5 Sep 2016 22:37:07 +0200 From: Maxime Ripard To: Chen-Yu Tsai Cc: Icenowy Zheng , Daniel Vetter , David Airlie , Thierry Reding , Thomas Petazzoni , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "linux-sunxi@googlegroups.com" , Rob Herring , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH 0/7] drm/sun4i: Introduce A33 display driver Message-ID: <20160905203707.GF8596@lukather> References: <20160901153204.11217-1-maxime.ripard@free-electrons.com> <18951472779805@web6g.yandex.ru> <20160902190605.GF6313@lukather> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="gneEPciiIl/aKvOT" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --gneEPciiIl/aKvOT Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Sep 03, 2016 at 09:43:59AM +0800, Chen-Yu Tsai wrote: > On Sat, Sep 3, 2016 at 3:06 AM, Maxime Ripard > wrote: > > Hi Icenowy, > > > > On Fri, Sep 02, 2016 at 09:30:05AM +0800, Icenowy Zheng wrote: > >> > >> > >> 01.09.2016, 23:40, "Maxime Ripard" : > >> > Hi everyone, > >> > > >> > This serie introduces the support in the sun4i-drm driver for the A3= 3. > >> > > >> > Beside the new IPs and special cases for the A33 new IPs, there's > >> > nothing really outstanding, and is now at feature parity with the A1= 3. > >> > >> How can the driver be modified to support LVDS screen? > >> > >> I have an A33 tablet with a 1024x768 LVDS panel. (iNet D978 Rev2 board, > >> which I pushed a dt a few days ago) > > > > Awesome, I don't have such a screen myself, so feel free to work on > > it. > > > > I haven't looked into the details of LVDS, but it should be something > > along the lines of commit 29e57fab97fc ("drm: sun4i: Add RGB output"). >=20 > The implementation might be along the lines of >=20 > 1. having multiple output ports, each for a different interface type. > (Some platforms go this route) >=20 > Or >=20 > 2. having a DT property describe what the output interface is. >=20 > The RGB/TCON driver would then setup the registers accordingly. Hmmm, yeah, we would need to adjust the bindings too... I guess I'd prefer 1), but that would also be the most invasive solution. I'm not sure how the DT maintainers feel about that. Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --gneEPciiIl/aKvOT Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXzddzAAoJEBx+YmzsjxAgQLEQAK5hlSCNih7G9IDo8QhPFLIL cOILPi8PxnS8VsthISnA/J5RJon3lexldQKTDuidIaFKRYdgwqN79XFykh35U5eD 2JiGoqK5tTUNSwLaiL+rKCST3GmGsv99VoadmEGqtUW6w4OX+dOjpL0CZvVzmvfW 4JBGoT44mdlvvf3VAZJCufyeVrufbeq2LiriG1RfI+XlpS+SwqX2fnjv0aWaYU+z ZWSPEovJm1wuPTYeGwO2v/AEqsrDom7YFeyHRdvVDaFXyk+bYvGcIAdeFKsIUD3P YGQ2z5lY4UVtn5Ac6LguqFSEch28QDujTW7MtL2heNzP7EeWvad84v1W/k1Ddorl A5qZiRCo5aa7CCP+hsLKrzSe1r715aScDTjOiJVDC4/+cYzH+RyMzAERNqDyIaeN enqcCu4hT84u74G9LgtMi8DfHGFrUFWnuHgNQXRxL+VMtWlDYXIg/Foru8fO6d3c WayxcRNkmSTYKe/mwRlwfELdnp2iDDs36L5exgrjwDdKP3rfNwv5h53z/a4HVMmO cZshUCS7OTbErCfJtdxRuEQQGLRwGxlJQVWwlTtm3imgNbJneHFd9oTYsUHz2/bW 0KtD0J8gvT+bRSa5eLiO8ckNmNZNEFDtFuUTgSeqcCEv70JlRSQXadmUYOwlaVpo 1DmjTUfoV0t+2HZhRHge =pYRv -----END PGP SIGNATURE----- --gneEPciiIl/aKvOT-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.ripard@free-electrons.com (Maxime Ripard) Date: Mon, 5 Sep 2016 22:37:07 +0200 Subject: [PATCH 0/7] drm/sun4i: Introduce A33 display driver In-Reply-To: References: <20160901153204.11217-1-maxime.ripard@free-electrons.com> <18951472779805@web6g.yandex.ru> <20160902190605.GF6313@lukather> Message-ID: <20160905203707.GF8596@lukather> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sat, Sep 03, 2016 at 09:43:59AM +0800, Chen-Yu Tsai wrote: > On Sat, Sep 3, 2016 at 3:06 AM, Maxime Ripard > wrote: > > Hi Icenowy, > > > > On Fri, Sep 02, 2016 at 09:30:05AM +0800, Icenowy Zheng wrote: > >> > >> > >> 01.09.2016, 23:40, "Maxime Ripard" : > >> > Hi everyone, > >> > > >> > This serie introduces the support in the sun4i-drm driver for the A33. > >> > > >> > Beside the new IPs and special cases for the A33 new IPs, there's > >> > nothing really outstanding, and is now at feature parity with the A13. > >> > >> How can the driver be modified to support LVDS screen? > >> > >> I have an A33 tablet with a 1024x768 LVDS panel. (iNet D978 Rev2 board, > >> which I pushed a dt a few days ago) > > > > Awesome, I don't have such a screen myself, so feel free to work on > > it. > > > > I haven't looked into the details of LVDS, but it should be something > > along the lines of commit 29e57fab97fc ("drm: sun4i: Add RGB output"). > > The implementation might be along the lines of > > 1. having multiple output ports, each for a different interface type. > (Some platforms go this route) > > Or > > 2. having a DT property describe what the output interface is. > > The RGB/TCON driver would then setup the registers accordingly. Hmmm, yeah, we would need to adjust the bindings too... I guess I'd prefer 1), but that would also be the most invasive solution. I'm not sure how the DT maintainers feel about that. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PATCH 0/7] drm/sun4i: Introduce A33 display driver Date: Mon, 5 Sep 2016 22:37:07 +0200 Message-ID: <20160905203707.GF8596@lukather> References: <20160901153204.11217-1-maxime.ripard@free-electrons.com> <18951472779805@web6g.yandex.ru> <20160902190605.GF6313@lukather> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1250595000==" Return-path: Received: from mail.free-electrons.com (down.free-electrons.com [37.187.137.238]) by gabe.freedesktop.org (Postfix) with ESMTP id 0F47A6E205 for ; Mon, 5 Sep 2016 20:37:19 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Chen-Yu Tsai Cc: Thomas Petazzoni , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Rob Herring , "linux-sunxi@googlegroups.com" , Icenowy Zheng , Daniel Vetter , "linux-arm-kernel@lists.infradead.org" List-Id: dri-devel@lists.freedesktop.org --===============1250595000== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="gneEPciiIl/aKvOT" Content-Disposition: inline --gneEPciiIl/aKvOT Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Sep 03, 2016 at 09:43:59AM +0800, Chen-Yu Tsai wrote: > On Sat, Sep 3, 2016 at 3:06 AM, Maxime Ripard > wrote: > > Hi Icenowy, > > > > On Fri, Sep 02, 2016 at 09:30:05AM +0800, Icenowy Zheng wrote: > >> > >> > >> 01.09.2016, 23:40, "Maxime Ripard" : > >> > Hi everyone, > >> > > >> > This serie introduces the support in the sun4i-drm driver for the A3= 3. > >> > > >> > Beside the new IPs and special cases for the A33 new IPs, there's > >> > nothing really outstanding, and is now at feature parity with the A1= 3. > >> > >> How can the driver be modified to support LVDS screen? > >> > >> I have an A33 tablet with a 1024x768 LVDS panel. (iNet D978 Rev2 board, > >> which I pushed a dt a few days ago) > > > > Awesome, I don't have such a screen myself, so feel free to work on > > it. > > > > I haven't looked into the details of LVDS, but it should be something > > along the lines of commit 29e57fab97fc ("drm: sun4i: Add RGB output"). >=20 > The implementation might be along the lines of >=20 > 1. having multiple output ports, each for a different interface type. > (Some platforms go this route) >=20 > Or >=20 > 2. having a DT property describe what the output interface is. >=20 > The RGB/TCON driver would then setup the registers accordingly. Hmmm, yeah, we would need to adjust the bindings too... I guess I'd prefer 1), but that would also be the most invasive solution. I'm not sure how the DT maintainers feel about that. Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --gneEPciiIl/aKvOT Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXzddzAAoJEBx+YmzsjxAgQLEQAK5hlSCNih7G9IDo8QhPFLIL cOILPi8PxnS8VsthISnA/J5RJon3lexldQKTDuidIaFKRYdgwqN79XFykh35U5eD 2JiGoqK5tTUNSwLaiL+rKCST3GmGsv99VoadmEGqtUW6w4OX+dOjpL0CZvVzmvfW 4JBGoT44mdlvvf3VAZJCufyeVrufbeq2LiriG1RfI+XlpS+SwqX2fnjv0aWaYU+z ZWSPEovJm1wuPTYeGwO2v/AEqsrDom7YFeyHRdvVDaFXyk+bYvGcIAdeFKsIUD3P YGQ2z5lY4UVtn5Ac6LguqFSEch28QDujTW7MtL2heNzP7EeWvad84v1W/k1Ddorl A5qZiRCo5aa7CCP+hsLKrzSe1r715aScDTjOiJVDC4/+cYzH+RyMzAERNqDyIaeN enqcCu4hT84u74G9LgtMi8DfHGFrUFWnuHgNQXRxL+VMtWlDYXIg/Foru8fO6d3c WayxcRNkmSTYKe/mwRlwfELdnp2iDDs36L5exgrjwDdKP3rfNwv5h53z/a4HVMmO cZshUCS7OTbErCfJtdxRuEQQGLRwGxlJQVWwlTtm3imgNbJneHFd9oTYsUHz2/bW 0KtD0J8gvT+bRSa5eLiO8ckNmNZNEFDtFuUTgSeqcCEv70JlRSQXadmUYOwlaVpo 1DmjTUfoV0t+2HZhRHge =pYRv -----END PGP SIGNATURE----- --gneEPciiIl/aKvOT-- --===============1250595000== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1250595000==--