Re: [PATCH] backlight: adp8860: Mark expected switch fall-through

2018-07-09 Thread Lee Jones
On Mon, 09 Jul 2018, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Signed-off-by: Gustavo A. R. Silva > --- > drivers/video/backlight/adp8860_bl.c | 1 + > 1 file changed, 1 insertion(+) Looks

Re: [PATCH v5 19/40] drm/i915: hdcp_check_link only on CP_IRQ

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:10:08PM +0530, Ramalingam C wrote: > HDCP check link is invoked only on CP_IRQ detection, instead of all > short pulses. > > v3: > No Changes. > v4: > Added sean in cc and collected the reviewed-by received. > v5: > No Change. > > Signed-off-by: Ramalingam C

Re: [Intel-gfx] [PATCH v5 13/40] drm/i915: Implement HDCP2.2 Enable and Disable

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:10:02PM +0530, Ramalingam C wrote: > Implements a sequence of enabling and disabling the HDCP2.2 > (auth and encryption). This is really hard to review, since all I see are stubs. I'd much rather have each patch do something useful, instead of just call stubs. That

Re: [PATCH 21/21] arm64: dts: qcom: sdm845: Add dpu to sdm845 dts file

2018-07-09 Thread Sean Paul
On Mon, Jul 09, 2018 at 12:07:11PM -0600, Rob Herring wrote: > On Mon, Jul 9, 2018 at 11:40 AM Sean Paul wrote: > > > > Signed-off-by: Sean Paul > > --- > > arch/arm64/boot/dts/qcom/sdm845.dtsi | 194 +++ > > 1 file changed, 194 insertions(+) > > > > diff --git

Re: [Intel-gfx] [PATCH v5 10/40] drm/i915: Pullout the bksv read and validation

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:59PM +0530, Ramalingam C wrote: > For reusability purpose, this patch implements the hdcp1.4 bksv's > read and validation as a functions. > > For detecting the HDMI panel's HDCP capability this fucntions will be > used. > > v2: > Rebased. > v3: > No Changes. >

Re: [Intel-gfx] [PATCH v5 02/40] drm: HDMI and DP specific HDCP2.2 defines

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:51PM +0530, Ramalingam C wrote: > This patch adds HDCP register definitions for HDMI and DP HDCP > adaptations. > > HDMI specific HDCP2.2 register definitions are added into drm_hdcp.h, > where as HDCP2.2 register offsets in DPCD offsets are defined at >

Re: [PATCH v5 11/40] drm/i915: Enable superior HDCP ver that is capable

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:10:00PM +0530, Ramalingam C wrote: > Considering that HDCP2.2 is more secure than HDCP1.4, When a setup > supports HDCP2.2 and HDCP1.4, HDCP2.2 will be enabled. > > v2: > Included few optimization suggestions [Chris Wilson] > Commit message is updated as per the

Re: [Intel-gfx] [PATCH v5 12/40] drm/i915: Enable HDCP1.4 incase of HDCP2.2 failure

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:10:01PM +0530, Ramalingam C wrote: > When HDCP2.2 enabling fails and HDCP1.4 is supported, HDCP1.4 is > enabled. > Just squash this into patch 11, no need for a separate patch. > v2: > Rebased. > v3: > No Changes. > v4: > Reviewed-by is collected. > v5: > No

Re: [PATCH v5 05/40] drm/i915: wrapping all hdcp var into intel_hdcp

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:54PM +0530, Ramalingam C wrote: > Considering significant number of HDCP specific variables, it will > be clean to have separate struct for HDCP. > > New structure called intel_hdcp is added within intel_connector. > > v2: > struct hdcp statically allocated. [Sean

Re: [Intel-gfx] [PATCH v5 06/40] drm/i915: Define HDCP2.2 related variables

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:55PM +0530, Ramalingam C wrote: > For upcoming implementation of HDCP2.2 in I915, important variable > required for HDCP2.2 are defined. Please just introduce them when you use them. I can't provide useful review on this patch unless I can see how the variables are

Re: [Intel-gfx] [PATCH v5 07/40] drm/i915: Define Intel HDCP2.2 registers

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:56PM +0530, Ramalingam C wrote: > Intel HDCP2.2 registers are defined with addr offsets and bit details. > > v2: > Replaced the arith calc with _PICK [Sean Paul] > v3: > No changes. > v4: > %s/HDCP2_CTR_DDI/HDCP2_CTL_DDI [Uma] > v5: > Added parentheses for

Re: [PATCH v5 20/40] drm/i915: Check HDCP 1.4 and 2.2 link on CP_IRQ

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:10:09PM +0530, Ramalingam C wrote: > On DP HDCP1.4 and 2.2, when CP_IRQ is received, start the link > integrity check for the HDCP version that is enabled. > > v2: > Rebased. Function name is changed. > v3: > No Changes. > v4: > No Changes. > v5: > No Changes. >

Re: [PATCH 10/12] pci: use for_each_if

2018-07-09 Thread Bjorn Helgaas
On Mon, Jul 09, 2018 at 10:36:48AM +0200, Daniel Vetter wrote: > Avoids the inverted condition compared to the open-coded version. > > Signed-off-by: Daniel Vetter > Cc: Bjorn Helgaas > Cc: linux-...@vger.kernel.org Acked-by: Bjorn Helgaas I assume you'll merge this with the rest of the

Re: [PATCH] cpufreq: use for_each_if

2018-07-09 Thread Rafael J. Wysocki
On Mon, Jul 9, 2018 at 6:11 PM, Daniel Vetter wrote: > Avoids the inverted condition compared to the open coded version. > > Signed-off-by: Daniel Vetter > Cc: "Rafael J. Wysocki" > Cc: Viresh Kumar > Cc: linux...@vger.kernel.org > Cc: Eric Engestrom > -- > v2: Fix the logic fumble in the 2nd

Re: [PATCH] kernel.h: Add for_each_if()

2018-07-09 Thread Andrew Morton
On Mon, 9 Jul 2018 18:25:09 +0200 Daniel Vetter wrote: > To avoid compilers complainig about ambigious else blocks when putting > an if condition into a for_each macro one needs to invert the > condition and add a dummy else. We have a nice little convenience > macro for that in drm headers,

Re: [PATCH 21/21] arm64: dts: qcom: sdm845: Add dpu to sdm845 dts file

2018-07-09 Thread Rob Clark
On Mon, Jul 9, 2018 at 2:35 PM, Sean Paul wrote: > On Mon, Jul 09, 2018 at 12:07:11PM -0600, Rob Herring wrote: >> On Mon, Jul 9, 2018 at 11:40 AM Sean Paul wrote: >> > >> > Signed-off-by: Sean Paul >> > --- >> > arch/arm64/boot/dts/qcom/sdm845.dtsi | 194 +++ >> > 1

Re: [Intel-gfx] [PATCH v5 01/40] drm: hdcp2.2 authentication msg definitions

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:50PM +0530, Ramalingam C wrote: > This patch defines the hdcp2.2 protocol messages for authentication. > > v2: > bit_fields are removed. Instead bitmasking used. [Tomas and Jani] > prefix HDCP_2_2_ is added to the macros. [Tomas] > v3: > No Changes. > v4: >

Re: [PATCH v5 09/40] drm/i915: Schedule hdcp_check_link in _intel_hdcp_enable

2018-07-09 Thread Sean Paul
On Wed, Jun 27, 2018 at 02:09:58PM +0530, Ramalingam C wrote: > As a preparation for making the intel_hdcp_enable as common function > for both HDCP1.4 and HDCP2.2, HDCP1.4 check_link scheduling is moved > into _intel_hdcp_enable() function. > > v3: > No Changes. > v4: > Style fix. > v5: >

[git pull] drm fixes for 4.18-rc5

2018-07-09 Thread Dave Airlie
Hey Linus, School holidays here, and I'm not sure if I'll be inclined to work until next week, (I might and if so there might be another fixes), but thought it best to just dequeue the bits I had now. Otherwise the rest of the fixes for rc5 might arrive just before rc5 or just after. This just

Re: [PATCH] fb: fix lost console when the user unplugs a USB adapter

2018-07-09 Thread Mikulas Patocka
On Wed, 4 Jul 2018, Noralf Trønnes wrote: > AFAIU calling unregister_framebuffer() with open fd's is just fine as > long as fb_info with buffers stay intact. All it does is to remove the > fbX from userspace. Cleanup can be done in fb_ops->fb_destroy. > > I have been working on generic fbdev

Re: [PATCH] fb: fix lost console when the user unplugs a USB adapter

2018-07-09 Thread Mikulas Patocka
On Wed, 4 Jul 2018, Bartlomiej Zolnierkiewicz wrote: > On Tuesday, July 03, 2018 01:18:57 PM Mikulas Patocka wrote: > > > > On Tue, 3 Jul 2018, Bartlomiej Zolnierkiewicz wrote: > > > > > Hi, > > > > > > On Sunday, June 03, 2018 11:46:29 AM Mikulas Patocka wrote: > > > > I have a USB display

Re: [linux-sunxi] Re: [PATCH v3 09/24] drm/sun4i: Don't skip TCONs if they don't have channel 0

2018-07-09 Thread Maxime Ripard
On Thu, Jul 05, 2018 at 10:03:35PM +0200, Jernej Škrabec wrote: > Dne četrtek, 05. julij 2018 ob 09:03:58 CEST je Maxime Ripard napisal(a): > > On Sun, Jul 01, 2018 at 11:11:06PM +0800, Chen-Yu Tsai wrote: > > > On Sun, Jul 1, 2018 at 4:27 PM, Jernej Škrabec > wrote: > > > > Dne četrtek, 28.

答复: [PATCH] vgaswitchroo: set audio client id according to bound gpu client id

2018-07-09 Thread Qu, Jim
Hi Lukas, Thanks to your explanation, and see comments in line. Do you need to runtime resume the HDA controller even if user space isn't streaming audio? Why, and in which situation exactly? Jim: OEM system uses pactl to queiry audio card and audio output sink, since the audio has power

Re: [PATCH] drm/sun4i: fix build failure with CONFIG_DRM_SUN8I_MIXER=m

2018-07-09 Thread Chen-Yu Tsai
On Mon, Jul 9, 2018 at 4:07 PM, Maxime Ripard wrote: > On Fri, Jul 06, 2018 at 02:45:53PM +0200, Arnd Bergmann wrote: >> Having DRM_SUN4I built-in but DRM_SUN8I_MIXER as a loadable module results in >> a link error, as we try to access a symbol from the sun8i_tcon_top.ko module: >> >> ERROR:

Re: [PATCH] drm/sun4i: fix build failure with CONFIG_DRM_SUN8I_MIXER=m

2018-07-09 Thread Maxime Ripard
On Mon, Jul 09, 2018 at 04:58:48PM +0800, Chen-Yu Tsai wrote: > On Mon, Jul 9, 2018 at 4:07 PM, Maxime Ripard > wrote: > > On Fri, Jul 06, 2018 at 02:45:53PM +0200, Arnd Bergmann wrote: > >> Having DRM_SUN4I built-in but DRM_SUN8I_MIXER as a loadable module results > >> in > >> a link error, as

Re: [PATCH 11/12] sched: use for_each_if in topology.h

2018-07-09 Thread Peter Zijlstra
On Mon, Jul 09, 2018 at 10:36:49AM +0200, Daniel Vetter wrote: > Avoids complaints from gcc about ambiguous else clauses. Is that a new thing? I'm fairly sure I've never seen it do that, > Signed-off-by: Daniel Vetter > Cc: Andrew Morton > Cc: Peter Zijlstra > --- > include/linux/topology.h

[Bug 107154] [drm] GPU recovery disabled.

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107154 --- Comment #4 from Christian König --- Do you have a full dmesg? -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list dri-devel@lists.freedesktop.org

Re: [PATCH] vgaswitchroo: set audio client id according to bound gpu client id

2018-07-09 Thread Lukas Wunner
On Mon, Jul 09, 2018 at 08:52:33AM +, Qu, Jim wrote: > Now, I found the audio device will auto suspend even if the gpu is active, > and if I plug in a HDMI device it also do not resume back. > > 1. Did you encounter similar issue before? > 2. audio will auto suspend as default at beginning

Re: [PATCH 04/12] cpufreq: use for_each_if

2018-07-09 Thread Eric Engestrom
On Monday, 2018-07-09 10:36:42 +0200, Daniel Vetter wrote: > Avoids the inverted condition compared to the open coded version. > > Signed-off-by: Daniel Vetter > Cc: "Rafael J. Wysocki" > Cc: Viresh Kumar > Cc: linux...@vger.kernel.org > --- > include/linux/cpufreq.h | 8 ++-- > 1 file

Re: [PATCH] MAINTAINERS: update drm tree

2018-07-09 Thread Daniel Stone
Hi, On Mon, 9 Jul 2018 at 08:38, Daniel Vetter wrote: > On Fri, Jul 06, 2018 at 04:43:09PM +0100, Mike Lothian wrote: > > Any change of this moving to https or the gitlab instance where its on as > > default? > > Moving all the drm repos over to gitlab is somewhere on the plans, but we > need to

[PATCH v3 0/2] dt-bindings: mipi-dsi: dual-channel DSI bindings

2018-07-09 Thread Archit Taneja
DSI host controllers these days can be ganged together to drive larger displays. Every SoC vendor supporting this is trying to add their own DT property so that the corresponding drivers can identify that they need to operate in the dual DSI mode. If we use the graph bindings, we don't really need

[PATCH v3 2/2] dt-bindings: mipi-dsi: Add dual-channel DSI related info

2018-07-09 Thread Archit Taneja
Add binding info for peripherals that support dual-channel DSI. Add corresponding optional bindings for DSI host controllers that may be configured in this mode. Add an example of an I2C controlled device operating in dual-channel DSI mode. Reviewed-by: Philippe Cornu Reviewed-by: Sean Paul

[PATCH v3 1/2] dt-bindings: mipi-dsi: Add info about peripherals with non-DSI control bus

2018-07-09 Thread Archit Taneja
Add a section that describes dt-bindings for peripherals that support MIPI DSI, but have a different bus as the primary control bus, or no control bus at all. Add an example for a peripheral with a non-DSI control bus. Reviewed-by: Rob Herring Reviewed-by: Boris Brezillon Reviewed-by: Philippe

Re: [PATCH] drm/sun4i: fix build failure with CONFIG_DRM_SUN8I_MIXER=m

2018-07-09 Thread Chen-Yu Tsai
On Mon, Jul 9, 2018 at 5:15 PM, Maxime Ripard wrote: > On Mon, Jul 09, 2018 at 04:58:48PM +0800, Chen-Yu Tsai wrote: >> On Mon, Jul 9, 2018 at 4:07 PM, Maxime Ripard >> wrote: >> > On Fri, Jul 06, 2018 at 02:45:53PM +0200, Arnd Bergmann wrote: >> >> Having DRM_SUN4I built-in but DRM_SUN8I_MIXER

[Bug 107153] 4.18-rc3 crash on hdmi (0010:dm_update_crtcs_state+0x41e/0x4a0)

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107153 --- Comment #3 from Patrik Kullman --- Also present in dmesg now after leaving the system on: [44775.903197] [drm] amdgpu_dm_irq_schedule_work FAILED src 6 [44777.523222] [drm] amdgpu_dm_irq_schedule_work FAILED src 6 [44778.253228] [drm]

[Bug 107153] 4.18-rc3 crash on hdmi (0010:dm_update_crtcs_state+0x41e/0x4a0)

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107153 --- Comment #4 from Patrik Kullman --- Same with 4.18-rc4: [ 44.170327] [drm] fb mappable at 0x2000A28000 [ 44.170328] [drm] vram apper at 0x20 [ 44.170329] [drm] size 35389440 [ 44.170329] [drm] fb depth is 24 [ 44.170329]

Re: [PATCH] drm/amdgpu/acp: Fix slab-out-of-bounds in mfd_add_device in acp_hw_init

2018-07-09 Thread Agrawal, Akshu
Was this patch ever picked up? I can't find it in agd5f/linux. >>> >>> >>> It wasn't applied. I don't see 51f7415039d4 ("drm/amd/amdgpu: >>> creating two I2S instances for stoney/cz") upstream yet either. >>> Daniel, Vijendar, which ones do you want applied? Can you send me the >>>

Re: [v3 03/10] mfd: mtk-mmsys: Add mmsys driver

2018-07-09 Thread Matthias Brugger
On 05/07/18 14:22, Lee Jones wrote: > On Thu, 05 Jul 2018, Matthias Brugger wrote: >> On 04/07/18 18:45, Lee Jones wrote: >>> On Wed, 04 Jul 2018, Matthias Brugger wrote: On 03/07/18 09:11, Lee Jones wrote: > On Mon, 25 Jun 2018, Matthias Brugger wrote: >> On 30/04/18 12:18, Lee

Re: [RFC PATCH v3 1/2] drm: Add generic colorkey properties for DRM planes

2018-07-09 Thread Dmitry Osipenko
On Friday, 6 July 2018 17:10:10 MSK Ville Syrjälä wrote: > On Fri, Jul 06, 2018 at 04:05:21PM +0300, Dmitry Osipenko wrote: > > On 06.07.2018 15:23, Ville Syrjälä wrote: > > > On Fri, Jul 06, 2018 at 02:11:44PM +0200, Maarten Lankhorst wrote: > > >> Hey, > > >> > > >> Op 04-06-18 om 00:00 schreef

Re: [PATCH v5 5/7] drm/i2c: tda998x: split tda998x_encoder_dpms into enable/disable

2018-07-09 Thread Russell King - ARM Linux
On Wed, May 23, 2018 at 11:31:20AM +0200, Peter Rosin wrote: > This fits better with the drm_bridge callbacks for when this > driver becomes a drm_bridge. > > Suggested-by: Laurent Pinchart > Signed-off-by: Peter Rosin > --- > drivers/gpu/drm/i2c/tda998x_drv.c | 64 >

[PATCH v2] drm/sun4i: Implement zpos for DE2

2018-07-09 Thread Jernej Skrabec
Initial implementation of DE2 planes only supported fixed zpos. Expand implementation with configurable zpos property. Implementation background: Channel in DE2 driver represents one DRM plane, whereas pipe is just mapped channel to known Z position. Pipe 0 will always be at the bottom, pipe 1

[PATCH v5 3/4] dt-bindings: display: amlogic: DT fix s/resets-names/reset-names/

2018-07-09 Thread Geert Uytterhoeven
Fixes: 8cceda5349377e30 ("dt-bindings: Add bindings for the Amlogic Meson dw-hdmi extension") Signed-off-by: Geert Uytterhoeven Acked-by: Rob Herring Acked-by: Neil Armstrong --- v5: - Add Acked-by, v4: - Add Acked-by, v3: - New. ---

[PATCH v2] drm: rcar-du: Add r8a77995 device support

2018-07-09 Thread Ulrich Hecht
From: Koji Matsuoka Add support for the R-Car D3 (R8A77995) SoC to the R-Car DU driver. Signed-off-by: Koji Matsuoka Signed-off-by: Ulrich Hecht --- This is a break-out from the "[PATCH 0/5] R-Car D3 LVDS/HDMI support" series. Changes since v1: - use the .channels_mask field in

[PATCH 4/6] drm/i2c: tda998x: convert to bridge driver

2018-07-09 Thread Russell King
Convert tda998x to a bridge driver with built-in encoder support for compatibility with existing component drivers. Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 147 +++--- 1 file changed, 75 insertions(+), 72 deletions(-) diff --git

[PATCH 15/17] ARM: dts: sun8i: r40: Disable TCONs by default.

2018-07-09 Thread Jernej Skrabec
R40 has 4 TCONs, but only 2 of them can receive some kind of output at the same time. Let's disable them by default, so only those which are really connected on board can be enabled in board dts file. Signed-off-by: Jernej Skrabec --- arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts | 4

[PATCH 07/17] drm/sun4i: DW HDMI: Release nodes if error happens during CRTC search

2018-07-09 Thread Jernej Skrabec
If error happens in sun8i_dw_hdmi_find_possible_crtcs(), nodes are not released with of_node_put() before returning. Fix that by calling of_node_put() when necessary. While on it, clean up the code by using of_graph_get_remote_node() which also lowers number of cases where error handling has to

[PATCH 03/17] ARM: dts: sun8i: r40: Remove fallback display engine compatible

2018-07-09 Thread Jernej Skrabec
R40 has pretty unique display pipeline. Because of that, H3 display engine compatible fallback should be removed. Fixes: 05a43a262d03 ("ARM: dts: sun8i: r40: Add HDMI pipeline") Signed-off-by: Jernej Skrabec --- arch/arm/boot/dts/sun8i-r40.dtsi | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH v5 4/4] dt-bindings: net: btusb: DT fix s/interrupt-name/interrupt-names/

2018-07-09 Thread Geert Uytterhoeven
Fixes: fd913ef7ce619467 ("Bluetooth: btusb: Add out-of-band wakeup support") Signed-off-by: Geert Uytterhoeven Acked-by: Rob Herring --- v4: - Add Acked-by, v3: - New. --- Documentation/devicetree/bindings/net/btusb.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] fbdev/via: fix defined but not used warning

2018-07-09 Thread Randy Dunlap
From: Randy Dunlap Fix a build warning in viafbdev.c when CONFIG_PROC_FS is not enabled by marking the unused function as __maybe_unused. ../drivers/video/fbdev/via/viafbdev.c:1471:12: warning: 'viafb_sup_odev_proc_show' defined but not used [-Wunused-function] Signed-off-by: Randy Dunlap

[PATCH 01/17] dt-bindings: display: sun4i-drm: Add R40 display engine compatible

2018-07-09 Thread Jernej Skrabec
R40 has pretty unique display pipeline. It supports two outputs at the same time. Possible outputs: - 1x HDMI, - 2x TV output - 1x VGA, - 1x MIPI DSI and - 2x LCD outputs That is the biggest number of possible outputs from all Allwinner SoC. Because of that, add new compatible for it.

Re: [PATCH v5 7/7] drm/i2c: tda998x: register as a drm bridge

2018-07-09 Thread Russell King - ARM Linux
On Fri, Jul 06, 2018 at 02:36:37PM +0100, Russell King - ARM Linux wrote: > On Wed, May 23, 2018 at 11:31:22AM +0200, Peter Rosin wrote: > > This makes this driver work with all(?) drivers that are not > > componentized and instead expect to connect to a panel/bridge. That > > said, the only one

Re: [PATCH v5 7/7] drm/i2c: tda998x: register as a drm bridge

2018-07-09 Thread Russell King - ARM Linux
On Wed, May 23, 2018 at 11:31:22AM +0200, Peter Rosin wrote: > This makes this driver work with all(?) drivers that are not > componentized and instead expect to connect to a panel/bridge. That > said, the only one tested is atmel-hlcdc. > > This hooks the relevant work function previously called

Re: [RFC PATCH v3 1/2] drm: Add generic colorkey properties for DRM planes

2018-07-09 Thread Dmitry Osipenko
On Friday, 6 July 2018 18:40:27 MSK Russell King - ARM Linux wrote: > On Fri, Jul 06, 2018 at 05:58:50PM +0300, Dmitry Osipenko wrote: > > On Friday, 6 July 2018 17:10:10 MSK Ville Syrjälä wrote: > > > IIRC my earlier idea was to have different colorkey modes for the > > > min+max and value+mask

[PATCH] video: goldfishfb: fix memory leak on driver remove

2018-07-09 Thread Anton Vasilyev
goldfish_fb_probe() allocates memory for fb, but goldfish_fb_remove() does not have deallocation of fb, which leads to memory leak on probe/remove. The patch adds deallocation into goldfish_fb_remove(). Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Vasilyev

[PATCH v5 2/4] arm64: dts: lg1313: DT fix s/#interrupts-cells/#interrupt-cells/

2018-07-09 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven Acked-by: Chanho Min --- v3: - Add Acked-by, v2: - New. --- arch/arm64/boot/dts/lg/lg1313.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/lg/lg1313.dtsi b/arch/arm64/boot/dts/lg/lg1313.dtsi index

[PATCH v5 1/4] arm64: dts: lg1312: DT fix s/#interrupts-cells/#interrupt-cells/

2018-07-09 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven Acked-by: Rob Herring Acked-by: Chanho Min --- v3: - Add Acked-by, v2: - Add Acked-by, - Rebased. --- arch/arm64/boot/dts/lg/lg1312.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/lg/lg1312.dtsi

[PATCH 02/17] drm/sun4i: Add R40 display engine compatible

2018-07-09 Thread Jernej Skrabec
R40 has versatile display pipeline. It supports two simultanious outputs on various outputs (TVE, VGA, HDMI, MIPI DSI, LCD). Signed-off-by: Jernej Skrabec --- drivers/gpu/drm/sun4i/sun4i_drv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c

Re: [PATCH] drm/cirrus: flip default to 32bpp

2018-07-09 Thread Daniel Vetter
On Fri, Jul 06, 2018 at 02:35:07PM -0400, Adam Jackson wrote: > On Fri, 2018-07-06 at 11:12 +0200, Gerd Hoffmann wrote: > > cirrus can handle 1024x768 (and slightly higher) with 24bpp depth. > > cirrus can handle up to 800x600 with 32bpp. > > 16bpp is maybe a better choice? Nobody's using cirrus

Re: [PATCH] MAINTAINERS: update drm tree

2018-07-09 Thread Daniel Vetter
On Fri, Jul 06, 2018 at 04:43:09PM +0100, Mike Lothian wrote: > Hi > > Any change of this moving to https or the gitlab instance where its on as > default? Moving all the drm repos over to gitlab is somewhere on the plans, but we need to figure out a transition strategy for the scripts and all

[PATCH 08/12] netdev: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids complaints from gcc about ambiguous else clauses. Signed-off-by: Daniel Vetter Cc: "David S. Miller" Cc: net...@vger.kernel.org --- include/linux/netdevice.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index

[PATCH 12/12] usb: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids the inverted condition compared to the open-coded version. Signed-off-by: Daniel Vetter Cc: Greg Kroah-Hartman Cc: linux-...@vger.kernel.org --- include/linux/usb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/usb.h b/include/linux/usb.h index

[Bug 199269] amdgpu/DC on Kaveri results in corrupt color output on HDMI

2018-07-09 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=199269 --- Comment #1 from Bernd Steinhauser (li...@bernd-steinhauser.de) --- Did a brief test on 4.18-rc3 and with that version it seems to be fixed. -- You are receiving this mail because: You are watching the assignee of the bug.

Re: [PATCH v4 2/2] drm/nouveau: tegra: Detach from ARM DMA/IOMMU mapping

2018-07-09 Thread Nicolas Chauvet
2018-05-30 16:06 GMT+02:00 Thierry Reding : > From: Thierry Reding > > Depending on the kernel configuration, early ARM architecture setup code > may have attached the GPU to a DMA/IOMMU mapping that transparently uses > the IOMMU to back the DMA API. Tegra requires special handling for IOMMU >

Re: [RFC PATCH] drm/atomic: add ASYNC_UPDATE flag to the Atomic IOCTL.

2018-07-09 Thread Gustavo Padovan
Hi Ville, On Thu, 2018-06-28 at 16:35 +0300, Ville Syrjälä wrote: > On Wed, Jun 27, 2018 at 11:25:06PM +0200, Enric Balletbo i Serra > wrote: > > From: Gustavo Padovan > > > > This flag tells core to jump ahead the queued update if the > > conditions > > in drm_atomic_async_check() are met.

[PATCH 1/6] drm/i2c: tda998x: find the drm_device via the drm_connector

2018-07-09 Thread Russell King
From: Peter Rosin This prepares for being a drm_bridge which will not register the encoder. That makes the connector the better choice. Reviewed-by: Laurent Pinchart Signed-off-by: Peter Rosin Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 2 +- 1 file changed, 1

[PATCH 04/17] drm/sun4i: tcon-top: Cleanup clock handling

2018-07-09 Thread Jernej Skrabec
There is no need to acquire reference to clock just to get its name. This commit just cleans up the code. There is no functional change. Signed-off-by: Jernej Skrabec --- drivers/gpu/drm/sun4i/sun8i_tcon_top.c | 48 ++ 1 file changed, 11 insertions(+), 37 deletions(-)

[PATCH 5/6] drm/i2c: tda998x: allocate tda998x_priv inside tda998x_create()

2018-07-09 Thread Russell King
Move the tda998x_priv allocation inside tda998x_create() and simplify the tda998x_create()'s arguments. Pass the same to tda998x_destroy(). Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 32 +--- 1 file changed, 17 insertions(+), 15 deletions(-)

Re: [RFC PATCH v3 1/2] drm: Add generic colorkey properties for DRM planes

2018-07-09 Thread Russell King - ARM Linux
On Fri, Jul 06, 2018 at 05:58:50PM +0300, Dmitry Osipenko wrote: > On Friday, 6 July 2018 17:10:10 MSK Ville Syrjälä wrote: > > IIRC my earlier idea was to have different colorkey modes for the > > min+max and value+mask modes. That way userspace might actually have > > some chance of figuring out

[PATCH 6/6] drm/i2c: tda998x: cleanup from previous changes

2018-07-09 Thread Russell King
Cleanup the code a little from the effects of the previous changes: - Move tda998x_destroy() to be above tda998x_create() - Use 'dev' directly in tda998x_create() where appropriate. Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 76 +++ 1

Re: [PATCH v4 7/8] drm/i2c: tda998x: register as a drm bridge

2018-07-09 Thread Russell King - ARM Linux
On Wed, Apr 25, 2018 at 11:01:15PM +0300, Jyri Sarha wrote: > On 25/04/18 02:25, Russell King - ARM Linux wrote: > > On Tue, Apr 24, 2018 at 09:25:44PM +0300, Jyri Sarha wrote: > >> On 24/04/18 20:06, Russell King - ARM Linux wrote: > >>> On Tue, Apr 24, 2018 at 07:04:16PM +0300, Jyri Sarha wrote:

[PATCH] gpu: host1x: Check whether size of unpin isn't 0

2018-07-09 Thread Dmitry Osipenko
Only gather pins are mapped by the Host1x driver, regular BO relocations are not. Check whether size of unpin isn't 0, otherwise IOVA allocation at 0x0 could be erroneously released. Signed-off-by: Dmitry Osipenko --- drivers/gpu/host1x/job.c | 3 ++- 1 file changed, 2 insertions(+), 1

Re: [PATCH 06/17] dt-bindings: display: sun4i-drm: Add R40 TV TCON description

2018-07-09 Thread Jernej Škrabec
Dne petek, 06. julij 2018 ob 22:40:52 CEST je Rob Herring napisal(a): > On Fri, Jul 06, 2018 at 07:51:02PM +0200, Jernej Skrabec wrote: > > TCON description is expanded with R40 TV TCON compatible. It is a bit > > special, because it is connected to TCON TOP instead directly to mixer > > and it

[PATCH 17/17] dt-bindings: display: sun4i-drm: Fix order of DW HDMI PHY compatibles

2018-07-09 Thread Jernej Skrabec
They are currently sorted alphabetically. However, they should be sorted by release date of the family and then alphabetically. Fixes: 03c35dbf73e0 ("dt-bindings: display: sun4i-drm: Add description of A64 HDMI PHY") Signed-off-by: Jernej Skrabec ---

Re: [RFC PATCH v3 1/2] drm: Add generic colorkey properties for DRM planes

2018-07-09 Thread Dmitry Osipenko
On Friday, 6 July 2018 20:01:36 MSK Russell King - ARM Linux wrote: > On Fri, Jul 06, 2018 at 07:33:14PM +0300, Dmitry Osipenko wrote: > > On Friday, 6 July 2018 18:40:27 MSK Russell King - ARM Linux wrote: > > > On Fri, Jul 06, 2018 at 05:58:50PM +0300, Dmitry Osipenko wrote: > > > > On Friday, 6

[PATCH 08/17] ARM: dts: sun8i: r40: Add mixer ids to TCON TOP

2018-07-09 Thread Jernej Skrabec
Mixer has to have a way to read its id, if needed. Add them in R40 DT. Signed-off-by: Jernej Skrabec --- arch/arm/boot/dts/sun8i-r40.dtsi | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi index

[PATCH 05/17] drm/sun4i: tcon: Release node when traversing of graph

2018-07-09 Thread Jernej Skrabec
Function sun4i_tcon_find_engine_traverse() doesn't release node if it needs to traverse of graph deeper than 1 level. Fix this by calling of_node_put(). Fixes: 49836b11fe71 ("drm/sun4i: tcon: Generalize engine search algorithm") Signed-off-by: Jernej Skrabec ---

Re: [RFC PATCH v3 1/2] drm: Add generic colorkey properties for DRM planes

2018-07-09 Thread Russell King - ARM Linux
On Fri, Jul 06, 2018 at 07:33:14PM +0300, Dmitry Osipenko wrote: > On Friday, 6 July 2018 18:40:27 MSK Russell King - ARM Linux wrote: > > On Fri, Jul 06, 2018 at 05:58:50PM +0300, Dmitry Osipenko wrote: > > > On Friday, 6 July 2018 17:10:10 MSK Ville Syrjälä wrote: > > > > IIRC my earlier idea

[PATCH 10/17] drm/sun4i: tcon-top: Add helpers for switching mux

2018-07-09 Thread Jernej Skrabec
We want to be able to set TCON TOP muxes at runtime. Add helpers for that. Old, static configuration of muxes at probe time is preserved for now. It will be removed when R40 TCON starts using them. Signed-off-by: Jernej Skrabec --- drivers/gpu/drm/sun4i/sun8i_tcon_top.c | 74

[PATCH v5 0/4] DT: Fix spelling of standard properties

2018-07-09 Thread Geert Uytterhoeven
Hi all, This patch series fixes misspellings of various standard DT properties in DT binding documentation, DTS files, and examples. While most of these are harmless, some of them may cause hard-to-debug failures. Changes compared to v5: - Add Acked-by. Changes compared to v4: -

[PATCH 2/3 v4] ARM: defconfig: Update the vexpress defconfig

2018-07-09 Thread Linus Walleij
Update the Versatile Express defconfig to match the Kconfig changes in the kernel. Cc: Sudeep Holla Cc: Lorenzo Pieralisi Signed-off-by: Linus Walleij --- ChangeLog v3->v4: - Resending. ChangeLog v1->v3: - Rebased --- arch/arm/configs/vexpress_defconfig | 12 1 file changed, 12

[PATCH 3/3 v4] ARM: defconfig: Enable the PL111 DRM driver on vexpress

2018-07-09 Thread Linus Walleij
This updates the Versatile defconfig to use the new P111 DRM driver that is merged in the DRM subsystem. We deactivate the old CLCD driver and activate the Pl111 DRM driver and the SiI9022 HDMI bridge. We activate DMA memory allocation using CMA so that the special graphics memory for the

[PATCH 03/12] cgroup: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids the need to invert the condition instead of the open-coded version. Signed-off-by: Daniel Vetter Cc: Tejun Heo Cc: Li Zefan Cc: Johannes Weiner Cc: cgro...@vger.kernel.org --- include/linux/cgroup.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH 04/12] cpufreq: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids the inverted condition compared to the open coded version. Signed-off-by: Daniel Vetter Cc: "Rafael J. Wysocki" Cc: Viresh Kumar Cc: linux...@vger.kernel.org --- include/linux/cpufreq.h | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/include/linux/cpufreq.h

[PATCH 06/12] mm: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids the inverted condition of the open-coded version. Signed-off-by: Daniel Vetter Cc: Andrew Morton Cc: Michal Hocko Cc: Vlastimil Babka Cc: Mel Gorman Cc: David Rientjes Cc: Kemi Wang Cc: Pavel Tatashin Cc: Petr Tesarik Cc: YASUAKI ISHIMATSU Cc: Andrey Ryabinin Cc: Nikolay Borisov

[PATCH 02/12] blk: use for_each_if

2018-07-09 Thread Daniel Vetter
Makes the macros resilient against if {} else {} blocks right afterwards. Signed-off-by: Daniel Vetter Cc: Tejun Heo Cc: Jens Axboe Cc: Shaohua Li Cc: Kate Stewart Cc: Greg Kroah-Hartman Cc: Joseph Qi Cc: Daniel Vetter Cc: Arnd Bergmann --- include/linux/blk-cgroup.h | 4 ++-- 1 file

[PATCH 05/12] dmar: Use for_each_If

2018-07-09 Thread Daniel Vetter
Avoids the inverted conditions compared to the open coded version. Signed-off-by: Daniel Vetter Cc: Joerg Roedel --- include/linux/dmar.h | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/include/linux/dmar.h b/include/linux/dmar.h index e2433bc50210..99397504e75e

[PATCH 07/12] ide: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids complaints from gcc about ambigious else clauses. Not that any of those are likely to show up in ide ... Signed-off-by: Daniel Vetter Cc: "David S. Miller" Cc: linux-...@vger.kernel.org --- include/linux/ide.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 11/12] sched: use for_each_if in topology.h

2018-07-09 Thread Daniel Vetter
Avoids complaints from gcc about ambiguous else clauses. Signed-off-by: Daniel Vetter Cc: Andrew Morton Cc: Peter Zijlstra --- include/linux/topology.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/topology.h b/include/linux/topology.h index

[PATCH 10/12] pci: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids the inverted condition compared to the open-coded version. Signed-off-by: Daniel Vetter Cc: Bjorn Helgaas Cc: linux-...@vger.kernel.org --- include/linux/pci.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/pci.h b/include/linux/pci.h index

[PATCH 09/12] nubus: use for_each_if

2018-07-09 Thread Daniel Vetter
Avoids the inverted check compared to the open-coded version. Signed-off-by: Daniel Vetter Cc: Finn Thain Cc: linux-m...@lists.linux-m68k.org --- include/linux/nubus.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/nubus.h b/include/linux/nubus.h index

[Bug 107154] [drm] GPU recovery disabled.

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107154 --- Comment #3 from Michel Dänzer --- Please attach the full dmesg output. Can you bisect between 4.14 and 4.15? -- You are receiving this mail because: You are the assignee for the bug.___

答复: [PATCH] vgaswitchroo: set audio client id according to bound gpu client id

2018-07-09 Thread Qu, Jim
Hi Lukas, Our OEM report the issue also can be observed with v4.17rc7 of test platform, and I also backported these 7 patches to v4.15, have confirmed the issue. Now, I found the audio device will auto suspend even if the gpu is active, and if I plug in a HDMI device it also do not resume

[PATCH 00/17] Allwinner R40 HDMI refactoring

2018-07-09 Thread Jernej Skrabec
This series fixes several issues found in R40 HDMI patch series after it was applied. Conversation can be found here: http://lists.infradead.org/pipermail/linux-arm-kernel/2018-June/586011.html One issue raised in that thread is probably not solved (conversation still on going), but I want to

[PATCH 06/17] dt-bindings: display: sun4i-drm: Add R40 TV TCON description

2018-07-09 Thread Jernej Skrabec
TCON description is expanded with R40 TV TCON compatible. It is a bit special, because it is connected to TCON TOP instead directly to mixer and it needs special handling. Signed-off-by: Jernej Skrabec --- Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 3 ++- 1 file changed, 2

Re: [RFC PATCH v3 1/2] drm: Add generic colorkey properties for DRM planes

2018-07-09 Thread Dmitry Osipenko
On 06.07.2018 15:23, Ville Syrjälä wrote: > On Fri, Jul 06, 2018 at 02:11:44PM +0200, Maarten Lankhorst wrote: >> Hey, >> >> Op 04-06-18 om 00:00 schreef Dmitry Osipenko: >>> From: Laurent Pinchart >>> >>> Color keying is the action of replacing pixels matching a given color >>> (or range of

[PATCH 2/6] drm/i2c: tda998x: split tda998x_encoder_dpms into enable/disable

2018-07-09 Thread Russell King
From: Peter Rosin This fits better with the drm_bridge callbacks for when this driver becomes a drm_bridge. Suggested-by: Laurent Pinchart Signed-off-by: Peter Rosin [edited by rmk to just split the tda998x_encoder_dpms() function and restore the double-disable protection we originally had,

[PATCH 11/17] drm/sun4i: tcon: Add another way for matching mixers with tcon

2018-07-09 Thread Jernej Skrabec
Till now, new way of matching engines with TCONs was reading their respective ids and match them by those ids. However, with introduction of TCON TOP, that might not be so straightforward anymore. - there might be more TCONs that engines (mixers) - TCON ids might have non-consecutive ids

[PATCH 16/17] drm/sun4i: tcon-top: Remove mux configuration at probe time

2018-07-09 Thread Jernej Skrabec
Now that R40 TCON migrated to runtime mux configuration, old code can be removed. Signed-off-by: Jernej Skrabec --- drivers/gpu/drm/sun4i/sun8i_tcon_top.c | 81 +++--- 1 file changed, 7 insertions(+), 74 deletions(-) diff --git a/drivers/gpu/drm/sun4i/sun8i_tcon_top.c

Re: [PATCH v4 7/8] drm/i2c: tda998x: register as a drm bridge

2018-07-09 Thread Russell King - ARM Linux
On Fri, Jul 06, 2018 at 11:03:46AM +0100, Russell King - ARM Linux wrote: > On Wed, Apr 25, 2018 at 11:01:15PM +0300, Jyri Sarha wrote: > > Oh yes. But in this case the substandard solution is already there and > > it is already widely used, despite it being severely broken. I am merely > > trying

[PATCH 3/6] drm/i2c: tda998x: move tda998x_set_config() into tda998x_create()

2018-07-09 Thread Russell King
Move the non-DT configuration of the TDA998x into tda998x_create() so that we do all setup in one place. Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 75 --- 1 file changed, 38 insertions(+), 37 deletions(-) diff --git

  1   2   3   >