Re: [PATCH v5] drm/i915: Init lspcon chip dynamically

2020-06-08 Thread Kai-Heng Feng
> On May 6, 2020, at 18:28, Kai-Heng Feng wrote: > > On HP 800 G4 DM, if HDMI cable isn't plugged before boot, the HDMI port > becomes useless and never responds to cable hotplugging: > [3.031904] [drm:lspcon_init [i915]] *ERROR* Failed to probe lspcon > [3.031945] [drm:intel_ddi_init [

[PATCH] drm/msm/dpu: fix error return code in dpu_encoder_init

2020-06-08 Thread Chen Tao
Fix to return negative error code -ENOMEM with the use of ERR_PTR from dpu_encoder_init. Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support") Signed-off-by: Chen Tao --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu

Re: [PATCH v2] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
> On Jun 7, 2020, at 11:16 PM, Laurent Pinchart > wrote: > > I forgot to mention, I think the subject line should be > > drm/rcar-du: Make DRM_RCAR_WRITEBACK depend on DRM_RCAR_DU > > Could you please let me know if you're OK with these two small changes ? Yes, I am fine with those. __

[PATCH v2 00/18] seqlock: Extend seqcount API with associated locks

2020-06-08 Thread Ahmed S. Darwish
Hi, This is v2 of the seqlock patch series: [PATCH v1 00/25] seqlock: Extend seqcount API with associated locks https://lore.kernel.org/lkml/20200519214547.352050-1-a.darw...@linutronix.de Patches 1=>3 of this v2 series add documentation for the existing seqlock.h datatypes and APIs. Hopef

[PATCH v2 05/18] dma-buf: Remove custom seqcount lockdep class key

2020-06-08 Thread Ahmed S. Darwish
Commit 3c3b177a9369 ("reservation: add support for read-only access using rcu") introduced a sequence counter to manage updates to reservations. Back then, the reservation object initializer reservation_object_init() was always inlined. Having the sequence counter initialization inlined meant that

Re: [PATCH v2 03/15] pwm: lpss: Add range limit check for the base_unit register value

2020-06-08 Thread Andy Shevchenko
On Sun, Jun 07, 2020 at 08:18:28PM +0200, Hans de Goede wrote: > When the user requests a high enough period ns value, then the > calculations in pwm_lpss_prepare() might result in a base_unit value of 0. > > But according to the data-sheet the way the PWM controller works is that > each input clo

[PATCH 2/2] iommu: Add gfp parameter to io_pgtable_ops->map()

2020-06-08 Thread Baolin Wang
Now the ARM page tables are always allocated by GFP_ATOMIC parameter, but the iommu_ops->map() function has been added a gfp_t parameter by commit 781ca2de89ba ("iommu: Add gfp parameter to iommu_ops::map"), thus io_pgtable_ops->map() should use the gfp parameter passed from iommu_ops->map() to all

[PATCH v2] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
There is no need to select DRM_RCAR_WRITEBACK if DRM=n which just make the generated .config a bit ugly. # ARM devices # # end of ARM devices CONFIG_DRM_RCAR_WRITEBACK=y # # Frame buffer Devices Let DRM_RCAR_WRITEBACK depend on DRM_RCAR_DU instead. Signed-off-by: Qian Cai --- drivers/g

[PATCH 1/2] iommu: Mark __iommu_map/__iommu_map_sg as static

2020-06-08 Thread Baolin Wang
Now the __iommu_map() and __iommu_map_sg() are used only in iommu.c file, so mark them as static. Signed-off-by: Baolin Wang --- drivers/iommu/iommu.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c index ac91024..04ba8

[PATCH 0/2] Some improvements for iommu

2020-06-08 Thread Baolin Wang
Hi, The first patch masks some functions as static, and the second patch changes to use the gfp parameter from iommu_ops->map() to allocate ARM page pages. Any comments are welcome. Thanks. Baolin Wang (2): iommu: Mark __iommu_map/__iommu_map_sg as static iommu: Add gfp parameter to io_pgtabl

Re: [PATCH v2 04/15] pwm: lpss: Fix off by one error in base_unit math in pwm_lpss_prepare()

2020-06-08 Thread Andy Shevchenko
On Sun, Jun 07, 2020 at 08:18:29PM +0200, Hans de Goede wrote: > According to the data-sheet the way the PWM controller works is that > each input clock-cycle the base_unit gets added to a N bit counter and > that counter overflowing determines the PWM output frequency. > > So assuming e.g. a 16 b

Re: [PATCH v4 1/3] virtio: add dma-buf support for exported objects

2020-06-08 Thread David Stevens
On Sun, Jun 7, 2020 at 5:04 AM Michael S. Tsirkin wrote: > > On Fri, Jun 05, 2020 at 10:28:42AM +0900, David Stevens wrote: > > On Fri, Jun 5, 2020 at 4:05 AM Michael S. Tsirkin wrote: > > > > > > On Tue, May 26, 2020 at 07:58:09PM +0900, David Stevens wrote: > > > > This change adds a new flavor

[PATCH v2 06/18] dma-buf: Use sequence counter with associated wound/wait mutex

2020-06-08 Thread Ahmed S. Darwish
A sequence counter write side critical section must be protected by some form of locking to serialize writers. If the serialization primitive is not disabling preemption implicitly, preemption has to be explicitly disabled before entering the sequence counter write side critical section. The dma-b

[PATCH] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
There is no need to select DRM_RCAR_WRITEBACK if DRM=n which just make the generated .config a bit ugly. # ARM devices # # end of ARM devices CONFIG_DRM_RCAR_WRITEBACK=y # # Frame buffer Devices Signed-off-by: Qian Cai --- drivers/gpu/drm/rcar-du/Kconfig | 1 + 1 file changed, 1 inserti

Re: [PATCH] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-06-08 Thread Qian Cai
> On Jun 7, 2020, at 10:02 PM, Laurent Pinchart > wrote: > > How about depending on DRM_RCAR_DU instead, as DRM_RCAR_WRITEBACK is > used to select compilation of rcar_du_writeback.c that is part of the > rcar-du driver ? Sure. I’ll send a v2. ___ dr

Re: [PATCH v3 31/39] memory: tegra20-emc: Register as interconnect provider

2020-06-08 Thread Dmitry Osipenko
07.06.2020 21:55, Dmitry Osipenko пишет: ... > + if (IS_ENABLED(CONFIG_INTERCONNECT)) { > + err = tegra_emc_interconnect_init(emc); > + if (err) > + dev_err(&pdev->dev, "failed to initialize ICC: %d\n", > + err); > + }

Re: A panic and a hang in the i915 drm driver

2020-06-08 Thread Jani Nikula
On Sun, 07 Jun 2020, David Howells wrote: > Hi, > > I'm seeing the attached oops and panic from the i915 drm driver. I've tried > bisecting it, but there's a problem in that one of the merged branches causes > the machine to hang without output. Cc: Ville and GG, I thought this was fixed (revert

Re: [PATCH] drm/panfrost: Use kvfree() to free bo->sgts in panfrost_mmu_map_fault_addr()

2020-06-08 Thread Steven Price
On 05/06/2020 19:52, Denis Efremov wrote: Use kvfree() to free bo->sgts, because the memory is allocated with kvmalloc_array(). Fixes: 187d2929206e ("drm/panfrost: Add support for GPU heap allocations") Cc: sta...@vger.kernel.org Signed-off-by: Denis Efremov Well spotted, but there's another

RE: [Intel-gfx] A panic and a hang in the i915 drm driver

2020-06-08 Thread Saarinen, Jani
HI, > -Original Message- > From: Intel-gfx On Behalf Of Jani > Nikula > Sent: maanantai 8. kesäkuuta 2020 10.49 > To: David Howells ; Joonas Lahtinen > ; Vivi, Rodrigo > Cc: intel-...@lists.freedesktop.org; linux-ker...@vger.kernel.org; dri- > de...@lists.freedesktop.org; dhowe...@redha

Re: [PATCH v4 1/3] virtio: add dma-buf support for exported objects

2020-06-08 Thread Michael S. Tsirkin
On Mon, Jun 08, 2020 at 05:32:26PM +0900, David Stevens wrote: > On Mon, Jun 8, 2020 at 3:00 PM Michael S. Tsirkin wrote: > > > > On Mon, Jun 08, 2020 at 10:33:09AM +0900, David Stevens wrote: > > > On Sun, Jun 7, 2020 at 5:04 AM Michael S. Tsirkin wrote: > > > > > > > > On Fri, Jun 05, 2020 at 1

Re: [PATCH 1/2] drm: drm_fourcc: add NV20 YUV format

2020-06-08 Thread Brian Starkey
Hi Jonas, On Sun, Jun 07, 2020 at 08:25:25PM +, Jonas Karlman wrote: > DRM_FORMAT_NV20 is a 2 plane format suitable for linear memory layout. > The format is similar to P210 with 4:2:2 sub-sampling but has no padding > between components. Instead, luminance and chrominance samples are grouped

Re: [PATCH v3 4/4] drm/virtio: Support virtgpu exported resources

2020-06-08 Thread Michael S. Tsirkin
On Fri, May 15, 2020 at 04:26:15PM +0900, David Stevens wrote: > > > + if (virtio_has_feature(vgdev->vdev, VIRTIO_GPU_F_RESOURCE_UUID)) { > > > + vgdev->has_resource_assign_uuid = true; > > > + } > > > > > > Just a question: this relies on DMA bufs so I assume it is > > not real

[Bug 71891] 3.13 fails to boot with the radeon module

2020-06-08 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=71891 Mike Diehl (noahwittman0...@gmail.com) changed: What|Removed |Added CC||noahwittman0...@gm

Re: [PATCH v2 03/15] pwm: lpss: Add range limit check for the base_unit register value

2020-06-08 Thread Hans de Goede
Hi, On 6/8/20 5:50 AM, Andy Shevchenko wrote: On Sun, Jun 07, 2020 at 08:18:28PM +0200, Hans de Goede wrote: When the user requests a high enough period ns value, then the calculations in pwm_lpss_prepare() might result in a base_unit value of 0. But according to the data-sheet the way the PWM

Re: [PATCH v2 04/15] pwm: lpss: Fix off by one error in base_unit math in pwm_lpss_prepare()

2020-06-08 Thread Hans de Goede
Hi, On 6/8/20 5:55 AM, Andy Shevchenko wrote: On Sun, Jun 07, 2020 at 08:18:29PM +0200, Hans de Goede wrote: According to the data-sheet the way the PWM controller works is that each input clock-cycle the base_unit gets added to a N bit counter and that counter overflowing determines the PWM ou

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-08 Thread Dan Carpenter
Hi Lukasz, I love your patch! Perhaps something to improve: url: https://github.com/0day-ci/linux/commits/Lukasz-Luba/Add-support-for-devices-in-the-Energy-Model/20200527-180614 base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next config: i386-randconfig-m0

Re: [PATCH 08/15] drm/panfrost: move devfreq_init()/fini() in device

2020-06-08 Thread Tomeu Vizoso
On 5/29/20 2:38 PM, Clément Péron wrote: Hi Steven On Thu, 28 May 2020 at 15:22, Steven Price wrote: On 10/05/2020 17:55, Clément Péron wrote: Later we will introduce devfreq probing regulator if they are present. As regulator should be probe only one time we need to get this logic in the de

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-08 Thread Dan Carpenter
On Mon, Jun 08, 2020 at 01:34:37PM +0100, Lukasz Luba wrote: > Hi Dan, > > Thank you for your analyzes. > > On 6/8/20 12:51 PM, Dan Carpenter wrote: > > Hi Lukasz, > > > > I love your patch! Perhaps something to improve: > > > > url: > > https://github.com/0day-ci/linux/commits/Lukasz-Luba/

Re: [PATCH 07/14] drm/mgag200: Switch to managed MM

2020-06-08 Thread Thomas Zimmermann
Hi Am 05.06.20 um 18:22 schrieb Daniel Vetter: > On Fri, Jun 05, 2020 at 03:57:56PM +0200, Thomas Zimmermann wrote: >> The memory-management code now cleans up automatically as part of >> device destruction. >> >> Signed-off-by: Thomas Zimmermann >> --- >> drivers/gpu/drm/mgag200/mgag200_drv.h

Re: [PATCH 06/14] drm/mgag200: Merge VRAM setup into MM initialization

2020-06-08 Thread Thomas Zimmermann
Hi Am 05.06.20 um 16:39 schrieb Sam Ravnborg: > Hi Thomas. > > Some parts I did not understand here. > > On Fri, Jun 05, 2020 at 03:57:55PM +0200, Thomas Zimmermann wrote: >> The VRAM setup in mgag200_drv.c is part of memory management and >> should be done in the same place. Merge the code into

Re: [PATCH 09/14] drm/mgag200: Prefix global names in mgag200_drv.c with mgag200_

2020-06-08 Thread Thomas Zimmermann
Hi Am 05.06.20 um 16:42 schrieb Sam Ravnborg: > On Fri, Jun 05, 2020 at 03:57:58PM +0200, Thomas Zimmermann wrote: >> The naming of global symbols in mgag200_drv.c is inconsistent. Fix >> that by prefixing all names with mgag200_. > > Hmm, static symbols are hardly global symbols. > Patch is fine

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-08 Thread Dan Carpenter
It's not really a proper bug report so it doesn't deserve a reported-by. Thanks, though! regards, dan carpenter ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH] drm/amd/display: Fix indenting in dcn30_set_output_transfer_func()

2020-06-08 Thread Dan Carpenter
These lines are a part of the if statement and they are supposed to be indented one more tab. Signed-off-by: Dan Carpenter --- drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hwseq.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_h

Re: [PATCH v2 03/15] pwm: lpss: Add range limit check for the base_unit register value

2020-06-08 Thread Hans de Goede
Hi, On 6/8/20 2:51 PM, Andy Shevchenko wrote: On Mon, Jun 08, 2020 at 01:07:12PM +0200, Hans de Goede wrote: On 6/8/20 5:50 AM, Andy Shevchenko wrote: On Sun, Jun 07, 2020 at 08:18:28PM +0200, Hans de Goede wrote: When the user requests a high enough period ns value, then the calculations in

Re: [PATCH v2 06/18] dma-buf: Use sequence counter with associated wound/wait mutex

2020-06-08 Thread Daniel Vetter
On Mon, Jun 08, 2020 at 02:57:17AM +0200, Ahmed S. Darwish wrote: > A sequence counter write side critical section must be protected by some > form of locking to serialize writers. If the serialization primitive is > not disabling preemption implicitly, preemption has to be explicitly > disabled be

Re: pwm/i915: Convert pwm-crc and i915 driver's PWM code to use the atomic PWM API

2020-06-08 Thread Daniel Vetter
On Sat, Jun 06, 2020 at 10:25:45PM +0200, Hans de Goede wrote: > Hi All, > > This patch series converts the i915 driver's cpde for controlling the > panel's backlight with an external PWM controller to use the atomic PWM API. > > Initially the plan was for this series to consist of 2 parts: > 1.

Re: [PATCH 8/9] drm/shmem-helpers: Ensure get_pages is not called on imported dma-buf

2020-06-08 Thread Thomas Zimmermann
Hi Am 03.06.20 um 15:12 schrieb Daniel Vetter: > On Thu, May 14, 2020 at 09:30:04AM +0200, Thomas Zimmermann wrote: >> Hi >> >> Am 11.05.20 um 11:35 schrieb Daniel Vetter: >>> Just a bit of light paranoia. Also sprinkle this check over >>> drm_gem_shmem_get_sg_table, which should only be called wh

Re: [PATCH 8/9] drm/shmem-helpers: Ensure get_pages is not called on imported dma-buf

2020-06-08 Thread Daniel Vetter
On Mon, Jun 08, 2020 at 04:40:26PM +0200, Thomas Zimmermann wrote: > Hi > > Am 03.06.20 um 15:12 schrieb Daniel Vetter: > > On Thu, May 14, 2020 at 09:30:04AM +0200, Thomas Zimmermann wrote: > >> Hi > >> > >> Am 11.05.20 um 11:35 schrieb Daniel Vetter: > >>> Just a bit of light paranoia. Also spri

[PATCH] drm/mm: remove invalid entry based optimization

2020-06-08 Thread Christian König
When the current entry is rejected as candidate for the search it does not mean that we can abort the subtree search. It is perfectly possible that only the alignment, but not the size is the reason for the rejection. Signed-off-by: Christian König --- drivers/gpu/drm/drm_mm.c | 6 ++ 1 fil

Re: [PATCH v3 4/4] drm/virtio: Support virtgpu exported resources

2020-06-08 Thread Michael S. Tsirkin
On Mon, Jun 08, 2020 at 07:36:55PM +0900, David Stevens wrote: > On Mon, Jun 8, 2020 at 6:43 PM Michael S. Tsirkin wrote: > > > > On Fri, May 15, 2020 at 04:26:15PM +0900, David Stevens wrote: > > > > > + if (virtio_has_feature(vgdev->vdev, > > > > > VIRTIO_GPU_F_RESOURCE_UUID)) { > > > > > +

Re: [PATCH v3] drm/doc: device hot-unplug for userspace

2020-06-08 Thread Pekka Paalanen
On Mon, 1 Jun 2020 17:32:03 +0300 Pekka Paalanen wrote: > Set up the expectations on how hot-unplugging a DRM device should look like to > userspace. > > Written by Daniel Vetter's request and largely based on his comments in IRC > and > from https://lists.freedesktop.org/archives/dri-devel/20

Re: [PATCH] drm/mm: remove invalid entry based optimization

2020-06-08 Thread Nirmoy
On 6/8/20 5:15 PM, Christian König wrote: When the current entry is rejected as candidate for the search it does not mean that we can abort the subtree search. It is perfectly possible that only the alignment, but not the size is the reason for the rejection. I know why I  did that, I was te

Re: [Intel-gfx] A panic and a hang in the i915 drm driver

2020-06-08 Thread David Howells
Jani Nikula wrote: > David, please try [1]. Assuming you mean this: https://patchwork.freedesktop.org/patch/366958/?series=77635&rev=1 yes, that works. Tested-by: David Howells ___ dri-devel mailing list dri-devel@lists.freedesktop.org https:/

Re: [PATCH] drm/amd/display: Fix indenting in dcn30_set_output_transfer_func()

2020-06-08 Thread Joe Perches
On Mon, 2020-06-08 at 17:16 +0300, Dan Carpenter wrote: > These lines are a part of the if statement and they are supposed to > be indented one more tab. > > Signed-off-by: Dan Carpenter > --- > drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hwseq.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 d

[PATCH 2/4] drm/bridge: ti-sn65dsi86: Don't use kernel-doc comment for local array

2020-06-08 Thread Douglas Anderson
When building we were getting an error: warning: cannot understand function prototype: 'const unsigned int ti_sn_bridge_dp_rate_lut[] = ' Arrays aren't supposed to be marked with "/**" kerneldoc comments. Fix. Fixes: a095f15c00e2 ("drm/bridge: add support for sn65dsi86 bridge driver") Sig

[PATCH 1/4] drm/bridge: ti-sn65dsi86: Don't compile GPIO bits if not CONFIG_OF_GPIO

2020-06-08 Thread Douglas Anderson
The kernel test robot noted that if "OF" is defined (which is needed to select DRM_TI_SN65DSI86 at all) but not OF_GPIO that we'd get compile failures because some of the members that we access in "struct gpio_chip" are only defined "#if defined(CONFIG_OF_GPIO)". All the GPIO bits in the driver ar

[PATCH 3/4] drm/bridge: ti-sn65dsi86: Fix kernel-doc typo ln_polr => ln_polrs

2020-06-08 Thread Douglas Anderson
This fixes a kernel doc warning due to a typo: warning: Function parameter or member 'ln_polrs' not described in 'ti_sn_bridge' Fixes: 5bebaeadb30e ("drm/bridge: ti-sn65dsi86: Implement lane reordering + polarity") Signed-off-by: Douglas Anderson --- drivers/gpu/drm/bridge/ti-sn65dsi86.c |

[PATCH 4/4] drm/bridge: ti-sn65dsi86: Check the regmap return value when setting a GPIO

2020-06-08 Thread Douglas Anderson
The ti_sn_bridge_gpio_set() got the return value of regmap_update_bits() but didn't check it. The function can't return an error value, but we should at least print a warning if it didn't work. This fixes a compiler warning about setting "ret" but not using it. Fixes: 27ed2b3f22ed ("drm/bridge:

Re: [PATCH] drm/amd/display: Fix indenting in dcn30_set_output_transfer_func()

2020-06-08 Thread Dan Carpenter
On Mon, Jun 08, 2020 at 10:16:27AM -0700, Joe Perches wrote: > On Mon, 2020-06-08 at 17:16 +0300, Dan Carpenter wrote: > > These lines are a part of the if statement and they are supposed to > > be indented one more tab. > > > > Signed-off-by: Dan Carpenter > > --- > > drivers/gpu/drm/amd/displa

Re: [git pull] drm next fixes for 5.7-rc1

2020-06-08 Thread pr-tracker-bot
The pull request you sent on Mon, 8 Jun 2020 13:15:58 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-next-2020-06-08 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/107821669a9cbf234f260d576039983b64c7cb6d Thank you! -- Deet-doot-dot, I am a bot. https://kor

Re: [git pull] drm msm next for 5.8-rc1

2020-06-08 Thread pr-tracker-bot
The pull request you sent on Mon, 8 Jun 2020 13:33:51 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-next-msm-5.8-2020-06-08 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/9413b9a690ec8aeaedea74bb875079d36f295304 Thank you! -- Deet-doot-dot, I am a bot. htt

Re: [PATCH] drm/amd/display: Fix indenting in dcn30_set_output_transfer_func()

2020-06-08 Thread Joe Perches
On Mon, 2020-06-08 at 20:49 +0300, Dan Carpenter wrote: > On Mon, Jun 08, 2020 at 10:16:27AM -0700, Joe Perches wrote: > > On Mon, 2020-06-08 at 17:16 +0300, Dan Carpenter wrote: > > > These lines are a part of the if statement and they are supposed to > > > be indented one more tab. > > > > > > S

[PATCH] drm/edid: add support for min horizontal rate equal to max horizontal rate

2020-06-08 Thread Cyrus Lien
According to EDID spec, table 3.26, byte #6 and #8, which said "Minimum rate value shall be less than or equal to maximum rate value". The minimum horizontal/vertical rate value is able to be equal to maximum horizontal/ veritcal rate value. This change check if h/v-sync excess maximum horizontal/

Re: [PATCH] drm/amd/display: Fix indenting in dcn30_set_output_transfer_func()

2020-06-08 Thread Alex Deucher
On Mon, Jun 8, 2020 at 10:17 AM Dan Carpenter wrote: > > These lines are a part of the if statement and they are supposed to > be indented one more tab. > > Signed-off-by: Dan Carpenter Applied. thanks! Alex > --- > drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hwseq.c | 6 +++--- > 1 file chan

[PATCH v5 00/13] drm/trace: Mirror DRM debug logs to tracefs

2020-06-08 Thread Sean Paul
From: Sean Paul This series is the latest in my journey to create a lightweight, always-on "flight recorder" (name credit Weston) of drm logs. This incarnation uses a trace_array to keep logs in memory exposed through tracefs. Users and distros can enable drm logs by using the drm.trace module pa

[PATCH v5 03/13] drm/i915/utils: Replace dev_printk with drm helpers

2020-06-08 Thread Sean Paul
From: Sean Paul Use drm logging helpers to add support for the upcoming tracefs implementation. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/i915/i915_utils.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_ut

[PATCH v5 02/13] drm/sil164: Convert dev_printk to drm_dev_dbg

2020-06-08 Thread Sean Paul
From: Sean Paul Use the drm debug helper instead of dev_printk in order to leverage the upcoming tracefs support Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/i2c/sil164_drv.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/driv

[PATCH v5 04/13] drm/msm/dpu: Replace definitions for dpu debug macros

2020-06-08 Thread Sean Paul
From: Sean Paul The debug messages shouldn't be logged as errors when debug categories are enabled. Use the drm logging helpers to do the right thing Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 20 1 file changed

[PATCH v5 01/13] drm/mipi_dbi: Convert pr_debug calls to DRM_DEBUG_DRIVER

2020-06-08 Thread Sean Paul
From: Sean Paul Use the drm logging helpers to output these messages to ensure they'll be included by the drm tracefs instance. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/drm_mipi_dbi.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --gi

[PATCH v5 07/13] drm/etnaviv: Change buffer dump checks to target syslog

2020-06-08 Thread Sean Paul
From: Sean Paul Since the logs protected by these checks specifically target syslog, use the new drm_debug_syslog_enabled() call to avoid triggering these prints when only trace is enabled. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/etnaviv/etnaviv_buffer.c

[PATCH v5 06/13] drm/amd: Gate i2c transaction logs on drm_debug_syslog

2020-06-08 Thread Sean Paul
From: Sean Paul Since the logs protected by these checks specifically target syslog, use the new drm_debug_syslog_enabled() call to avoid triggering these prints when only trace is enabled. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/amd/amdgpu/smu_v11_0_i2c.

[PATCH v5 05/13] drm/print: rename drm_debug* to be more syslog-centric

2020-06-08 Thread Sean Paul
From: Sean Paul In preparation for tracefs support, rename drm_debug related functions to reflect that it targets the syslog. This will allow us to selectively target syslog and/or tracefs. No functional changes here. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/

[PATCH v5 12/13] drm/i915: Use debug category printer for welcome message

2020-06-08 Thread Sean Paul
From: Sean Paul The welcome printer is meant to be gated on DRM_UT_DRIVER, so use the debug category printer to avoid dumping the message in the wrong place. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/i915/i915_drv.c | 3 ++- 1 file changed, 2 insertions(+),

[PATCH v5 13/13] drm/print: Add tracefs support to the drm logging helpers

2020-06-08 Thread Sean Paul
From: Sean Paul This patch adds a new module parameter called drm.trace which accepts the same mask as drm.debug. When a debug category is enabled, log messages will be put in a new tracefs instance called drm for consumption. Using the new tracefs instance will allow distros to enable drm loggi

[PATCH v5 09/13] drm/i915: Change infoframe debug checks to specify syslog

2020-06-08 Thread Sean Paul
From: Sean Paul Since the logs protected by these checks specifically target syslog, use the new drm_debug_syslog_enabled() call to avoid triggering these prints when only trace is enabled. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/i915/display/intel_displa

[PATCH v5 08/13] drm/nouveau: Change debug checks to specifically target syslog

2020-06-08 Thread Sean Paul
From: Sean Paul Since the logs protected by these checks specifically target syslog, use the new drm_debug_syslog_enabled() call to avoid triggering these prints when only trace is enabled. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/nouveau/dispnv50/disp.h |

[PATCH v5 10/13] drm/print: Add drm_debug_category_printer

2020-06-08 Thread Sean Paul
From: Sean Paul This patch adds a new printer which will select the appropriate output for a given debug category. Currently there is only one output target, which is syslog. However in the future we'll have tracefs and it will be useful to print to syslog, tracefs, or both. Drivers just need to

[PATCH v5 11/13] drm/mst: Convert debug printers to debug category printers

2020-06-08 Thread Sean Paul
From: Sean Paul The printers in dp_mst are meant to be gated on DRM_UT_DP, so use the debug category printer to avoid dumping mst transactions to the wrong place. Signed-off-by: Sean Paul Changes in v5: -Added to the set --- drivers/gpu/drm/drm_dp_mst_topology.c | 9 ++--- 1 file changed,

Re: [PATCH 1/2] drm/msm/dpu: add support for color processing blocks in dpu driver

2020-06-08 Thread John Stultz
On Wed, Mar 25, 2020 at 1:17 AM Kalyan Thota wrote: > > This change adds support to configure dspp blocks in > the dpu driver. > > Macro description of the changes coming in this patch. > 1) Add dspp definitions in the hw catalog. > 2) Add capability to reserve dspp blocks in the display data path

Re: [PATCH 1/2] drm/msm/dpu: add support for color processing blocks in dpu driver

2020-06-08 Thread John Stultz
On Mon, Jun 8, 2020 at 3:25 PM John Stultz wrote: > > On Wed, Mar 25, 2020 at 1:17 AM Kalyan Thota wrote: > > > > This change adds support to configure dspp blocks in > > the dpu driver. > > > > Macro description of the changes coming in this patch. > > 1) Add dspp definitions in the hw catalog.

Re: [PATCH 1/2] drm/msm/dpu: add support for color processing blocks in dpu driver

2020-06-08 Thread Rob Clark
On Mon, Jun 8, 2020 at 3:37 PM John Stultz wrote: > > On Mon, Jun 8, 2020 at 3:25 PM John Stultz wrote: > > > > On Wed, Mar 25, 2020 at 1:17 AM Kalyan Thota > > wrote: > > > > > > This change adds support to configure dspp blocks in > > > the dpu driver. > > > > > > Macro description of the cha

Re: [PATCH v3 104/105] dt-bindings: display: vc4: hdmi: Add BCM2711 HDMI controllers bindings

2020-06-08 Thread Rob Herring
On Tue, Jun 2, 2020 at 9:08 AM Maxime Ripard wrote: > > Hi Rob, > > On Fri, May 29, 2020 at 12:18:33PM -0600, Rob Herring wrote: > > On Wed, May 27, 2020 at 05:49:14PM +0200, Maxime Ripard wrote: > > > The HDMI controllers found in the BCM2711 SoC need some adjustments to the > > > bindings, espec

[PATCH AUTOSEL 5.7 019/274] drm/amdgpu: Init data to avoid oops while reading pp_num_states.

2020-06-08 Thread Sasha Levin
From: limingyu [ Upstream commit 6f81b2d047c59eb77cd04795a44245d6a52cdaec ] For chip like CHIP_OLAND with si enabled(amdgpu.si_support=1), the amdgpu will expose pp_num_states to the /sys directory. In this moment, read the pp_num_states file will excute the amdgpu_get_pp_num_states func. In our

[PATCH AUTOSEL 5.7 020/274] drm/bridge: panel: Return always an error pointer in drm_panel_bridge_add()

2020-06-08 Thread Sasha Levin
From: Enric Balletbo i Serra [ Upstream commit 30be3031087139061de4421bf52015931eaab569 ] Since commit 89958b7cd955 ("drm/bridge: panel: Infer connector type from panel by default"), drm_panel_bridge_add() and their variants can return NULL and an error pointer. This is fine but none of the actu

[PATCH AUTOSEL 5.7 017/274] drm/amd/display: fix virtual signal dsc setup

2020-06-08 Thread Sasha Levin
From: Dmytro Laktyushkin [ Upstream commit d5bef51f084fccafa984b114ff74a01a64a0e2e3 ] This prevents dpcd access on virtual links. Signed-off-by: Dmytro Laktyushkin Reviewed-by: Eric Bernstein Acked-by: Rodrigo Siqueira Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu

[PATCH AUTOSEL 5.7 006/274] drm: bridge: adv7511: Extend list of audio sample rates

2020-06-08 Thread Sasha Levin
From: Bogdan Togorean [ Upstream commit b97b6a1f6e14a25d1e1ca2a46c5fa3e2ca374e22 ] ADV7511 support sample rates up to 192kHz. CTS and N parameters should be computed accordingly so this commit extend the list up to maximum supported sample rate. Signed-off-by: Bogdan Togorean Reviewed-by: Andr

[PATCH AUTOSEL 5.7 016/274] drm/amd/display: Force watermark value propagation

2020-06-08 Thread Sasha Levin
From: Joshua Aberback [ Upstream commit 868149c9a072cbdc22a73ce25a487f9fbfa171ef ] [Why] The HUBBUB watermark registers are in an area that cannot be power gated, but the HUBP copies of the watermark values are in areas that can be power gated. When we power on a pipe, it will not automatically

[PATCH AUTOSEL 5.7 001/274] drm/amdgpu: fix and cleanup amdgpu_gem_object_close v4

2020-06-08 Thread Sasha Levin
From: Christian König [ Upstream commit 82c416b13cb7d22b96ec0888b296a48dff8a09eb ] The problem is that we can't add the clear fence to the BO when there is an exclusive fence on it since we can't guarantee the the clear fence will complete after the exclusive one. To fix this refactor the funct

[PATCH AUTOSEL 5.7 046/274] drm: rcar-du: Set primary plane zpos immutably at initializing

2020-06-08 Thread Sasha Levin
From: Tomohito Esaki [ Upstream commit 7982471d01aac33994276bf567c8f1f3a137648a ] According to drm_plane_create_zpos_property() function documentation, all planes zpos range should be set if zpos property is supported. However, the rcar-du driver didn't set primary plane zpos range. Since the pr

[PATCH AUTOSEL 5.7 060/274] drm/dp: Lenovo X13 Yoga OLED panel brightness fix

2020-06-08 Thread Sasha Levin
From: Mark Pearson [ Upstream commit 0df3ff451287d71c620384eb7bb2cd3a8106412c ] Add another panel that needs the edid quirk to the list so that brightness control works correctly. Fixes issue seen on Lenovo X13 Yoga with OLED panel Co-developed-by: jendr...@lenovo.com Signed-off-by: Mark Pearso

[PATCH AUTOSEL 5.7 116/274] drm/amd/display: Correct updating logic of dcn21's pipe VM flags

2020-06-08 Thread Sasha Levin
From: Dale Zhao [ Upstream commit 2a28fe92220a116735ef45939b7edcfee83cc6b0 ] [Why]: Renoir's pipe VM flags are not correctly updated if pipe strategy has changed during some scenarios. It will result in watermarks mistakenly calculation, thus underflow and garbage appear. [How]: Correctly updat

[PATCH AUTOSEL 5.7 087/274] staging: android: ion: use vmap instead of vm_map_ram

2020-06-08 Thread Sasha Levin
From: Christoph Hellwig [ Upstream commit 5bf9917452112694b2c774465ee4dbe441c84b77 ] vm_map_ram can keep mappings around after the vm_unmap_ram. Using that with non-PAGE_KERNEL mappings can lead to all kinds of aliasing issues. Signed-off-by: Christoph Hellwig Signed-off-by: Andrew Morton Ac

[PATCH AUTOSEL 5.7 106/274] drm/mediatek: set dpi pin mode to gpio low to avoid leakage current

2020-06-08 Thread Sasha Levin
From: Jitao Shi [ Upstream commit 6bd4763fd532cff43f9b15704f324c45a9806f53 ] Config dpi pins mode to output and pull low when dpi is disabled. Aovid leakage current from some dpi pins (Hsync Vsync DE ... ). Signed-off-by: Jitao Shi Signed-off-by: Chun-Kuang Hu Signed-off-by: Sasha Levin ---

[PATCH AUTOSEL 5.7 090/274] drm/hisilicon: Enforce 128-byte stride alignment to fix the hardware limitation

2020-06-08 Thread Sasha Levin
From: Tian Tao [ Upstream commit 9c9a8468de21895abc43f45fc86346467217c986 ] because the hardware limitation,The initial color depth must set to 32bpp and must set the FB Offset of the display hardware to 128Byte alignment, which is used to solve the display problem at 800x600 and 1440x900 resolu

[PATCH AUTOSEL 5.7 104/274] drm/ast: Allocate initial CRTC state of the correct size

2020-06-08 Thread Sasha Levin
From: Thomas Zimmermann [ Upstream commit f0adbc382b8bb46a2467c4e5e1027763a197c8e1 ] The ast driver inherits from DRM's CRTC state, but still uses the atomic helper for struct drm_crtc_funcs.reset, drm_atomic_helper_crtc_reset(). The helper only allocates enough memory for the core CRTC state.

[PATCH AUTOSEL 5.7 095/274] drm/amd/display: Revert to old formula in set_vtg_params

2020-06-08 Thread Sasha Levin
From: Alvin Lee [ Upstream commit a1a0e61f3c43c610f0a3c109348c14ce930c1977 ] [Why] New formula + cursor change causing underflow on certain configs [How] Rever to old formula Signed-off-by: Alvin Lee Reviewed-by: Yongqiang Sun Acked-by: Rodrigo Siqueira Signed-off-by: Alex Deucher Signed-o

[PATCH AUTOSEL 5.7 105/274] dt-bindings: display: mediatek: control dpi pins mode to avoid leakage

2020-06-08 Thread Sasha Levin
From: Jitao Shi [ Upstream commit b0ff9b590733079f7f9453e5976a9dd2630949e3 ] Add property "pinctrl-names" to swap pin mode between gpio and dpi mode. Set the dpi pins to gpio mode and output-low to avoid leakage current when dpi disabled. Acked-by: Rob Herring Signed-off-by: Jitao Shi Signed-

[PATCH AUTOSEL 5.7 127/274] drm/amd/display: Do not disable pipe split if mode is not supported

2020-06-08 Thread Sasha Levin
From: Sung Lee [ Upstream commit 1dfedb39d38f813357885e19badd1971c17f79a7 ] [WHY] If mode is not supported, pipe split should not be disabled. This may cause more modes to fail. [HOW] Check for mode support before disabling pipe split. This commit was previously reverted as it was thought to h

[PATCH AUTOSEL 5.7 117/274] drm/amd/display: dmcu wait loop calculation is incorrect in RV

2020-06-08 Thread Sasha Levin
From: Paul Hsieh [ Upstream commit 7fc5c319efceaed1a23b7ef35c333553ce39fecf ] [Why] Driver already get display clock from SMU base on MHz, but driver read again and mutiple 1000 cause wait loop value is overflow. [How] remove coding error Signed-off-by: Paul Hsieh Reviewed-by: Eric Yang Acke

[PATCH AUTOSEL 5.7 138/274] drm/bridge: fix stack usage warning on old gcc

2020-06-08 Thread Sasha Levin
From: Arnd Bergmann [ Upstream commit 78b0d99a68ecdc84728c99f4fef71942e9ecf35a ] Some older versions of gcc badly optimize code that passes an inline function argument into another function by reference, causing huge stack usage: drivers/gpu/drm/bridge/tc358768.c: In function 'tc358768_bridge_p

[PATCH AUTOSEL 5.7 169/274] drm/vkms: Hold gem object while still in-use

2020-06-08 Thread Sasha Levin
From: Ezequiel Garcia [ Upstream commit 0ea2ea42b31abc1141f2fd3911f952a97d401fcb ] We need to keep the reference to the drm_gem_object until the last access by vkms_dumb_create. Therefore, the put the object after it is used. This fixes a use-after-free issue reported by syzbot. While here, c

[PATCH AUTOSEL 5.7 188/274] drm/mcde: dsi: Fix return value check in mcde_dsi_bind()

2020-06-08 Thread Sasha Levin
From: Wei Yongjun [ Upstream commit 761e9f4f80a21a4b845097027030bef863001636 ] The of_drm_find_bridge() function returns NULL on error, it doesn't return error pointers so this check doesn't work. Fixes: 5fc537bfd000 ("drm/mcde: Add new driver for ST-Ericsson MCDE") Signed-off-by: Wei Yongjun

[PATCH AUTOSEL 5.7 223/274] drm/amd/powerpay: Disable gfxoff when setting manual mode on picasso and raven

2020-06-08 Thread Sasha Levin
From: chen gong [ Upstream commit cbd2d08c7463e78d625a69e9db27ad3004cbbd99 ] [Problem description] 1. Boot up picasso platform, launches desktop, Don't do anything (APU enter into "gfxoff" state) 2. Remote login to platform using SSH, then type the command line: sudo su -c "echo manual

[PATCH AUTOSEL 5.7 224/274] drm/amdgpu: Sync with VM root BO when switching VM to CPU update mode

2020-06-08 Thread Sasha Levin
From: Felix Kuehling [ Upstream commit 90ca78deb004abe75b5024968a199acb96bb70f9 ] This fixes an intermittent bug where a root PD clear operation still in progress could overwrite a PDE update done by the CPU, resulting in a VM fault. Fixes: 108b4d928c03 ("drm/amd/amdgpu: Update VM function poin

[PATCH AUTOSEL 5.6 003/606] drm/i915: Handle idling during i915_gem_evict_something busy loops

2020-06-08 Thread Sasha Levin
From: Chris Wilson [ Upstream commit 955da9d77435acac066139e9d7f7723ce7204a1d ] i915_gem_evict_something() is charged with finding a slot within the GTT that we may reuse. Since our goal is not to stall, we first look for a slot that only overlaps idle vma. To this end, on the first pass we move

[PATCH AUTOSEL 5.6 037/606] drm/i915/tgl+: Fix interrupt handling for DP AUX transactions

2020-06-08 Thread Sasha Levin
From: Imre Deak commit 4457a9db2bdec2360ddb15242341696108167886 upstream. Unmask/enable AUX interrupts on all ports on TGL+. So far the interrupts worked only on port A, which meant each transaction on other ports took 10ms. Cc: # v5.4+ Signed-off-by: Imre Deak Reviewed-by: Ville Syrjälä Lin

[PATCH AUTOSEL 5.6 031/606] Make the "Reducing compressed framebufer size" message be DRM_INFO_ONCE()

2020-06-08 Thread Sasha Levin
From: Peter Jones commit 82152d424b6cb6fc1ede7d03d69c04e786688740 upstream. This was sort of annoying me: random:~$ dmesg | tail -1 [523884.039227] [drm] Reducing the compressed framebuffer size. This may lead to less power savings than a non-reduced-size. Try to increase stolen memory size i

[PATCH AUTOSEL 5.6 036/606] drm/amd/amdgpu: add raven1 part to the gfxoff quirk list

2020-06-08 Thread Sasha Levin
From: Tom St Denis commit 975f543e7522e17b8a4bf34d7daeac44819aee5a upstream. On my raven1 system (rev c6) with VBIOS 113-RAVEN-114 GFXOFF is not stable (resulting in large block tiling noise in some applications). Disabling GFXOFF via the quirk list fixes the problems for me. Signed-off-by: To

[PATCH AUTOSEL 5.6 035/606] drm/amd/display: add basic atomic check for cursor plane

2020-06-08 Thread Sasha Levin
From: Simon Ser commit 626bf90fe03fa080d8df06bb0397c95c53ae8e27 upstream. This patch adds a basic cursor check when an atomic test-only commit is performed. The position and size of the cursor plane is checked. This should fix user-space relying on atomic checks to assign buffers to planes. Si

[PATCH AUTOSEL 5.6 129/606] drm/amd/display: fix counter in wait_for_no_pipes_pending

2020-06-08 Thread Sasha Levin
From: Roman Li [ Upstream commit 80797dd6f1a525d1160c463d6a9f9d29af182cbb ] [Why] Wait counter is not being reset for each pipe. [How] Move counter reset into pipe loop scope. Signed-off-by: Roman Li Reviewed-by: Zhan Liu Acked-by: Aurabindo Pillai Signed-off-by: Alex Deucher Signed-off-by

  1   2   >