Re: [PATCH v8 5/6] drm/imx: Introduce i.MX8qm/qxp DPU DRM

2021-03-02 Thread Laurentiu Palcu
Hi Liu Ying, One comment below. On Tue, Mar 02, 2021 at 02:33:15PM +0800, Liu Ying wrote: > This patch introduces i.MX8qm/qxp Display Processing Unit(DPU) DRM support. > > DPU is comprised of two main components that include a blit engine for > 2D graphics accelerations(with composition

Re: [PATCH v6 5/6] drm/imx: Introduce i.MX8qm/qxp DPU DRM

2021-01-25 Thread Laurentiu Palcu
Hi Liu Ying, Just some minor comments below. On Thu, Jan 21, 2021 at 03:14:22PM +0800, Liu Ying wrote: > This patch introduces i.MX8qm/qxp Display Processing Unit(DPU) DRM support. > > DPU is comprised of two main components that include a blit engine for > 2D graphics accelerations(with

Re: [PATCH v5 5/6] drm/imx: Introduce i.MX8qm/qxp DPU DRM

2021-01-20 Thread Laurentiu Palcu
Hi Liu Ying, On Wed, Jan 20, 2021 at 04:42:50PM +0800, Liu Ying wrote: > Hi Laurentiu, > > On Fri, 2021-01-15 at 19:27 +0200, Laurentiu Palcu wrote: > > Hi Liu Ying, > > > > I promised I would have a second, more in-depth, look at this and I finally > > managed

Re: [PATCH v5 5/6] drm/imx: Introduce i.MX8qm/qxp DPU DRM

2021-01-15 Thread Laurentiu Palcu
Hi Liu Ying, I promised I would have a second, more in-depth, look at this and I finally managed to do it. I have to admit it was a challenge. Partially because I'm not very familiar with DPU but mostly because of the abundance of macros used. It's true, macros make the code more compact.

[PATCH] arm64: dts: imx8mq: add DCSS node

2020-11-25 Thread Laurentiu Palcu
This patch adds the node for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- Hi, This is, actually, a resend of the patch because we decided to drop it from the main DCSS patchset until the driver gets merged. Now that the driver is in mainline, we can finally add DCSS

Re: [RFC] dcss: fix attaching to sn56dsi86 bridge

2020-11-24 Thread Laurentiu Palcu
Hi Lukas, On Tue, Nov 24, 2020 at 06:19:57PM +0100, Lukas F. Hartmann wrote: > The sn56dsi86 DSI to eDP bridge driver does not support attaching > without a drm connector. I think the SN65DSI86 driver is exactly what you should focus on, so that it works when connector is optional. The

Re: [PATCH 5/8] drm/imx: Introduce i.MX8qxp DPU DRM

2020-11-23 Thread Laurentiu Palcu
Hi Liu Ying, On Mon, Nov 23, 2020 at 10:45:38AM +0800, Liu Ying wrote: > Hi Laurentiu, > > On Fri, 2020-11-20 at 16:38 +0200, Laurentiu Palcu wrote: > > Hi Liu Ying, > > > > I gave this a first look but, since this is a huge piece of code and I'm not > > very

Re: [PATCH 5/8] drm/imx: Introduce i.MX8qxp DPU DRM

2020-11-20 Thread Laurentiu Palcu
Hi Liu Ying, I gave this a first look but, since this is a huge piece of code and I'm not very familiar with DPU, I'll probably give it another pass next week. Anyway, some comments/questions inline. On Thu, Nov 19, 2020 at 11:22:22AM +0200, Liu Ying wrote: > This patch introduces i.MX8qxp

Re: [PATCH 0/8] drm/imx: Introduce i.MX8qxp DPU DRM

2020-11-19 Thread Laurentiu Palcu
Hi Liu Ying, On Thu, Nov 19, 2020 at 05:22:17PM +0800, Liu Ying wrote: > Hi, > > > This patch set introduces i.MX8qxp Display Processing Unit(DPU) DRM support. Glad to see this series out. However, something went wrong with it as patch 5/8 didn't make it to dri-devel mailing list... :/

[PATCH] drm/imx/dcss: allow using nearest neighbor interpolation scaling

2020-11-05 Thread Laurentiu Palcu
This patch adds support for using NN interpolation scaling by setting the SCALING_FILTER plane property to 1. Otherwise, the default method is used. Signed-off-by: Laurentiu Palcu --- I had no retro pixel art games to test this with, so I used modetest to see the results: To test, I used

[PATCH 2/2] drm/imx/dcss: fix coccinelle warning

2020-11-05 Thread Laurentiu Palcu
X8MQ") Signed-off-by: Laurentiu Palcu --- drivers/gpu/drm/imx/dcss/dcss-plane.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/imx/dcss/dcss-plane.c b/drivers/gpu/drm/imx/dcss/dcss-plane.c index 46a188dd02ad..5db093aada2f 100644 --- a/drivers/gpu/drm/imx

[PATCH 0/2] drm/imx/dcss: a couple of fixes

2020-11-05 Thread Laurentiu Palcu
Hi, This patchset fixes 90/270 rotations for Vivante tiled and super-tiled formats and a Coccinelle warning. Thanks, laurentiu Laurentiu Palcu (2): drm/imx/dcss: fix rotations for Vivante tiled formats drm/imx/dcss: fix coccinelle warning drivers/gpu/drm/imx/dcss/dcss-plane.c | 14

[PATCH 1/2] drm/imx/dcss: fix rotations for Vivante tiled formats

2020-11-05 Thread Laurentiu Palcu
width and height need to be swapped since DPR is sending the buffer to scaler already rotated. Also, make sure to allow full rotations for DRM_FORMAT_MOD_VIVANTE_SUPER_TILED. Fixes: 9021c317b770 ("drm/imx: Add initial support for DCSS on iMX8MQ") Signed-off-by: Laurentiu Palcu --- d

Re: [PATCH v4 06/11] Add maintainer for IMX jpeg v4l2 driver

2020-11-02 Thread Laurentiu Palcu
Hi Mirela, On Mon, Nov 02, 2020 at 05:08:16AM +0200, Mirela Rabulea (OSS) wrote: > From: Mirela Rabulea > > Added as: > FREESCALE IMX / MXC JPEG V4L2 DRIVER > > Signed-off-by: Mirela Rabulea > --- > MAINTAINERS | 8 > 1 file changed, 8 insertions(+) > > diff --git a/MAINTAINERS

Re: [PATCH v4 04/11] media: imx-jpeg: Add V4L2 driver for i.MX8 JPEG Encoder/Decoder

2020-11-02 Thread Laurentiu Palcu
Hi Mirela, I wanted to give it a more in-depth look but I then saw that patch 11 deletes a lot of code from this file. So, the review on the deleted parts would be pointless... :/ I suggest you squash 4 and 11 together. However, there are a few comments below. On Mon, Nov 02, 2020 at

[PATCH] drm/imx/dcss: fix 64-bit divisions

2020-09-11 Thread Laurentiu Palcu
Use div_s64() for the 64-bit divisions. This would allow the driver to compile on 32-bit architectures, if needed. Signed-off-by: Laurentiu Palcu --- drivers/gpu/drm/imx/dcss/dcss-scaler.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/imx/dcss/dcss

Re: [PATCH -next] drm/imx: fix unused but set variable warnings

2020-09-11 Thread Laurentiu Palcu
plit the patches. But, I guess, we could remove it for now to make the bots happy. Reviewed-by: Laurentiu Palcu > --- > drivers/gpu/drm/imx/dcss/dcss-plane.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/imx/dcss/dcss-plane.c > b/drivers/gpu/dr

Re: [PATCH] drm/imx/dcss: fix compilation issue on 32bit

2020-09-10 Thread Laurentiu Palcu
Hi Lucas, On Thu, Sep 10, 2020 at 12:24:44PM +0200, Lucas Stach wrote: > On Do, 2020-09-10 at 13:21 +0300, Laurentiu Palcu wrote: > > On Thu, Sep 10, 2020 at 11:57:10AM +0200, Daniel Vetter wrote: > > > On Thu, Sep 10, 2020 at 11:53 AM Laurentiu Palcu > > > wr

Re: [PATCH] drm/imx/dcss: fix compilation issue on 32bit

2020-09-10 Thread Laurentiu Palcu
On Thu, Sep 10, 2020 at 11:57:10AM +0200, Daniel Vetter wrote: > On Thu, Sep 10, 2020 at 11:53 AM Laurentiu Palcu > wrote: > > > > When compiling for 32bit platforms, the compilation fails with: > > > > ERROR: modpost: "__aeabi_ldivmod" > > [d

[PATCH] drm/imx/dcss: fix compilation issue on 32bit

2020-09-10 Thread Laurentiu Palcu
e no 32bit SoCs have DCSS, so far. Signed-off-by: Laurentiu Palcu Reported-by: Daniel Vetter --- drivers/gpu/drm/imx/dcss/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/imx/dcss/Kconfig b/drivers/gpu/drm/imx/dcss/Kconfig index 69860de8861f.

Re: [PATCH v9 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-09-10 Thread Laurentiu Palcu
Hi Daniel, On Thu, Sep 10, 2020 at 08:47:34AM +0200, Daniel Vetter wrote: > On Wed, Sep 9, 2020 at 5:03 PM Lucas Stach wrote: > > > > Hi Laurentiu, > > > > On Mo, 2020-08-31 at 14:24 +0300, Laurentiu Palcu wrote: > > > Hi Lucas, Sam, > > > > >

Re: [PATCH v9 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-08-31 Thread Laurentiu Palcu
Hi Lucas, Sam, On Mon, Aug 31, 2020 at 12:37:23PM +0200, Lucas Stach wrote: > Hi Laurentiu, > > On Fr, 2020-08-28 at 11:36 +0300, Laurentiu Palcu wrote: > > Hi Lucas, > > > > I was wondering about the plans to merge this series. Since not many > > people can tes

Re: [PATCH v9 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-08-28 Thread Laurentiu Palcu
haven't been very active in this area... :/ On that note, I will probably need help with the merging, provided it's still happenning. Will you be able to help me out with this? Thanks, laurentiu On Fri, Jul 31, 2020 at 11:18:28AM +0300, Laurentiu Palcu wrote: > From: Laurentiu Palcu >

[PATCH v9 3/5] drm/imx/dcss: use drm_bridge_connector API

2020-07-31 Thread Laurentiu Palcu
From: Laurentiu Palcu Make use of drm_bridge_connector API to have the connector initialized by the display controller. Signed-off-by: Laurentiu Palcu --- drivers/gpu/drm/imx/dcss/dcss-dev.c | 17 +--- drivers/gpu/drm/imx/dcss/dcss-kms.c | 31 - drivers

[PATCH v9 2/5] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-31 Thread Laurentiu Palcu
From: Laurentiu Palcu This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). Some of its capabilities include: * 4K@60fps; * HDR10; * one graphics and 2 video pipelines; * on-the-fly decompression of compressed video and graphics; The reference manual can be found here

[PATCH v9 5/5] dt-bindings: display: imx: add bindings for DCSS

2020-07-31 Thread Laurentiu Palcu
From: Laurentiu Palcu Add bindings for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- Changes in v9: * Include imx8mq-clock.h in the example so we can use clock names instead of their values; .../bindings/display/imx/nxp,imx8mq-dcss.yaml | 108

[PATCH v9 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-07-31 Thread Laurentiu Palcu
From: Laurentiu Palcu Hi, This patchset adds initial DCSS support for iMX8MQ chip. Initial support includes only graphics plane support (no video planes), no HDR10 capabilities, no graphics decompression (only linear, tiled and super-tiled buffers allowed). Support for the rest of the features

[PATCH v9 4/5] MAINTAINERS: Add entry for i.MX 8MQ DCSS driver

2020-07-31 Thread Laurentiu Palcu
From: Laurentiu Palcu The driver is part of DRM subsystem and is located in drivers/gpu/drm/imx/dcss. Signed-off-by: Laurentiu Palcu --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2c669c07fa35..1a22038f2869 100644 --- a/MAINTAINERS

[PATCH v9 1/5] drm/imx: compile imx directory by default

2020-07-31 Thread Laurentiu Palcu
From: Laurentiu Palcu Currently the drm/imx/ directory is compiled only if DRM_IMX is set. Adding a new IMX related IP in the same directory would need DRM_IMX to be set, which would bring in also IPUv3 core driver... The current patch would allow adding new IPs in the imx/ directory without

Re: [PATCH v8 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-07-30 Thread Laurentiu Palcu
Hi Guido, On Wed, Jul 29, 2020 at 05:09:52PM +0200, Guido Günther wrote: > Hi, > On Wed, Jul 29, 2020 at 05:16:47PM +0300, Laurentiu Palcu wrote: > > Hi Guido, > > > > On Wed, Jul 29, 2020 at 03:59:48PM +0200, Guido Günther wrote: > > > Hi, > > >

Re: [PATCH v8 5/5] dt-bindings: display: imx: add bindings for DCSS

2020-07-29 Thread Laurentiu Palcu
On Wed, Jul 29, 2020 at 03:27:30PM +0200, Guido Günther wrote: > Hi, > On Fri, Jul 24, 2020 at 12:07:34PM +0300, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > Add bindings for iMX8MQ Display Controller Subsystem. > > > > Signed-off-by: Laurentiu

Re: [PATCH v8 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-07-29 Thread Laurentiu Palcu
Hi Guido, On Wed, Jul 29, 2020 at 03:59:48PM +0200, Guido Günther wrote: > Hi, > On Fri, Jul 24, 2020 at 12:07:29PM +0300, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > Hi, > > > > This patchset adds initial DCSS support for iMX8MQ chip. Initia

Re: [PATCH v8 2/5] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-29 Thread Laurentiu Palcu
Hi Guido, On Wed, Jul 29, 2020 at 03:11:21PM +0200, Guido Günther wrote: > Hi Laurentiu, > On Fri, Jul 24, 2020 at 12:07:31PM +0300, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). >

[PATCH v8 2/5] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-24 Thread Laurentiu Palcu
From: Laurentiu Palcu This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). Some of its capabilities include: * 4K@60fps; * HDR10; * one graphics and 2 video pipelines; * on-the-fly decompression of compressed video and graphics; The reference manual can be found here

[PATCH v8 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-07-24 Thread Laurentiu Palcu
From: Laurentiu Palcu Hi, This patchset adds initial DCSS support for iMX8MQ chip. Initial support includes only graphics plane support (no video planes), no HDR10 capabilities, no graphics decompression (only linear, tiled and super-tiled buffers allowed). Support for the rest of the features

[PATCH v8 5/5] dt-bindings: display: imx: add bindings for DCSS

2020-07-24 Thread Laurentiu Palcu
From: Laurentiu Palcu Add bindings for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu Reviewed-by: Rob Herring --- .../bindings/display/imx/nxp,imx8mq-dcss.yaml | 104 ++ 1 file changed, 104 insertions(+) create mode 100644 Documentation/devicetree

[PATCH v8 4/5] MAINTAINERS: Add entry for i.MX 8MQ DCSS driver

2020-07-24 Thread Laurentiu Palcu
From: Laurentiu Palcu The driver is part of DRM subsystem and is located in drivers/gpu/drm/imx/dcss. Signed-off-by: Laurentiu Palcu --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index dad5a62d21a7..200c5985b41f 100644 --- a/MAINTAINERS

[PATCH v8 3/5] drm/imx/dcss: use drm_bridge_connector API

2020-07-24 Thread Laurentiu Palcu
From: Laurentiu Palcu Make use of drm_bridge_connector API to have the connector initialized by the display controller. Signed-off-by: Laurentiu Palcu CC: Sam Ravnborg CC: Laurent Pinchart --- drivers/gpu/drm/imx/dcss/dcss-dev.c | 17 +--- drivers/gpu/drm/imx/dcss/dcss-kms.c

[PATCH v8 1/5] drm/imx: compile imx directory by default

2020-07-24 Thread Laurentiu Palcu
From: Laurentiu Palcu Currently the drm/imx/ directory is compiled only if DRM_IMX is set. Adding a new IMX related IP in the same directory would need DRM_IMX to be set, which would bring in also IPUv3 core driver... The current patch would allow adding new IPs in the imx/ directory without

Re: [PATCH v7 2/5] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-21 Thread Laurentiu Palcu
Hi Philipp, On Tue, Jul 21, 2020 at 02:43:28PM +0200, Philipp Zabel wrote: > Hi Laurentiu, > > On Tue, 2020-07-21 at 13:20 +0300, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). &

[PATCH v7 2/5] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-21 Thread Laurentiu Palcu
From: Laurentiu Palcu This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). Some of its capabilities include: * 4K@60fps; * HDR10; * one graphics and 2 video pipelines; * on-the-fly decompression of compressed video and graphics; The reference manual can be found here

[PATCH v7 4/5] MAINTAINERS: Add entry for i.MX 8MQ DCSS driver

2020-07-21 Thread Laurentiu Palcu
From: Laurentiu Palcu The driver is part of DRM subsystem and is located in drivers/gpu/drm/imx/dcss. Signed-off-by: Laurentiu Palcu --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index dad5a62d21a7..200c5985b41f 100644 --- a/MAINTAINERS

[PATCH v7 1/5] drm/imx: compile imx directory by default

2020-07-21 Thread Laurentiu Palcu
From: Laurentiu Palcu Currently the drm/imx/ directory is compiled only if DRM_IMX is set. Adding a new IMX related IP in the same directory would need DRM_IMX to be set, which would bring in also IPUv3 core driver... The current patch would allow adding new IPs in the imx/ directory without

[PATCH v7 5/5] dt-bindings: display: imx: add bindings for DCSS

2020-07-21 Thread Laurentiu Palcu
From: Laurentiu Palcu Add bindings for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu Reviewed-by: Rob Herring --- .../bindings/display/imx/nxp,imx8mq-dcss.yaml | 104 ++ 1 file changed, 104 insertions(+) create mode 100644 Documentation/devicetree

[PATCH v7 3/5] drm/imx/dcss: use drm_bridge_connector API

2020-07-21 Thread Laurentiu Palcu
From: Laurentiu Palcu Make use of drm_bridge_connector API to have the connector initialized by the display controller. Signed-off-by: Laurentiu Palcu CC: Sam Ravnborg CC: Laurent Pinchart --- drivers/gpu/drm/imx/dcss/dcss-dev.c | 17 +--- drivers/gpu/drm/imx/dcss/dcss-kms.c

[PATCH v7 0/5] Add support for iMX8MQ Display Controller Subsystem

2020-07-21 Thread Laurentiu Palcu
From: Laurentiu Palcu Hi, This patchset adds initial DCSS support for iMX8MQ chip. Initial support includes only graphics plane support (no video planes), no HDR10 capabilities, no graphics decompression (only linear, tiled and super-tiled buffers allowed). Support for the rest of the features

Re: [PATCH v6 4/4] dt-bindings: display: imx: add bindings for DCSS

2020-07-20 Thread Laurentiu Palcu
Hi Rob, On Mon, Jul 20, 2020 at 10:49:27AM -0600, Rob Herring wrote: > On Fri, 17 Jul 2020 17:41:29 +0300, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > Add bindings for iMX8MQ Display Controller Subsystem. > > > > Signed-off-by: Laurentiu Palcu

[PATCH] drm/bridge/adv7511: set the bridge type properly

2020-07-20 Thread Laurentiu Palcu
From: Laurentiu Palcu After the drm_bridge_connector_init() helper function has been added, the ADV driver has been changed accordingly. However, the 'type' field of the bridge structure was left unset, which makes the helper function always return -EINVAL. Signed-off-by: Laurentiu Palcu

Re: [PATCH v6 2/4] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-20 Thread Laurentiu Palcu
Hi Sam, On Fri, Jul 17, 2020 at 09:48:49PM +0200, Sam Ravnborg wrote: > Hi Laurentiu. > > On Fri, Jul 17, 2020 at 05:41:27PM +0300, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). &

[PATCH v6 0/4] Add support for iMX8MQ Display Controller Subsystem

2020-07-17 Thread Laurentiu Palcu
From: Laurentiu Palcu Hi, This patchset adds initial DCSS support for iMX8MQ chip. Initial support includes only graphics plane support (no video planes), no HDR10 capabilities, no graphics decompression (only linear, tiled and super-tiled buffers allowed). Support for the rest of the features

[PATCH v6 2/4] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-17 Thread Laurentiu Palcu
From: Laurentiu Palcu This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). Some of its capabilities include: * 4K@60fps; * HDR10; * one graphics and 2 video pipelines; * on-the-fly decompression of compressed video and graphics; The reference manual can be found here

[PATCH v6 4/4] dt-bindings: display: imx: add bindings for DCSS

2020-07-17 Thread Laurentiu Palcu
From: Laurentiu Palcu Add bindings for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- .../bindings/display/imx/nxp,imx8mq-dcss.yaml | 104 ++ 1 file changed, 104 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/imx/nxp

[PATCH v6 3/4] MAINTAINERS: Add entry for i.MX 8MQ DCSS driver

2020-07-17 Thread Laurentiu Palcu
From: Laurentiu Palcu The driver is part of DRM subsystem and is located in drivers/gpu/drm/imx/dcss. Signed-off-by: Laurentiu Palcu --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index dad5a62d21a7..200c5985b41f 100644 --- a/MAINTAINERS

[PATCH v6 1/4] drm/imx: compile imx directory by default

2020-07-17 Thread Laurentiu Palcu
From: Laurentiu Palcu Currently the drm/imx/ directory is compiled only if DRM_IMX is set. Adding a new IMX related IP in the same directory would need DRM_IMX to be set, which would bring in also IPUv3 core driver... The current patch would allow adding new IPs in the imx/ directory without

Re: [PATCH v5 0/4] Add support for iMX8MQ Display Controller Subsystem

2020-07-17 Thread Laurentiu Palcu
> > wrote: > > > > Hi Laurentiu, > > > > > > > > Am Donnerstag, den 09.07.2020, 19:47 +0300 schrieb Laurentiu Palcu: > > > > > From: Laurentiu Palcu > > > > > > > > > > Hi, > > > > > > > > &g

[PATCH v5 2/5] drm/imx: Add initial support for DCSS on iMX8MQ

2020-07-09 Thread Laurentiu Palcu
From: Laurentiu Palcu This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). Some of its capabilities include: * 4K@60fps; * HDR10; * one graphics and 2 video pipelines; * on-the-fly decompression of compressed video and graphics; The reference manual can be found here

[PATCH v5 4/5] dt-bindings: display: imx: add bindings for DCSS

2020-07-09 Thread Laurentiu Palcu
From: Laurentiu Palcu Add bindings for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- .../bindings/display/imx/nxp,imx8mq-dcss.yaml | 84 +++ 1 file changed, 84 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/imx/nxp,imx8mq

[PATCH v5 5/5] arm64: dts: imx8mq: add DCSS node

2020-07-09 Thread Laurentiu Palcu
From: Laurentiu Palcu This patch adds the node for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 23 +++ 1 file changed, 23 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch

[PATCH v5 0/4] Add support for iMX8MQ Display Controller Subsystem

2020-07-09 Thread Laurentiu Palcu
From: Laurentiu Palcu Hi, This patchset adds initial DCSS support for iMX8MQ chip. Initial support includes only graphics plane support (no video planes), no HDR10 capabilities, no graphics decompression (only linear, tiled and super-tiled buffers allowed). Support for the rest of the features

[PATCH v5 3/5] drm/imx/dcss: add component framework functionality

2020-07-09 Thread Laurentiu Palcu
From: Laurentiu Palcu Component framework is needed by HDP driver. Signed-off-by: Laurentiu Palcu --- drivers/gpu/drm/imx/dcss/dcss-drv.c | 89 ++--- drivers/gpu/drm/imx/dcss/dcss-kms.c | 14 - drivers/gpu/drm/imx/dcss/dcss-kms.h | 4 +- 3 files changed, 80

[PATCH v5 1/5] drm/imx: compile imx directory by default

2020-07-09 Thread Laurentiu Palcu
From: Laurentiu Palcu Currently the drm/imx/ directory is compiled only if DRM_IMX is set. Adding a new IMX related IP in the same directory would need DRM_IMX to be set, which would bring in also IPUv3 core driver... The current patch would allow adding new IPs in the imx/ directory without

Re: [PATCH v4 2/4] drm/imx: Add initial support for DCSS on iMX8MQ

2020-05-15 Thread Laurentiu Palcu
Hi Guido, On Fri, May 15, 2020 at 11:27:19AM +0200, Guido Günther wrote: > Hi Laurentiu, > On Fri, Mar 06, 2020 at 02:49:26PM +0200, Laurentiu Palcu wrote: > > From: Laurentiu Palcu > > > > This adds initial support for iMX8MQ's Display Controller Subsystem (DCSS). >

[PATCH v2 1/5] clk: imx8mq: Add VIDEO2_PLL clock

2019-10-02 Thread Laurentiu Palcu
This clock is needed by DCSS when high resolutions are used. Signed-off-by: Laurentiu Palcu CC: Abel Vesa --- drivers/clk/imx/clk-imx8mq.c | 4 include/dt-bindings/clock/imx8mq-clock.h | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/clk/imx/clk

[PATCH v2 5/5] arm64: dts: imx8mq: add DCSS node

2019-10-02 Thread Laurentiu Palcu
This patch adds the node for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 25 + 1 file changed, 25 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts

Re: Re: [PATCH 5/5] arm64: dts: imx8mq: add DCSS node

2019-09-24 Thread Laurentiu Palcu
Hi Fabio, On Mon, Sep 23, 2019 at 01:12:42PM -0300, Fabio Estevam wrote: > Hi Laurentiu, > > On Mon, Sep 23, 2019 at 11:14 AM Laurentiu Palcu > wrote: > > > + > > + dcss: dcss@0x32e0 { > > Node names should be generic, so: > >

Re: Re: [PATCH 5/5] arm64: dts: imx8mq: add DCSS node

2019-09-24 Thread Laurentiu Palcu
On Mon, Sep 23, 2019 at 09:54:42AM -0700, Stephen Boyd wrote: > Quoting Laurentiu Palcu (2019-09-23 07:13:19) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi > > b/arch/arm64/boot/dts/freescale/imx8mq.dtsi > > index 52aae34..d4aa778 100644 > > --- a/ar

[PATCH 5/5] arm64: dts: imx8mq: add DCSS node

2019-09-23 Thread Laurentiu Palcu
This patch adds the node for iMX8MQ Display Controller Subsystem. Signed-off-by: Laurentiu Palcu --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 25 + 1 file changed, 25 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts

[PATCH 3/5] drm/imx: Add initial support for DCSS on iMX8MQ

2019-09-23 Thread Laurentiu Palcu
-plane de-gamma/CSC/gamma support is in. Signed-off-by: Laurentiu Palcu --- drivers/gpu/drm/imx/Kconfig| 2 + drivers/gpu/drm/imx/Makefile | 1 + drivers/gpu/drm/imx/dcss/Kconfig | 7 + drivers/gpu/drm/imx/dcss/Makefile | 6 + drivers/gpu/drm/imx/dcss/dcss

[PATCH 1/5] clk: imx8mq: Add VIDEO2_PLL clock

2019-09-23 Thread Laurentiu Palcu
This clock is needed by DCSS when high resolutions are used. Signed-off-by: Laurentiu Palcu CC: Abel Vesa --- drivers/clk/imx/clk-imx8mq.c | 4 include/dt-bindings/clock/imx8mq-clock.h | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/clk/imx/clk

[PATCH] drm/imx: fix typo in ipu_plane_formats[]

2017-06-28 Thread Laurentiu Palcu
The BGRA appears twice in the ipu_plane_formats[] list. The duplicate should be BGRX. The original commit is: commit 59d6b7189a96 ("drm/imx: ipuv3-plane: enable support for RGBX and RGBA pixel formats") Signed-off-by: Laurentiu Palcu <laurentiu.pa...@nxp.com>

[PATCH] drm/imx: fix typo in ipu_plane_formats[]

2017-06-28 Thread Laurentiu Palcu
The BGRA appears twice in the ipu_plane_formats[] list. The duplicate should be BGRX. The original commit is: commit 59d6b7189a96 ("drm/imx: ipuv3-plane: enable support for RGBX and RGBA pixel formats") Signed-off-by: Laurentiu Palcu --- drivers/gpu/drm/imx/ipuv3-p

Re: [PATCH] spi: dln2: Pass of_node to spi master

2016-04-15 Thread Laurentiu Palcu
ff-by: Crestez Dan Leonard <leonard.cres...@intel.com> Signed-off-by: Laurentiu Palcu <laurentiu.pa...@intel.com> > --- > drivers/spi/spi-dln2.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/spi/spi-dln2.c b/drivers/spi/spi-dln2.c > index 3b7d91

Re: [PATCH] spi: dln2: Pass of_node to spi master

2016-04-15 Thread Laurentiu Palcu
ff-by: Crestez Dan Leonard Signed-off-by: Laurentiu Palcu > --- > drivers/spi/spi-dln2.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/spi/spi-dln2.c b/drivers/spi/spi-dln2.c > index 3b7d91d..4e8a862 100644 > --- a/drivers/spi/spi-dln2.c > +++ b

Re: [PATCH v4 2/7] power: bq27x00_battery: Renaming for consistency

2015-09-15 Thread Laurentiu Palcu
On Mon, Sep 14, 2015 at 04:26:08PM -0500, Andrew F. Davis wrote: > Rename functions that are used by multiple devices. New devices > have been added and the function names and driver name are no longer > general enough for the functionality they provide. > > Signed-off-by: Andrew F. Davis > ---

Re: [PATCH v4 2/7] power: bq27x00_battery: Renaming for consistency

2015-09-15 Thread Laurentiu Palcu
On Mon, Sep 14, 2015 at 04:26:08PM -0500, Andrew F. Davis wrote: > Rename functions that are used by multiple devices. New devices > have been added and the function names and driver name are no longer > general enough for the functionality they provide. > > Signed-off-by: Andrew F. Davis

Re: [PATCH] power: bq24261_charger: Add support for TI BQ24261 charger

2015-09-10 Thread Laurentiu Palcu
On Thu, Sep 10, 2015 at 08:49:26AM +0900, Krzysztof Kozlowski wrote: > On 10.09.2015 02:31, Andreas Dannenberg wrote: > > On Wed, Sep 09, 2015 at 01:17:11PM +0900, Krzysztof Kozlowski wrote: > >> On 09.09.2015 11:26, Andreas Dannenberg wrote: > >>> Krzysztof, good observation! In bq2425x_charger.c

Re: [PATCH] power: bq24261_charger: Add support for TI BQ24261 charger

2015-09-10 Thread Laurentiu Palcu
On Thu, Sep 10, 2015 at 08:49:26AM +0900, Krzysztof Kozlowski wrote: > On 10.09.2015 02:31, Andreas Dannenberg wrote: > > On Wed, Sep 09, 2015 at 01:17:11PM +0900, Krzysztof Kozlowski wrote: > >> On 09.09.2015 11:26, Andreas Dannenberg wrote: > >>> Krzysztof, good observation! In bq2425x_charger.c

Re: [PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-06-24 Thread Laurentiu Palcu
On Wed, Jun 24, 2015 at 10:30:58AM +0200, Pavel Machek wrote: > On Wed 2015-06-24 10:32:54, Laurentiu Palcu wrote: > > On Sat, May 02, 2015 at 04:59:34PM +0200, Pavel Machek wrote: > > > > > > Should this link > > > > > > > More detail

Re: [PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-06-24 Thread Laurentiu Palcu
On Sat, May 02, 2015 at 04:59:34PM +0200, Pavel Machek wrote: > > Should this link > > > More details about the chip can be found here: > > http://www.ti.com/product/bq25890 > > > @@ -0,0 +1,998 @@ > > +/* > > + * TI BQ25890 charger driver > > + * > > + * Copyright (C) 2015 Intel

Re: [PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-06-24 Thread Laurentiu Palcu
On Sat, May 02, 2015 at 04:59:34PM +0200, Pavel Machek wrote: Should this link More details about the chip can be found here: http://www.ti.com/product/bq25890 @@ -0,0 +1,998 @@ +/* + * TI BQ25890 charger driver + * + * Copyright (C) 2015 Intel Corporation + * + * This

Re: [PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-06-24 Thread Laurentiu Palcu
On Wed, Jun 24, 2015 at 10:30:58AM +0200, Pavel Machek wrote: On Wed 2015-06-24 10:32:54, Laurentiu Palcu wrote: On Sat, May 02, 2015 at 04:59:34PM +0200, Pavel Machek wrote: Should this link More details about the chip can be found here: http://www.ti.com/product/bq25890

[PATCH] power_supply: bq25890: make chip_id int

2015-06-02 Thread Laurentiu Palcu
Signed-off-by: Laurentiu Palcu Reported-by: Dan Carpenter --- drivers/power/bq25890_charger.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/power/bq25890_charger.c b/drivers/power/bq25890_charger.c index c7b4903..16b7c7bc 100644 --- a/drivers/power/bq25890_charger.c +

[PATCH] power_supply: bq25890: make chip_id int

2015-06-02 Thread Laurentiu Palcu
-by: Laurentiu Palcu laurentiu.pa...@intel.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/power/bq25890_charger.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/power/bq25890_charger.c b/drivers/power/bq25890_charger.c index c7b4903..16b7c7bc 100644

[PATCH v2 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-05-19 Thread Laurentiu Palcu
More details about the chip can be found here: http://www.ti.com/product/bq25890 Signed-off-by: Laurentiu Palcu --- drivers/power/Kconfig | 7 + drivers/power/Makefile | 1 + drivers/power/bq25890_charger.c | 999 3 files changed

[PATCH v2 1/2] Documentation: devicetree: Add TI BQ25890 bindings

2015-05-19 Thread Laurentiu Palcu
Signed-off-by: Laurentiu Palcu --- .../devicetree/bindings/power/bq25890.txt | 46 ++ 1 file changed, 46 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/bq25890.txt diff --git a/Documentation/devicetree/bindings/power/bq25890.txt b

[PATCH v2 0/2] Add support for BQ25890 charger chip

2015-05-19 Thread Laurentiu Palcu
ds support for BQ25890 USB charger chip. Information about the chip can be found here: http://www.ti.com/product/bq25890 Thanks, laurentiu Laurentiu Palcu (2): Documentation: devicetree: Add TI BQ25890 bindings power_supply: Add support for TI BQ25890 charger chip .../devicetree/bindings/pow

Re: [PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-05-19 Thread Laurentiu Palcu
Hi Krzysztof, On Tue, May 19, 2015 at 05:40:25PM +0900, Krzysztof Kozlowski wrote: > > + > > +static int bq25890_chip_reset(struct bq25890_device *bq) > > +{ > > + int ret; > > + > > + ret = bq25890_field_write(bq, F_REG_RST, 1); > > + if (ret < 0) > > + return

Re: [PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-05-19 Thread Laurentiu Palcu
Hi Krzysztof, On Tue, May 19, 2015 at 05:40:25PM +0900, Krzysztof Kozlowski wrote: + +static int bq25890_chip_reset(struct bq25890_device *bq) +{ + int ret; + + ret = bq25890_field_write(bq, F_REG_RST, 1); + if (ret 0) + return ret; + +

[PATCH v2 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-05-19 Thread Laurentiu Palcu
More details about the chip can be found here: http://www.ti.com/product/bq25890 Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com --- drivers/power/Kconfig | 7 + drivers/power/Makefile | 1 + drivers/power/bq25890_charger.c | 999

[PATCH v2 0/2] Add support for BQ25890 charger chip

2015-05-19 Thread Laurentiu Palcu
for BQ25890 USB charger chip. Information about the chip can be found here: http://www.ti.com/product/bq25890 Thanks, laurentiu Laurentiu Palcu (2): Documentation: devicetree: Add TI BQ25890 bindings power_supply: Add support for TI BQ25890 charger chip .../devicetree/bindings/power/bq25890.txt

[PATCH v2 1/2] Documentation: devicetree: Add TI BQ25890 bindings

2015-05-19 Thread Laurentiu Palcu
Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com --- .../devicetree/bindings/power/bq25890.txt | 46 ++ 1 file changed, 46 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/bq25890.txt diff --git a/Documentation/devicetree/bindings

Re: [PATCH 1/2] Documentation: devicetree: Add TI BQ25890 bindings

2015-05-18 Thread Laurentiu Palcu
Hi Krzysztof, On Sat, May 16, 2015 at 08:53:42PM +0900, Krzysztof Kozlowski wrote: > 2015-05-15 23:06 GMT+09:00 Laurentiu Palcu : > > Signed-off-by: Laurentiu Palcu > > --- > > .../devicetree/bindings/power/bq25890.txt | 45 > > ++ > &g

Re: [PATCH 1/2] Documentation: devicetree: Add TI BQ25890 bindings

2015-05-18 Thread Laurentiu Palcu
Hi Krzysztof, On Sat, May 16, 2015 at 08:53:42PM +0900, Krzysztof Kozlowski wrote: 2015-05-15 23:06 GMT+09:00 Laurentiu Palcu laurentiu.pa...@intel.com: Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com --- .../devicetree/bindings/power/bq25890.txt | 45

[PATCH 0/2] Add support for BQ25890 charger chip

2015-05-15 Thread Laurentiu Palcu
Hi, This series adds support for BQ25890 USB charger chip. Information about the chip can be found here: http://www.ti.com/product/bq25890 Thanks, laurentiu Laurentiu Palcu (2): Documentation: devicetree: Add TI BQ25890 bindings power_supply: Add support for TI BQ25890 charger chip

[PATCH 1/2] Documentation: devicetree: Add TI BQ25890 bindings

2015-05-15 Thread Laurentiu Palcu
Signed-off-by: Laurentiu Palcu --- .../devicetree/bindings/power/bq25890.txt | 45 ++ 1 file changed, 45 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/bq25890.txt diff --git a/Documentation/devicetree/bindings/power/bq25890.txt b

[PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-05-15 Thread Laurentiu Palcu
More details about the chip can be found here: http://www.ti.com/product/bq25890 Signed-off-by: Laurentiu Palcu --- drivers/power/Kconfig | 7 + drivers/power/Makefile | 1 + drivers/power/bq25890_charger.c | 998 3 files changed

[PATCH 1/2] Documentation: devicetree: Add TI BQ25890 bindings

2015-05-15 Thread Laurentiu Palcu
Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com --- .../devicetree/bindings/power/bq25890.txt | 45 ++ 1 file changed, 45 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/bq25890.txt diff --git a/Documentation/devicetree/bindings

[PATCH 2/2] power_supply: Add support for TI BQ25890 charger chip

2015-05-15 Thread Laurentiu Palcu
More details about the chip can be found here: http://www.ti.com/product/bq25890 Signed-off-by: Laurentiu Palcu laurentiu.pa...@intel.com --- drivers/power/Kconfig | 7 + drivers/power/Makefile | 1 + drivers/power/bq25890_charger.c | 998

[PATCH 0/2] Add support for BQ25890 charger chip

2015-05-15 Thread Laurentiu Palcu
Hi, This series adds support for BQ25890 USB charger chip. Information about the chip can be found here: http://www.ti.com/product/bq25890 Thanks, laurentiu Laurentiu Palcu (2): Documentation: devicetree: Add TI BQ25890 bindings power_supply: Add support for TI BQ25890 charger chip

Re: [PATCH v3] power_supply: Add support for Richtek rt9455 battery charger

2015-05-07 Thread Laurentiu Palcu
Hi Anda, On Wed, May 06, 2015 at 06:41:00PM +0300, Anda-Maria Nicolae wrote: (...) > >>Updates from v2 version: > >>- removed unused masks and keep the used ones. I have tried to access mask > >>field > >>from struct regmap_field, but I have received the following compilation > >>error: >

  1   2   3   >