From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756385AbcIFN7x (ORCPT ); Tue, 6 Sep 2016 09:59:53 -0400 Received: from down.free-electrons.com ([37.187.137.238]:53753 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754815AbcIFN7v (ORCPT ); Tue, 6 Sep 2016 09:59:51 -0400 Date: Tue, 6 Sep 2016 15:59:39 +0200 From: Maxime Ripard To: Peter Korsgaard Cc: Daniel Vetter , David Airlie , Thierry Reding , Chen-Yu Tsai , Rob Herring , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Thomas Petazzoni Subject: Re: [linux-sunxi] [PATCH 3/7] drm/sun4i: Add SAT and DRC drivers Message-ID: <20160906135939.GF9040@lukather> References: <20160901153204.11217-1-maxime.ripard@free-electrons.com> <20160901153204.11217-4-maxime.ripard@free-electrons.com> <8737lfwhz9.fsf@dell.be.48ers.dk> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="cpvLTH7QU4gwfq3S" Content-Disposition: inline In-Reply-To: <8737lfwhz9.fsf@dell.be.48ers.dk> 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 --cpvLTH7QU4gwfq3S Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Peter, On Sun, Sep 04, 2016 at 10:03:06PM +0200, Peter Korsgaard wrote: > >>>>> "Maxime" =3D=3D Maxime Ripard wr= ites: >=20 > Hi, >=20 > > The A33 pipeline also has some new components called SAT and DRC. Even > > though their exact features and programming model is not known (or > > documented), they need to be clocked for the pipeline to carry the vid= eo > > signal all the way. >=20 > > Add minimal drivers for those that just claim the needed resources for= the > > pipeline to operate properly. >=20 > > Signed-off-by: Maxime Ripard > > --- > > .../bindings/display/sunxi/sun4i-drm.txt | 37 +++++++ > > drivers/gpu/drm/sun4i/Makefile | 3 +- > > drivers/gpu/drm/sun4i/sun6i_drc.c | 117 ++++++++++++= +++++++++ > > drivers/gpu/drm/sun4i/sun8i_sat.c | 105 ++++++++++++= ++++++ > > 4 files changed, 261 insertions(+), 1 deletion(-) > > create mode 100644 drivers/gpu/drm/sun4i/sun6i_drc.c > > create mode 100644 drivers/gpu/drm/sun4i/sun8i_sat.c >=20 > > diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm= =2Etxt b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > index d467ea93ac08..87c3c8dd34cb 100644 > > --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > @@ -51,6 +51,43 @@ Required properties: > > second the block connected to the TCON channel 1 (usually the TV > > encoder) > =20 > > +SAT > > +--- > > + > > +The SAT, found in the A33, allows to do some color correction. > > + > > +Required properties: > > + - compatible: value must be one of: > > + * allwinner,sun8i-a33-sat > > + - reg: base address and size of the memory-mapped region. > > + - clock: phandles to bus clock feeding the SAT > > + - resets: phandles to the reset line driving the SAT > > + > > +- ports: A ports node with endpoint definitions as defined in > > + Documentation/devicetree/bindings/media/video-interfaces.txt. The > > + first port should be the input endpoints, the second one the outputs > > + > > +DRC > > +--- > > + > > +The DRC, found in the latest Allwinner SoCs (A31, A23, A33), allows to > > +do some backlight control to enhance the power consumption. >=20 > 'Enhance the power consumption'? That doesn't sound like something you > would want ;) Presumably it is something to allow you to save power by > dynamically adjusting LCD backlight and pixel brightness/contrast > depending on screen content? I believe this is typically called content > adaptive backlight control: >=20 > https://www.ecnmag.com/article/2010/04/content-adaptive-lcd-backlight-con= trol >=20 > You spell out what DRC and SAT stands for in the driver source code, > perhaps it also makes sense to do it here? >=20 > Perhaps rewording it to something like this is clearer: >=20 > .. allows to dynamically adjust pixel brightness/contrast based on > histogram measurements for LCD content adaptive backlight control. You're right, I changed it. Thanks! Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --cpvLTH7QU4gwfq3S Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXzsvLAAoJEBx+YmzsjxAg5I8P/33jextUCXlHevY6jXSrbI6H vHhkraz2mBM3WwQCf0A9V4dPC2KqTojI16twX0LF7Q6jwvlE2RtJXWMiYsiSBjIZ ijQxUWgm0KYVAGlD1uTNcCoz9EzkIEibYDbEOIYFLe0ZP7UZC8wbp6pgH1/6lsnr Wzdhwlo4t8Qg1fsheIB8lmZzHVrEIGgb9kLD64VEp53nAqfArln6cXOD1ywJI6Bm Gf/fyUXy5NWq1M2m/o1827mKLr04MGkN0HoZ2USjUutA2QcnYDKcjWsm1ZgZVfDF 46g2D7gp3IqvmmT05e/4zKxRHv50xhuV3ya4NmA9n8xhP3vBwTi962lErjqLwARc QNYIB5JUi6647ntqIQzdT5GTJQP/zkm6O4eP/XL9t34c2LFZmJEZadUuLo+YSKXX SzQWrLOw2hyOKrbA99TwyK430bO5IZGjAtvex2Ps14Bb3xeKFcqdyRbasvFO6kYF Tz+z5hyiz65zKBlkNQCEd05N+65rOFz44wCDo0pDuaD8v7cW19xeeBPx1ef9TfU0 nQfCr0c9x4hqP27593309m+2MN0nruNmrLXXuZ6wp8m83uIPzp9lWWyQSqQgBjIO uDZFB7hxJXAM0EHJKxjgsx54Ce5XcqypldAoV43ckeBFklh2x3P7fSzSP4+hHbph 7u15CXsN/LcWROsIxJQ0 =EDfP -----END PGP SIGNATURE----- --cpvLTH7QU4gwfq3S-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.ripard@free-electrons.com (Maxime Ripard) Date: Tue, 6 Sep 2016 15:59:39 +0200 Subject: [linux-sunxi] [PATCH 3/7] drm/sun4i: Add SAT and DRC drivers In-Reply-To: <8737lfwhz9.fsf@dell.be.48ers.dk> References: <20160901153204.11217-1-maxime.ripard@free-electrons.com> <20160901153204.11217-4-maxime.ripard@free-electrons.com> <8737lfwhz9.fsf@dell.be.48ers.dk> Message-ID: <20160906135939.GF9040@lukather> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Peter, On Sun, Sep 04, 2016 at 10:03:06PM +0200, Peter Korsgaard wrote: > >>>>> "Maxime" == Maxime Ripard writes: > > Hi, > > > The A33 pipeline also has some new components called SAT and DRC. Even > > though their exact features and programming model is not known (or > > documented), they need to be clocked for the pipeline to carry the video > > signal all the way. > > > Add minimal drivers for those that just claim the needed resources for the > > pipeline to operate properly. > > > Signed-off-by: Maxime Ripard > > --- > > .../bindings/display/sunxi/sun4i-drm.txt | 37 +++++++ > > drivers/gpu/drm/sun4i/Makefile | 3 +- > > drivers/gpu/drm/sun4i/sun6i_drc.c | 117 +++++++++++++++++++++ > > drivers/gpu/drm/sun4i/sun8i_sat.c | 105 ++++++++++++++++++ > > 4 files changed, 261 insertions(+), 1 deletion(-) > > create mode 100644 drivers/gpu/drm/sun4i/sun6i_drc.c > > create mode 100644 drivers/gpu/drm/sun4i/sun8i_sat.c > > > diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > index d467ea93ac08..87c3c8dd34cb 100644 > > --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > @@ -51,6 +51,43 @@ Required properties: > > second the block connected to the TCON channel 1 (usually the TV > > encoder) > > > +SAT > > +--- > > + > > +The SAT, found in the A33, allows to do some color correction. > > + > > +Required properties: > > + - compatible: value must be one of: > > + * allwinner,sun8i-a33-sat > > + - reg: base address and size of the memory-mapped region. > > + - clock: phandles to bus clock feeding the SAT > > + - resets: phandles to the reset line driving the SAT > > + > > +- ports: A ports node with endpoint definitions as defined in > > + Documentation/devicetree/bindings/media/video-interfaces.txt. The > > + first port should be the input endpoints, the second one the outputs > > + > > +DRC > > +--- > > + > > +The DRC, found in the latest Allwinner SoCs (A31, A23, A33), allows to > > +do some backlight control to enhance the power consumption. > > 'Enhance the power consumption'? That doesn't sound like something you > would want ;) Presumably it is something to allow you to save power by > dynamically adjusting LCD backlight and pixel brightness/contrast > depending on screen content? I believe this is typically called content > adaptive backlight control: > > https://www.ecnmag.com/article/2010/04/content-adaptive-lcd-backlight-control > > You spell out what DRC and SAT stands for in the driver source code, > perhaps it also makes sense to do it here? > > Perhaps rewording it to something like this is clearer: > > .. allows to dynamically adjust pixel brightness/contrast based on > histogram measurements for LCD content adaptive backlight control. You're right, I changed it. Thanks! 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 3/7] drm/sun4i: Add SAT and DRC drivers Date: Tue, 6 Sep 2016 15:59:39 +0200 Message-ID: <20160906135939.GF9040@lukather> References: <20160901153204.11217-1-maxime.ripard@free-electrons.com> <20160901153204.11217-4-maxime.ripard@free-electrons.com> <8737lfwhz9.fsf@dell.be.48ers.dk> Reply-To: maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="cpvLTH7QU4gwfq3S" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Content-Disposition: inline In-Reply-To: <8737lfwhz9.fsf-D6SC8u56vOOJDPpyT6T3/w@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Peter Korsgaard Cc: Daniel Vetter , David Airlie , Thierry Reding , Chen-Yu Tsai , Rob Herring , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Thomas Petazzoni List-Id: dri-devel@lists.freedesktop.org --cpvLTH7QU4gwfq3S Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Hi Peter, On Sun, Sep 04, 2016 at 10:03:06PM +0200, Peter Korsgaard wrote: > >>>>> "Maxime" == Maxime Ripard writes: > > Hi, > > > The A33 pipeline also has some new components called SAT and DRC. Even > > though their exact features and programming model is not known (or > > documented), they need to be clocked for the pipeline to carry the video > > signal all the way. > > > Add minimal drivers for those that just claim the needed resources for the > > pipeline to operate properly. > > > Signed-off-by: Maxime Ripard > > --- > > .../bindings/display/sunxi/sun4i-drm.txt | 37 +++++++ > > drivers/gpu/drm/sun4i/Makefile | 3 +- > > drivers/gpu/drm/sun4i/sun6i_drc.c | 117 +++++++++++++++++++++ > > drivers/gpu/drm/sun4i/sun8i_sat.c | 105 ++++++++++++++++++ > > 4 files changed, 261 insertions(+), 1 deletion(-) > > create mode 100644 drivers/gpu/drm/sun4i/sun6i_drc.c > > create mode 100644 drivers/gpu/drm/sun4i/sun8i_sat.c > > > diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > index d467ea93ac08..87c3c8dd34cb 100644 > > --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > @@ -51,6 +51,43 @@ Required properties: > > second the block connected to the TCON channel 1 (usually the TV > > encoder) > > > +SAT > > +--- > > + > > +The SAT, found in the A33, allows to do some color correction. > > + > > +Required properties: > > + - compatible: value must be one of: > > + * allwinner,sun8i-a33-sat > > + - reg: base address and size of the memory-mapped region. > > + - clock: phandles to bus clock feeding the SAT > > + - resets: phandles to the reset line driving the SAT > > + > > +- ports: A ports node with endpoint definitions as defined in > > + Documentation/devicetree/bindings/media/video-interfaces.txt. The > > + first port should be the input endpoints, the second one the outputs > > + > > +DRC > > +--- > > + > > +The DRC, found in the latest Allwinner SoCs (A31, A23, A33), allows to > > +do some backlight control to enhance the power consumption. > > 'Enhance the power consumption'? That doesn't sound like something you > would want ;) Presumably it is something to allow you to save power by > dynamically adjusting LCD backlight and pixel brightness/contrast > depending on screen content? I believe this is typically called content > adaptive backlight control: > > https://www.ecnmag.com/article/2010/04/content-adaptive-lcd-backlight-control > > You spell out what DRC and SAT stands for in the driver source code, > perhaps it also makes sense to do it here? > > Perhaps rewording it to something like this is clearer: > > .. allows to dynamically adjust pixel brightness/contrast based on > histogram measurements for LCD content adaptive backlight control. You're right, I changed it. Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --cpvLTH7QU4gwfq3S--