Re: [PATCH v7 2/3] drm: Add support for the LogiCVC display controller

2020-12-02 Thread Paul Kocialkowski
Hi Sam, On Wed 04 Nov 20, 22:22, Sam Ravnborg wrote: > Hi Paul. > > A few comments in the following. I did not find time to read all of the > driver. Thanks for taking a look at the driver! > > Sam > > On Mon, Nov 02, 2020 at 04:53:07PM +0100, Paul Kocialkowski

Re: [PATCH v7 2/3] drm: Add support for the LogiCVC display controller

2020-12-02 Thread Paul Kocialkowski
Hi, On Tue 03 Nov 20, 10:46, Maxime Ripard wrote: > On Mon, Nov 02, 2020 at 04:53:07PM +0100, Paul Kocialkowski wrote: > > Introduces a driver for the LogiCVC display controller, a programmable > > logic controller optimized for use in Xilinx Zynq-7000 SoCs and other > &

Re: [PATCH v7 2/3] drm: Add support for the LogiCVC display controller

2020-12-11 Thread Paul Kocialkowski
Hi, On Mon 07 Dec 20, 11:42, Maxime Ripard wrote: > On Wed, Dec 02, 2020 at 05:06:40PM +0100, Paul Kocialkowski wrote: > > > > +static void logicvc_crtc_atomic_begin(struct drm_crtc *drm_crtc, > > > > + str

[PATCH] drm/rockchip: Avoid uninitialized use of endpoint id in LVDS

2020-11-10 Thread Paul Kocialkowski
iable in that case. Fixes: 34cc0aa25456 ("drm/rockchip: Add support for Rockchip Soc LVDS") Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/rockchip/rockchip_lvds.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_lvds.c b/

Re: [PATCH v8 1/4] dt-bindings: display: Document the Xylon LogiCVC display controller

2021-09-14 Thread Paul Kocialkowski
Hi Rob, I just found out as I'm about to send a new revision that I had not yet responded to your concerns here. On Tue 12 Jan 21, 09:17, Rob Herring wrote: > On Wed, Dec 23, 2020 at 10:29:44PM +0100, Paul Kocialkowski wrote: > > The Xylon LogiCVC is a display controller

[PATCH v9 0/4] drm: LogiCVC display controller support

2021-09-14 Thread Paul Kocialkowski
ribed more possible dt parameters; - Added support for the lvds-3bit interface; - Added support for grabbing syscon regmap from parent node; - Removed layers count property and count layers child nodes instead. Paul Kocialkowski (4): dt-bindings: display: Document the Xylon LogiCVC display contr

[PATCH v9 1/4] dt-bindings: display: Document the Xylon LogiCVC display controller

2021-09-14 Thread Paul Kocialkowski
The Xylon LogiCVC is a display controller implemented as programmable logic in Xilinx FPGAs. Signed-off-by: Paul Kocialkowski Acked-by: Rob Herring --- .../display/xylon,logicvc-display.yaml| 302 ++ 1 file changed, 302 insertions(+) create mode 100644 Documentation

[PATCH v9 2/4] dt-bindings: mfd: logicvc: Add patternProperties for the display

2021-09-14 Thread Paul Kocialkowski
The LogiCVC multi-function device has a display part which is now described in its binding. Add a patternProperties match for it. Signed-off-by: Paul Kocialkowski --- Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation

[PATCH v9 3/4] drm: Add support for the LogiCVC display controller

2021-09-14 Thread Paul Kocialkowski
implementation of logicvc_layer_buffer_find_setup for specifics. Version 4 allows configuring each buffer address directly, which guarantees that any buffer can be configured. Signed-off-by: Paul Kocialkowski Reviewed-by: Maxime Ripard --- MAINTAINERS | 6 + drivers

[PATCH v9 4/4] NOTFORMERGE: drm/logicvc: Add plane colorkey support

2021-09-14 Thread Paul Kocialkowski
Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/logicvc/logicvc_drm.h | 3 + drivers/gpu/drm/logicvc/logicvc_layer.c | 151 +++- drivers/gpu/drm/logicvc/logicvc_layer.h | 7 ++ 3 files changed, 155 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/logicvc

Re: [PATCH v9 0/4] drm: LogiCVC display controller support

2021-10-01 Thread Paul Kocialkowski
Hi, On Tue 14 Sep 21, 22:05, Paul Kocialkowski wrote: > This series introduces support for the LogiCVC display controller. > The controller is a bit unusual since it is usually loaded as > programmable logic on Xilinx FPGAs or Zynq-7000 SoCs. > More details are presented on the mai

[PATCH v10 3/6] gpio: logicvc: Support compatible with major version only

2022-01-20 Thread Paul Kocialkowski
Support the newly-introduced common compatible for version 3. Signed-off-by: Paul Kocialkowski --- drivers/gpio/gpio-logicvc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpio/gpio-logicvc.c b/drivers/gpio/gpio-logicvc.c index 992cc958a43f..9b27343f5242

[PATCH v10 2/6] dt-bindings: gpio: logicvc: Add a compatible with major version only

2022-01-20 Thread Paul Kocialkowski
There are lots of different versions of the logicvc block and it makes little sense to list them all in compatibles since all versions with the same major are found to be register-compatible. Introduce a new compatible with the major version only. Signed-off-by: Paul Kocialkowski

[PATCH v10 4/6] dt-bindings: display: Add compatibles with major versions only

2022-01-20 Thread Paul Kocialkowski
There are lots of different versions of the logicvc block and it makes little sense to list them all in compatibles since all versions with the same major are found to be register-compatible. Add common compatibles that only list the major version instead. Signed-off-by: Paul Kocialkowski

[PATCH v10 0/6] drm: LogiCVC display controller support

2022-01-20 Thread Paul Kocialkowski
t property and count layers child nodes instead. Paul Kocialkowski (6): dt-bindings: mfd: logicvc: Add a compatible with the major version only dt-bindings: gpio: logicvc: Add a compatible with major version only gpio: logicvc: Support compatible with major version only dt-bindings: di

[PATCH v10 1/6] dt-bindings: mfd: logicvc: Add a compatible with the major version only

2022-01-20 Thread Paul Kocialkowski
There are lots of different versions of the logicvc block and it makes little sense to list them all in compatibles since all versions with the same major are found to be register-compatible. Introduce a new compatible with the major version only. Signed-off-by: Paul Kocialkowski

[PATCH v10 6/6] NOTFORMERGE: drm/logicvc: Add plane colorkey support

2022-01-20 Thread Paul Kocialkowski
Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/logicvc/logicvc_drm.h | 3 + drivers/gpu/drm/logicvc/logicvc_layer.c | 151 +++- drivers/gpu/drm/logicvc/logicvc_layer.h | 7 ++ 3 files changed, 155 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/logicvc

[PATCH v10 5/6] drm: Add support for the LogiCVC display controller

2022-01-20 Thread Paul Kocialkowski
implementation of logicvc_layer_buffer_find_setup for specifics. Version 4 allows configuring each buffer address directly, which guarantees that any buffer can be configured. Signed-off-by: Paul Kocialkowski Reviewed-by: Maxime Ripard --- MAINTAINERS | 6 + drivers

[PATCH v8 0/4] drm: LogiCVC display controller support

2020-12-23 Thread Paul Kocialkowski
t bindings documentation to dt schema; - Described more possible dt parameters; - Added support for the lvds-3bit interface; - Added support for grabbing syscon regmap from parent node; - Removed layers count property and count layers child nodes instead. Paul Kocialkowski (4): dt-bindings: display

[PATCH v8 1/4] dt-bindings: display: Document the Xylon LogiCVC display controller

2020-12-23 Thread Paul Kocialkowski
The Xylon LogiCVC is a display controller implemented as programmable logic in Xilinx FPGAs. Signed-off-by: Paul Kocialkowski Acked-by: Rob Herring --- .../display/xylon,logicvc-display.yaml| 313 ++ 1 file changed, 313 insertions(+) create mode 100644 Documentation

[PATCH v8 2/4] dt-bindings: mfd: logicvc: Add patternProperties for the display

2020-12-23 Thread Paul Kocialkowski
The LogiCVC multi-function device has a display part which is now described in its binding. Add a patternProperties match for it. Signed-off-by: Paul Kocialkowski --- Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation

[PATCH v8 3/4] drm: Add support for the LogiCVC display controller

2020-12-23 Thread Paul Kocialkowski
implementation of logicvc_layer_buffer_find_setup for specifics. Version 4 allows configuring each buffer address directly, which guarantees that any buffer can be configured. Signed-off-by: Paul Kocialkowski Reviewed-by: Maxime Ripard --- MAINTAINERS | 6 + drivers

[PATCH v8 4/4] NOTFORMERGE: drm/logicvc: Add plane colorkey support

2020-12-23 Thread Paul Kocialkowski
Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/logicvc/logicvc_drm.h | 3 + drivers/gpu/drm/logicvc/logicvc_layer.c | 150 +++- drivers/gpu/drm/logicvc/logicvc_layer.h | 7 ++ 3 files changed, 154 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/logicvc

Re: [PATCH v8 1/4] dt-bindings: display: Document the Xylon LogiCVC display controller

2020-12-25 Thread Paul Kocialkowski
Hi, On Thu 24 Dec 20, 10:01, Rob Herring wrote: > On Wed, 23 Dec 2020 22:29:44 +0100, Paul Kocialkowski wrote: > > The Xylon LogiCVC is a display controller implemented as programmable > > logic in Xilinx FPGAs. > > > > Signed-off-by: Paul Kocialkowski

Re: [PATCH v8 4/4] NOTFORMERGE: drm/logicvc: Add plane colorkey support

2021-01-05 Thread Paul Kocialkowski
mething for colorkey: > > https://drmdb.emersion.fr/properties/4008636142/colorkey > > > > I know this is marked "not for merge", but it would be nice to discuss > > with them and come up with a standardized property. -- Paul Kocialkowski, B

Re: [PATCH v7 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-05-02 Thread Paul Kocialkowski
Hi, On Thu, 2019-04-25 at 10:42 -0700, Eric Anholt wrote: > Paul Kocialkowski writes: > > > The binner BO is not required until the V3D is in use, so avoid > > allocating it at probe and do it on the first non-dumb BO allocation. > > > > Keep track of which

[PATCH v8 3/4] drm/vc4: Check for the binner bo before handling OOM interrupt

2019-05-03 Thread Paul Kocialkowski
Since the OOM interrupt directly deals with the binner bo, it doesn't make sense to try and handle it without a binner buffer registered. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_irq.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_ir

[PATCH v8 2/4] drm/vc4: Check for V3D before binner bo alloc

2019-05-03 Thread Paul Kocialkowski
Check that we have a V3D device registered before attempting to allocate a binner buffer object. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_v3d.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_v3d.c b/drivers/gpu/drm/vc4/vc4_v3d.c index

[PATCH v8 0/4] drm/vc4: Binner BO management improvements

2019-05-03 Thread Paul Kocialkowski
patches into one. Paul Kocialkowski (4): drm/vc4: Reformat and the binner bo allocation helper drm/vc4: Check for V3D before binner bo alloc drm/vc4: Check for the binner bo before handling OOM interrupt drm/vc4: Allocate binner bo when starting to use the V3D drivers/gpu/drm/vc4/vc4_bo.c

[PATCH v8 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-05-03 Thread Paul Kocialkowski
runtime suspend/resume to avoid failing to allocate it at resume. This happens when the CMA pool is full at that point and results in a hard crash. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_bo.c | 31 - drivers/gpu/drm/vc4/vc4_drv.c | 6 drivers/gpu/drm

[PATCH v8 1/4] drm/vc4: Reformat and the binner bo allocation helper

2019-05-03 Thread Paul Kocialkowski
In preparation for wrapping the binner bo allocation helper with put/get helpers, pass the vc4 dev directly and drop the vc4 prefix. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_v3d.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm

Re: Support for 2D engines/blitters in V4L2 and DRM

2019-05-09 Thread Paul Kocialkowski
Hi Pekka, Le lundi 06 mai 2019 à 11:28 +0300, Pekka Paalanen a écrit : > On Wed, 17 Apr 2019 20:10:15 +0200 > Paul Kocialkowski wrote: > > > There's also the possibility of writing up a drm-render DDX to handle > > these 2D blitters that can make things a lot faster

Improving DRM connector hotplug uevents

2019-05-10 Thread Paul Kocialkowski
g like: HOTPLUG=1 CONNECTOR_ID=47 STATUS=Connected CONNECTOR_ID=48 STATUS=Disconnected Where each STATUS entry would refer to the previous CONNECTOR_ID entry. What do you think? Cheers, Paul > > On Thu, 2019-05-09 at 14:24 +0200, Paul Kocialkowski wrote: > > > Hi, > > > > &g

Re: Improving DRM connector hotplug uevents

2019-05-10 Thread Paul Kocialkowski
Hi, On Fri, 2019-05-10 at 09:56 +0200, Daniel Vetter wrote: > On Fri, May 10, 2019 at 9:42 AM Paul Kocialkowski > wrote: > > Hi, > > > > So this thread has drifted off from IGT to a general DRM improvement, > > so renaming the thread and adding some CCs. >

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-10 Thread Paul Kocialkowski
,13 @@ > > struct drm_device; > struct device; > +struct drm_connector; > +struct drm_property; > > int drm_class_device_register(struct device *dev); > void drm_class_device_unregister(struct device *dev); > > void drm_sysfs_hotplug_event(struct drm_device *dev); > - > +void drm_sysfs_connector_status_event(struct drm_connector *connector, > + struct drm_property *property); > #endif -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: Improving DRM connector hotplug uevents

2019-05-10 Thread Paul Kocialkowski
Hi, Resending this with the right CC. Also changed my mind in the meantime and my latest proposal is at: https://lists.freedesktop.org/archives/dri-devel/2019-May/217442.html On Fri, 2019-05-10 at 10:06 +0200, Daniel Vetter wrote: > On Fri, May 10, 2019 at 10:01 AM Paul Kocialkowski >

Re: [PATCH v8 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-05-10 Thread Paul Kocialkowski
Hi, On Thu, 2019-05-09 at 11:39 -0700, Eric Anholt wrote: > Paul Kocialkowski writes: > > > The binner BO is not required until the V3D is in use, so avoid > > allocating it at probe and do it on the first non-dumb BO allocation. > > > > Keep track of which

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Paul Kocialkowski
Hi, On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > On Fri, May 10, 2019 at 2:12 PM Paul Kocialkowski > wrote: > > Hi, > > > > On Tue, 2019-05-07 at 21:57 +0530, Ramalingam C wrote: > > > DRM API for generating uevent for a status chan

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-13 Thread Paul Kocialkowski
Hey, Le lundi 13 mai 2019 à 11:34 +0200, Daniel Vetter a écrit : > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > wrote: > > Hi, > > > > On Fri, 2019-05-10 at 16:54 +0200, Daniel Vetter wrote: > > > On Fri, May 10, 2019 at 2:12 PM Paul Koci

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Paul Kocialkowski
; On Mon, 13 May 2019 11:34:58 +0200 > > > > Daniel Vetter wrote: > > > > > > > > > On Mon, May 13, 2019 at 11:02 AM Paul Kocialkowski > > > > > wrote: > > > > > > Hi, > > > > > > > > > > > >

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-14 Thread Paul Kocialkowski
Hi, On Tue, 2019-05-14 at 13:09 +0200, Daniel Vetter wrote: > On Mon, May 13, 2019 at 7:14 PM Paul Kocialkowski > wrote: > > Hey, > > > > Le lundi 13 mai 2019 à 11:34 +0200, Daniel Vetter a écrit : > > > On Mon, May 13, 2019 at 11:02 AM Paul Koci

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Paul Kocialkowski
Hi, On Tue, 2019-05-14 at 16:28 +0200, Daniel Vetter wrote: > On Tue, May 14, 2019 at 4:13 PM Paul Kocialkowski > wrote: > > Hi, > > > > On Tue, 2019-05-14 at 13:09 +0200, Daniel Vetter wrote: > > > On Mon, May 13, 2019 at 7:14 PM Paul Kocialkowski > > >

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-15 Thread Paul Kocialkowski
; > gets back to dispatching. The terminology in libudev API confused me as > > > an event is a device. Squashing together would make sense if the > > > uevent were just updating a device attribute list. Previously when we > > > had just a single kind of uevent, that

[PATCH v9 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-05-16 Thread Paul Kocialkowski
runtime suspend/resume to avoid failing to allocate it at resume. This happens when the CMA pool is full at that point and results in a hard crash. Signed-off-by: Paul Kocialkowski Reviewed-by: Eric Anholt --- drivers/gpu/drm/vc4/vc4_bo.c | 31 +- drivers/gpu/drm/vc4/vc4_drv.c

[PATCH v9 3/4] drm/vc4: Check for the binner bo before handling OOM interrupt

2019-05-16 Thread Paul Kocialkowski
Since the OOM interrupt directly deals with the binner bo, it doesn't make sense to try and handle it without a binner buffer registered. Signed-off-by: Paul Kocialkowski Reviewed-by: Eric Anholt --- drivers/gpu/drm/vc4/vc4_irq.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dr

[PATCH v9 1/4] drm/vc4: Reformat and the binner bo allocation helper

2019-05-16 Thread Paul Kocialkowski
In preparation for wrapping the binner bo allocation helper with put/get helpers, pass the vc4 dev directly and drop the vc4 prefix. Signed-off-by: Paul Kocialkowski Reviewed-by: Eric Anholt --- drivers/gpu/drm/vc4/vc4_v3d.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions

[PATCH v9 2/4] drm/vc4: Check for V3D before binner bo alloc

2019-05-16 Thread Paul Kocialkowski
Check that we have a V3D device registered before attempting to allocate a binner buffer object. Signed-off-by: Paul Kocialkowski Reviewed-by: Eric Anholt --- drivers/gpu/drm/vc4/vc4_v3d.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_v3d.c b/drivers/gpu/drm

[PATCH v9 0/4] drm/vc4: Binner BO management improvements

2019-05-16 Thread Paul Kocialkowski
sentence about fristopen; * Added collected Reviewed-By tags. Changes since v1: * Squashed the two final patches into one. Paul Kocialkowski (4): drm/vc4: Reformat and the binner bo allocation helper drm/vc4: Check for V3D before binner bo alloc drm/vc4: Check for the binner bo before handling

[PATCH] backlight: gpio-backlight: Set power state instead of brightness at probe

2019-05-17 Thread Paul Kocialkowski
by setting the brightness to 1 at probe and managing the power state accordingly, a bit like it's done in pwm-backlight. Fixes: 8b770e3c9824 ("backlight: Add GPIO-based backlight driver") Signed-off-by: Paul Kocialkowski --- drivers/video/backlight/gpio_backlight.c | 19 +

Re: [PATCH v3 1/7] drm/rockchip: Change the scl_vop_cal_scl_fac to pass drm_format_info

2019-05-20 Thread Paul Kocialkowski
_info structure > it has access to to retrieve that fourcc. Change the prototype of that > function to pass the drm_format_info structure directly, removing the need > for an extra lookup. > > Suggested-by: Philipp Zabel > Signed-off-by: Maxime Ripard Reviewed-by:

Re: [PATCH v3 4/7] drm/fourcc: Pass the format_info pointer to drm_format_plane_cpp

2019-05-20 Thread Paul Kocialkowski
ut we really should have an unsigned plane index wherever it's used. We're only checking for plane >= info->num_planes but seldom for plane < 0. Either way, this is: Reviewed-by: Paul Kocialkowski Cheers, Paul > Reviewed-by: Emil Velikov > Signed-off-by: Maxime Ripar

Re: [PATCH v3 7/7] drm: Remove users of drm_format_num_planes

2019-05-20 Thread Paul Kocialkowski
zx_writel(paddr_reg, paddr); > paddr_reg += 4; > } > diff --git a/include/drm/drm_fourcc.h b/include/drm/drm_fourcc.h > index 4ef8ccb5d236..405466692bd2 100644 > --- a/include/drm/drm_fourcc.h > +++ b/include/drm/drm_fourcc.h > @@ -261,23 +261,6 @@ drm_format_info_is_yuv_sampling_444(const struct > drm_format_info *info) > } > > /** > - * drm_format_info_plane_cpp - determine the bytes per pixel value > - * @format: pixel format info > - * @plane: plane index > - * > - * Returns: > - * The bytes per pixel value for the specified plane. > - */ > -static inline > -int drm_format_info_plane_cpp(const struct drm_format_info *info, int plane) > -{ > - if (!info || plane >= info->num_planes) > - return 0; > - > - return info->cpp[plane]; > -} > - > -/** > * drm_format_info_plane_width - width of the plane given the first plane > * @format: pixel format info > * @width: width of the first plane > -- > git-series 0.9.1 > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v3 5/7] drm/fourcc: Pass the format_info pointer to drm_format_plane_width/height

2019-05-20 Thread Paul Kocialkowski
also be changed to match the > other functions prototype. Same comment about plane being int instead of unsigned int, but I think we can fix that later. Another thing that I find odd is that the division by vsub/hsub is not rounded up, but again, it's something independent from your patch. Revi

Re: [PATCH v7 09/11] drm: uevent for connector status change

2019-05-20 Thread Paul Kocialkowski
Le lundi 20 mai 2019 à 18:11 +0200, Daniel Vetter a écrit : > On Fri, May 17, 2019 at 01:08:24PM +0300, Pekka Paalanen wrote: > > On Thu, 16 May 2019 14:24:55 +0200 > > Daniel Vetter wrote: > > > > > On Thu, May 16, 2019 at 11:22:11AM +0300, Pekka Paalanen wrote: > > > > On Wed, 15 May 2019 10:24

Re: [PATCH] drm/todo: Add bootsplash entry

2019-05-23 Thread Paul Kocialkowski
Hi, On Thu, 2019-05-23 at 16:26 +0200, Noralf Trønnes wrote: > Ease entry for anyone wanting to pick up the bootsplash work by providing > a couple of pointers. > > Signed-off-by: Noralf Trønnes I think a native DRM bootsplash would be a great thing to have! Reviewed-by: Paul

[PATCH v5 0/4] drm/vc4: Binner BO management improvements

2019-04-15 Thread Paul Kocialkowski
condition as discussed. Changes since v2: * Removed deprecated sentence about fristopen; * Added collected Reviewed-By tags. Changes since v1: * Squashed the two final patches into one. Paul Kocialkowski (4): drm/vc4: Reformat and export binner bo allocation helper drm/vc4: Check for V3D

[PATCH v5 3/4] drm/vc4: Check for the binner bo before handling OOM interrupt

2019-04-15 Thread Paul Kocialkowski
Since the OOM interrupt directly deals with the binner bo, it doesn't make sense to try and handle it without a binner buffer registered. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_irq.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_ir

[PATCH v5 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-04-15 Thread Paul Kocialkowski
hard crash. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_bo.c | 30 ++ drivers/gpu/drm/vc4/vc4_drv.c | 17 + drivers/gpu/drm/vc4/vc4_drv.h | 9 +++ drivers/gpu/drm/vc4/vc4_irq.c | 6 +++-- drivers/gpu/drm/vc4/vc4_v3d.c | 47

[PATCH v5 1/4] drm/vc4: Reformat and export binner bo allocation helper

2019-04-15 Thread Paul Kocialkowski
Since we'll be using the binner bo allocation helper in other parts of the driver, reformat it with a vc4_v3d prefix and pass the vc4 dev directly to match other functions. Make the function visible to the whole driver too. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_

[PATCH v5 2/4] drm/vc4: Check for V3D before binner bo alloc

2019-04-15 Thread Paul Kocialkowski
Check that we have a V3D device registered before attempting to allocate a binner buffer object. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_v3d.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_v3d.c b/drivers/gpu/drm/vc4/vc4_v3d.c index

[PATCH] drm/sun4i: Use DRM_GEM_CMA_VMAP_DRIVER_OPS for GEM operations

2019-04-15 Thread Paul Kocialkowski
can also get rid of free_object_unlocked and gem_vm_ops, which are superseded by the object funcs. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/sun4i/sun4i_drv.c | 16 +--- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b

Re: [PATCH] drm/sun4i: Use DRM_GEM_CMA_VMAP_DRIVER_OPS for GEM operations

2019-04-16 Thread Paul Kocialkowski
Hi, Le lundi 15 avril 2019 à 18:15 +0200, Paul Kocialkowski a écrit : > Our driver makes a typical use of CMA, with GEM object allocated as > GEM CMA objects. Use DRM_GEM_CMA_VMAP_DRIVER_OPS to describe the ops > instead of duplicating them. > > Because DRM_GEM_CMA_VMAP_DRIVER_O

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-17 Thread Paul Kocialkowski
3 +- > lib/image-formats-selftests.c | 325 +- > lib/image-formats.c | 717 ++- > 70 files changed, 1740 insertions(+), 1174 deletions(-) > delete mode 100644 drivers/gpu/drm/selftests/test-drm_form

Re: [PATCH 06/20] lib: Add video format information library

2019-04-17 Thread Paul Kocialkowski
Hi, On Wed, 2019-04-17 at 09:54 +0200, Maxime Ripard wrote: > Move the DRM formats API to turn this into a more generic image formats API > to be able to leverage it into some other places of the kernel, such as > v4l2 drivers. See a few comments below. > Signed-off-by: Maxime Ripard > --- > i

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-17 Thread Paul Kocialkowski
| 438 ++- > lib/Kconfig| 7 +- > lib/Makefile | 3 +- > lib/image-formats-selftests.c | 325 +- > lib/image-formats.c

Re: [PATCH 19/20] lib: image-formats: Add more functions

2019-04-17 Thread Paul Kocialkowski
int plane) > +{ > + if (!format || plane >= format->num_planes) > + return 0; > + > + return image_format_info_plane_stride(format, width, plane) * > + image_format_info_plane_height(format, height, plane); > +} > + > co

Re: [PATCH 06/20] lib: Add video format information library

2019-04-17 Thread Paul Kocialkowski
Hi, On Wed, 2019-04-17 at 14:48 +0200, Maxime Ripard wrote: > Hi, > > On Wed, Apr 17, 2019 at 02:34:54PM +0200, Paul Kocialkowski wrote: > > > +struct image_format_info { > > > + union { > > > + /** > > > + * @drm_fmt: > &g

Support for 2D engines/blitters in V4L2 and DRM

2019-04-17 Thread Paul Kocialkowski
Hi Nicolas, I'm detaching this thread from our V4L2 stateless decoding spec since it has drifted off and would certainly be interesting to DRM folks as well! For context: I was initially talking about writing up support for the Allwinner 2D engine as a DRM render driver, where I'd like to be able

Re: Support for 2D engines/blitters in V4L2 and DRM

2019-04-18 Thread Paul Kocialkowski
Hi Daniel, On Thu, 2019-04-18 at 10:18 +0200, Daniel Vetter wrote: > On Wed, Apr 17, 2019 at 08:10:15PM +0200, Paul Kocialkowski wrote: > > Hi Nicolas, > > > > I'm detaching this thread from our V4L2 stateless decoding spec since > > it has drifted off and would

Re: Support for 2D engines/blitters in V4L2 and DRM

2019-04-18 Thread Paul Kocialkowski
Hi, On Thu, 2019-04-18 at 18:09 +0900, Tomasz Figa wrote: > On Thu, Apr 18, 2019 at 5:55 PM Paul Kocialkowski > wrote: > > Hi Daniel, > > > > On Thu, 2019-04-18 at 10:18 +0200, Daniel Vetter wrote: > > > On Wed, Apr 17, 2019 at 08:10:15PM +0200, Paul Kocia

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-18 Thread Paul Kocialkowski
nvenient for v4l uapi. > > - Add a conversion library that tries to best-effort map between drm > > and v4l formats. On the drm side that most likely means you need > > offsets for planes, and modifiers too (since those are implied in some > > v4l fourcc). Emphasis on &

[PATCH v2] drm/sun4i: Use DRM_GEM_CMA_VMAP_DRIVER_OPS for GEM operations

2019-04-18 Thread Paul Kocialkowski
can also get rid of free_object_unlocked and gem_vm_ops, which are superseded by the object funcs. Signed-off-by: Paul Kocialkowski --- Changes since v1: * Made sure our dumb_create overwrites the OPS's dumb_create. drivers/gpu/drm/sun4i/sun4i_drv.c | 16 +--- 1 file chang

[PATCH 1/3] drm/sun4i: Add missing drm_atomic_helper_shutdown at driver unbind

2019-04-18 Thread Paul Kocialkowski
A call to drm_atomic_helper_shutdown is required to properly release the internal references taken by the core and avoid warnings about leaking objects. Add it since it was missing. Fixes: 9026e0d122ac ("drm: Add Allwinner A10 Display Engine support") Signed-off-by: Paul Ko

[PATCH 0/3] sun4i/drm: Fixes for removing and re-probing the driver

2019-04-18 Thread Paul Kocialkowski
This series brings-in some fixes that are necessary to be able to remove the driver at run-time (when built as a module) and properly re-probe it afterwards. Cheers, Paul Paul Kocialkowski (3): drm/sun4i: Add missing drm_atomic_helper_shutdown at driver unbind drm/sun4i: Set device driver

[PATCH 3/3] drm/sun4i: Fix component unbinding and component master deletion

2019-04-18 Thread Paul Kocialkowski
lay Engine support") Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/sun4i/sun4i_drv.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c index af07291544a4..0ea365e54de1 100644 --- a/drivers/gpu/drm/sun4i/su

[PATCH 2/3] drm/sun4i: Set device driver data at bind time for use in unbind

2019-04-18 Thread Paul Kocialkowski
Our sun4i_drv_unbind gets the drm device using dev_get_drvdata. However, that driver data is never set in sun4i_drv_bind. Set it there to avoid getting a NULL pointer at unbind time. Fixes: 9026e0d122ac ("drm: Add Allwinner A10 Display Engine support") Signed-off-by: Paul Ko

Re: [linux-sunxi] [PATCH 3/3] drm/sun4i: Fix component unbinding and component master deletion

2019-04-18 Thread Paul Kocialkowski
Hi, Le jeudi 18 avril 2019 à 08:03 -0700, Chen-Yu Tsai a écrit : > On Thu, Apr 18, 2019 at 6:27 AM Paul Kocialkowski > wrote: > > For our component-backed driver to be properly removed, we need to > > delete the component master in sun4i_drv_remove and make sure to call > &

[PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind

2019-04-19 Thread Paul Kocialkowski
component unbinding and component master deletion") Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c index 843b86661833..29258b404e

Re: Support for 2D engines/blitters in V4L2 and DRM

2019-04-19 Thread Paul Kocialkowski
e > overhead. Note that as we speak, I have colleagues who are > experimenting with frame timestamp prediction that slaves to the > effective timestamp (catching up over time). But we still have issues > when the capture driver skipped a frame (missed a capture window). > > I hope

Re: [PATCH 1/4] drm/v3d: Fix debugfs reads of MMU regs.

2019-04-19 Thread Paul Kocialkowski
Hi, On Thu, 2019-04-18 at 17:10 -0700, Eric Anholt wrote: > They're in the hub, not the individual cores. Although I don't have docs to check, looks sane: Reviewed-by: Paul Kocialkowski Cheers, Paul > Signed-off-by: Eric Anholt > --- > drivers/gpu/drm/v3d/v3d_debu

Re: [PATCH 2/4] drm/v3d: Set the correct DMA mask according to the MMU's limits.

2019-04-19 Thread Paul Kocialkowski
Hi, On Thu, 2019-04-18 at 17:10 -0700, Eric Anholt wrote: > On 7278, we've got 40 bits to work with. Although I don't have docs to check, looks sane: Reviewed-by: Paul Kocialkowski Cheers, Paul > Signed-off-by: Eric Anholt > --- > drivers/gpu/drm/v3d/v3d_debugfs.c |

Re: [PATCH 3/4] drm/v3d: Dump V3D error debug registers in debugfs, and one at reset.

2019-04-19 Thread Paul Kocialkowski
oks sane: Reviewed-by: Paul Kocialkowski Cheers, Paul > Signed-off-by: Eric Anholt > --- > drivers/gpu/drm/v3d/v3d_debugfs.c | 5 > drivers/gpu/drm/v3d/v3d_gem.c | 4 +++- > drivers/gpu/drm/v3d/v3d_regs.h| 38 +++ > 3 files

Re: [PATCH 4/4] drm/v3d: Fix and extend MMU error handling.

2019-04-19 Thread Paul Kocialkowski
L2T, CLE, PTB, and TLB > faults. Didn't check the docs but looks sane too! Reviewed-by: Paul Kocialkowski Cheers, Paul > Signed-off-by: Eric Anholt > --- > drivers/gpu/drm/v3d/v3d_drv.c | 1 + > drivers/gpu/drm/v3d/v3d_drv.h | 2 ++ > drivers/gpu/drm/v3d/v3d_irq

Re: [git pull] drm fixes for 5.1-rc6

2019-04-19 Thread Paul Kocialkowski
+-- > drivers/gpu/host1x/hw/channel_hw.c | 8 ++-- > 10 files changed, 58 insertions(+), 18 deletions(-) > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind

2019-04-19 Thread Paul Kocialkowski
Hi, On Fri, 2019-04-19 at 09:02 -0700, Chen-Yu Tsai wrote: > On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski > wrote: > > Our components may still be using the DRM device driver (if only to > > access our driver's private data), so make sure to unbind them before &g

Re: [PATCH v5 3/4] drm/vc4: Check for the binner bo before handling OOM interrupt

2019-04-23 Thread Paul Kocialkowski
Hi, On Mon, 2019-04-15 at 13:48 -0700, Eric Anholt wrote: > Paul Kocialkowski writes: > > > Since the OOM interrupt directly deals with the binner bo, it doesn't > > make sense to try and handle it without a binner buffer registered. > > > &g

Re: [PATCH v5 3/4] drm/vc4: Check for the binner bo before handling OOM interrupt

2019-04-23 Thread Paul Kocialkowski
On Tue, 2019-04-23 at 10:30 +0200, Paul Kocialkowski wrote: > Hi, > > On Mon, 2019-04-15 at 13:48 -0700, Eric Anholt wrote: > > Paul Kocialkowski writes: > > > > > Since the OOM interrupt directly deals with the binner bo, it doesn't > > > make sense

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-23 Thread Paul Kocialkowski
Hi, On Tue, 2019-04-23 at 09:30 +0200, Daniel Vetter wrote: > On Sun, Apr 21, 2019 at 01:40:45AM +0300, Laurent Pinchart wrote: > > Hi Paul, > > > > On Thu, Apr 18, 2019 at 01:49:54PM +0200, Paul Kocialkowski wrote: > > > On Thu, 2019-04-18 at 11:02 +0200, Maxim

Re: [PATCH v5 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-04-23 Thread Paul Kocialkowski
Hi, On Mon, 2019-04-15 at 13:50 -0700, Eric Anholt wrote: > Paul Kocialkowski writes: > > > The binner bo is not required until the V3D is in use, so avoid > > allocating it at probe and do it on the first non-dumb BO allocation. > > Keep track of which clients are us

[PATCH v6 2/4] drm/vc4: Check for V3D before binner bo alloc

2019-04-23 Thread Paul Kocialkowski
Check that we have a V3D device registered before attempting to allocate a binner buffer object. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_v3d.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_v3d.c b/drivers/gpu/drm/vc4/vc4_v3d.c index

[PATCH v6 4/4] drm/vc4: Allocate binner bo when starting to use the V3D

2019-04-23 Thread Paul Kocialkowski
runtime suspend/resume to avoid failing to allocate it at resume. This happens when the CMA pool is full at that point and results in a hard crash. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_bo.c | 45 +-- drivers/gpu/drm/vc4/vc4_drv.c | 6 drivers

[PATCH v6 0/4] drm/vc4: Binner BO management improvements

2019-04-23 Thread Paul Kocialkowski
. Changes since v1: * Squashed the two final patches into one. Paul Kocialkowski (4): drm/vc4: Reformat and the binner bo allocation helper drm/vc4: Check for V3D before binner bo alloc drm/vc4: Check for the binner bo before handling OOM interrupt drm/vc4: Allocate binner bo when starting

[PATCH v6 1/4] drm/vc4: Reformat and the binner bo allocation helper

2019-04-23 Thread Paul Kocialkowski
In preparation for wrapping the binner bo allocation helper with put/get helpers, pass the vc4 dev directly and drop the vc4 prefix. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_v3d.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm

[PATCH v6 3/4] drm/vc4: Check for the binner bo before handling OOM interrupt

2019-04-23 Thread Paul Kocialkowski
Since the OOM interrupt directly deals with the binner bo, it doesn't make sense to try and handle it without a binner buffer registered. Signed-off-by: Paul Kocialkowski --- drivers/gpu/drm/vc4/vc4_irq.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_ir

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-23 Thread Paul Kocialkowski
Hi, On Tue, 2019-04-23 at 10:28 -0400, Nicolas Dufresne wrote: > Le mardi 23 avril 2019 à 14:33 +0200, Paul Kocialkowski a écrit : > > Hi, > > > > On Tue, 2019-04-23 at 09:30 +0200, Daniel Vetter wrote: > > > On Sun, Apr 21, 2019 at 01:40:45AM +0300, Laurent P

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-23 Thread Paul Kocialkowski
Hi, Le mardi 23 avril 2019 à 17:02 +0100, Daniel Stone a écrit : > Hi Laurent, > > On Tue, 23 Apr 2019 at 16:54, Laurent Pinchart > wrote: > > On Tue, Apr 23, 2019 at 09:59:37AM +0100, Daniel Stone wrote: > > > On Tue, 23 Apr 2019 at 08:26, Daniel Vetter wrote: > > > Totally. Let's take DRM_FOR

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-23 Thread Paul Kocialkowski
Hi, Le mardi 23 avril 2019 à 18:45 +0300, Laurent Pinchart a écrit : > Hi Daniel, > > On Tue, Apr 23, 2019 at 09:25:54AM +0200, Daniel Vetter wrote: > > On Sun, Apr 21, 2019 at 01:59:04AM +0300, Laurent Pinchart wrote: > > > On Thu, Apr 18, 2019 at 12:07:44PM +0200, Daniel Vetter wrote: > > > > O

Re: [PATCH 00/20] drm: Split out the formats API and move it to a common place

2019-04-23 Thread Paul Kocialkowski
Hi, Le dimanche 21 avril 2019 à 01:40 +0300, Laurent Pinchart a écrit : > Hi Paul, > > On Thu, Apr 18, 2019 at 01:49:54PM +0200, Paul Kocialkowski wrote: > > On Thu, 2019-04-18 at 11:02 +0200, Maxime Ripard wrote: > > > On Thu, Apr 18, 2019 at 09:52:10AM +0200, Daniel Vet

Re: [PATCH 06/20] lib: Add video format information library

2019-04-23 Thread Paul Kocialkowski
Hi, Le mardi 23 avril 2019 à 13:22 +0200, Thomas Zimmermann a écrit : > Hi > > Am 17.04.19 um 09:54 schrieb Maxime Ripard: > > Move the DRM formats API to turn this into a more generic image formats API > > to be able to leverage it into some other places of the kernel, such as > > v4l2 drivers.

Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind

2019-04-23 Thread Paul Kocialkowski
Hi, Le vendredi 19 avril 2019 à 19:10 +0200, Paul Kocialkowski a écrit : > Hi, > > On Fri, 2019-04-19 at 09:02 -0700, Chen-Yu Tsai wrote: > > On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski > > wrote: > > > Our components may still be using the DRM device drive

  1   2   3   4   5   6   >