Re: [EXT] Re: [PATCH] drm: fix HDR static metadata type field numbering

2019-11-28 Thread Laurentiu Palcu
On Wed, Nov 27, 2019 at 05:17:03PM +0200, Ville Syrjälä wrote: > Caution: EXT Email > > On Wed, Nov 27, 2019 at 02:42:35PM +, Laurentiu Palcu wrote: > > According to CTA-861 specification, HDR static metadata data block allows a > > sink to indicate which HDR metadata types it supports by

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

2019-11-28 Thread Laurent Pinchart
Hi Geert, On Thu, Nov 28, 2019 at 08:56:14AM +0100, Geert Uytterhoeven wrote: > On Wed, Nov 13, 2019 at 11:04 AM Jacopo Mondi > wrote: > > Add a driver for the R-Car Display Unit Color Correction Module. > > In most of Gen3 SoCs, each DU output channel is provided with a CMM unit > > to perform

Re: [PATCH v2 0/4] drm/rect: Bugfixes and selftests

2019-11-28 Thread Benjamin GAIGNARD
On 11/22/19 6:56 PM, Ville Syrjala wrote: > From: Ville Syrjälä > > My earlier fixes for drm_rect + div-by-zero fix + some > selftests that Daniel requested. > > Cc: Maarten Lankhorst > Cc: Benjamin Gaignard > Cc: Daniel Vetter Thanks to have handle this. Reviewed-by: Benjamin Gaignard > >

[GIT PULL] drm/arc: Yet another set of minor fixes

2019-11-28 Thread Alexey Brodkin
Hi David, Daniel! The following changes since commit 8082731830a0b95f7f7a63b78de67de446013c80: drm/vram: remove unused declaration (2019-11-27 07:51:49 +0100) are available in the Git repository at: g...@github.com:abrodkin/linux.git tags/arcpgu-updates-2019.11.27 for you to fetch changes

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

2019-11-28 Thread Geert Uytterhoeven
Hi Laurent, On Thu, Nov 28, 2019 at 9:09 AM Laurent Pinchart wrote: > On Thu, Nov 28, 2019 at 08:56:14AM +0100, Geert Uytterhoeven wrote: > > On Wed, Nov 13, 2019 at 11:04 AM Jacopo Mondi > > wrote: > > > Add a driver for the R-Car Display Unit Color Correction Module. > > > In most of Gen3

RE: [PATCH] drm/bridge/synopsys: dsi: read status error during transfer

2019-11-28 Thread Angelo Ribeiro
Hi Yannick, Could you please share why you are adding this retry? Are you trying to solve an issue with the generic interface buffs/fifos overflow? I think that it could be nice read the CMD_PKT_STATUS register to ensure that buffs/fifos are not full or empty before a read or write. > From:

[PATCH] msm:disp:dpu1: add scaler support on SC7180 display

2019-11-28 Thread Shubhashree Dhar
Add scaler support for display driver. This patch has dependency on the below series https://patchwork.kernel.org/patch/11260267/ Co-developed-by: Raviteja Tamatam Signed-off-by: Raviteja Tamatam Signed-off-by: Shubhashree Dhar --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 21

[PATCH 5/5] drm/amd/powerplay: Remove unneeded variable 'ret' in amdgpu_smu.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/powerplay/amdgpu_smu.c:1192:5-8: Unneeded variable: "ret". Return "0" on line 1195 drivers/gpu/drm/amd/powerplay/amdgpu_smu.c:1945:5-8: Unneeded variable: "ret". Return "0" on line 1961 Reported-by: Hulk Robot Signed-off-by: zhengbin ---

[PATCH 2/5] drm/amd/powerplay: Remove unneeded variable 'result' in vega10_hwmgr.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c:4363:5-11: Unneeded variable: "result". Return "0" on line 4370 Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH 4/4] drm/amd/display: Remove unneeded semicolon in display_rq_dlg_calc_21.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:1525:144-145: Unneeded semicolon drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:1526:142-143: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: zhengbin ---

[PATCH] msm:disp:dpu1: Fix core clk rate in display driver

2019-11-28 Thread Shubhashree Dhar
Fix max core clk rate during dt parsing in display driver. Signed-off-by: Shubhashree Dhar --- drivers/gpu/drm/msm/disp/dpu1/dpu_io_util.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_io_util.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_io_util.c index

[PATCH 3/5] drm/amd/powerplay: Remove unneeded variable 'ret' in smu7_hwmgr.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c:5188:5-8: Unneeded variable: "ret". Return "0" on line 5196 Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)

[PATCH 3/4] drm/amd/display: Remove unneeded semicolon in hdcp.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/display/modules/hdcp/hdcp.c:506:2-3: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/display/modules/hdcp/hdcp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: Re: [PATCH v2 2/2] drm: bridge: adv7511: Add support for ADV7535

2019-11-28 Thread Schrempf Frieder
Hi Bogdan, On 21.08.19 07:34, Togorean, Bogdan wrote: > On Tue, 2019-08-20 at 10:53 +0200, Daniel Vetter wrote: >> [External] >> >> On Mon, Aug 19, 2019 at 12:46:16PM +0200, Sam Ravnborg wrote: >>> Hi Bogdan. >>> >> adv7533_detach_dsi(adv7511); >>

[PATCH 0/4] drm/amd/display: Remove unneeded semicolon

2019-11-28 Thread zhengbin
zhengbin (4): drm/amd/display: Remove unneeded semicolon in bios_parser.c drm/amd/display: Remove unneeded semicolon in bios_parser2.c drm/amd/display: Remove unneeded semicolon in hdcp.c drm/amd/display: Remove unneeded semicolon in display_rq_dlg_calc_21.c

Re: [PATCH v2 2/2] drm: bridge: adv7511: Add support for ADV7535

2019-11-28 Thread Schrempf Frieder
On 27.11.19 15:22, Togorean, Bogdan wrote: > Hi Frieder, > > I'm glad to find there are other persons interested in this driver and > especially support for ADV7535. Unfortunately I had to put on hold the > development due to other activities but I'll send V3 tomorrow. Great to hear that. Thanks

[PATCH 0/5] drm/amd/powerplay: Remove unneeded variable

2019-11-28 Thread zhengbin
zhengbin (5): drm/amd/powerplay: Remove unneeded variable 'result' in smu10_hwmgr.c drm/amd/powerplay: Remove unneeded variable 'result' in vega10_hwmgr.c drm/amd/powerplay: Remove unneeded variable 'ret' in smu7_hwmgr.c drm/amd/powerplay: Remove unneeded variable 'result' in

RE: [GIT PULL] drm/arc: Yet another set of minor fixes

2019-11-28 Thread Alexey Brodkin
Hi Daniel, > -Original Message- > From: Daniel Vetter > Sent: Wednesday, November 27, 2019 1:07 PM > To: Alexey Brodkin > Cc: Daniel Vetter ; David Airlie ; arcml > a...@lists.infradead.org>; Eugeniy Paltsev ; > dri-devel@lists.freedesktop.org > Subject: Re: [GIT PULL] drm/arc: Yet

Re: [PATCH 2/3] drm/hibmc: Use drm_gem_fb_create

2019-11-28 Thread kbuild test robot
to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Daniel-Vetter/drm-rockchip-Use-drm_gem_fb_create_with_dirty/20191128-023917 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: arm64

Re: [PATCH 24/30] drm/mcde: dsi: Use drm_bridge_init()

2019-11-28 Thread Linus Walleij
On Tue, Nov 26, 2019 at 2:16 PM Mihail Atanassov wrote: > No functional change. > > Signed-off-by: Mihail Atanassov Acked-by: Linus Walleij Yours, Linus Walleij ___ dri-devel mailing list dri-devel@lists.freedesktop.org

Re: [Intel-gfx] [PATCH 01/13] video: fb_defio: preserve user fb_ops

2019-11-28 Thread Jani Nikula
On Wed, 27 Nov 2019, Daniel Vetter wrote: > On Wed, Nov 27, 2019 at 07:17:41PM +0100, Daniel Vetter wrote: >> On Wed, Nov 27, 2019 at 06:31:57PM +0200, Jani Nikula wrote: >> > Modifying fb_ops directly to override fb_mmap with fb_deferred_io_mmap >> > and then resetting it to NULL afterwards

[PATCH 1/5] drm/amd/powerplay: Remove unneeded variable 'result' in smu10_hwmgr.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/powerplay/hwmgr/smu10_hwmgr.c:1154:5-11: Unneeded variable: "result". Return "0" on line 1159 Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/powerplay/hwmgr/smu10_hwmgr.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH 4/5] drm/amd/powerplay: Remove unneeded variable 'result' in vega12_hwmgr.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c:502:5-11: Unneeded variable: "result". Return "0" on line 515 Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c | 4 +--- 1 file changed, 1 insertion(+), 3

RE: [GIT PULL] drm/arc: Minor improvements

2019-11-28 Thread Alexey Brodkin
Hi all, As Jose suggested I'm adding "drm-misc" maintainers as that tree might be a better fit for ARC PGU patches. -Alexey > -Original Message- > From: linux-snps-arc On Behalf > Of Alexey Brodkin > Sent: Wednesday, November 27, 2019 10:25 AM > To: Daniel Vetter ; David Airlie > Cc:

[PATCH] drm/mediatek: Fix can't get component for external display plane.

2019-11-28 Thread Pi-Hsun Shih
From: Yongqiang Niu The original logic is ok for primary display, but will not find out component for external display. For example, plane->index is 6 for external display, but there are only 2 layer nr in external display, and this condition will never happen: if (plane->index < (count +

[PATCH 1/4] drm/amd/display: Remove unneeded semicolon in bios_parser.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/display/dc/bios/bios_parser.c:2192:2-3: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/display/dc/bios/bios_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 0/3] drm/omap: fix am4 evm lcd

2019-11-28 Thread Tony Lindgren
Hi * Tomi Valkeinen [191127 12:59]: > Hi Tony, Thierry, Laurent, > > Any thoughts on the below points? > I think yet another option is to write some omap boot time quirks code, > which looks at the DT data, and changes the panel compatible string (for 1), > and removes the timings node (for

[PATCH 2/4] drm/amd/display: Remove unneeded semicolon in bios_parser2.c

2019-11-28 Thread zhengbin
Fixes coccicheck warning: drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c:995:2-3: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: zhengbin --- drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: Re: [PATCH v2 2/2] drm: bridge: adv7511: Add support for ADV7535

2019-11-28 Thread Togorean, Bogdan
Hi Frieder, I'm glad to find there are other persons interested in this driver and especially support for ADV7535. Unfortunately I had to put on hold the development due to other activities but I'll send V3 tomorrow. I also started work on HDCP support for this driver and hope to send soon a

RE: [PATCH v2] video: hyperv: hyperv_fb: Use physical memory for fb on HyperV Gen 1 VMs.

2019-11-28 Thread Michael Kelley
From: Wei Hu Sent: Friday, November 22, 2019 12:24 AM > > On Hyper-V, Generation 1 VMs can directly use VM's physical memory for > their framebuffers. This can improve the efficiency of framebuffer and > overall performence for VM. The physical memory assigned to framebuffer > must be

[PATCH v6] drm/panel: Add generic DSI display controller YAML bindings

2019-11-28 Thread Linus Walleij
This adds a starting point for processing and defining generic bindings used by DSI display controllers and panels attached to the virtual DSI ports. Cc: Stephan Gerhold Cc: devicet...@vger.kernel.org Suggested-by: Rob Herring Signed-off-by: Linus Walleij --- ChangeLog v5->v6: - Rename subject

Re: [PATCH 13/13] samples: vfio-mdev: constify fb ops

2019-11-28 Thread Jani Nikula
On Wed, 27 Nov 2019, Daniel Vetter wrote: > On Wed, Nov 27, 2019 at 06:32:09PM +0200, Jani Nikula wrote: >> Now that the fbops member of struct fb_info is const, we can star making >> the ops const as well. >> >> Cc: Kirti Wankhede >> Cc: k...@vger.kernel.org >> Signed-off-by: Jani Nikula > >

Re: [PATCH 1/2] PM / runtime: Allow drivers to override runtime PM behaviour on sleep

2019-11-28 Thread Rafael J. Wysocki
On Thursday, November 28, 2019 5:50:26 PM CET Thierry Reding wrote: > > --0F1p//8PRICkK4MW > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > Content-Transfer-Encoding: quoted-printable > > On Thu, Nov 28, 2019 at 05:14:51PM +0100, Rafael J. Wysocki wrote: > > On Thu,

Re: [git pull] drm for 5.5-rc1

2019-11-28 Thread Stephen Rothwell
Hi Dave, On Fri, 29 Nov 2019 09:13:16 +1000 Dave Airlie wrote: > > On Fri, 29 Nov 2019 at 07:55, Stephen Rothwell wrote: > > > > Hi Dave, > > > > On Thu, 28 Nov 2019 12:37:06 +1000 Dave Airlie wrote: > > > > > > > And apparently nobody bothered to tell me about the semantic conflict > > >

Re: [git pull] drm for 5.5-rc1

2019-11-28 Thread Stephen Rothwell
Hi Dave, On Thu, 28 Nov 2019 12:37:06 +1000 Dave Airlie wrote: > > > And apparently nobody bothered to tell me about the semantic conflict > > with the media tree due to the changed calling convention of > > cec_notifier_cec_adap_unregister(). Didn't that show up in linux-next? > > I can see

Re: [PATCH 1/2] PM / runtime: Allow drivers to override runtime PM behaviour on sleep

2019-11-28 Thread Rafael J. Wysocki
On Thursday, November 28, 2019 11:03:57 PM CET Rafael J. Wysocki wrote: > On Thursday, November 28, 2019 5:50:26 PM CET Thierry Reding wrote: > > > > --0F1p//8PRICkK4MW > > Content-Type: text/plain; charset=us-ascii > > Content-Disposition: inline > > Content-Transfer-Encoding: quoted-printable >

[git pull] mm + drm vmwgfx coherent

2019-11-28 Thread Dave Airlie
Hi Linus, This is just a separated pull for the mm pagewalking + drm/vmwgfx work Thomas did and you were involved in, I've left it separate in case you don't feel as comfortable with it as the other stuff. It has mm acks/r-b in the right places from what I can see. Dave.

[PATCH] drm/panfrost: Register devfreq cooling device

2019-11-28 Thread Robin Murphy
When we have devfreq, also try to register a basic cooling device in case GPU workloads manage to hit thermal throttling thresholds. Signed-off-by: Robin Murphy --- drivers/gpu/drm/panfrost/panfrost_devfreq.c | 32 ++--- 1 file changed, 21 insertions(+), 11 deletions(-) diff

Re: [git pull] drm for 5.5-rc1

2019-11-28 Thread Dave Airlie
On Fri, 29 Nov 2019 at 07:55, Stephen Rothwell wrote: > > Hi Dave, > > On Thu, 28 Nov 2019 12:37:06 +1000 Dave Airlie wrote: > > > > > And apparently nobody bothered to tell me about the semantic conflict > > > with the media tree due to the changed calling convention of > > >

[Bug 205585] [Regression] [amdgpu] AMD Vega 64 GPU invalid access and EEH under load

2019-11-28 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205585 --- Comment #3 from Timothy Pearson (tpear...@raptorengineering.com) --- Just had a chance to test on 5.4.0, still fails (haven't had a chance to bisect yet; I suspect it's more related to the 64-bit enablement on POWER in 5.4 than anything

Re: [PATCH RFC v4 02/16] cgroup: Introduce cgroup for drm subsystem

2019-11-28 Thread Kenny Ho
On Tue, Oct 1, 2019 at 10:31 AM Michal Koutný wrote: > On Thu, Aug 29, 2019 at 02:05:19AM -0400, Kenny Ho wrote: > > +struct cgroup_subsys drm_cgrp_subsys = { > > + .css_alloc = drmcg_css_alloc, > > + .css_free = drmcg_css_free, > > + .early_init = false, > > +

Re: [PATCH RFC v4 07/16] drm, cgroup: Add total GEM buffer allocation limit

2019-11-28 Thread Kenny Ho
On Tue, Oct 1, 2019 at 10:30 AM Michal Koutný wrote: > On Thu, Aug 29, 2019 at 02:05:24AM -0400, Kenny Ho wrote: > > drm.buffer.default > > A read-only flat-keyed file which exists on the root cgroup. > > Each entry is keyed by the drm device's major:minor. > > > >

[PATCH] rnndb: Add GBIF registers for a6xx GPU

2019-11-28 Thread Sharat Masetty
Add GBIF register definitions required to implement a618 GPU revision Signed-off-by: Sharat Masetty --- rnndb/adreno/a6xx.xml | 26 ++ 1 file changed, 26 insertions(+) diff --git a/rnndb/adreno/a6xx.xml b/rnndb/adreno/a6xx.xml index 747f071..2d2063a 100644 ---

Re: [PATCH 3/7] drm/udl: Remove unused encoder and CRTC code

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 02:47:03PM +0100, Thomas Zimmermann wrote: > The removed functionality is provided by simple-pipe helpers. > > Signed-off-by: Thomas Zimmermann Needs to be squashed into the previous patch, ideally with the new functions in the same place as the old ones (as much as

Re: [PATCH 7/7] drm/udl: Move udl_handle_damage() into udl_modeset.c

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 02:47:07PM +0100, Thomas Zimmermann wrote: > The only caller of udl_handle_damage() in the plane-update function > in udl_modeset.c. Move udl_handle_damage() there, make it static, and > remove several left-over macros. > > Signed-off-by: Thomas Zimmermann Reviewed-by:

[PULL] drm-intel-next-fixes

2019-11-28 Thread Joonas Lahtinen
Hi Dave & Daniel, Most importantly we have the fix to power regression that was introduced by the security fixes. Then fix for query uAPI and increase in request pre-emption timeout to accommodate super heavy benchmarks. Couple of display voltage programming fixes too. Thanks to Chris for

Re: [Intel-gfx] [PATCH 01/13] video: fb_defio: preserve user fb_ops

2019-11-28 Thread Daniel Vetter
On Thu, Nov 28, 2019 at 11:05:57AM +0100, Daniel Vetter wrote: > On Thu, Nov 28, 2019 at 11:09:46AM +0200, Jani Nikula wrote: > > On Wed, 27 Nov 2019, Daniel Vetter wrote: > > > On Wed, Nov 27, 2019 at 07:17:41PM +0100, Daniel Vetter wrote: > > >> On Wed, Nov 27, 2019 at 06:31:57PM +0200, Jani

Re: [PATCH 2/5] udmabuf: allow userspace to set map attributes

2019-11-28 Thread Gerd Hoffmann
Hi, > diff --git a/include/uapi/linux/udmabuf.h b/include/uapi/linux/udmabuf.h > index 46b6532ed855..f90831f2bb0d 100644 > --- a/include/uapi/linux/udmabuf.h > +++ b/include/uapi/linux/udmabuf.h > @@ -6,6 +6,8 @@ > #include > > #define UDMABUF_FLAGS_CLOEXEC0x01 > +#define

Re: [Intel-gfx] [PATCH 02/13] drm/fb-helper: don't preserve fb_ops across deferred IO use

2019-11-28 Thread Jani Nikula
On Thu, 28 Nov 2019, Noralf Trønnes wrote: > Den 27.11.2019 17.31, skrev Jani Nikula: >> Deferred IO now preserves the fb_ops. >> >> Cc: Noralf Trønnes >> Cc: dri-devel@lists.freedesktop.org >> Signed-off-by: Jani Nikula >> --- >> drivers/gpu/drm/drm_fb_helper.c | 18 ++ >> 1

Re: [Intel-gfx] [RFC 03/13] drm/i915/svm: Runtime (RT) allocator support

2019-11-28 Thread Joonas Lahtinen
Quoting Niranjan Vishwanathapura (2019-11-27 21:23:56) > >We should try to have the uAPI as final as early as possible. The > >change process is harder the later it is done, even for RFC :) > > > >On the same note, I'm inclined to have I915_SVM_MIGRATE called > >I915_GEM_VM_PREFAULT from the

Re: [PATCH] drm/panel: Add Boe Himax8279d MIPI-DSI LCD panel

2019-11-28 Thread Emil Velikov
Hi Jerry, On Tue, 26 Nov 2019 at 08:14, Jerry Han wrote: > > Support Boe Himax8279d 8.0" 1200x1920 TFT LCD panel, it is a MIPI DSI > panel. > > V9: > - Adjust init code, make the format more concise > - kill off default_off_cmds (Emil) > - use mipi_dsi_dcs_set_display_{on,off} in their

[PATCH] drm/i915: Provide ddc symlink in hdmi connector sysfs directory

2019-11-28 Thread Andrzej Pietrasiewicz
Use the ddc pointer provided by the generic connector. Signed-off-by: Andrzej Pietrasiewicz Acked-by: Sam Ravnborg Reviewed-by: Emil Velikov --- Rebased onto drm-misc-next as of 28 Nov 2019 drivers/gpu/drm/i915/display/intel_hdmi.c | 12 1 file changed, 8 insertions(+), 4

Re: [PATCH v2] drm/dsc: Return unsigned long on compute offset

2019-11-28 Thread Jani Nikula
On Fri, 22 Nov 2019, wrote: > From: Mikita Lipski > > We shouldn't compare int with unsigned long to find the max value > and since we are not expecting negative value returned from > compute_offset we should make this function return unsigned long > so we can compare the values when computing

Re: [PATCH 2/7] drm/udl: Convert to struct drm_simple_display_pipe

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 02:47:02PM +0100, Thomas Zimmermann wrote: > Udl has a single display pipeline with aprimary plane; perfect for > simple-pipe helpers. Convert it over. The old encoder and CRTC code > becomes unused and obsolete. > > Exported formats for the primary plane are RGB565 and

Re: [PATCH v2] drm/dp_mst: Fix W=1 warnings

2019-11-28 Thread Jani Nikula
On Thu, 28 Nov 2019, Benjamin GAIGNARD wrote: > On 11/28/19 12:21 PM, Jani Nikula wrote: >> On Thu, 28 Nov 2019, Benjamin Gaignard wrote: >>> Fix the warnings that show up with W=1. >>> They are all about unused but set variables. >>> If functions returns are not used anymore make them void. >>>

Re: [PATCH v2] drm/dp_mst: Fix W=1 warnings

2019-11-28 Thread Daniel Vetter
On Thu, Nov 28, 2019 at 03:34:21PM +0200, Jani Nikula wrote: > On Thu, 28 Nov 2019, Benjamin GAIGNARD wrote: > > On 11/28/19 12:21 PM, Jani Nikula wrote: > >> On Thu, 28 Nov 2019, Benjamin Gaignard wrote: > >>> Fix the warnings that show up with W=1. > >>> They are all about unused but set

Re: [PATCH] drm/i915: Provide ddc symlink in hdmi connector sysfs directory

2019-11-28 Thread Jani Nikula
On Thu, 28 Nov 2019, Andrzej Pietrasiewicz wrote: > Use the ddc pointer provided by the generic connector. > > Signed-off-by: Andrzej Pietrasiewicz > Acked-by: Sam Ravnborg > Reviewed-by: Emil Velikov > --- > Rebased onto drm-misc-next as of 28 Nov 2019 The conflict is in

Re: [PATCH 1/7] drm/udl: Init connector before encoder and CRTC

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 02:47:01PM +0100, Thomas Zimmermann wrote: > To mimic simple-pipe, we initialize the connector before the rest of > the display pipeline. > > Signed-off-by: Thomas Zimmermann > --- > drivers/gpu/drm/udl/udl_connector.c | 7 +++ > drivers/gpu/drm/udl/udl_drv.h

Re: [Intel-gfx] [PATCH 02/13] drm/fb-helper: don't preserve fb_ops across deferred IO use

2019-11-28 Thread Noralf Trønnes
Den 28.11.2019 13.05, skrev Jani Nikula: > On Thu, 28 Nov 2019, Noralf Trønnes wrote: >> Den 27.11.2019 17.31, skrev Jani Nikula: >>> Deferred IO now preserves the fb_ops. >>> >>> Cc: Noralf Trønnes >>> Cc: dri-devel@lists.freedesktop.org >>> Signed-off-by: Jani Nikula >>> --- >>>

Re: [PATCH 5/7] drm/udl: Convert to drm_atomic_helper_dirtyfb()

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 02:47:05PM +0100, Thomas Zimmermann wrote: > The infrastruture for atomic modesetting allows us to use the generic > code for dirty-FB and damage handling. Switch over udl and remove the > driver's implementation. > > Signed-off-by: Thomas Zimmermann > --- >

Re: [PATCH 6/7] drm/udl: Remove struct udl_device.active_fb_16

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 02:47:06PM +0100, Thomas Zimmermann wrote: > The udl driver stores the currently active framebuffer to know from > where to accept damage updates. > > With the conversion to plane-state damage handling, this is not necessary > any longer. The currently active framebuffer

Re: [PATCH v2 3/3] drm/mgag200: Add workaround for HW that does not support 'startadd'

2019-11-28 Thread Sasha Levin
Hi, [This is an automated email] This commit has been processed because it contains a "Fixes:" tag, fixing commit: 81da87f63a1e ("drm: Replace drm_gem_vram_push_to_system() with kunmap + unpin"). The bot has tested the following trees: v5.3.13. v5.3.13: Build failed! Errors:

Re: [PATCH v2 2/3] drm/mgag200: Store flags from PCI driver data in device structure

2019-11-28 Thread Sasha Levin
Hi, [This is an automated email] This commit has been processed because it contains a "Fixes:" tag, fixing commit: 81da87f63a1e ("drm: Replace drm_gem_vram_push_to_system() with kunmap + unpin"). The bot has tested the following trees: v5.3.13. v5.3.13: Failed to apply! Possible dependencies:

Re: [PATCH 0/4] drm/amd/display: Remove unneeded semicolon

2019-11-28 Thread Harry Wentland
Series is Reviewed-by: Harry Wentland Harry On 2019-11-27 9:31 p.m., zhengbin wrote: > zhengbin (4): > drm/amd/display: Remove unneeded semicolon in bios_parser.c > drm/amd/display: Remove unneeded semicolon in bios_parser2.c > drm/amd/display: Remove unneeded semicolon in hdcp.c >

Re: [PATCH] drm/dp_mst: Fix W=1 warnings

2019-11-28 Thread Jani Nikula
On Tue, 12 Nov 2019, Benjamin Gaignard wrote: > Fix the warnings that show up with W=1. > They are all about unused but set variables. > > Signed-off-by: Benjamin Gaignard > --- > drivers/gpu/drm/drm_dp_mst_topology.c | 50 > +-- > 1 file changed, 19

Re: [Intel-gfx] [PATCH 01/13] video: fb_defio: preserve user fb_ops

2019-11-28 Thread Jani Nikula
On Thu, 28 Nov 2019, Daniel Vetter wrote: > On Thu, Nov 28, 2019 at 11:05:57AM +0100, Daniel Vetter wrote: >> On Thu, Nov 28, 2019 at 11:09:46AM +0200, Jani Nikula wrote: >> > On Wed, 27 Nov 2019, Daniel Vetter wrote: >> > > On Wed, Nov 27, 2019 at 07:17:41PM +0100, Daniel Vetter wrote: >> > >>

Re: [PATCH 13/13] samples: vfio-mdev: constify fb ops

2019-11-28 Thread Jani Nikula
On Thu, 28 Nov 2019, Christophe de Dinechin wrote: >> On 27 Nov 2019, at 17:32, Jani Nikula wrote: >> >> Now that the fbops member of struct fb_info is const, we can star making > s/star/start/ Ooops, thanks. BR, Jani. > >> the ops const as well. >> >> Cc: Kirti Wankhede >> Cc:

Re: [PATCHv3/RFC 1/4] drm/arm: Factor out generic afbc helpers

2019-11-28 Thread james qian wang (Arm Technology China)
On Mon, Nov 25, 2019 at 09:55:06AM +0100, Daniel Vetter wrote: > On Thu, Nov 21, 2019 at 06:22:44PM +0100, Andrzej Pietrasiewicz wrote: > > These are useful for other users of afbc, e.g. rockchip. > > > > Signed-off-by: Andrzej Pietrasiewicz > > --- > > drivers/gpu/drm/Makefile | 2 +-

Re: [PATCH 08/13] video: fbdev: make fbops member of struct fb_info a const pointer

2019-11-28 Thread kbuild test robot
to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Jani-Nikula/video-drm-constify-fbops-in-struct-fb_info/20191128-022047 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: x86_64

Re: [PATCH] drm/rect: update kerneldoc for drm_rect_clip_scaled()

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 05:10:14PM +0200, Ville Syrjälä wrote: > On Tue, Nov 26, 2019 at 03:52:13PM +0100, Daniel Vetter wrote: > > This was forgotten in f96bdf564f3e ("drm/rect: Handle rounding errors > > in drm_rect_clip_scaled, v3.") > > > > Spotted while reviewing patches from Ville touching

Re: [PATCH v2 1/3] drm/mgag200: Extract device type from flags

2019-11-28 Thread Emil Velikov
On 2019/11/27, Thomas Zimmermann wrote: > Hi Emil > > Am 27.11.19 um 17:29 schrieb Emil Velikov: > > Hi Thomas, > > > > On Tue, 26 Nov 2019 at 10:15, Thomas Zimmermann wrote: > >> > >> Adds a conversion function that extracts the device type from the > >> PCI id-table flags. Allows for storing

Re: [Intel-gfx] [PATCH 01/13] video: fb_defio: preserve user fb_ops

2019-11-28 Thread Daniel Vetter
On Thu, Nov 28, 2019 at 11:09:46AM +0200, Jani Nikula wrote: > On Wed, 27 Nov 2019, Daniel Vetter wrote: > > On Wed, Nov 27, 2019 at 07:17:41PM +0100, Daniel Vetter wrote: > >> On Wed, Nov 27, 2019 at 06:31:57PM +0200, Jani Nikula wrote: > >> > Modifying fb_ops directly to override fb_mmap with

Re: [PATCH 2/3] drm/hibmc: Use drm_gem_fb_create

2019-11-28 Thread Daniel Vetter
ckchip-Use-drm_gem_fb_create_with_dirty/20191128-023917 > base: git://anongit.freedesktop.org/drm-intel for-linux-next > config: arm64-randconfig-a001-20191128 (attached as .config) > compiler: aarch64-linux-gcc (GCC) 7.4.0 > reproduce: > wget > https://raw.github

Re: Next round of associating ddc adapters with connectors

2019-11-28 Thread Jani Nikula
On Fri, 22 Nov 2019, Andrzej Pietrasiewicz wrote: > Dear Maintainers, > > Can you please express your opinion on these patches: > > [tegra] https://patchwork.freedesktop.org/patch/327099/?series=65831=1 > [vc4] https://patchwork.freedesktop.org/patch/327102/?series=65831=1 > [zte]

Re: [PATCH 02/13] drm/fb-helper: don't preserve fb_ops across deferred IO use

2019-11-28 Thread Noralf Trønnes
Den 27.11.2019 17.31, skrev Jani Nikula: > Deferred IO now preserves the fb_ops. > > Cc: Noralf Trønnes > Cc: dri-devel@lists.freedesktop.org > Signed-off-by: Jani Nikula > --- > drivers/gpu/drm/drm_fb_helper.c | 18 ++ > 1 file changed, 2 insertions(+), 16 deletions(-) > >

Re: [PATCH 13/13] samples: vfio-mdev: constify fb ops

2019-11-28 Thread Daniel Vetter
On Thu, Nov 28, 2019 at 11:22:23AM +0200, Jani Nikula wrote: > On Wed, 27 Nov 2019, Daniel Vetter wrote: > > On Wed, Nov 27, 2019 at 06:32:09PM +0200, Jani Nikula wrote: > >> Now that the fbops member of struct fb_info is const, we can star making > >> the ops const as well. > >> > >> Cc: Kirti

Re: [PATCH 2/2] drm/fourcc: Fill out all block sizes for P210

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 11:46:54AM +, Liviu Dudau wrote: > On Tue, Nov 26, 2019 at 10:14:14AM +0100, Daniel Vetter wrote: > > 0 means 1 as the default, but it's mighty confusing if the block size > > for the first plane is spelled out explicitly, but not for the 2nd > > plane. > > > > No cc:

Re: [PATCH 08/13] video: fbdev: make fbops member of struct fb_info a const pointer

2019-11-28 Thread kbuild test robot
to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Jani-Nikula/video-drm-constify-fbops-in-struct-fb_info/20191128-022047 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: x86_64

Re: [EXT] Re: [PATCH] drm: fix HDR static metadata type field numbering

2019-11-28 Thread Ville Syrjälä
On Thu, Nov 28, 2019 at 08:39:41AM +, Laurentiu Palcu wrote: > On Wed, Nov 27, 2019 at 05:17:03PM +0200, Ville Syrjälä wrote: > > Caution: EXT Email > > > > On Wed, Nov 27, 2019 at 02:42:35PM +, Laurentiu Palcu wrote: > > > According to CTA-861 specification, HDR static metadata data

Re: [PATCH v2] drm/dp_mst: Fix W=1 warnings

2019-11-28 Thread Jani Nikula
On Thu, 28 Nov 2019, Benjamin Gaignard wrote: > Fix the warnings that show up with W=1. > They are all about unused but set variables. > If functions returns are not used anymore make them void. > > Signed-off-by: Benjamin Gaignard > CC: Jani Nikula > --- > changes in version 2: > - fix

Re: [PATCH v3 1/2] drm: call drm_gem_object_funcs.mmap with fake offset

2019-11-28 Thread Gerd Hoffmann
On Wed, Nov 27, 2019 at 10:25:22AM +0100, Gerd Hoffmann wrote: > The fake offset is going to stay, so change the calling convention for > drm_gem_object_funcs.mmap to include the fake offset. Update all users > accordingly. > > Note that this reverts 83b8a6f242ea ("drm/gem: Fix mmap fake offset

Re: [PATCH 0/2] PM / runtime: Allow drivers to override runtime PM behaviour on sleep

2019-11-28 Thread Thierry Reding
On Thu, Nov 28, 2019 at 05:47:05PM +0100, Daniel Vetter wrote: > On Thu, Nov 28, 2019 at 5:03 PM Thierry Reding > wrote: > > > > From: Thierry Reding > > > > This is a result of looking into a more formal way of doing what was > > proposed here: > > > >

Re: [PATCH 1/2] dt-bindings: display: DT schema for rocktech,rk101ii01d-ct panel

2019-11-28 Thread Jyri Sarha
On 28/11/2019 18:45, Jyri Sarha wrote: > Add DT schema binding for Rocktech Displays Limited RK101II01D-CT > 10.1" TFT 1280x800 Pixels with LVDS interface, LED Backlight and > capacitive touch panel. > > Signed-off-by: Jyri Sarha > --- > .../display/panel/rocktech,rk101ii01d-ct.yaml | 48

[PATCH] drm/i915: Provide ddc symlink in hdmi connector sysfs directory

2019-11-28 Thread Andrzej Pietrasiewicz
Use the ddc pointer provided by the generic connector. Signed-off-by: Andrzej Pietrasiewicz Acked-by: Sam Ravnborg Reviewed-by: Emil Velikov --- Rebased onto drm-intel-next-queued. drivers/gpu/drm/i915/display/intel_hdmi.c | 12 1 file changed, 8 insertions(+), 4 deletions(-)

[PATCH 6/9] drm/tegra: vic: Export module device table

2019-11-28 Thread Thierry Reding
From: Thierry Reding Export the module device table to ensure the VIC compatible strings are listed in the module's aliases table. This in turn causes the driver to be automatically loaded on boot if VIC is the only enabled subdevice of the logical host1x DRM device. Reviewed-by: Daniel Vetter

[PATCH 4/9] drm/tegra: Use proper IOVA address for cursor image

2019-11-28 Thread Thierry Reding
From: Thierry Reding The IOVA address for the cursor is the result of mapping the buffer object for the given display controller. Make sure to use the proper IOVA address as stored in the cursor's plane state. Signed-off-by: Thierry Reding --- drivers/gpu/drm/tegra/dc.c | 16

[PATCH 5/9] drm/tegra: sor: Implement system suspend/resume

2019-11-28 Thread Thierry Reding
From: Thierry Reding Upon system suspend, make sure the +5V HDMI regulator is disabled. This avoids potentially leaking current to the HDMI connector. This also makes sure that upon resume the regulator is enabled again, which in some cases is necessary to properly restore the state of the

[PATCH 2/9] drm/tegra: gem: Properly pin imported buffers

2019-11-28 Thread Thierry Reding
From: Thierry Reding Buffers that are imported from a DMA-BUF don't have pages allocated with them. At the same time an SG table for them can't be derived using the DMA API helpers because the necessary information doesn't exist. However there's already an SG table that was created during

[PATCH 8/9] drm/tegra: dpaux: Add missing runtime PM references

2019-11-28 Thread Thierry Reding
From: Thierry Reding Ensure that a runtime PM reference is acquired each time the DPAUX registers are accessed. Otherwise the code may end up running without the controller being powered, out-of-reset or clocked in some corner cases, resulting in a crash. Signed-off-by: Thierry Reding ---

[PATCH 1/9] drm/tegra: hub: Remove bogus connection mutex check

2019-11-28 Thread Thierry Reding
From: Thierry Reding I have no recollection why that check is there, but it seems to trigger all the time, so remove it. Everything works fine without. Signed-off-by: Thierry Reding --- drivers/gpu/drm/tegra/hub.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 3/9] drm/tegra: gem: Remove premature import restrictions

2019-11-28 Thread Thierry Reding
From: Thierry Reding It's not known at import time whether or not all users of a DMA-BUF will be able to deal with non-contiguous memory. Each user needs to verify at map-time whether it can access the buffer. Signed-off-by: Thierry Reding --- drivers/gpu/drm/tegra/gem.c | 7 --- 1 file

[PATCH 9/9] drm/tegra: sor: Make the +5V HDMI supply optional

2019-11-28 Thread Thierry Reding
From: Thierry Reding The SOR supports multiple display modes, but only when driving an HDMI monitor does it make sense to control the +5V power supply. eDP and DP don't need this, so make it optional. This fixes a crash observed during system suspend/resume. Signed-off-by: Thierry Reding ---

Re: [PATCH 1/3] drm/rockchip: Use drm_gem_fb_create_with_dirty

2019-11-28 Thread Andrzej Pietrasiewicz
W dniu 27.11.2019 o 19:00, Daniel Vetter pisze: If rockchip would switch over to the generic fbdev setup we could grabage collect even more of all this code (all of the remaining fb handling code really). v2: Actually use _with_dirty like the patch subject promised (Andrzej) Cc: Andrzej

[PATCH 2/2] drm/panel: simple: Add Rocktech RK101II01D-CT panel

2019-11-28 Thread Jyri Sarha
Add support for Rocktech RK101II01D-CT, 10.1" 1280x800 TFT with LVDS interface, LED backlight and integrated Goodix GT928 capacitive touch panel. Signed-off-by: Jyri Sarha --- drivers/gpu/drm/panel/panel-simple.c | 32 1 file changed, 32 insertions(+) diff --git

Re: [PATCH 0/2] PM / runtime: Allow drivers to override runtime PM behaviour on sleep

2019-11-28 Thread Daniel Vetter
On Thu, Nov 28, 2019 at 5:03 PM Thierry Reding wrote: > > From: Thierry Reding > > This is a result of looking into a more formal way of doing what was > proposed here: > > http://patchwork.ozlabs.org/patch/1145363/ > > The Tegra DRM driver is written such that runtime PM controls all >

Re: [PATCH 1/2] PM / runtime: Allow drivers to override runtime PM behaviour on sleep

2019-11-28 Thread Thierry Reding
On Thu, Nov 28, 2019 at 05:14:51PM +0100, Rafael J. Wysocki wrote: > On Thu, Nov 28, 2019 at 5:03 PM Thierry Reding > wrote: > > > > From: Thierry Reding > > > > Currently the driver PM core will automatically acquire a runtime PM > > reference for devices before system sleep is entered. This

[Bug 205661] Upgrade to 5.4 from K5.3.13 fails x2 attempts

2019-11-28 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205661 --- Comment #1 from Jani Nikula (jani.nik...@intel.com) --- FYI, the bugzilla is for upstream kernel, not out-of-tree code. -- You are receiving this mail because: You are watching the assignee of the bug.

Re: [PATCH 29/30] drm/bridge: add support for device links to bridge

2019-11-28 Thread Mihail Atanassov
On Tuesday, 26 November 2019 14:35:34 GMT Daniel Vetter wrote: > On Tue, Nov 26, 2019 at 01:16:26PM +, Mihail Atanassov wrote: > > From: Russell King > > > > Bridge devices have been a potential for kernel oops as their lifetime > > is independent of the DRM device that they are bound to.

[PATCH 2/2] drm/tegra: Allow runtime suspend on system sleep

2019-11-28 Thread Thierry Reding
From: Thierry Reding By default the PM core prevents devices from being runtime suspended during system sleep. This is needed to avoid some common cases where devices cannot be properly resumed because their parents are runtime suspended at an unfortunate point in time. However, there are cases

[PATCH 1/2] PM / runtime: Allow drivers to override runtime PM behaviour on sleep

2019-11-28 Thread Thierry Reding
From: Thierry Reding Currently the driver PM core will automatically acquire a runtime PM reference for devices before system sleep is entered. This is needed to avoid potential issues related to devices' parents getting put to runtime suspend at the wrong time and causing problems with their

  1   2   >