Re: [PATCH 02/10] drm: Add Plane Degamma properties

2018-02-19 Thread Daniel Vetter
On Thu, Feb 15, 2018 at 12:32:52AM -0500, Daniele Castagna wrote: > From: "uma.shankar at intel.com (Uma Shankar)" > > Add Plane Degamma as a blob property and plane > degamma size as a range property. > > (am from https://patchwork.kernel.org/patch/10046515/) > >

Re: [PATCH 04/10] drm: Add Plane Gamma properties

2018-02-19 Thread Daniel Vetter
On Thu, Feb 15, 2018 at 12:32:54AM -0500, Daniele Castagna wrote: > From: "uma.shankar at intel.com (Uma Shankar)" > > Add plane gamma as blob property and size as a > range property. > > (am from https://patchwork.kernel.org/patch/9971325/) > > Change-Id:

Re: [PATCH 0/8] drm: Add COLOR_ENCODING and COLOR_RANGE plane properties

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:19PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Here's a refresh of Jyri's COLOR_ENCODING and COLOR_RANGE properties, > and the i915 implementation I did on top. I tossed in a few core > updates as well: plane state dump,

Re: [PATCH v3 3/6] dt-bindings: Add panel-timing subnode to simple-panel

2018-02-19 Thread Thierry Reding
On Thu, Feb 08, 2018 at 12:48:50PM -0500, Sean Paul wrote: > This patch adds a new subnode to simple-panel allowing us to override > the typical timing expressed in the panel's display_timing. > > Changes in v2: > - Split out the binding into a new patch (Rob) > - display-timings is a new

Re: [PATCH 3/8] drm/atomic: Include color encoding/range in plane state dump

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:22PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Include color_enconding and color_range in the plane state dump. > > Cc: Harry Wentland > Cc: Daniel Vetter > Cc: Daniel Stone

Re: [PATCH 1/8] drm: Add optional COLOR_ENCODING and COLOR_RANGE properties to drm_plane

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:20PM +0200, Ville Syrjala wrote: > From: Jyri Sarha > > Add a standard optional properties to support different non RGB color > encodings in DRM planes. COLOR_ENCODING select the supported non RGB > color encoding, for instance ITU-R BT.709 YCbCr.

Re: [PATCH 0/8] drm: Add COLOR_ENCODING and COLOR_RANGE plane properties

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:19PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Here's a refresh of Jyri's COLOR_ENCODING and COLOR_RANGE properties, > and the i915 implementation I did on top. I tossed in a few core > updates as well: plane state dump,

Re: [PATCH v3 1/6] dt-bindings: Clarify timing subnode use as panel-timing

2018-02-19 Thread Thierry Reding
On Thu, Feb 08, 2018 at 12:48:48PM -0500, Sean Paul wrote: > Add a note in the documentation explaining when it's appropriate to use > the display-timings subnode on its own, as well as the preferred name to > use (panel-timing). panel-timing seems rather redundant, but I guess that's the de

[PATCH] drm/todo: Add idr_init_base todo

2018-02-19 Thread Daniel Vetter
Suggested-by: Chris Wilson Cc: Chris Wilson Acked-by: Chris Wilson Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 10 ++ 1 file changed, 10 insertions(+) diff --git

Re: [Intel-gfx] [PATCH] Forward Error Correction is supported on DP 1.4. This patch adds corresponding DPCD register definitions.

2018-02-19 Thread Daniel Vetter
On Tue, Feb 13, 2018 at 11:55:35PM -0800, Anusha Srivatsa wrote: > v2: Add dri-devel mailing list to the CC list(Jani) > > v3: Change names, add missing masks (Manasi) > > v4: Add missing shifts to mask (Manasi) > > v5: Arrange the definitions in ascending order > of the address (Jani) > > Cc:

Re: [Intel-gfx] [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 03:47:42PM +0100, Lukas Wunner wrote: > On Mon, Feb 19, 2018 at 03:05:53PM +0100, Daniel Vetter wrote: > > On Mon, Feb 19, 2018 at 12:58:17PM +0100, Lukas Wunner wrote: > > > On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > > > > Well, userspace expects

Re: [PATCH] drm: Use idr_init_base(1) when using id==0 for invalid

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 02:35:38PM +, Chris Wilson wrote: > Quoting Christian König (2018-02-13 13:48:24) > > Am 12.02.2018 um 18:14 schrieb Ville Syrjälä: > > > On Mon, Feb 12, 2018 at 02:55:33PM +, Chris Wilson wrote: > > >> Use the new idr_init_base() function to create an IDR that

Re: [Intel-gfx] [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Lukas Wunner
On Mon, Feb 19, 2018 at 03:05:53PM +0100, Daniel Vetter wrote: > On Mon, Feb 19, 2018 at 12:58:17PM +0100, Lukas Wunner wrote: > > On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > > > Well, userspace expects hotplug events, even when we runtime suspend > > > stuff. Hence waking

Re: [RFC PULL] Add Display Support for Qualcomm SDM845

2018-02-19 Thread Daniel Vetter
On Tue, Feb 13, 2018 at 03:00:09PM -0500, Rob Clark wrote: > On Tue, Feb 13, 2018 at 2:18 PM, Sean Paul wrote: > > Hi dri-devel, > > Qualcomm has been working for the past few weeks on forward porting their > > downstream drm driver from 4.14 to mainline. Please consider

Re: [PATCH] drm: Handle unexpected holes in color-eviction

2018-02-19 Thread Chris Wilson
Quoting Joonas Lahtinen (2018-02-19 14:40:32) > Quoting Chris Wilson (2018-02-19 13:35:43) > > +++ b/drivers/gpu/drm/drm_mm.c > > @@ -836,9 +836,24 @@ struct drm_mm_node *drm_mm_scan_color_evict(struct > > drm_mm_scan *scan) > > if (!mm->color_adjust) > > return NULL; > >

Re: [PATCH] drm: Handle unexpected holes in color-eviction

2018-02-19 Thread Joonas Lahtinen
Quoting Chris Wilson (2018-02-19 13:35:43) > +++ b/drivers/gpu/drm/drm_mm.c > @@ -836,9 +836,24 @@ struct drm_mm_node *drm_mm_scan_color_evict(struct > drm_mm_scan *scan) > if (!mm->color_adjust) > return NULL; > > - hole = list_first_entry(>hole_stack,

Re: [PATCH] drm: Use idr_init_base(1) when using id==0 for invalid

2018-02-19 Thread Chris Wilson
Quoting Christian König (2018-02-13 13:48:24) > Am 12.02.2018 um 18:14 schrieb Ville Syrjälä: > > On Mon, Feb 12, 2018 at 02:55:33PM +, Chris Wilson wrote: > >> Use the new idr_init_base() function to create an IDR that knows id==0 > >> is never allocated as it maps to an invalid identifier.

Re: [PATCH v3 6/6] arm64: dts: rockchip: Specify override mode for kevin panel

2018-02-19 Thread Enric Balletbo Serra
2018-02-08 18:48 GMT+01:00 Sean Paul : > This patch adds an override mode for kevin devices. The mode increases > both back porches to allow a pixel clock of 2kHz as opposed to the > 'typical' value of 252750kHz. This is needed to avoid interference with > the touch

Re: [RFC] drm/atomic: Abuse legacy cursor update flag for legacy gamma update too

2018-02-19 Thread Daniel Vetter
On Tue, Feb 13, 2018 at 10:12:01AM +0100, Maarten Lankhorst wrote: > Programs like redshift set the legacy gamma for X.org every 5 seconds. > Because atomic commits wait for vblank completion, we get a frame drop > every 5 seconds because of the legacy gamma update. > > Work around this by

[Bug 105163] *-symbol-checks hard-code the nm executable

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105163 Bug ID: 105163 Summary: *-symbol-checks hard-code the nm executable Product: DRI Version: XOrg git Hardware: All OS: Linux (All) Status: NEW Severity:

Re: [PATCH v3 5/6] drm/panel: simple: Use display_timing for lq123p1jx31

2018-02-19 Thread Enric Balletbo Serra
Hi, 2018-02-08 18:48 GMT+01:00 Sean Paul : > Convert the sharp lq123p1jx31 from using a fixed mode to specifying a > display timing with min/typ/max values. This allows us to capture the > timings set forth in the datasheet as well as the additional values that > we've

Re: [PATCH v3 4/6] drm/panel: simple: Add ability to override typical timing

2018-02-19 Thread Enric Balletbo Serra
Hi, 2018-02-08 18:48 GMT+01:00 Sean Paul : > This patch adds the ability to override the typical display timing for a > given panel. This is useful for devices which have timing constraints > that do not apply across the entire display driver (eg: to avoid > crosstalk

Re: [RFC PATCH v2 2/6] dt-bindings: add bindings for Samsung micro-USB 11-pin connector

2018-02-19 Thread Rob Herring
On Thu, Feb 15, 2018 at 11:39:16AM +0100, Andrzej Hajda wrote: > Samsung micro-USB 11-pin connector beside standard micro-USB pins, > has pins dedicated to route MHL traffic. > > Signed-off-by: Andrzej Hajda > --- > .../connector/samsung,usb-connector-11pin.txt | 51 >

Re: [PATCH] drm/simple_kms_helper: Fix NULL pointer dereference with no active CRTC

2018-02-19 Thread Daniel Vetter
On Tue, Feb 13, 2018 at 10:44:16AM +0200, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko > > It is possible that drm_simple_kms_plane_atomic_check called > with no CRTC set, e.g. when user-space application sets CRTC_ID/FB_ID > to 0 before doing

Re: [RFC PATCH v2 1/6] dt-bindings: add bindings for USB physical connector

2018-02-19 Thread Rob Herring
On Thu, Feb 15, 2018 at 11:39:15AM +0100, Andrzej Hajda wrote: > These bindings allow to describe most known standard USB connectors > and it should be possible to extend it if necessary. > USB connectors, beside USB can be used to route other protocols, > for example UART, Audio, MHL. In such

Re: [PATCH RFC v3 4/6] drm: drm_fourcc: Add new formats for Xilinx IPs

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:54PM -0800, Hyun Kwon wrote: > This patch adds new formats needed by Xilinx IP. Pixels are not > byte-aligned in these formats, and the drm_format_info for these > formats has macro-pixel information. > > Signed-off-by: Jeffrey Mouroux >

Re: [PATCH] gpu: ipu-v3: make const arrays int_reg static, shrinks object size

2018-02-19 Thread Philipp Zabel
Hi Colin, On Wed, 2018-02-14 at 18:45 +, Colin King wrote: > From: Colin Ian King > > Don't populate the const read-only arrays int_reg on the stack but instead > make them static. Makes the object code smaller by over 80 bytes: > > Before: >text data

Re: [PATCH] gpu: ipu-v3: prg: fix device node leak in ipu_prg_lookup_by_phandle

2018-02-19 Thread Philipp Zabel
Hi Tobias, On Thu, 2018-02-15 at 15:35 +0100, Tobias Jordan wrote: > Before returning, call of_node_put() for the device node returned by > of_parse_phandle(). > > Fixes: ea9c260514c1 ("gpu: ipu-v3: add driver for Prefetch Resolve Gasket") > Signed-off-by: Tobias Jordan

Re: [Intel-gfx] [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 12:58:17PM +0100, Lukas Wunner wrote: > On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > > On Sun, Feb 11, 2018 at 10:38:28AM +0100, Lukas Wunner wrote: > > > Fix a deadlock on hybrid graphics laptops that's been present since 2013: > > > > > > DRM drivers

Re: [PATCH v5 05/12] drm/bridge/synopsys: dw-hdmi: don't clobber drvdata

2018-02-19 Thread Philipp Zabel
On Wed, 2018-02-14 at 21:08 +0100, Jernej Skrabec wrote: > dw_hdmi shouldn't set drvdata since some drivers might need to store > it's own data there. Rework dw_hdmi in a way to return struct dw_hdmi > instead to store it in drvdata. This way drivers are responsible to > store and pass structure

Re: [PATCH] s390/console: enable dummy console for vt

2018-02-19 Thread Christian Borntraeger
On 02/19/2018 02:35 PM, Farhan Ali wrote: > > > On 02/15/2018 07:02 AM, Christian Borntraeger wrote: >> >> >> On 02/15/2018 12:57 PM, Thomas Huth wrote: >>> On 15.02.2018 12:26, Geert Uytterhoeven wrote: Hi Christian, On Thu, Feb 15, 2018 at 12:14 PM, Christian Borntraeger

Re: [PATCH v2 4/4] drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

2018-02-19 Thread Daniel Vetter
On Mon, Feb 12, 2018 at 02:51:44PM -0500, Joe Moriarty wrote: > The Parfait (version 2.1.0) static code analysis tool found the > following NULL pointer derefernce problem. > > - drivers/gpu/drm/drm_vblank.c > Null pointer checks were added to return values from calls to > drm_crtc_from_index().

[PATCH libdrm] meson: do not use cairo/valgrind if disabled

2018-02-19 Thread Igor Gnatenko
-Dcairo-tests=false currently results into enabling cairo support if it was found. Same for valgrind. v2: * Use underscore-prefixed variables to not change type of variable * Use empty array for "fake" dependency instead of real empty object v3: * Fix typo Reviewed-by: Eric Engestrom

Re: [PATCH libdrm] meson: do not use cairo/valgrind if disabled

2018-02-19 Thread Igor Gnatenko
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Mon, 2018-02-19 at 12:15 +, Eric Engestrom wrote: > On Sunday, 2018-02-18 14:00:50 +0100, Igor Gnatenko wrote: > > -Dcairo-tests=false currently results into enabling cairo support if it > > was found. Same for valgrind. > > Indeed, this was

[PATCH libdrm] meson: do not use cairo/valgrind if disabled

2018-02-19 Thread Igor Gnatenko
-Dcairo-tests=false currently results into enabling cairo support if it was found. Same for valgrind. v2: * Use underscore-prefixed variables to not change type of variable * Use empty array for "fake" dependency instead of real empty object Signed-off-by: Igor Gnatenko

[Bug 104345] Playing video hangs X-Server with showing scrambled picture, sound still playing.

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104345 --- Comment #10 from bernhardu --- Same happens with vanilla kernel 4.15.2. -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list

Re: [PATCH RFC 4/9] drm/omap: Make omapdss API more generic

2018-02-19 Thread Tomi Valkeinen
Hi Jyri, On 16/02/18 13:25, Jyri Sarha wrote: > The new omapdss API is HW independent and cleans up some of the DSS5 > specific hacks from the omapdrm side and gets rid off the DSS5 IRQ > register bits and replace them with HW independent generic u64 based > macros. The new macros make it more

Re: [Intel-gfx] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Lukas Wunner
On Mon, Feb 19, 2018 at 12:48:04PM +0100, Daniel Vetter wrote: > On Thu, Feb 15, 2018 at 06:38:44AM +0100, Lukas Wunner wrote: > > On Wed, Feb 14, 2018 at 09:58:43AM -0500, Sean Paul wrote: > > > On Wed, Feb 14, 2018 at 03:43:56PM +0100, Michel Dänzer wrote: > > > > On 2018-02-14 03:08 PM, Sean

Re: [PATCH libdrm] meson: do not use cairo/valgrind if disabled

2018-02-19 Thread Eric Engestrom
On Sunday, 2018-02-18 14:00:50 +0100, Igor Gnatenko wrote: > -Dcairo-tests=false currently results into enabling cairo support if it > was found. Same for valgrind. Indeed, this was wrong; thanks for the fix! Reviewed-by: Eric Engestrom Do you have commit access, or

Re: [PATCH RFC 5/9] drm/omap: move common stuff from dss.h to omapdss.h

2018-02-19 Thread Tomi Valkeinen
Hi Jyri, On 16/02/18 13:25, Jyri Sarha wrote: > The new DSS6 driver needs some structs and defines which are currently > in dss.h, which is for the old DSS driver. > > Move the required structs and defines from dss.h to omapdss.h. > > Signed-off-by: Tomi Valkeinen >

[Bug 105005] No image after downtime (RX460)

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105005 Dmitry changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH v2 3/4] drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

2018-02-19 Thread Daniel Vetter
On Mon, Feb 12, 2018 at 02:51:43PM -0500, Joe Moriarty wrote: > The Parfait (version 2.1.0) static code analysis tool found the > following NULL pointer derefernce problem. > > - drivers/gpu/drm/drm_edid.c > The call to drm_cvt_mode() in function drm_mode_std() for the > HDTV hack resulted in the

Re: [PATCH RFC 1/9] drm/omap: Update omapdss API to allow alternative DSS implementations

2018-02-19 Thread Tomi Valkeinen
Hi Jyri, Laurent, On 16/02/18 13:25, Jyri Sarha wrote: > After this patch OMAP_DSS_BASE module is not including any OMAP2_DSS > headers, only the API omapdss.h. "sturct dss_data", the piece of the > data structure needed for base.c is defined in omapdss.h and added as > a member to struct

Re: [PATCH v2 2/4] drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

2018-02-19 Thread Daniel Vetter
On Mon, Feb 12, 2018 at 02:51:42PM -0500, Joe Moriarty wrote: > The Parfait (version 2.1.0) static code analysis tool found the > following NULL pointer derefernce problem. > > - drivers/gpu/drm/drm_dp_mst_topology.c > The call to drm_dp_calculate_rad() in function drm_dp_port_setup_pdt() > could

Re: [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Lukas Wunner
On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > On Sun, Feb 11, 2018 at 10:38:28AM +0100, Lukas Wunner wrote: > > Fix a deadlock on hybrid graphics laptops that's been present since 2013: > > > > DRM drivers poll connectors in 10 sec intervals. The poll worker is > > stopped on

Re: [PATCH v2 1/4] drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

2018-02-19 Thread Daniel Vetter
On Mon, Feb 12, 2018 at 02:51:41PM -0500, Joe Moriarty wrote: > The Parfait (version 2.1.0) static code analysis tool found the > following NULL pointer dereference problem. > > - drivers/gpu/drm/drm_drv.c > Any calls to drm_minor_get_slot() could result in the return of a NULL > pointer when an

Re: [PATCH v2 4/8] drm/rockchip: dw_hdmi: Allow outputs that don't need output switching

2018-02-19 Thread Heiko Stuebner
Hi Robin, Am Montag, 19. Februar 2018, 12:43:11 CET schrieb Robin Murphy: > Hi Heiko, > > On 16/02/18 20:41, Heiko Stuebner wrote: > > So far we always encountered socs with 2 output crtcs needing the driver > > to tell the hdmi block which output to connect to. But there also exist > > socs

Re: [Intel-gfx] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Daniel Vetter
On Thu, Feb 15, 2018 at 06:38:44AM +0100, Lukas Wunner wrote: > On Wed, Feb 14, 2018 at 09:58:43AM -0500, Sean Paul wrote: > > On Wed, Feb 14, 2018 at 03:43:56PM +0100, Michel Dänzer wrote: > > > On 2018-02-14 03:08 PM, Sean Paul wrote: > > > > On Wed, Feb 14, 2018 at 10:26:35AM +0100, Maarten

[PATCH] drm/tinydrm: Remove chunk splitting in tinydrm_spi_transfer

2018-02-19 Thread Meghana Madhyastha
-Remove chunk splitting in tinydrm_spi_transfer in tinydrm-helpers as The spi core will split a buffer into max_dma_len chunks for the spi controller driver to handle. -Remove automatic byte swapping in tinydrm_spi_transfer as it doesn't have users. -Remove the upper bound check on dma transfer

Re: [PATCH v2 7/8] drm/rockchip: dw_hdmi: store rockchip_hdmi reference in phy_data object

2018-02-19 Thread Robin Murphy
On 16/02/18 20:41, Heiko Stuebner wrote: When using special phy handling operations we'll often need access to the rockchip_hdmi struct. As the chip-data that occupies the phy_data pointer initially gets assigned to the rockchip_hdmi struct we can not s/not/now/ (or s/not//) ? Again, this

Re: [PATCH v2 4/8] drm/rockchip: dw_hdmi: Allow outputs that don't need output switching

2018-02-19 Thread Robin Murphy
Hi Heiko, On 16/02/18 20:41, Heiko Stuebner wrote: So far we always encountered socs with 2 output crtcs needing the driver to tell the hdmi block which output to connect to. But there also exist socs with only one crtc like the rk3228, rk3328 and rk3368. So adapt the register field to simply

[PATCH] drm: Handle unexpected holes in color-eviction

2018-02-19 Thread Chris Wilson
During eviction, the driver may free more than one hole in the drm_mm due to the side-effects in evicting the scanned nodes. However, drm_mm_scan_color_evict() expects that the scan result is the first available hole (in the mru freed hole_stack list): kernel BUG at

Re: [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Daniel Vetter
On Sun, Feb 11, 2018 at 10:38:28AM +0100, Lukas Wunner wrote: > Fix a deadlock on hybrid graphics laptops that's been present since 2013: > > DRM drivers poll connectors in 10 sec intervals. The poll worker is > stopped on ->runtime_suspend with cancel_delayed_work_sync(). However > the poll

Re: [PATCH RFC v3 3/6] drm: fourcc: Add drm_format_plane_width_bytes()

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:53PM -0800, Hyun Kwon wrote: > drm_format_plane_width_bytes() calculates and returns the number of bytes > for given width of specified format. The calculation uses @cpp > in drm format info for byte-aligned formats. If the format isn't > byte-aligned, @cpp should 0,

Re: [PATCH RFC v3 2/6] drm: drm_fourcc: Introduce macro-pixel info to drm_format_info

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:52PM -0800, Hyun Kwon wrote: > Multiple pixels can be grouped as a single unit and form a 'macro-pixel'. > This is to model formats where multiple non-byte aligned pixels are stored > together in a byte-aligned way. For example, if 3 - 10 bit > pixels are stored in 32

Re: [PATCH RFC v3 1/6] drm: fourcc.h: Use inline kern-doc style for struct drm_format_info

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:51PM -0800, Hyun Kwon wrote: > Use the inline kern-doc style for struct drm_format_info for better > readability. This is just a preliminary change for further table update. > > Signed-off-by: Hyun Kwon > --- > v3 > - This is added > --- > ---

Re: [PATCH] drm/edid: Add 6 bpc quirk for CPT panel in Asus UX303LA

2018-02-19 Thread Jani Nikula
On Sun, 18 Feb 2018, Kai-Heng Feng wrote: > Similar to commit e10aec652f31 ("drm/edid: Add 6 bpc quirk for display > AEO model 0."), the EDID reports "DFP 1.x compliant TMDS" but it support > 6bpc instead of 8 bpc. > > Hence, use 6 bpc quirk for this panel. > > Fixes:

[PATCH] drm: add documentation for tv connector state margins

2018-02-19 Thread Jani Nikula
A bit boring documentation fix, but gets rid of the warnings: ./include/drm/drm_connector.h:370: warning: Function parameter or member 'margins.left' not described in 'drm_tv_connector_state' ./include/drm/drm_connector.h:370: warning: Function parameter or member 'margins.right' not described

[4.9-stable 2/9] drm/i915: hide unused intel_panel_set_backlight function

2018-02-19 Thread Arnd Bergmann
commit fd94d53e55bd487368dfee9f1af24da78b2bb582 upstream. Building i915 without backlight support results in a harmless warning for intel_panel_set_backlight: drivers/gpu/drm/i915/intel_panel.c:653:13: error: 'intel_panel_set_backlight' defined but not used [-Werror=unused-function] This moves

[Bug 105156] kdg2kfd_probe failed

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105156 Oded Gabbay changed: What|Removed |Added Status|NEW |ASSIGNED -- You

[Bug 105156] kdg2kfd_probe failed

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105156 Oded Gabbay changed: What|Removed |Added Assignee|dri-devel@lists.freedesktop

Re: kdg2kfd_probe failed

2018-02-19 Thread Oded Gabbay
HI Sen, Those messages are expected and you can ignore them. In kernel 4.14 a couple of warning/error messages were added to amdkfd. If you use the amdgpu driver you will not see them but because you use radeon you are seeing them. In kernel 4.15 we removed support of amdkfd from radeon, so once

Re: [PATCH 0/6] Add support for in-line nested struct comments

2018-02-19 Thread Jani Nikula
On Sun, 18 Feb 2018, Jonathan Corbet wrote: > On Fri, 16 Feb 2018 11:48:14 -0200 > Mauro Carvalho Chehab wrote: > >> his series fix two bugs at kernel-doc.rst examples and add support >> for in-line nested struct comments. >> >> It also converts one

Re: [PATCH] gpu/drm/udl: Replace struct_mutex with driver private lock

2018-02-19 Thread Daniel Vetter
On Sat, Feb 10, 2018 at 06:47:31PM +0530, Shreeya Patel wrote: > On Fri, 2018-02-09 at 12:18 +, Chris Wilson wrote: > > Quoting Shreeya Patel (2018-02-09 12:10:56) > > > > > > dev->struct_mutex is the Big DRM Lock and the only bit where > > > it’s mandatory is serializing GEM buffer object

Re: [PATCH 1/3] drm: add func to get max iomem address v2

2018-02-19 Thread Daniel Vetter
On Fri, Feb 16, 2018 at 02:40:11PM +, Chris Wilson wrote: > Quoting Chunming Zhou (2018-02-09 02:44:08) > > it will be used to check if the driver needs swiotlb > > v2: Don't use inline, instead, move function to drm_memory.c (Mechel > > Daenzer ) > > > > Change-Id:

[PATCH 4/4] drm/ttm: keep BOs reserved until end of eviction

2018-02-19 Thread Christian König
This avoids problems when BOs are evicted but directly moved back into the domain from other threads. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 37 + 1 file changed, 29 insertions(+), 8 deletions(-) diff

[PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout.

2018-02-19 Thread Christian König
This solves the problem that when we swapout a BO from a domain we sometimes couldn't make room for it because holding the lock blocks all other BOs with this reservation object. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 33

[PATCH 2/4] drm/amdgpu: use new ww_mutex_is_owned_by function

2018-02-19 Thread Christian König
Instead of accessing ww_mutex internals directly use the provided function to check if the ww_mutex was indeed locked by the current command submission. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 ++- 1 file changed, 2 insertions(+),

[PATCH 1/4] locking/ww_mutex: add ww_mutex_is_owned_by function

2018-02-19 Thread Christian König
amdgpu needs to verify if userspace sends us valid addresses and the simplest way of doing this is to check if the buffer object is locked with the ticket of the current submission. Clean up the access to the ww_mutex internals by providing a function for this and extend the check to the thread

Re: [PATCH] dma-buf/sw_sync: Fix kerneldoc warnings

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 11:38:16AM +, Chris Wilson wrote: > drivers/dma-buf/sw_sync.c:248: warning: No description found for parameter > 'obj' > drivers/dma-buf/sw_sync.c:248: warning: No description found for parameter > 'value' > drivers/dma-buf/sw_sync.c:248: warning: Excess function

Re: [PATCH] drm: Fix kerneldoc warnings for drm_lease

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 11:08:17AM +, Chris Wilson wrote: > drivers/gpu/drm/drm_lease.c:56: warning: No description found for parameter > 'lessee_id' > drivers/gpu/drm/drm_lease.c:56: warning: Excess function parameter 'id' > description in '_drm_find_lessee' >

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-19 Thread Rafael J. Wysocki
On Sun, Feb 18, 2018 at 9:38 AM, Lukas Wunner wrote: > PCI devices not bound to a driver are supposed to stay in D0 during > runtime suspend. But they may have a parent which is bound and can be > transitioned to D3cold at runtime. Once the parent goes to D3cold, the > unbound

Re: [PATCH 1/3] drm: add func to get max iomem address

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 02:00:15PM +0800, Chunming Zhou wrote: > it will be used to check if the driver needs swiotlb > > Change-Id: Idbe47af8f12032d4803bb3d47273e807f19169c3 > Signed-off-by: Chunming Zhou > --- > include/drm/drm_cache.h | 13 + > 1 file

Re: [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()

2018-02-19 Thread Joonas Lahtinen
Quoting Jani Nikula (2018-02-19 11:34:34) > On Fri, 16 Feb 2018, Bjorn Helgaas wrote: > > On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote: > >> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as > >> where a PCI device is present. This restricts

Re: [PATCH] drm/bochs: make structure bochs_bo_driver static

2018-02-19 Thread Daniel Vetter
On Wed, Feb 07, 2018 at 11:13:53AM +, Colin King wrote: > From: Colin Ian King > > The structure bochs_bo_driver is local to the source and does not need to > be in global scope, so make it static. > > Cleans up sparse warning: >

Re: [PATCH v5 1/5] drm: xlnx: Xilinx DRM KMS module

2018-02-19 Thread Daniel Vetter
On Tue, Feb 06, 2018 at 05:36:36PM -0800, Hyun Kwon wrote: > Xilinx has various platforms for display, where users can create > using multiple IPs in the programmable FPGA fabric, or where > some hardened piepline is available on the chip. Furthermore, > hardened pipeline can also interact with

Re: Questions on page flips and atomic modeset

2018-02-19 Thread Oleksandr Andrushchenko
On 02/19/2018 11:36 AM, Daniel Vetter wrote: On Tue, Feb 06, 2018 at 11:59:37AM +0200, Oleksandr Andrushchenko wrote: Hello, Ville! Thank you very much for such a comprehensive answer. Just noticed a few of your questions threads. We'd very much appreciate (for the next person creating an

Re: [PATCH v3 33/43] drm/panel: simple: Change mode for Sharp lq123p1jx31

2018-02-19 Thread Enric Balletbo Serra
Hi, 2018-02-16 21:54 GMT+01:00 Doug Anderson : > Hi, > > On Fri, Feb 16, 2018 at 4:34 AM, Enric Balletbo Serra > wrote: >> Hi, >> >> 2018-01-31 17:52 GMT+01:00 Doug Anderson : >>> Hi, >>> >>> >>> On Wed, Jan 31, 2018 at 7:16 AM,

Re: [PATCH 1/3] drm: simple_kms_helper: Add mode_valid() callback support

2018-02-19 Thread Daniel Vetter
On Sat, Feb 10, 2018 at 05:08:34PM +, Eric Anholt wrote: > Linus Walleij writes: > > > The PL111 needs to filter valid modes based on memory bandwidth. > > I guess it is a pretty simple operation, so we can still claim > > the DRM KMS helper pipeline is simple after

Re: Questions on page flips and atomic modeset

2018-02-19 Thread Daniel Vetter
On Tue, Feb 06, 2018 at 11:59:37AM +0200, Oleksandr Andrushchenko wrote: > Hello, Ville! > > Thank you very much for such a comprehensive answer. Just noticed a few of your questions threads. We'd very much appreciate (for the next person creating an atomic driver) if you check out our kernel

Re: Thinkpad X1 Carbon 3rd - Reducing the compressed framebuffer size

2018-02-19 Thread Pali Rohár
On Tuesday 13 February 2018 19:45:56 Ville Syrjälä wrote: > On Tue, Feb 13, 2018 at 06:43:41PM +0100, Pali Rohár wrote: > > On Tuesday 13 February 2018 18:12:21 Ville Syrjälä wrote: > > > On Tue, Feb 13, 2018 at 05:04:37PM +0100, Pali Rohár wrote: > > > > So it can be done only once after reboot?

Re: [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()

2018-02-19 Thread Jani Nikula
On Fri, 16 Feb 2018, Bjorn Helgaas wrote: > On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote: >> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as >> where a PCI device is present. This restricts the device drivers to be >> reused for other

Re: [PATCH] drm/nouveau: prefer XBGR2101010 for addfb ioctl

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 10:21:54AM +0100, Daniel Vetter wrote: > On Mon, Feb 05, 2018 at 09:10:12AM -0500, Ilia Mirkin wrote: > > On Mon, Feb 5, 2018 at 8:58 AM, Ville Syrjälä > > wrote: > > > On Sat, Feb 03, 2018 at 02:11:23PM -0500, Ilia Mirkin wrote: > > >>

Re: [PATCH] drm/nouveau: prefer XBGR2101010 for addfb ioctl

2018-02-19 Thread Daniel Vetter
On Mon, Feb 05, 2018 at 09:10:12AM -0500, Ilia Mirkin wrote: > On Mon, Feb 5, 2018 at 8:58 AM, Ville Syrjälä > wrote: > > On Sat, Feb 03, 2018 at 02:11:23PM -0500, Ilia Mirkin wrote: > >> Nouveau only exposes support for XBGR2101010. Prior to the atomic > >>

Re: [PATCH] drm/ast: Add option to initialize palette on driver load

2018-02-19 Thread Daniel Vetter
On Fri, Feb 02, 2018 at 01:59:51AM -0600, Timothy Pearson wrote: > > Non-x86, such as OpenPOWER and ARM machines, do not execute the > ASPEED-provided > option ROM on system start. As a result, the VGA palette registers remain > uninitialized, leading to odd colors and generally hard to read

Re: [PATCH] drm/fb-helper: Scale back depth to supported maximum

2018-02-19 Thread Daniel Vetter
On Fri, Feb 02, 2018 at 04:03:43PM +0200, Ville Syrjälä wrote: > On Fri, Feb 02, 2018 at 02:56:30PM +0100, Linus Walleij wrote: > > On Thu, Feb 1, 2018 at 2:19 PM, Ville Syrjälä > > wrote: > > > [Me] > > >> + /* > > >> + * If we run into a situation where,

Re: [PATCH 1/4] drm/simple_kms_helper: Add {enable|disable}_vblank callback support

2018-02-19 Thread Oleksandr Andrushchenko
ping On 02/12/2018 10:52 AM, Oleksandr Andrushchenko wrote: From: Oleksandr Andrushchenko If simple_kms_helper based driver needs to work with vblanks, then it has to provide drm_driver.{enable|disable}_vblank callbacks, because

Re: [RFC v2 02/10] drm: drm_fourcc: Introduce macro-pixel info to drm_format_info

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 06:19:42PM -0800, Hyun Kwon wrote: > Hi Daniel, > > On Tue, 2018-01-30 at 02:27:07 -0800, Daniel Vetter wrote: > > On Thu, Jan 25, 2018 at 06:03:59PM -0800, Hyun Kwon wrote: > > > Multiple pixels can be grouped as a single unit and form a 'macro-pixel'. > > > This is to

Re: [RFC v2 02/10] drm: drm_fourcc: Introduce macro-pixel info to drm_format_info

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 06:19:32PM -0800, Hyun Kwon wrote: > Hi Daniel, > > On Tue, 2018-01-30 at 02:22:40 -0800, Daniel Vetter wrote: > > On Thu, Jan 25, 2018 at 06:03:59PM -0800, Hyun Kwon wrote: > > > Multiple pixels can be grouped as a single unit and form a 'macro-pixel'. > > > This is to

[PATCH 1/2] drm/mm: Fix caching of leftmost node in the interval tree

2018-02-19 Thread Chris Wilson
When we descend the tree to find our slot, if we step to the right, we are no longer the leftmost node. Fixes: f808c13fd373 ("lib/interval_tree: fast overlap detection") Signed-off-by: Chris Wilson Cc: Davidlohr Bueso Cc: Jérôme Glisse

[PATCH 2/2] drm/mm: Micro-optimise updating the upper layers of the interval tree

2018-02-19 Thread Chris Wilson
When we insert a node into a hole inside the interval tree, we need to climb the layers above us to update the cached interval. When doing so, we know that the initial node exists as it is our starting hole. add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-10 (-10) Function

[Bug 198123] Console is the wrong color at boot with radeon 6670

2018-02-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198123 --- Comment #31 from Daniel Vetter (dan...@ffwll.ch) --- Created attachment 274229 --> https://bugzilla.kernel.org/attachment.cgi?id=274229=edit another test patch for Deposite Pirate -- You are receiving this mail because: You are watching

[Bug 198123] Console is the wrong color at boot with radeon 6670

2018-02-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198123 --- Comment #30 from Daniel Vetter (dan...@ffwll.ch) --- There was not a single backtrace in that attachement. Was this only up to fbcon, or did you start X too? Where the colors correct afterwards? It's also strange that it fails to compile on

kdg2kfd_probe failed

2018-02-19 Thread Sen Dion
For some unknown reason, I started to get the "kgd2kfd_probe failed" message after the update of Fedora-27 (limited to security advisories). Here is dmesg output: Console: switching to colour frame buffer device 240x67 radeon :01:05.0: fb0: radeondrmfb frame buffer device kfd kfd: DID

<    1   2