Re: [PATCH 4/4] drm: rcar-du: Add support for R8A779H0

2024-06-20 Thread Jacopo Mondi
Hi Geert On Thu, Jun 20, 2024 at 02:48:49PM GMT, Geert Uytterhoeven wrote: > Hi Laurent, Jacopo, > > On Wed, Jun 19, 2024 at 9:46 PM Laurent Pinchart > wrote: > > On Wed, Jun 19, 2024 at 12:22:18PM +0200, Jacopo Mondi wrote: > > > Add support for R-Car R8A

Re: [PATCH 3/4] drm: rcar-mipi-dsi: Add support for R8A779H0

2024-06-19 Thread Jacopo Mondi
Hi Krzysztof On Wed, Jun 19, 2024 at 01:07:24PM GMT, Krzysztof Kozlowski wrote: > On 19/06/2024 12:22, Jacopo Mondi wrote: > > Add support for R-Car R8A779H0 V4M which has the same characteristics > > of the already supported R-Car V4H R8A779G0. > > > > Signed-off-by: J

[PATCH 2/4] drm: ti-sn65dsi86: Check bridge connection failure

2024-06-19 Thread Jacopo Mondi
From: Phong Hoang Add a check to the register access function when attaching a bridge device. Signed-off-by: Phong Hoang Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/ti

[PATCH 3/4] drm: rcar-mipi-dsi: Add support for R8A779H0

2024-06-19 Thread Jacopo Mondi
Add support for R-Car R8A779H0 V4M which has the same characteristics of the already supported R-Car V4H R8A779G0. Signed-off-by: Jacopo Mondi --- BSP patch: https://github.com/renesas-rcar/linux-bsp/commit/61b876a8fa2c5d0f8049ecf29f24e3dd73ba9f8b --- .../devicetree/bindings/display/bridge

[PATCH 4/4] drm: rcar-du: Add support for R8A779H0

2024-06-19 Thread Jacopo Mondi
Add support for R-Car R8A779H0 V4M which has similar characteristics as the already supported R-Car V4H R8A779G0, but with a single output channel. Signed-off-by: Jacopo Mondi --- BSP patch https://github.com/renesas-rcar/linux-bsp/commit/f2fc3314dab2052240653c1a31ba3d7c7190038e

[PATCH 1/4] drm: rcar-mipi-dsi: Fix CLOCKSET1_LOCK definition

2024-06-19 Thread Jacopo Mondi
simply called "lock" in the datasheet. Signed-off-by: Takeshi Kihara Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi_regs.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi_regs.h b/drive

[PATCH 0/4] drm: rcar-du: Add support for R8A779H0

2024-06-19 Thread Jacopo Mondi
Hawk board available yet. Once the board will be available, DTS integration will follow. Jacopo Mondi (2): drm: rcar-mipi-dsi: Add support for R8A779H0 drm: rcar-du: Add support for R8A779H0 Phong Hoang (1): drm: ti-sn65dsi86: Check bridge connection failure Takeshi Kihara (1): drm: rcar-mip

Re: [PATCH] drm/fourcc: Add RGB161616 and BGR161616 formats

2024-05-02 Thread Jacopo Mondi
Hello which tree should this patch be collected from now that it has been reviewed ? On Mon, Feb 26, 2024 at 02:25:43PM GMT, Jacopo Mondi wrote: > Add FourCC definitions for the 48-bit RGB/BGR formats to the > DRM/KMS uapi. > > The format will be used by the Raspberry Pi P

Re: [PATCH 0/2] drm/fourcc.h: Add libcamera to Open Source Waiver

2024-03-14 Thread Jacopo Mondi
Hello gentle nudge for *) libcamera: are we ok being listed here ? *) DRM/KMS: is it ok splitting the list of projects in the way I've done ? Thanks j On Wed, Feb 28, 2024 at 11:22:42AM +0100, Jacopo Mondi wrote: > As suggested by Sima, add libcamera to the list of projects to w

Re: [RFC] drm/fourcc: Add RPI modifiers

2024-02-28 Thread Jacopo Mondi
Hi Laurent On Tue, Feb 27, 2024 at 03:08:27PM +0200, Laurent Pinchart wrote: > On Mon, Feb 26, 2024 at 04:46:24PM +0100, Daniel Vetter wrote: > > On Mon, 26 Feb 2024 at 16:39, Jacopo Mondi wrote: > > > > > > Add modifiers for the Raspberry Pi PiSP compressed formats.

[PATCH 2/2] drm/fourcc.h: Add libcamera to Open Source Waiver

2024-02-28 Thread Jacopo Mondi
he library. Add libcamera to the list projects to which the 'Open Source User Waiver' notice applies. Signed-off-by: Jacopo Mondi --- include/uapi/drm/drm_fourcc.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.

[PATCH 1/2] drm/fourcc.h: List of Open Source Waiver projects

2024-02-28 Thread Jacopo Mondi
Signed-off-by: Jacopo Mondi --- include/uapi/drm/drm_fourcc.h | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h index 84d502e42961..4e6df826946a 100644 --- a/include/uapi/drm/drm_fourcc.h +++ b/includ

[PATCH 0/2] drm/fourcc.h: Add libcamera to Open Source Waiver

2024-02-28 Thread Jacopo Mondi
As suggested by Sima, add libcamera to the list of projects to which the Open Source Waiver notice applies. To maintain the paragraph readable, make a list out of the projects to which such notice applies. Jacopo Mondi (2): drm/fourcc.h: List of Open Source Waiver projects drm/fourcc.h: Add

Re: [RFC] drm/fourcc: Add RPI modifiers

2024-02-26 Thread Jacopo Mondi
Hi Sima On Mon, Feb 26, 2024 at 04:46:24PM +0100, Daniel Vetter wrote: > On Mon, 26 Feb 2024 at 16:39, Jacopo Mondi > wrote: > > > > Add modifiers for the Raspberry Pi PiSP compressed formats. > > > > The compressed formats are documented at: > > Documentation

[RFC] drm/fourcc: Add RPI modifiers

2024-02-26 Thread Jacopo Mondi
-off-by: Jacopo Mondi --- Background: --- The Raspberry Pi PiSP camera subsystem is on its way to upstream through the Video4Linux2 subsystem: https://patchwork.linuxtv.org/project/linux-media/list/?series=12310 The PiSP camera system is composed by a "Front End" and a "

[PATCH] drm/fourcc: Add RGB161616 and BGR161616 formats

2024-02-26 Thread Jacopo Mondi
Add FourCC definitions for the 48-bit RGB/BGR formats to the DRM/KMS uapi. The format will be used by the Raspberry Pi PiSP Back End, supported by a V4L2 driver in kernel space and by libcamera in userspace, which uses the DRM FourCC identifiers. Signed-off-by: Jacopo Mondi --- drivers/gpu/drm

Re: [PATCH v12 3/4] drm: renesas: Add RZ/G2L DU Support

2023-11-15 Thread Jacopo Mondi
Hi Biju On Wed, Nov 15, 2023 at 12:36:55PM +, Biju Das wrote: > Hi Jacopo Mondi, > > Thanks for the feedback. > > > Subject: Re: [PATCH v12 3/4] drm: renesas: Add RZ/G2L DU Support > > > > Hi Biju > > > > the series does not apply on v6.3. What i

Re: [PATCH v12 3/4] drm: renesas: Add RZ/G2L DU Support

2023-11-07 Thread Jacopo Mondi
Hi Biju the series does not apply on v6.3. What is the base for the series ? On Thu, Nov 02, 2023 at 03:07:43PM +, Biju Das wrote: > The LCD controller is composed of Frame Compression Processor (FCPVD), > Video Signal Processor (VSPD), and Display Unit (DU). > > It has DPI/DSI interfaces a

Re: [PATCH v10 3/4] drm: renesas: Add RZ/G2L DU Support

2023-09-19 Thread Jacopo Mondi
Hi Biju On Mon, Sep 18, 2023 at 08:09:58AM +, Biju Das wrote: > Hi Jacopo Mondi, > > Looks like you are happy with my response for V10. I will send V11 soon. Sorry for the late reply.. See below, I only see two "controversial" points > > Cheers, > Biju

Re: [PATCH v10 3/4] drm: renesas: Add RZ/G2L DU Support

2023-09-07 Thread Jacopo Mondi
Hi Biju On Tue, Jul 04, 2023 at 10:04:46AM +0100, Biju Das wrote: > The LCD controller is composed of Frame Compression Processor (FCPVD), > Video Signal Processor (VSPD), and Display Unit (DU). > > It has DPI/DSI interfaces and supports a maximum resolution of 1080p > along with 2 RPFs to support

[RFC 6/9] drm: rcar-du: cmm: Refactor LUT configuration

2023-06-21 Thread Jacopo Mondi
Bingham Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_cmm.c | 60 -- drivers/gpu/drm/rcar-du/rcar_cmm.h | 19 +--- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 22 +++--- 3 files changed, 55 insertions(+), 46 deletions(-) diff --git a/drivers/gpu

[RFC 8/9] drm: rcar-du: kms: Configure the CLU

2023-06-21 Thread Jacopo Mondi
From: Kieran Bingham Link the DRM 3D-CLU configuration to the CMM setup configuration. Signed-off-by: Kieran Bingham Signed-off-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 23 ++- 1 file

[RFC 7/9] drm: rcar-du: cmm: Provide 3D-CLU support

2023-06-21 Thread Jacopo Mondi
specific color adjustment. Signed-off-by: Kieran Bingham Signed-off-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_cmm.c | 71 +- drivers/gpu/drm/rcar-du/rcar_cmm.h | 11 +++-- 2 files changed, 76

[RFC 9/9] drm: rcar-du: crtc: Enable 3D LUT

2023-06-21 Thread Jacopo Mondi
Enable the 3D LUT in rcar_du_crtc by first creating a property for the supported 3d lut modes and by calling the drm_crtc_enable_lut3d() helper. Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_cmm.h | 14 ++ drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 23

[RFC 5/9] drm/drm_color_mgmt: add function to attach 3D LUT props

2023-06-21 Thread Jacopo Mondi
From: Melissa Wen If the driver supports user 3D LUT then it calls a drm function to attach 3D LUT related properties according to HW caps. Signed-off-by: Melissa Wen --- drivers/gpu/drm/drm_color_mgmt.c | 35 include/drm/drm_color_mgmt.h | 3 +++ 2 files

[RFC 4/9] drm/drm_color_mgmt: add function to create 3D LUT modes supported

2023-06-21 Thread Jacopo Mondi
From: Melissa Wen DRM color function to create modes for lut3d mode property from an array of drm_color_lut3d_mode modes supported by the HW and advertise to userspace. Userspace can get the description of a specific mode in the enum list from its blob data. Signed-off-by: Melissa Wen --- driv

[RFC 2/9] drm/drm_color_mgmt: add shaper LUT to color mgmt properties

2023-06-21 Thread Jacopo Mondi
From: Melissa Wen Shaper LUT is used to shape the content after blending, i.e., de-linearize or normalize space before applying a 3D LUT color correction. In the next patch, we add 3D LUT property to DRM color management after this shaper LUT and before the current gamma LUT. Signed-off-by: Meli

[RFC 3/9] drm/drm_color_mgmt: add 3D LUT props to DRM color mgmt

2023-06-21 Thread Jacopo Mondi
From: Melissa Wen Add 3D LUT for gammar correction using a 3D lookup table. The position in the color correction pipeline where 3D LUT is applied depends on hw design, being after CTM or gamma. If just after CTM, a shaper lut must be set to shape the content for a non-linear space. That details s

[RFC 1/9] drm: Add 3D LUT mode and its attributes

2023-06-21 Thread Jacopo Mondi
From: Alex Hung A struct is defined for 3D LUT modes to be supported by hardware. The elements includes lut_size, lut_stride, bit_depth, color_format and flags. Note: A patchset "IGT tests for pre-blending 3D LUT interfaces" for this proposal is sent to IGT mailing list. Signed-off-by: Alex Hun

[RFC 0/9] drm: rcar-du: cmm: Enable 3D LUT support

2023-06-21 Thread Jacopo Mondi
test) which are available at: https://git.sr.ht/~jmondi_/kms-test The series validates the usage of the HW interface in the hope of re-starting discussions and interests in the definition of CRTC color management properties. Thanks j Alex Hung (1): drm: Add 3D LUT mode and its attributes Jac

Re: [RFC PATCH v2 00/18] Add DRM CRTC 3D LUT interface

2023-06-15 Thread Jacopo Mondi
Hi Pekka thanks for the reply On Thu, Jun 15, 2023 at 10:14:05AM +0300, Pekka Paalanen wrote: > On Tue, 13 Jun 2023 17:43:55 +0200 > Jacopo Mondi wrote: > > > Hello > > > >I'm completing the support for 3D LUT on R-Car DU peripheral > > and I have u

Re: [RFC PATCH v2 00/18] Add DRM CRTC 3D LUT interface

2023-06-14 Thread Jacopo Mondi
Hello I'm completing the support for 3D LUT on R-Car DU peripheral and I have used this series as a base. I'm wondering, since quite some time has passed without any update if this series is still a thing and it makes any sense for me to try to bring it forward. I'm asking as I've noticed: "[

Re: [PATCH v3 1/6] dt-bindings: media: Add macros for video interface bus types

2022-06-28 Thread Jacopo Mondi
gt; > Signed-off-by: Laurent Pinchart Just putting a tag here as we have more flags to add here (in example the device orientation) Reviewed-by: Jacopo Mondi > --- > Changes since v2: > > - Go back to PARALLEL > > Changes since v1: > > - Dual-license under GPL-2.0-o

Re: [PATCH v2 1/4] drm: rcar-du: lvds: Convert to DRM panel bridge helper

2020-12-16 Thread Jacopo Mondi
Hi Laurent, On Wed, Dec 16, 2020 at 03:49:24PM +0200, Laurent Pinchart wrote: > Hi Jacopo, > > > > + if (lvds->panel) { > > > + lvds->next_bridge = devm_drm_panel_bridge_add(lvds->dev, > > > + lvds->panel); > > > > Reading the devm_drm_

Re: [PATCH v2] drm: rcar-du: Fix leak of CMM platform device reference

2020-12-16 Thread Jacopo Mondi
Hi Laurent On Wed, Dec 16, 2020 at 04:24:11PM +0200, Laurent Pinchart wrote: > Hi Jacopo, > > On Wed, Dec 16, 2020 at 03:16:28PM +0100, Jacopo Mondi wrote: > > On Wed, Dec 16, 2020 at 04:08:36PM +0200, Laurent Pinchart wrote: > > > The device references acquired by of_find_

Re: [PATCH v2] drm: rcar-du: Fix leak of CMM platform device reference

2020-12-16 Thread Jacopo Mondi
Hi Laurent, On Wed, Dec 16, 2020 at 04:08:36PM +0200, Laurent Pinchart wrote: > The device references acquired by of_find_device_by_node() are not > released by the driver. Fix this by registering a cleanup action. > > Fixes: 8de707aeb452 ("drm: rcar-du: kms: Initialize CMM instances") > Signed-of

Re: [PATCH] dt-bindings: display: bridge: renesas, lvds: RZ/G2E needs renesas,companion too

2020-12-16 Thread Jacopo Mondi
Hi Laurent, Fabrizio On Wed, Dec 16, 2020 at 12:59:27AM +0200, Laurent Pinchart wrote: > From: Fabrizio Castro > > Document RZ/G2E support for property renesas,companion. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Laurent Pinchart > Signed-off-by: Laurent Pinchart

Re: [PATCH] drm: rcar-du: Fix leak of CMM platform device reference

2020-12-16 Thread Jacopo Mondi
Hi Laurent, I wonder if 'leaked' is correct in subject. It probably is, un-balanced ref-counting will prevent the device from being released. It should however happen only at system tear-down, doesn't it ? On Wed, Dec 16, 2020 at 03:22:18AM +0200, Laurent Pinchart wrote: > The device references a

Re: [PATCH v2 1/4] drm: rcar-du: lvds: Convert to DRM panel bridge helper

2020-12-16 Thread Jacopo Mondi
Hi Laurent, On Wed, Dec 16, 2020 at 02:50:18AM +0200, Laurent Pinchart wrote: > Replace the manual panel handling with usage of the DRM panel bridge > helper. This simplifies the driver, and brings support for > DRM_BRIDGE_ATTACH_NO_CONNECTOR as an added bonus. > > Signed-off-by: Laurent Pinchart

Re: [PATCH v2 4/4] drm: rcar-du: Use drm_bridge_connector_init() helper

2020-12-16 Thread Jacopo Mondi
ase); > + if (IS_ERR(connector)) { > + dev_err(rcdu->dev, > + "failed to created connector for output %u\n", output); > + return PTR_ERR(connector); > + } > + > + return drm_connector_attach_encoder(connector, &re

Re: [PATCH] drm: rcar-du: fix reference leak in rcar_cmm_enable

2020-12-15 Thread Jacopo Mondi
hen applying if they feel to. > > Fixes: e08e934d6c28 ("drm: rcar-du: Add support for CMM") > Reported-by: Hulk Robot > Signed-off-by: Qinglang Miao Seems fair! Acked-by: Jacopo Mondi Thanks j > --- > drivers/gpu/drm/rcar-du/rcar_cmm.c | 2 +- > 1 file changed,

Re: [PATCH v2 4/9] drm: rcar-du: Use DRM-managed allocation for VSP planes

2020-12-15 Thread Jacopo Mondi
esult > in the planes being freed before being unregistered during the managed > cleanup of the DRM objects. Use a plain kcalloc(), and cleanup the > planes and free the memory in the existing rcar_du_vsp_cleanup() > handler. > > Signed-off-by: Laurent Pinchart > Reviewed-by: K

Re: [PATCH v2 5/9] drm: rcar-du: Use DRM-managed allocation for encoders

2020-12-15 Thread Jacopo Mondi
ld result > in the encoder being freed before being unregistered during the managed > cleanup of the DRM objects. Use a plain kzalloc(), and register a drmm > action to cleanup the encoder. > > Signed-off-by: Laurent Pinchart Reviewed-by: Jacopo Mondi Thanks j >

Re: [PATCH 9/9] drm: rcar-du: Drop local encoder variable

2020-12-14 Thread Jacopo Mondi
> Signed-off-by: Laurent Pinchart Reviewed-by: Jacopo Mondi Thanks j > --- > drivers/gpu/drm/rcar-du/rcar_du_encoder.c | 6 ++ > drivers/gpu/drm/rcar-du/rcar_du_encoder.h | 2 -- > 2 files changed, 2 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/rcar-d

Re: [PATCH 8/9] drm: rcar-du: Skip encoder allocation for LVDS1 in dual-link mode

2020-12-14 Thread Jacopo Mondi
t); > + > ret = drm_encoder_init(&rcdu->ddev, encoder, &rcar_du_encoder_funcs, > DRM_MODE_ENCODER_NONE, NULL); > if (ret < 0) Do you have any other caller of the 'done:' label left apart from the one after this last line ? In case you don't you can call devm_kfree() here With or without this addressed: Reviewed-by: Jacopo Mondi > -- > Regards, > > Laurent Pinchart > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 7/9] drm: rcar-du: Replace dev_private with container_of

2020-12-14 Thread Jacopo Mondi
off-by: Laurent Pinchart De gustibus non est disputandum Reviewed-by: Jacopo Mondi Thanks j > --- > drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 -- > drivers/gpu/drm/rcar-du/rcar_du_drv.h | 5 + > drivers/gpu/drm/rcar-du/rcar_du_kms.c | 8 > drivers/gpu/drm/rcar-

Re: [PATCH 6/9] drm: rcar-du: Embed drm_device in rcar_du_device

2020-12-14 Thread Jacopo Mondi
-sorting the headers rcar_du_drv without mentioning it too). Reviewed-by: Jacopo Mondi > --- > drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 2 +- > drivers/gpu/drm/rcar-du/rcar_du_drv.c | 33 + > drivers/gpu/drm/rcar-du/rcar_du_drv.h | 5 ++-- &g

Re: [PATCH 5/9] drm: rcar-du: Use DRM-managed allocation for encoders

2020-12-14 Thread Jacopo Mondi
Hi Laurent, On Sat, Dec 05, 2020 at 12:01:35AM +0200, Laurent Pinchart wrote: > devm_kzalloc() is the wrong API to allocate encoders, as the lifetime of > the encoders is tied to the DRM device, not the device to driver > binding. drmm_kzalloc() isn't a good option either, as it would result > in

Re: [PATCH 3/9] drm: rcar-du: Drop unneeded encoder cleanup in error path

2020-12-14 Thread Jacopo Mondi
encoder->funcs->destroy(encoder); This is probably worth a Fixes tag, as accessing encoder->func if drm_simple_encoder_init() has not completed might lead to a NULL pointer dereference. Apart from this, patch looks good Reviewed-by: Jacopo Mondi Thanks j > + if (

Re: [PATCH 1/9] drm: rcar-du: Fix crash when using LVDS1 clock for CRTC

2020-12-14 Thread Jacopo Mondi
encoder is skipped. > > Fix this by storing bridge pointers internally instead of retrieving > them from the encoder. > > Signed-off-by: Laurent Pinchart The patch itself looks good! Reviewed-by: Jacopo Mondi Thanks j > --- > drivers/gpu/drm/rcar-du/rcar_du_crtc.c| 1

Re: [PATCH 2/9] drm: rcar-du: Release vsp device reference in all error paths

2020-12-14 Thread Jacopo Mondi
er safe than sorry Reviewed-by: Jacopo Mondi > --- > drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > index f6a69aa116e6..4dcb1bfbe2

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2020-07-17 Thread Jacopo Mondi
Hello Eugeniu, On Mon, Jun 15, 2020 at 04:17:23PM +0200, Eugeniu Rosca wrote: > Hi Jacopo, > > On Fri, Jun 12, 2020 at 05:12:09PM +0200, Jacopo Mondi wrote: > > On Mon, Jun 08, 2020 at 11:44:32AM +0200, Eugeniu Rosca wrote: > > > FWIW, I seem to hit pre-existing i

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2020-06-12 Thread Jacopo Mondi
Hi Eugeniu, On Tue, Jun 09, 2020 at 04:29:59PM +0200, Eugeniu Rosca wrote: > Hi Laurent, > > On Sun, Jun 07, 2020 at 05:41:58AM +0300, Laurent Pinchart wrote: > > Note that the CMM driver is controlled by the DU driver. As the DU > > driver will reenable the display during resume, it will call > >

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2020-06-12 Thread Jacopo Mondi
Hi Eugeniu On Mon, Jun 08, 2020 at 11:44:32AM +0200, Eugeniu Rosca wrote: > Hello, > > Many thanks for your comments and involvement. > > On Sun, Jun 07, 2020 at 05:41:58AM +0300, Laurent Pinchart wrote: > > On Fri, Jun 05, 2020 at 03:53:15PM +0200, Jacopo Mondi wrote: >

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2020-06-05 Thread Jacopo Mondi
Hi Eugeniu On Fri, Jun 05, 2020 at 03:41:24PM +0200, Eugeniu Rosca wrote: > Hi Jacopo, > > On Fri, Jun 05, 2020 at 03:29:00PM +0200, Jacopo Mondi wrote: > > On Wed, May 27, 2020 at 09:15:55AM +0200, Eugeniu Rosca wrote: > > > Could you kindly share the cross compilation s

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2020-06-05 Thread Jacopo Mondi
Hello Eugeniu, sorry for the late reply On Wed, May 27, 2020 at 09:15:55AM +0200, Eugeniu Rosca wrote: > Hi Jacopo, > > On Tue, Oct 15, 2019 at 12:46:13PM +0200, Jacopo Mondi wrote: > 8<--- > > > * Testing > > I have tested by injecting a color inversi

Re: [PATCH 5/4] dt-bindings: display: bridge: thc63lvd1024: Document dual-output mode

2020-05-14 Thread Jacopo Mondi
Hi Laurent, On Thu, May 14, 2020 at 02:21:27AM +0300, Laurent Pinchart wrote: > The DT binding support both dual-input and dual-output mode, but only > dual-input is documented. Document dual-output mode. > > Suggested-by: Jacopo Mondi > Signed-off-by: Laurent Pinchart Thanks A

Re: [PATCH -next] drm/rcar-du: Fix return value check in rcar_du_cmm_init()

2020-04-28 Thread Jacopo Mondi
707aeb452 ("drm: rcar-du: kms: Initialize CMM instances") > Signed-off-by: Wei Yongjun Correct indeed! Also -ENODEV seems appropriate to me as return value. Thanks! Acked-by: Jacopo Mondi > --- > drivers/gpu/drm/rcar-du/rcar_du_kms.c | 8 > 1 file changed, 4 i

Re: [PATCH 3/4] dt-bindings: display: bridge: thc63lvd1024: Convert binding to YAML

2020-04-05 Thread Jacopo Mondi
> b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml > new file mode 100644 > index ..469ac4a34273 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml > @@ -0,0 +1,121 @@ > +# SPDX-License-Identifier

[PATCH v7 3/7] drm: rcar-du: Add support for CMM

2019-11-14 Thread Jacopo Mondi
advanced CMM features will be implemented on top of this initial one. Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- v6 -> v7 - Expand rcar_cmm_setup() function documentation to detail its relationship with rcar_cmm_enable() and their call or

[PATCH v7 6/7] drm: rcar-du: crtc: Register GAMMA_LUT properties

2019-11-14 Thread Jacopo Mondi
Enable the GAMMA_LUT KMS property using the framework helpers to register the property and set the associated gamma table maximum size. Reviewed-by: Kieran Bingham Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c

[PATCH v7 7/7] drm: rcar-du: kms: Expand comment in vsps parsing routine

2019-11-14 Thread Jacopo Mondi
Expand comment in the 'vsps' parsing routine to specify the LIF channel index defaults to 0 in case the second cell of the property is not specified to remain compatible with older DT bindings. Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --

[PATCH v7 2/7] dt-bindings: display, renesas, du: Document cmms property

2019-11-14 Thread Jacopo Mondi
Document the newly added 'cmms' property which accepts a list of phandle and channel index pairs that point to the CMM units available for each Display Unit output video channel. Reviewed-by: Rob Herring Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Ja

[PATCH v7 1/7] dt-bindings: display: renesas, cmm: Add R-Car CMM documentation

2019-11-14 Thread Jacopo Mondi
: Laurent Pinchart Signed-off-by: Jacopo Mondi --- .../bindings/display/renesas,cmm.yaml | 67 +++ 1 file changed, 67 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/renesas,cmm.yaml diff --git a/Documentation/devicetree/bindings/display/renesas

[PATCH v7 5/7] drm: rcar-du: crtc: Control CMM operations

2019-11-14 Thread Jacopo Mondi
Implement CMM handling in the crtc begin and enable atomic callbacks, and enable CMM unit through the Display Extensional Functions register at group setup time. Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- v6 -> v7 - Drop check for CMM

[PATCH v7 4/7] drm: rcar-du: kms: Initialize CMM instances

2019-11-14 Thread Jacopo Mondi
ess device link between the two. Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 6 ++ drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 + drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 + drivers/gpu/d

[PATCH v7 0/7] drm: rcar-du: Add Color Management Module (CMM)

2019-11-14 Thread Jacopo Mondi
5/7 I've sent as v6.1 against the v6 series - Make the DRM_RCAR_CMM symbol a tristate and make it implied by DRM_RCAR_DU to prevent having DU built-in and CMM has a module, which causes linkage errors due to unresolved symbols. Jacopo Mondi (7): dt-bindings: display: renesas,cmm: Add R-

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2019-11-11 Thread Jacopo Mondi
org > ow...@vger.kernel.org> On Behalf Of Jacopo Mondi > > Sent: Tuesday, October 15, 2019 4:16 PM > > To: laurent.pinch...@ideasonboard.com; > > kieran.bingham+rene...@ideasonboard.com; ge...@linux-m68k.org; > > ho...@verge.net.au; uli+rene...@fpond.eu; Kalakodima Ve

Re: [PATCH v3 2/7] drm/bridge: Repurpose lvds-encoder.c

2019-11-08 Thread Jacopo Mondi
Hi Laurent, On Fri, Nov 08, 2019 at 01:06:58PM +0200, Laurent Pinchart wrote: > Hello Jacopo, > > On Fri, Nov 08, 2019 at 10:39:27AM +0100, Jacopo Mondi wrote: > > On Fri, Nov 08, 2019 at 09:22:56AM +, Fabrizio Castro wrote: > > > On 07 November 2019 20:35 Laurent Pi

Re: [PATCH v3 2/7] drm/bridge: Repurpose lvds-encoder.c

2019-11-08 Thread Jacopo Mondi
Hello, On Fri, Nov 08, 2019 at 09:22:56AM +, Fabrizio Castro wrote: > Hello Laurent, > > Thank you for your feedback! > > > From: devicetree-ow...@vger.kernel.org > > On Behalf Of Laurent Pinchart > > Sent: 07 November 2019 20:35 > > Subject: Re: [PATCH v3 2/7] drm/bridge: Repurpose lvds-enc

Re: [PATCH v2 1/4] drm/bridge: Repurpose lvds-encoder.c

2019-11-07 Thread Jacopo Mondi
Hi Fabrizio, On Thu, Nov 07, 2019 at 08:02:25PM +, Fabrizio Castro wrote: > Hi Jacopo, > > Thank you for your feedback! > > > From: Jacopo Mondi > > Sent: 07 November 2019 18:19 > > Subject: Re: [PATCH v2 1/4] drm/bridge: Repurpose lvds-encoder.c > > &

Re: [PATCH v2 1/4] drm/bridge: Repurpose lvds-encoder.c

2019-11-07 Thread Jacopo Mondi
Hi Fabrizio, thanks for the patch. On Mon, Nov 04, 2019 at 04:58:00PM +, Fabrizio Castro wrote: > lvds-encoder.c implementation is also suitable for LVDS decoders, > not just LVDS encoders. > Instead of creating a new driver for addressing support for > transparent LVDS decoders, repurpose l

Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support

2019-11-07 Thread Jacopo Mondi
Hi Laurent, On Wed, Nov 06, 2019 at 02:56:53PM +0200, Laurent Pinchart wrote: > Hi Jacopo, > > On Wed, Nov 06, 2019 at 02:52:25PM +0200, Laurent Pinchart wrote: > > On Wed, Nov 06, 2019 at 11:46:28AM +0100, Jacopo Mondi wrote: > > > On Wed, Nov 06, 2019 at 12:02:05PM +0200

Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support

2019-11-06 Thread Jacopo Mondi
Hi Laurent, Dave, On Wed, Nov 06, 2019 at 12:02:05PM +0200, Laurent Pinchart wrote: > This time with Jacopo correctly CC'ed. > > On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote: > > Hi Dave, > > > > (CC'ing Jacopo) > > > > On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrot

Re: [PATCH v7 0/9] backlight: gpio: simplify the driver

2019-11-01 Thread Jacopo Mondi
Hello, as promised... On Fri, Nov 01, 2019 at 08:58:03AM +, Lee Jones wrote: > On Thu, 24 Oct 2019, Jacopo Mondi wrote: > > > Hello, > > > > On Thu, Oct 24, 2019 at 07:47:26AM +0100, Lee Jones wrote: > > > On Wed, 23 Oct 2019, Daniel Thompson wrote: > &g

Re: [PATCH v7 0/9] backlight: gpio: simplify the driver

2019-10-24 Thread Jacopo Mondi
Hello, On Thu, Oct 24, 2019 at 07:47:26AM +0100, Lee Jones wrote: > On Wed, 23 Oct 2019, Daniel Thompson wrote: > > > On Tue, Oct 22, 2019 at 11:29:54AM +0200, Bartosz Golaszewski wrote: > > > wt., 22 paź 2019 o 10:36 Bartosz Golaszewski napisał(a): > > > > > > > > From: Bartosz Golaszewski > >

Re: [PATCH v5 0/7] backlight: gpio: simplify the driver

2019-10-18 Thread Jacopo Mondi
Hi, On Thu, Oct 17, 2019 at 08:25:50AM +0100, Lee Jones wrote: > On Wed, 16 Oct 2019, Jacopo Mondi wrote: > > > Hi, sorry for not having replied earlier > > > > On Wed, Oct 16, 2019 at 02:56:57PM +0200, Linus Walleij wrote: > > > On Mon, Oct 14,

[PATCH v6.1 5/8] drm: rcar-du: crtc: Control CMM operations

2019-10-17 Thread Jacopo Mondi
Implement CMM handling in the crtc begin and enable atomic callbacks, and enable CMM unit through the Display Extensional Functions register at group setup time. Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- v6 -> v6.1 - Drop check for CMM

[PATCH 6.1 3/8] drm: rcar-du: Add support for CMM

2019-10-17 Thread Jacopo Mondi
advanced CMM features will be implemented on top of this initial one. Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- v6 -> v6.1 - Expand rcar_cmm_setup() function documentation to detail its relationship with rcar_cmm_enable() and their call or

Re: [PATCH v6 3/8] drm: rcar-du: Add support for CMM

2019-10-17 Thread Jacopo Mondi
Hi Laurent, On Wed, Oct 16, 2019 at 04:45:26PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Wed, Oct 16, 2019 at 10:55:43AM +0200, Jacopo Mondi wrote: > > Add a driver for the R-Car Display Unit Color Correction Module. > > > &

Re: [PATCH v5 0/7] backlight: gpio: simplify the driver

2019-10-16 Thread Jacopo Mondi
Hi, sorry for not having replied earlier On Wed, Oct 16, 2019 at 02:56:57PM +0200, Linus Walleij wrote: > On Mon, Oct 14, 2019 at 10:12 AM Lee Jones wrote: > > > > arch/sh/boards/mach-ecovec24/setup.c | 33 -- > > > > I guess we're just waiting for the SH Acks now? > > The one mainta

[PATCH v6 5/8] drm: rcar-du: crtc: Control CMM operations

2019-10-16 Thread Jacopo Mondi
Implement CMM handling in the crtc begin and enable atomic callbacks, and enable CMM unit through the Display Extensional Functions register at group setup time. Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 55

[PATCH v6 6/8] drm: rcar-du: crtc: Register GAMMA_LUT properties

2019-10-16 Thread Jacopo Mondi
Enable the GAMMA_LUT KMS property using the framework helpers to register the property and set the associated gamma table maximum size. Reviewed-by: Kieran Bingham Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c

[PATCH v6 1/8] dt-bindings: display: renesas,cmm: Add R-Car CMM documentation

2019-10-16 Thread Jacopo Mondi
-off-by: Jacopo Mondi --- .../bindings/display/renesas,cmm.yaml | 67 +++ 1 file changed, 67 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/renesas,cmm.yaml diff --git a/Documentation/devicetree/bindings/display/renesas,cmm.yaml b/Documentation

[PATCH v6 8/8] drm: rcar-du: kms: Expand comment in vsps parsing routine

2019-10-16 Thread Jacopo Mondi
Expand comment in the 'vsps' parsing routine to specify the LIF channel index defaults to 0 in case the second cell of the property is not specified to remain compatible with older DT bindings. Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --

[PATCH v6 4/8] drm: rcar-du: kms: Initialize CMM instances

2019-10-16 Thread Jacopo Mondi
ess device link between the two. Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 6 ++ drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 + drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 + drivers/gpu/d

[PATCH v6 2/8] dt-bindings: display, renesas,du: Document cmms property

2019-10-16 Thread Jacopo Mondi
Document the newly added 'cmms' property which accepts a list of phandle and channel index pairs that point to the CMM units available for each Display Unit output video channel. Reviewed-by: Rob Herring Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Ja

[PATCH v6 7/8] arm64: dts: renesas: Add CMM units to Gen3 SoCs

2019-10-16 Thread Jacopo Mondi
wed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- 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/r8a7799

[PATCH v6 3/8] drm: rcar-du: Add support for CMM

2019-10-16 Thread Jacopo Mondi
advanced CMM features will be implemented on top of this initial one. Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/Kconfig| 7 + drivers/gpu/drm/rcar-du/Makefile | 1 + drivers/gpu/drm/rcar-du/rcar_cmm.c | 212

[PATCH v6 0/8] drm: rcar-du: Add Color Management Module (CMM)

2019-10-16 Thread Jacopo Mondi
erday 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 ? Thanks j Jacopo Mondi (8): dt-bindings: display: renesas,cmm: Add R-Car CMM document

Re: [PATCH v5 5/8] drm: rcar-du: crtc: Control CMM operations

2019-10-15 Thread Jacopo Mondi
Hi Laurent, On Tue, Oct 15, 2019 at 08:54:22PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Tue, Oct 15, 2019 at 03:37:52PM +0200, Jacopo Mondi wrote: > > On Tue, Oct 15, 2019 at 02:15:35PM +0100, Kieran Bingham wrote: > > > On

Re: [PATCH v5 5/8] drm: rcar-du: crtc: Control CMM operations

2019-10-15 Thread Jacopo Mondi
Hi Kieran, On Tue, Oct 15, 2019 at 02:15:35PM +0100, Kieran Bingham wrote: > Hi Jacopo, > > On 15/10/2019 11:46, Jacopo Mondi wrote: > > Implement CMM handling in the crtc begin and enable atomic callbacks, > > and enable CMM unit through the Display Extensional Func

Re: [PATCH v5 3/8] drm: rcar-du: Add support for CMM

2019-10-15 Thread Jacopo Mondi
Hi Kieran, thanks for review On Tue, Oct 15, 2019 at 12:53:55PM +0100, Kieran Bingham wrote: > Hi Jacopo, > > On 15/10/2019 11:46, Jacopo Mondi wrote: > > Add a driver for the R-Car Display Unit Color Correction Module. > > > > In most of Gen3 SoCs, each DU output cha

[PATCH v5 8/8] drm: rcar-du: kms: Expand comment in vsps parsing routine

2019-10-15 Thread Jacopo Mondi
Expand comment in the 'vsps' parsing routine to specify the LIF channel index defaults to 0 in case the second cell of the property is not specified to remain compatible with older DT bindings. Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rca

[PATCH v5 7/8] arm64: dts: renesas: Add CMM units to Gen3 SoCs

2019-10-15 Thread Jacopo Mondi
Add CMM units to Renesas R-Car Gen3 SoC that support it, and reference them from the Display Unit they are connected to. Sort the 'vsps', 'renesas,cmm' and 'status' properties in the DU unit consistently in all the involved DTS. Signed-off-by: Jacopo Mondi --

[PATCH v5 3/8] drm: rcar-du: Add support for CMM

2019-10-15 Thread Jacopo Mondi
advanced CMM features will be implemented on top of this initial one. Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/Kconfig| 7 + drivers/gpu/drm/rcar-du/Makefile | 1 + drivers/gpu/drm/rcar-du/rcar_cmm.c | 198 + drivers/gpu/drm/rcar-du/rcar_cmm.h | 60

[PATCH v5 4/8] drm: rcar-du: kms: Initialize CMM instances

2019-10-15 Thread Jacopo Mondi
ess device link between the two. Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 6 ++ drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 + drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 + drivers/gpu/d

[PATCH v5 5/8] drm: rcar-du: crtc: Control CMM operations

2019-10-15 Thread Jacopo Mondi
Implement CMM handling in the crtc begin and enable atomic callbacks, and enable CMM unit through the Display Extensional Functions register at group setup time. Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 79 + drivers/gpu/drm/rcar-du

[PATCH v5 6/8] drm: rcar-du: crtc: Register GAMMA_LUT properties

2019-10-15 Thread Jacopo Mondi
Enable the GAMMA_LUT KMS property using the framework helpers to register the property and set the associated gamma table maximum size. Reviewed-by: Kieran Bingham Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart Signed-off-by: Jacopo Mondi --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c

[PATCH v5 2/8] dt-bindings: display, renesas,du: Document cmms property

2019-10-15 Thread Jacopo Mondi
Document the newly added 'cmms' property which accepts a list of phandle and channel index pairs that point to the CMM units available for each Display Unit output video channel. Reviewed-by: Rob Herring Reviewed-by: Kieran Bingham Reviewed-by: Laurent Pinchart Signed-off-by: Ja

  1   2   3   4   >