From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Jacopo Mondi <jacopo+renesas@jmondi.org> Cc: kieran.bingham+renesas@ideasonboard.com, geert@linux-m68k.org, horms@verge.net.au, uli+renesas@fpond.eu, airlied@linux.ie, daniel@ffwll.ch, linux-renesas-soc@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 0/8] drm: rcar-du: Add Color Management Module (CMM) Date: Thu, 17 Oct 2019 22:14:00 +0300 [thread overview] Message-ID: <20191017191400.GH4901@pendragon.ideasonboard.com> (raw) In-Reply-To: <20191016085548.105703-1-jacopo+renesas@jmondi.org> Hi Jacopo, Thank you for your work. On Wed, Oct 16, 2019 at 10:55:40AM +0200, Jacopo Mondi wrote: > Minimal increment to the CMM series, this time should really be the last one. > > Just missing Rob's ack on [1/8] and Laurent's one on [5/8]. > > Changelog is minimal: > CMM > - Remove the cmm_config.enable flag. The cmm_config.table field validity is > used to enable/disable the LUT operations > - Expand comments as suggested by Laurent > > CRTC > - use drm_color_lut_size() to check the LUT table size > - Inline calls to rcar_cmm_enable()/disable() > - Add TODO entries as suggested by Laurent > > For the record, the full series changelog is available at: > https://paste.debian.net/1107427/ > > v5 from yesterday with informations on testing is available at: > https://lkml.org/lkml/2019/10/15/337 > > Geert will you collect for DTS patches for the next release? > I assume the DU changes go through Laurent instead ? I've taken patch 1/8 to 6/8 and 8/8 in my tree. I expected Geert to take 7/8. > Jacopo Mondi (8): > dt-bindings: display: renesas,cmm: Add R-Car CMM documentation > dt-bindings: display, renesas,du: Document cmms property > drm: rcar-du: Add support for CMM > drm: rcar-du: kms: Initialize CMM instances > drm: rcar-du: crtc: Control CMM operations > drm: rcar-du: crtc: Register GAMMA_LUT properties > arm64: dts: renesas: Add CMM units to Gen3 SoCs > drm: rcar-du: kms: Expand comment in vsps parsing routine > > .../bindings/display/renesas,cmm.yaml | 67 ++++++ > .../bindings/display/renesas,du.txt | 5 + > arch/arm64/boot/dts/renesas/r8a7795.dtsi | 39 ++++ > arch/arm64/boot/dts/renesas/r8a7796.dtsi | 31 ++- > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 31 ++- > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 21 ++ > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 21 ++ > drivers/gpu/drm/rcar-du/Kconfig | 7 + > drivers/gpu/drm/rcar-du/Makefile | 1 + > drivers/gpu/drm/rcar-du/rcar_cmm.c | 212 ++++++++++++++++++ > drivers/gpu/drm/rcar-du/rcar_cmm.h | 58 +++++ > drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 65 ++++++ > drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 + > drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 + > drivers/gpu/drm/rcar-du/rcar_du_group.c | 10 + > drivers/gpu/drm/rcar-du/rcar_du_group.h | 2 + > drivers/gpu/drm/rcar-du/rcar_du_kms.c | 82 ++++++- > drivers/gpu/drm/rcar-du/rcar_du_regs.h | 5 + > 18 files changed, 658 insertions(+), 3 deletions(-) > create mode 100644 Documentation/devicetree/bindings/display/renesas,cmm.yaml > create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.c > create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.h > -- Regards, Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Jacopo Mondi <jacopo+renesas@jmondi.org> Cc: horms@verge.net.au, airlied@linux.ie, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, kieran.bingham+renesas@ideasonboard.com, geert@linux-m68k.org, uli+renesas@fpond.eu Subject: Re: [PATCH v6 0/8] drm: rcar-du: Add Color Management Module (CMM) Date: Thu, 17 Oct 2019 22:14:00 +0300 [thread overview] Message-ID: <20191017191400.GH4901@pendragon.ideasonboard.com> (raw) In-Reply-To: <20191016085548.105703-1-jacopo+renesas@jmondi.org> Hi Jacopo, Thank you for your work. On Wed, Oct 16, 2019 at 10:55:40AM +0200, Jacopo Mondi wrote: > Minimal increment to the CMM series, this time should really be the last one. > > Just missing Rob's ack on [1/8] and Laurent's one on [5/8]. > > Changelog is minimal: > CMM > - Remove the cmm_config.enable flag. The cmm_config.table field validity is > used to enable/disable the LUT operations > - Expand comments as suggested by Laurent > > CRTC > - use drm_color_lut_size() to check the LUT table size > - Inline calls to rcar_cmm_enable()/disable() > - Add TODO entries as suggested by Laurent > > For the record, the full series changelog is available at: > https://paste.debian.net/1107427/ > > v5 from yesterday with informations on testing is available at: > https://lkml.org/lkml/2019/10/15/337 > > Geert will you collect for DTS patches for the next release? > I assume the DU changes go through Laurent instead ? I've taken patch 1/8 to 6/8 and 8/8 in my tree. I expected Geert to take 7/8. > Jacopo Mondi (8): > dt-bindings: display: renesas,cmm: Add R-Car CMM documentation > dt-bindings: display, renesas,du: Document cmms property > drm: rcar-du: Add support for CMM > drm: rcar-du: kms: Initialize CMM instances > drm: rcar-du: crtc: Control CMM operations > drm: rcar-du: crtc: Register GAMMA_LUT properties > arm64: dts: renesas: Add CMM units to Gen3 SoCs > drm: rcar-du: kms: Expand comment in vsps parsing routine > > .../bindings/display/renesas,cmm.yaml | 67 ++++++ > .../bindings/display/renesas,du.txt | 5 + > arch/arm64/boot/dts/renesas/r8a7795.dtsi | 39 ++++ > arch/arm64/boot/dts/renesas/r8a7796.dtsi | 31 ++- > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 31 ++- > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 21 ++ > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 21 ++ > drivers/gpu/drm/rcar-du/Kconfig | 7 + > drivers/gpu/drm/rcar-du/Makefile | 1 + > drivers/gpu/drm/rcar-du/rcar_cmm.c | 212 ++++++++++++++++++ > drivers/gpu/drm/rcar-du/rcar_cmm.h | 58 +++++ > drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 65 ++++++ > drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 + > drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 + > drivers/gpu/drm/rcar-du/rcar_du_group.c | 10 + > drivers/gpu/drm/rcar-du/rcar_du_group.h | 2 + > drivers/gpu/drm/rcar-du/rcar_du_kms.c | 82 ++++++- > drivers/gpu/drm/rcar-du/rcar_du_regs.h | 5 + > 18 files changed, 658 insertions(+), 3 deletions(-) > create mode 100644 Documentation/devicetree/bindings/display/renesas,cmm.yaml > create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.c > create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.h > -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2019-10-17 19:14 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-16 8:55 [PATCH v6 0/8] drm: rcar-du: Add Color Management Module (CMM) Jacopo Mondi 2019-10-16 8:55 ` [PATCH v6 1/8] dt-bindings: display: renesas,cmm: Add R-Car CMM documentation Jacopo Mondi 2019-10-17 14:10 ` Rob Herring 2019-10-17 14:10 ` Rob Herring 2019-10-16 8:55 ` [PATCH v6 2/8] dt-bindings: display, renesas,du: Document cmms property Jacopo Mondi 2019-10-16 8:55 ` [PATCH v6 3/8] drm: rcar-du: Add support for CMM Jacopo Mondi 2019-10-16 13:45 ` Laurent Pinchart 2019-10-16 13:45 ` Laurent Pinchart 2019-10-17 12:43 ` Jacopo Mondi 2019-10-17 12:48 ` Laurent Pinchart 2019-10-17 12:48 ` Laurent Pinchart 2019-10-17 13:43 ` [PATCH 6.1 " Jacopo Mondi 2019-10-17 19:11 ` Laurent Pinchart 2019-10-16 8:55 ` [PATCH v6 4/8] drm: rcar-du: kms: Initialize CMM instances Jacopo Mondi 2019-10-16 8:55 ` [PATCH v6 5/8] drm: rcar-du: crtc: Control CMM operations Jacopo Mondi 2019-10-16 13:49 ` Laurent Pinchart 2019-10-16 13:49 ` Laurent Pinchart 2019-10-17 13:44 ` [PATCH v6.1 " Jacopo Mondi 2019-10-17 19:12 ` Laurent Pinchart 2019-10-16 8:55 ` [PATCH v6 6/8] drm: rcar-du: crtc: Register GAMMA_LUT properties Jacopo Mondi 2019-10-16 8:55 ` [PATCH v6 7/8] arm64: dts: renesas: Add CMM units to Gen3 SoCs Jacopo Mondi 2019-10-21 8:58 ` Geert Uytterhoeven 2019-10-21 8:58 ` Geert Uytterhoeven 2019-10-16 8:55 ` [PATCH v6 8/8] drm: rcar-du: kms: Expand comment in vsps parsing routine Jacopo Mondi 2019-10-17 19:14 ` Laurent Pinchart [this message] 2019-10-17 19:14 ` [PATCH v6 0/8] drm: rcar-du: Add Color Management Module (CMM) Laurent Pinchart 2019-11-05 16:14 ` Geert Uytterhoeven 2019-11-05 16:14 ` Geert Uytterhoeven 2019-11-05 19:52 ` Laurent Pinchart 2019-11-05 19:52 ` Laurent Pinchart
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20191017191400.GH4901@pendragon.ideasonboard.com \ --to=laurent.pinchart@ideasonboard.com \ --cc=airlied@linux.ie \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=geert@linux-m68k.org \ --cc=horms@verge.net.au \ --cc=jacopo+renesas@jmondi.org \ --cc=kieran.bingham+renesas@ideasonboard.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=uli+renesas@fpond.eu \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.