Re: [PATCH 2/2] drm/sun4i: backend: Disable alpha on the lowest plane on the A20

2020-08-28 Thread Chen-Yu Tsai
On Tue, Jul 28, 2020 at 9:48 PM Maxime Ripard wrote: > > Unlike we previously thought, the per-pixel alpha is just as broken on the > A20 as it is on the A10. Remove the quirk that says we can use it. > > Cc: Paul Kocialkowski > Fixes: dcf496a6a608 ("drm/sun4i: sun4i: Introduce a quirk for lowest

Re: [PATCH 1/2] drm/sun4i: backend: Support alpha property on lowest plane

2020-08-28 Thread Chen-Yu Tsai
On Tue, Jul 28, 2020 at 9:48 PM Maxime Ripard wrote: > > Unlike what we previously thought, only the per-pixel alpha is broken on > the lowest plane and the per-plane alpha isn't. Remove the check on the > alpha property being set on the lowest plane to reject a mode. > > Cc: Paul Kocialkowski >

Re: [PATCH v2 3/4] drm/sun4i: tcon: Support the LVDS Dual-Link on the A20

2020-08-28 Thread Chen-Yu Tsai
On Thu, Jul 30, 2020 at 5:35 PM Maxime Ripard wrote: > > The A20 can use its second TCON as the secondary LVDS link in a dual-link > setup, with the TCON0 being the main link. Extend a bit the parsing code to > leverage the DRM dual-link code, register only the LVDS output on the > primary TCON, a

Re: [PATCH v2 2/4] drm/sun4i: tcon: Refactor the LVDS and panel probing

2020-08-28 Thread Chen-Yu Tsai
On Thu, Jul 30, 2020 at 5:35 PM Maxime Ripard wrote: > > The current code to parse the DT, deal with the older device trees, and > register either the RGB or LVDS output has so far grown organically into > the bind function and has become quite hard to extend properly. > > Let's move it into a sin

Re: [PATCH] drm/i915/display: fix uninitialized variable

2020-08-28 Thread Souza, Jose
Just merged the first patch that fixed this issue, thanks anyways. 2034c2129bc4a91d471815d4dc7a2a69eaa5338d - drm/i915/display: Ensure that ret is always initialized in icl_combo_phy_verify_state On Tue, 2020-08-25 at 16:20 -0700, t...@redhat.com wrote: > From: Tom Rix < > t...@redhat.com > > >

[radeon-alex:amd-staging-drm-next 1403/1518] drivers/gpu/drm/amd/amdgpu/../pm/powerplay/smumgr/smu7_smumgr.c:408:44: sparse: sparse: incorrect type in argument 1 (different address spaces)

2020-08-28 Thread kernel test robot
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: 8252270c0b5fba4dd651473c5fee66ecaa3d9026 commit: 82e6bcf7c9a0c82f3d41a5837bb1dadd68037e55 [1403/1518] drm/amd/pm: optimize the power related source code layout config: powerpc64-randconfig-s032-20200828 (attached

Re: [PATCH 2/5] dt-bindings: display/bridge: nwl-dsi: Document video_pll clock

2020-08-28 Thread Rob Herring
On Fri, 28 Aug 2020 14:13:29 +0300, Robert Chiras (OSS) wrote: > From: Robert Chiras > > Add documentation for a new clock 'video_pll'. > > Signed-off-by: Robert Chiras > --- > Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml | 3 +++ > 1 file changed, 3 insertions(+) > My bot

Re: [PATCH 2/2] drm: panel: simple: Add AM-1280800N3TZQW-T00H

2020-08-28 Thread kernel test robot
Hi Jagan, I love your patch! Yet something to improve: [auto build test ERROR on robh/for-next] [also build test ERROR on drm-intel/for-linux-next drm-tip/drm-tip linus/master v5.9-rc2 next-20200828] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread John Stultz
On Fri, Aug 28, 2020 at 1:05 AM Greg Kroah-Hartman wrote: > > On Thu, Aug 27, 2020 at 11:54:12AM -0700, John Stultz wrote: > > On Thu, Aug 27, 2020 at 10:17 AM Greg Kroah-Hartman > > wrote: > > > On Thu, Aug 27, 2020 at 10:31:41PM +0530, Amit Pundir wrote: > > > > I don't know what is the right t

Re: [git pull] drm fixes for 5.9-rc3

2020-08-28 Thread pr-tracker-bot
The pull request you sent on Fri, 28 Aug 2020 12:35:20 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2020-08-28 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/5ec06b5c0d259a8c7c4376b121b2f62dfbfe57ef Thank you! -- Deet-doot-dot, I am a bot. https://k

Re: [PATCH v2 2/3] drm: bridge: add support for lontium LT9611UXC bridge

2020-08-28 Thread Sam Ravnborg
Hi Dmitry On Fri, Aug 28, 2020 at 06:49:05PM +0300, Dmitry Baryshkov wrote: > Add support for Lontium LT9611UXC HDMI bridge. Lontium LT9611UXC is a > DSI to HDMI bridge which supports two DSI ports and I2S port as an input > and HDMI port as output. Despite name being similar to LT9611, these > de

Re: [PATCH RFC v6 1/6] dt-bindings: display: add Unisoc's drm master bindings

2020-08-28 Thread Rob Herring
On Tue, Jul 28, 2020 at 4:08 AM Kevin Tang wrote: > > From: Kevin Tang > > The Unisoc DRM master device is a virtual device needed to list all > DPU devices or other display interface nodes that comprise the > graphics subsystem > > Cc: Orson Zhai > Cc: Chunyan Zhang > Signed-off-by: Kevin Tang

Re: [PATCH v2 1/3] dt-bindings: display: bridge: Add documentation for LT9611UXC

2020-08-28 Thread Sam Ravnborg
On Fri, Aug 28, 2020 at 06:49:04PM +0300, Dmitry Baryshkov wrote: > Lontium LT9611UXC is a DSI to HDMI bridge which supports 2 DSI ports > and I2S port as input and one HDMI port as output. The LT9611UXC chip is > handled by a separate driver, but the bindings used are fully compatible > with the L

Re: [PATCH 3/3] drm/vc4: hdmi: Add pixel bvb clock control

2020-08-28 Thread Stefan Wahren
Hi Maxime, Am 28.08.20 um 17:25 schrieb Maxime Ripard: > Hi, > > On Fri, Aug 28, 2020 at 02:45:49PM +0200, Stefan Wahren wrote: >> Am 28.08.20 um 08:30 schrieb Hoegeun Kwon: >>> On 8/27/20 6:49 PM, Stefan Wahren wrote: Am 27.08.20 um 06:35 schrieb Hoegeun Kwon: > Hi Stefan, > > Th

Re: [PATCH 2/2] drm: panel: simple: Add AM-1280800N3TZQW-T00H

2020-08-28 Thread Sam Ravnborg
Hi Jagan. On Fri, Aug 28, 2020 at 09:29:38PM +0530, Jagan Teki wrote: > Add Ampire, AM-1280800N3TZQW-T00H 10.1" TFT LCD panel timings. > > Signed-off-by: Jagan Teki > --- > drivers/gpu/drm/panel/panel-simple.c | 28 > 1 file changed, 28 insertions(+) > > diff --git

Re: [PATCH RFC v6 1/6] dt-bindings: display: add Unisoc's drm master bindings

2020-08-28 Thread Sam Ravnborg
Hi Kevin. > > > > Any specific reason why this is not a ports node like used by many other > > display bindings? > > In other words - I think this is too simple. > We only support one display pipeline now, other interface, like > DP(DisplayPort), HDMI...will be add later... > > ports: > $re

Re: [PATCH RFC v6 2/6] drm/sprd: add Unisoc's drm kms master

2020-08-28 Thread Sam Ravnborg
Hi Kevin > > > > > + > > > + drm->dev_private = sprd; > > dev_private is deprecated. Alwyas use upclassing. > dev_private is deprecated ? I see everyone is still using it, so it > will be deprecated in the future? It is deprecated - and a lot of drivers is in need up an update to get away from

Re: [PATCH 2/2] drm: panel: simple: Add AM-1280800N3TZQW-T00H

2020-08-28 Thread kernel test robot
Hi Jagan, I love your patch! Yet something to improve: [auto build test ERROR on robh/for-next] [also build test ERROR on drm-intel/for-linux-next drm-tip/drm-tip linus/master v5.9-rc2 next-20200828] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting

Re: Aw: Re: [PATCH v5 3/7] drm/mediatek: disable tmds on mt2701

2020-08-28 Thread Matthias Brugger
On 28/08/2020 01:46, Chun-Kuang Hu wrote: Hi, Frank: Matthias Brugger 於 2020年8月27日 週四 下午10:28寫道: On 27/08/2020 15:41, Frank Wunderlich wrote: Hi Matthias, any opinions about the dts-changes? they look good to me. maybe series except the tmds-Patch get merged...so i add it only to m

[PATCH 2/2] drm: panel: simple: Add AM-1280800N3TZQW-T00H

2020-08-28 Thread Jagan Teki
Add Ampire, AM-1280800N3TZQW-T00H 10.1" TFT LCD panel timings. Signed-off-by: Jagan Teki --- drivers/gpu/drm/panel/panel-simple.c | 28 1 file changed, 28 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index b

[PATCH 1/2] dt-bindings: display: simple: Add AM-1280800N3TZQW-T00H

2020-08-28 Thread Jagan Teki
Add dt-bindings for 10.1" TFT LCD module from Ampire Co. Ltd. as part of panel-simple. Signed-off-by: Jagan Teki --- .../devicetree/bindings/display/panel/panel-simple.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.

Re: [PATCH 3/3] drm/vc4: hdmi: Add pixel bvb clock control

2020-08-28 Thread Dave Stevenson
Hi Maxime, Stefan, and Hoegeun On Fri, 28 Aug 2020 at 16:25, Maxime Ripard wrote: > > Hi, > > On Fri, Aug 28, 2020 at 02:45:49PM +0200, Stefan Wahren wrote: > > Am 28.08.20 um 08:30 schrieb Hoegeun Kwon: > > > On 8/27/20 6:49 PM, Stefan Wahren wrote: > > >> Am 27.08.20 um 06:35 schrieb Hoegeun Kw

[Bug 203033] nouveau hung task

2020-08-28 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203033 James O'Beirne (james.obei...@pm.me) changed: What|Removed |Added CC||james.obei...@pm.me

Re: [PATCH v1 2/2] drm: bridge: add support for lontium LT9611UXC bridge

2020-08-28 Thread Vinod Koul
On 28-08-20, 18:01, Dmitry Baryshkov wrote: > On 28/08/2020 17:18, Vinod Koul wrote: > > On 28-08-20, 15:04, Dmitry Baryshkov wrote: > > > +static int lt9611uxc_bridge_attach(struct drm_bridge *bridge, > > > + enum drm_bridge_attach_flags flags) > > > +{ > > > + struct lt961

Re: [PATCH v1 2/2] drm: bridge: add support for lontium LT9611UXC bridge

2020-08-28 Thread Laurent Pinchart
On Fri, Aug 28, 2020 at 05:33:00PM +0300, Laurent Pinchart wrote: > On Fri, Aug 28, 2020 at 07:48:48PM +0530, Vinod Koul wrote: > > On 28-08-20, 15:04, Dmitry Baryshkov wrote: > > > > > +#define EDID_BLOCK_SIZE 128 > > > +#define EDID_NUM_BLOCKS 2 > > > > tab or space either one, not both ;) > >

Re: [PATCH RFC v6 1/6] dt-bindings: exynos-bus: Add documentation for interconnect properties

2020-08-28 Thread Sylwester Nawrocki
On 30.07.2020 14:28, Sylwester Nawrocki wrote: > On 09.07.2020 23:04, Rob Herring wrote: >> On Thu, Jul 02, 2020 at 06:37:19PM +0200, Sylwester Nawrocki wrote: >>> Add documentation for new optional properties in the exynos bus nodes: >>> samsung,interconnect-parent, #interconnect-cells, bus-width.

Re: [PATCH v1 2/2] drm: bridge: add support for lontium LT9611UXC bridge

2020-08-28 Thread Laurent Pinchart
On Fri, Aug 28, 2020 at 07:48:48PM +0530, Vinod Koul wrote: > On 28-08-20, 15:04, Dmitry Baryshkov wrote: > > > +#define EDID_BLOCK_SIZE128 > > +#define EDID_NUM_BLOCKS 2 > > tab or space either one, not both ;) > > > +static struct mipi_dsi_device *lt9611uxc_attach_dsi(struct lt9611uxc > >

Re: [PATCH v1 2/2] drm: bridge: add support for lontium LT9611UXC bridge

2020-08-28 Thread Vinod Koul
On 28-08-20, 15:04, Dmitry Baryshkov wrote: > +#define EDID_BLOCK_SIZE 128 > +#define EDID_NUM_BLOCKS 2 tab or space either one, not both ;) > +static struct mipi_dsi_device *lt9611uxc_attach_dsi(struct lt9611uxc > *lt9611uxc, > + struct device_

Re: [PATCH 2/2] drm/tve200: Fix handling of platform_get_irq() error

2020-08-28 Thread Linus Walleij
On Thu, Aug 27, 2020 at 9:11 AM Krzysztof Kozlowski wrote: > platform_get_irq() returns -ERRNO on error. In such case comparison > to 0 would pass the check. > > Fixes: 179c02fe90a4 ("drm/tve200: Add new driver for TVE200") > Signed-off-by: Krzysztof Kozlowski Acked-by: Linus Walleij Yours,

Re: [PATCH 1/2] drm/mcde: Fix handling of platform_get_irq() error

2020-08-28 Thread Linus Walleij
On Thu, Aug 27, 2020 at 9:11 AM Krzysztof Kozlowski wrote: > platform_get_irq() returns -ERRNO on error. In such case comparison > to 0 would pass the check. > > Fixes: 5fc537bfd000 ("drm/mcde: Add new driver for ST-Ericsson MCDE") > Signed-off-by: Krzysztof Kozlowski Acked-by: Linus Walleij

Re: [PATCH v1 1/2] dt-bindings: display: bridge: Add documentation for LT9611UXC

2020-08-28 Thread Vinod Koul
On 28-08-20, 15:04, Dmitry Baryshkov wrote: > Lontium LT9611UXC is a DSI to HDMI bridge which supports 2 DSI ports > and I2S port as input and one HDMI port as output. The LT9611UXC chip is > handled by a separate driver, but the bindings used are fully compatible > with the LT9611 chip, so let's r

Re: [PATCH v1 2/2] drm: bridge: add support for lontium LT9611UXC bridge

2020-08-28 Thread kernel test robot
Hi Dmitry, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on next-20200828] [cannot apply to robh/for-next drm-intel/for-linux-next tegra-drm/drm/tegra/for-next linus/master drm-exynos/exynos-drm-next drm/drm-next

Re: [PATCH 3/3] drm/vc4: hdmi: Add pixel bvb clock control

2020-08-28 Thread Stefan Wahren
Hi, Am 28.08.20 um 08:30 schrieb Hoegeun Kwon: > On 8/27/20 6:49 PM, Stefan Wahren wrote: >> Am 27.08.20 um 06:35 schrieb Hoegeun Kwon: >>> Hi Stefan, >>> >>> Thank you for your review. >>> >>> >>> On 8/26/20 7:04 PM, Stefan Wahren wrote: Hi Hoeguen, Am 21.08.20 um 09:10 schrieb Hoe

[PATCH v3] drm/vkms: avoid warning in vkms_get_vblank_timestamp

2020-08-28 Thread Sidong Yang
This patch avoid the warning in vkms_get_vblank_timestamp when vblanks aren't enabled. When running igt test kms_cursor_crc just after vkms module, the warning raised like below. Initial value of vblank time is zero and hrtimer.node.expires is also zero if vblank aren't enabled before. vkms module

Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips

2020-08-28 Thread Jim Quinlan
On Thu, Aug 27, 2020 at 2:35 AM Christoph Hellwig wrote: > > On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote: > > Hi, > > > > On 8/24/2020 12:30 PM, Jim Quinlan wrote: > >> > >> Patchset Summary: > >>Enhance a PCIe host controller driver. Because of its unusual design > >>

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread Hridya Valsaraju
On Thu, Aug 27, 2020 at 10:17 AM Greg Kroah-Hartman wrote: > > On Thu, Aug 27, 2020 at 10:31:41PM +0530, Amit Pundir wrote: > > On Thu, 27 Aug 2020 at 21:34, Greg Kroah-Hartman > > wrote: > > > > > > On Thu, Aug 27, 2020 at 09:31:27AM -0400, Laura Abbott wrote: > > > > On 8/27/20 8:36 AM, Greg Kr

Re: [PATCH 1/2] drm/virtio: fix unblank

2020-08-28 Thread Gerd Hoffmann
On Mon, Aug 24, 2020 at 09:24:40AM +0200, Jiri Slaby wrote: > On 18. 08. 20, 9:25, Gerd Hoffmann wrote: > > When going through a disable/enable cycle without changing the > > framebuffer the optimization added by commit 3954ff10e06e ("drm/virtio: > > skip set_scanout if framebuffer didn't change")

Re: [PATCH] drm: virtio: fix kconfig dependency warning

2020-08-28 Thread Gerd Hoffmann
Hi, > config DRM_VIRTIO_GPU > tristate "Virtio GPU driver" > - depends on DRM && VIRTIO && MMU > + depends on DRM && VIRTIO_MENU && MMU Shouldn't this depend on both VIRTIO and VIRTIO_MENU, simliar to the other virtio drivers? take care, Gerd _

Re: [PATCH 3/3 v3] backlight: Add Kinetic KTD253 backlight driver

2020-08-28 Thread Lee Jones
On Wed, 19 Aug 2020, Linus Walleij wrote: > The Kinetic KTD253 backlight driver is controlled with a > single GPIO line, but still supports a range of brightness > settings by sending fast pulses on the line. > > This is based off the source code release for the Samsung > GT-S7710 mobile phone. >

Re: [PATCH 2/3 v3] dt-bindings: backlight: Add Kinetic KTD253 bindings

2020-08-28 Thread Lee Jones
On Wed, 19 Aug 2020, Linus Walleij wrote: > This adds device tree bindings for the Kinetic KTD253 > white LED backlight driver. > > Cc: devicet...@vger.kernel.org > Cc: Sam Ravnborg > Signed-off-by: Linus Walleij > --- > ChangeLog v2->v3: > - Drop the pointless cargo-culted "default-on" propert

Re: [PATCH 1/3 v3] dt-bindings: backlight: Add some common backlight properties

2020-08-28 Thread Lee Jones
On Wed, 19 Aug 2020, Linus Walleij wrote: > Let's use a common.yaml include for the backlight like we do with > the LEDs. The LEDs are inherently incompatible so their bindings > cannot be reused for backlight. > > Cc: devicet...@vger.kernel.org > Suggested-by: Sam Ravnborg > Signed-off-by: Linu

[PATCH 5/6] drm/nouveau: Support DMA fence arrays

2020-08-28 Thread Thierry Reding
From: Thierry Reding A DMA fence can be composed of multiple fences in an array. Support this in the Nouveau driver by iteratively synchronizing to each DMA fence in the array. Signed-off-by: Thierry Reding --- drivers/gpu/drm/nouveau/nouveau_fence.c | 31 ++--- 1 file chan

[PATCH 1/6] drm/nouveau: Split nouveau_fence_sync()

2020-08-28 Thread Thierry Reding
From: Thierry Reding Turn nouveau_fence_sync() into a low-level helper that adds fence waits to the channel command stream. The new nouveau_bo_sync() helper replaces the previous nouveau_fence_sync() implementation. It passes each of the buffer object's fences to nouveau_fence_sync() in turn. Th

[PATCH 6/6] drm/nouveau: Allow zero pushbuffer submits

2020-08-28 Thread Thierry Reding
From: Thierry Reding These are useful in cases where only a fence is to be created to wait for existing jobs in the command stream. Signed-off-by: Thierry Reding --- drivers/gpu/drm/nouveau/nouveau_gem.c | 197 +- 1 file changed, 99 insertions(+), 98 deletions(-) diff

[PATCH 2/6] drm/nouveau: Add nouveau_fence_ref()

2020-08-28 Thread Thierry Reding
From: Thierry Reding This is a simple wrapper that increments the reference count of the backing DMA fence. Signed-off-by: Thierry Reding --- drivers/gpu/drm/nouveau/nouveau_fence.c | 9 + drivers/gpu/drm/nouveau/nouveau_fence.h | 1 + 2 files changed, 10 insertions(+) diff --git a/dr

[PATCH 4/6] drm/nouveau: Support sync FDs and syncobjs

2020-08-28 Thread Thierry Reding
From: Thierry Reding Extends the new NOUVEAU_GEM_PUSHBUF2 IOCTL to accept and emit one or more sync FDs and/or DRM native sync objects. Signed-off-by: Thierry Reding --- Note: If acceptable, this should be merged into the previous patch that adds the new IOCTL. drivers/gpu/drm/nouveau/nouveau

[PATCH 3/6] drm/nouveau: Support fence FDs at kickoff

2020-08-28 Thread Thierry Reding
From: Thierry Reding Add a new NOUVEAU_GEM_PUSHBUF2 IOCTL that accepts and emits a sync fence FD from/to userspace if requested by the corresponding flags. Based heavily on work by Lauri Peltonen Signed-off-by: Thierry Reding --- drivers/gpu/drm/nouveau/nouveau_drm.c | 1 + drivers/gpu/drm/

[PATCH 0/6] drm/nouveau: Support sync FDs and sync objects

2020-08-28 Thread Thierry Reding
From: Thierry Reding Hi, This series implements a new IOCTL to submit push buffers that can optionally return a sync FD or sync object to userspace. This is useful in cases where userspace wants to synchronize operations between the GPU and another driver (such as KMS for display). Among other t

Re: [bug report] drm/dp: annotate implicit fall throughs

2020-08-28 Thread Jani Nikula
On Tue, 25 Aug 2020, Dan Carpenter wrote: > Hello Mathieu Malaterre, > > The patch e9c0c874711b: "drm/dp: annotate implicit fall throughs" > from Jan 14, 2019, leads to the following static checker warning: > > drivers/gpu/drm/drm_dp_helper.c:495 drm_dp_downstream_max_bpc() > warn: ign

Re: [PATCH v2 0/24] backlight: add init macros and accessors

2020-08-28 Thread Linus Walleij
On Sun, Aug 23, 2020 at 12:45 PM Sam Ravnborg wrote: > The first patch trims backlight_update_status() so it can be called with a > NULL > backlight_device. Then the caller do not need to add this check just to avoid > a NULL reference. > > The backlight drivers uses several different patterns w

Re: [PATCH] video: backlight: sky81452-backlight: Fix reference count imbalance on error

2020-08-28 Thread Lee Jones
On Thu, 20 Aug 2020, Daniel Thompson wrote: > On Thu, Aug 20, 2020 at 08:49:16AM +0100, Lee Jones wrote: > > On Thu, 20 Aug 2020, dinghao@zju.edu.cn wrote: > > > > > > On Thu, 20 Aug 2020, dinghao@zju.edu.cn wrote: > > > > > > > > > > On Wed, 19 Aug 2020, Markus Elfring wrote: > > > > >

Re: [PATCH 3/3] drm/vc4: hdmi: Add pixel bvb clock control

2020-08-28 Thread Dave Stevenson
Hi Stefan & Hoegeun On Wed, 26 Aug 2020 at 11:04, Stefan Wahren wrote: > > Hi Hoeguen, > > Am 21.08.20 um 09:10 schrieb Hoegeun Kwon: > > There is a problem that the output does not work at a resolution > > exceeding FHD. To solve this, we need to adjust the bvb clock at a > > resolution exceedin

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

2020-08-28 Thread Laurentiu Palcu
Hi Lucas, I was wondering about the plans to merge this series. Since not many people can test it properly due to lack of DCSS support in the upstream NWL driver (which I heard it's coming soon) and a completely nonexistent HDP driver, are we going to take a leap of faith and merge it as is? Or sh

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread Greg Kroah-Hartman
On Thu, Aug 27, 2020 at 11:54:12AM -0700, John Stultz wrote: > On Thu, Aug 27, 2020 at 10:17 AM Greg Kroah-Hartman > wrote: > > On Thu, Aug 27, 2020 at 10:31:41PM +0530, Amit Pundir wrote: > > > I don't know what is the right thing to do here. I just want to > > > highlight that AOSP's audio (code

[PATCH] drm/bridge: Fix the dsi remote end-points

2020-08-28 Thread Vinod Koul
DSI end-points are supposed to be at node 0 and node 1 as per binding. So fix this and use node 0 and node 1 for dsi. Reported-by: Dmitry Baryshkov Fixes: 23278bf54afe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge") Signed-off-by: Vinod Koul --- drivers/gpu/drm/bridge/lontium-lt9611.c | 4 +

Re: [PATCH] drm/sun4i: lvds: Invert the LVDS polarity

2020-08-28 Thread Maxime Ripard
On Wed, Aug 26, 2020 at 01:13:33AM +0800, Chen-Yu Tsai wrote: > On Sat, Jul 4, 2020 at 9:38 PM Maxime Ripard wrote: > > > > The LVDS controller can invert the polarity / lanes of the LVDS output. > > The default polarity causes some issues on some panels. > > > > However, U-Boot has always used th

Re: [PATCH 0/8] Convert the intel iommu driver to the dma-iommu api

2020-08-28 Thread Logan Gunthorpe
On 2020-08-23 6:04 p.m., Tom Murphy wrote: > I have added a check for the sg_dma_len == 0 : > """ > } __sgt_iter(struct scatterlist *sgl, bool dma) { > struct sgt_iter s = { .sgp = sgl }; > > + if (sgl && sg_dma_len(sgl) == 0) > + s.sgp = NULL; > > if (s.sgp) {

Aw: [PATCH 00/18] Convert arch/arm to use iommu-dma

2020-08-28 Thread Frank Wunderlich
Tested full series on bananapi r2 (mt7623/mt2701, 5.9-rc1 + hdmi-patches), works so far fbcon+x without issues Tested-by: Frank Wunderlich regards Frank ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/l

[PATCH V2 2/8] drm/lima: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. Reviewed-by: Qiang Yu Signed-off-by: Viresh Kumar --- V2: Applied Reviewe

Re: [PATCH v5 13/36] PM / devfreq: tegra30: Use MC timings for building OPP table

2020-08-28 Thread Chanwoo Choi
Hi, On 8/15/20 1:47 AM, Dmitry Osipenko wrote: > 14.08.2020 05:02, Chanwoo Choi пишет: >> Hi Dmitry, >> >> I add the minor comment. Except of some comments, it looks good to me. > > Hello, Chanwoo! Thank you for the review! > > ... >>> +struct tegra_devfreq_soc_data { >>> + const char *mc_comp

Re: [PATCH 16/18] staging/media/tegra-vde: Clean up IOMMU workaround

2020-08-28 Thread Dmitry Osipenko
24.08.2020 17:01, Robin Murphy пишет: ... >> Robin, thank you very much for the clarifications! >> >> In accordance to yours comments, this patch can't be applied until Tegra >> SMMU will support IOMMU_DOMAIN_IDENTITY and implement def_domain_type() >> callback that returns IOMMU_DOMAIN_IDENTITY fo

[PATCH] video: fbdev: remove set but not used 'ulBestVCO'

2020-08-28 Thread Jason Yan
This addresses the following gcc warning with "make W=1": drivers/video/fbdev/kyro/STG4000InitDevice.c: In function ‘ProgramClock’: drivers/video/fbdev/kyro/STG4000InitDevice.c:123:6: warning: variable ‘ulBestVCO’ set but not used [-Wunused-but-set-variable] 123 | u32 ulBestVCO = 0, ulBestClk =

Re: drm/bridge: Synopsys DW-HDMI bridge driver for the Ingenic JZ4780 (was Re: Specialising the Synopsys DW-HDMI bridge driver for the Ingenic JZ4780)

2020-08-28 Thread H. Nikolaus Schaller
Hi Ezequiel, > Am 24.08.2020 um 19:38 schrieb Ezequiel Garcia > : > > On Mon, 24 Aug 2020 at 13:05, H. Nikolaus Schaller wrote: >> >> Hi Ezequiel, > > The fix is just re-organizing the code a bit. Just pushed a possible > fix for that (following the IPU handling by Paul Cercueil), > please fe

[PATCH v2 1/2] drm/ingenic: Fix leak of device_node pointer

2020-08-28 Thread Paul Cercueil
of_graph_get_remote_node() requires of_node_put() to be called on the device_node pointer when it's no more in use. Fixes: fc1acf317b01 ("drm/ingenic: Add support for the IPU") Signed-off-by: Paul Cercueil --- drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH v4] drm/msm/dp: Add DP compliance tests on Snapdragon Chipsets

2020-08-28 Thread Kuogee Hsieh
add event thread to execute events serially from event queue. Also timeout mode is supported which allow an event be deferred to be executed at later time. Both link and phy compliant tests had been done successfully. Changes in v2: - Fix potential deadlock by removing redundant connect_mutex - C

Aw: Re: [PATCH v5 3/7] drm/mediatek: disable tmds on mt2701

2020-08-28 Thread Frank Wunderlich
Hi Matthias, any opinions about the dts-changes? maybe series except the tmds-Patch get merged...so i add it only to my own repo till we find a better way? currently mainline does not support hdmi at all for the board. the tmds-patch is only a fix for specific resolutions which have a "flickeri

[PATCH] video: fbdev: remove set but not used 'ulCoreClock'

2020-08-28 Thread Jason Yan
This addresses the following gcc warning with "make W=1": drivers/video/fbdev/kyro/STG4000InitDevice.c: In function ‘SetCoreClockPLL’: drivers/video/fbdev/kyro/STG4000InitDevice.c:247:6: warning: variable ‘ulCoreClock’ set but not used [-Wunused-but-set-variable] // yanaijie fixed 247 | u32 ulC

[PATCH v3 0/1] drm/bridge: ps8640: Make sure all needed is powered to get the EDID

2020-08-28 Thread Enric Balletbo i Serra
The first 4 patches of the series version 2: - drm/bridge_connector: Set default status connected for eDP connectors - drm/bridge: ps8640: Get the EDID from eDP control - drm/bridge: ps8640: Return an error for incorrect attach flags - drm/bridge: ps8640: Print an error if VDO control fails

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread Joel Fernandes
On Thu, Aug 27, 2020 at 10:31:41PM +0530, Amit Pundir wrote: > On Thu, 27 Aug 2020 at 21:34, Greg Kroah-Hartman > wrote: > > > > On Thu, Aug 27, 2020 at 09:31:27AM -0400, Laura Abbott wrote: > > > On 8/27/20 8:36 AM, Greg Kroah-Hartman wrote: > > > > The ION android code has long been marked to be

[PATCH V2 0/8] opp: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
Hello, This cleans up some of the user code around calls to dev_pm_opp_of_remove_table(). All the patches can be picked by respective maintainers directly except for the last patch, which needs the previous two to get merged first. These are based for 5.9-rc1. Rajendra, Since most of these chan

Re: [PATCH] drm/amd/display: Fix memory leak in amdgpu_dm_mode_config_init()

2020-08-28 Thread Markus Elfring
> When amdgpu_display_modeset_create_props() fails, state and > state->context should be freed to prevent memleak. It's the > same when amdgpu_dm_audio_init() fails. * Can another imperative wording become helpful for the change description? * Would you like to consider the tag “Fixes” for the co

[PATCH V2 3/8] drm/msm: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. While at it, also create a label to put clkname. Signed-off-by: Viresh Kuma

[PATCH] drm/nouveau/svm: remove set but not used 'ret'

2020-08-28 Thread Jason Yan
This addresses the following gcc warning with "make W=1": drivers/gpu/drm/nouveau/nouveau_svm.c: In function ‘nouveau_pfns_map’: drivers/gpu/drm/nouveau/nouveau_svm.c:818:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable] 818 | int ret; | ^~~ Reported-by: Hulk R

Re: [PATCH V2] drm/sun4i: add missing put_device() call in sun8i_r40_tcon_tv_set_mux()

2020-08-28 Thread Maxime Ripard
On Wed, Aug 26, 2020 at 09:08:26AM +0800, Yu Kuai wrote: > If sun8i_r40_tcon_tv_set_mux() succeed, sun8i_r40_tcon_tv_set_mux() > doesn't have a corresponding put_device(). Thus add put_device() > to fix the exception handling for this function implementation. > > Fixes: 0305189afb32 ("drm/sun4i: t

Re: Aw: Re: [PATCH v5 3/7] drm/mediatek: disable tmds on mt2701

2020-08-28 Thread Frank Wunderlich
Without this patch i have flickering/horizontal distortion (looks line every second line has different x position as one above approx. 5 px) on my 1280x1024 tft. Fbcon is unreadable with this problem. Hard to describe by words only :( Am 28. August 2020 01:46:07 MESZ schrieb Chun-Kuang Hu : >

Re: [PATCH 0/8] Convert the intel iommu driver to the dma-iommu api

2020-08-28 Thread Tom Murphy
On Thu, 27 Aug 2020 at 22:36, Logan Gunthorpe wrote: > > > > On 2020-08-23 6:04 p.m., Tom Murphy wrote: > > I have added a check for the sg_dma_len == 0 : > > """ > > } __sgt_iter(struct scatterlist *sgl, bool dma) { > > struct sgt_iter s = { .sgp = sgl }; > > > > + if (sgl && sg_dm

Re: [PATCH] coccinelle: api: fix kobj_to_dev.cocci warnings

2020-08-28 Thread Julia Lawall
On Thu, 27 Aug 2020, Markus Elfring wrote: > > Generated by: scripts/coccinelle/api/kobj_to_dev.cocci > > > > Fixes: a2fc3718bc22 ("coccinelle: api: add kobj_to_dev.cocci script") > > I wonder about such a combination of information. > > I find it reasonable that two function implementations sho

[PATCH] drm/bridge/tc358775: Fix semicolon

2020-08-28 Thread Vinay Simha BN
Remove unneded semicolon Reported-by: kernel test robot Signed-off-by: Vinay Simha BN --- drivers/gpu/drm/bridge/tc358775.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c index d951cdc..2272adc 100644 --

Re: [PATCH] drm: Parse Colorimetry data block from EDID

2020-08-28 Thread crj
Hi Ville Syrjälä, 在 2020/8/27 18:57, Ville Syrjälä 写道: On Wed, Aug 26, 2020 at 10:23:28PM +0800, Algea Cao wrote: CEA 861.3 spec adds colorimetry data block for HDMI. Parsing the block to get the colorimetry data from panel. And what exactly do you want to do with that data? We can get colo

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread Shuah Khan
On 8/27/20 6:36 AM, Greg Kroah-Hartman wrote: The ION android code has long been marked to be removed, now that we dma-buf support merged into the real part of the kernel. It was thought that we could wait to remove the ion kernel at a later time, but as the out-of-tree Android fork of the ion c

[PULL] drm-misc-next

2020-08-28 Thread Maxime Ripard
Hi Daniel, Dave, Here's a re-run of last week's PR (without all the -rc1 churn) plus the patches that got in this week. Thanks! Maxime drm-misc-next-2020-08-27: drm-misc-next for 5.10: UAPI Changes: Cross-subsystem Changes: Core Changes: - ttm: various cleanups and reworks of the API Drive

Re: [PATCH] video: fbdev: controlfb: Fix build for COMPILE_TEST=y && PPC_PMAC=n

2020-08-28 Thread Michael Ellerman
On Fri, 21 Aug 2020 20:49:10 +1000, Michael Ellerman wrote: > The build is currently broken, if COMPILE_TEST=y and PPC_PMAC=n: > > linux/drivers/video/fbdev/controlfb.c: In function > ‘control_set_hardware’: > linux/drivers/video/fbdev/controlfb.c:276:2: error: implicit declaration of >

Re: [PATCH] coccinelle: api: fix kobj_to_dev.cocci warnings

2020-08-28 Thread Markus Elfring
> Generated by: scripts/coccinelle/api/kobj_to_dev.cocci > > Fixes: a2fc3718bc22 ("coccinelle: api: add kobj_to_dev.cocci script") I wonder about such a combination of information. I find it reasonable that two function implementations should be adjusted according to a generated patch. Thus I ima

[PATCH v2 2/2] drm/ingenic: Fix driver not probing when IPU port is missing

2020-08-28 Thread Paul Cercueil
Even if support for the IPU was compiled in, we may run on a device (e.g. the Qi LB60) where the IPU is not available, or simply with an old devicetree without the IPU node. In that case the ingenic-drm refused to probe. Fix the driver so that it will probe even if the IPU node is not present in d

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread Laura Abbott
On 8/27/20 8:36 AM, Greg Kroah-Hartman wrote: The ION android code has long been marked to be removed, now that we dma-buf support merged into the real part of the kernel. It was thought that we could wait to remove the ion kernel at a later time, but as the out-of-tree Android fork of the ion c

[PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-08-28 Thread Enric Balletbo i Serra
The get_edid() callback can be triggered anytime by an ioctl, i.e drm_mode_getconnector (ioctl) -> drm_helper_probe_single_connector_modes -> drm_bridge_connector_get_modes -> ps8640_bridge_get_edid Actually if the bridge pre_enable() function was not called before get_edid

Re: [PATCH v2 5/5] drm/bridge: ps8640: Rework power state handling

2020-08-28 Thread Enric Balletbo i Serra
Hi Sam, Thanks for your comments. On 26/8/20 20:46, Sam Ravnborg wrote: > Hi Enric. > > On Wed, Aug 26, 2020 at 10:15:26AM +0200, Enric Balletbo i Serra wrote: >> The get_edid() callback can be triggered anytime by an ioctl, i.e >> >> drm_mode_getconnector (ioctl) >> -> drm_helper_probe_si

Re: [PATCH] staging: ion: remove from the tree

2020-08-28 Thread Amit Pundir
On Thu, 27 Aug 2020 at 21:34, Greg Kroah-Hartman wrote: > > On Thu, Aug 27, 2020 at 09:31:27AM -0400, Laura Abbott wrote: > > On 8/27/20 8:36 AM, Greg Kroah-Hartman wrote: > > > The ION android code has long been marked to be removed, now that we > > > dma-buf support merged into the real part of