Need to start somewhere ...
Reviewed-by: seanpaul at chromium.org
Signed-off-by: Daniel Vetter
---
Makefile | 2 +-
drm-misc.rst | 45 +
2 files changed, 46 insertions(+), 1 deletion(-)
create mode 100644 drm-misc.rst
diff --git a/Makefile b/Mak
: git-0b626d7
$ grep "warning: array" media-libs:mesa-9999:20160719-211957.log
src/gallium/drivers/r600/r600_shader.c:1590:13: warning: array subscript is
above array bounds [-Warray-bounds]
src/gallium/drivers/r600/r600_shader.c:1591:13: warning: array subscript is
above array bounds [-War
m
kernel at all
Patch was compile tested with x86_64_defconfig + CONFIG_DRM_RADEON=m
Patch is against 4.7.0-rc7 (localversion-next is -next-20160719)
drivers/gpu/drm/radeon/ci_dpm.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/radeon/ci_dpm.c b
might add.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160719/17a02dca/attachment.html>
Arnd Bergmann writes:
>> > I think that's fine, a couple were already picked up, and what I have
>> > left now is
>> >
>> > a281bfa5713a [SUBMITTED 20160615] [EXPERIMENTAL] Kbuild: enable
>> > -Wmissing-include-dirs by default
>> > 83934921e68e [SUBMITTED 20160615] rtlwifi: don't add include pat
Add missing drm_crtc_vblank_on/off() calls so vblank irq
handling/updating/timestamping never runs with a crtc shut down
or during its shutdown/startup, as that causes large jumps in
vblank count and trouble for compositors.
Signed-off-by: Mario Kleiner
Cc: Eric Anholt
---
drivers/gpu/drm/vc4/v
On top of the interlaced video mode fix and with some additional
adjustments, this now works well. It has almost the same accuracy
as on regular progressive scan modes.
Signed-off-by: Mario Kleiner
Cc: Eric Anholt
---
drivers/gpu/drm/vc4/vc4_crtc.c | 21 -
1 file changed, 8
We can't handle doublescan modes at the moment, so if
userspace tries to set one, reject the mode set.
Signed-off-by: Mario Kleiner
Cc: Eric Anholt
---
drivers/gpu/drm/vc4/vc4_crtc.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/vc4/vc4_crtc.c b/drivers/gpu/drm/vc4/
We must not apply CRTC_INTERLACE_HALVE_V to interlaced modes during
mode enumeration, as drm_helper_probe_single_connector_modes
does, so wrap it and reset the effect of CRTC_INTERLACE_HALVE_V
on affected interlaced modes.
Also mode_fixup interlaced modes passed in from user space.
This fixes the
We already don't expose such modes to userspace, but make
sure userspace can't sneak some interlaced mode in.
Signed-off-by: Mario Kleiner
Cc: Eric Anholt
---
drivers/gpu/drm/vc4/vc4_dpi.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/vc4/vc4_dpi.c b/drivers/g
Hi Eric and all,
this batch prevents setting modes one shouldn't set,
adds precise vblank timestamping for interlaced video
modes, and one fix for vblank en/disable during crtc
en/disable.
All successfully tested, also with timing measurement equipment,
on the RPi 2B.
Thanks,
-mario
Hi Philipp, CK,
Thanks for the review.
On Mon, 2016-07-18 at 10:36 +0200, Philipp Zabel wrote:
> Hi CK, YT,
>
> Am Montag, den 18.07.2016, 14:32 +0800 schrieb CK Hu:
> > Hi, YT:
> >
> > One comment inline.
> >
> >
> > On Fri, 2016-07-15 at 18:07 +0800, YT Shen wrote:
> > > We need to acquire
Hi CK,
On Mon, 2016-07-18 at 14:58 +0800, CK Hu wrote:
> Hi, YT:
>
> One comment inline.
>
> On Fri, 2016-07-15 at 18:07 +0800, YT Shen wrote:
> > This patch add support for the Mediatek MT2701 DISP subsystem.
> > There is only one OVL engine in MT2701.
> >
> > Signed-off-by: YT Shen
> > ---
>
Hi CK,
On Tue, 2016-07-19 at 13:39 +0800, CK Hu wrote:
> Hi, YT:
>
> Some comments inline.
>
> On Fri, 2016-07-15 at 18:07 +0800, YT Shen wrote:
> > From: shaoming chen
> >
> > add dsi interrupt control
> >
> > Signed-off-by: shaoming chen
> > ---
> > drivers/gpu/drm/mediatek/mtk_dsi.c | 1
connector_id in the uapi actually means drm_connector->base.id, which
is something entirely different. And ->index is also consistent with
plane/encoder/CRTCS and the various drm_*_index() functions.
While at it also improve/align the kerneldoc comment.
v2: Mention where those ids are from ...
v
B.full = (Y << SHIFT_AMOUNT);
>> >>> > -
>> >>> > - Sum.full = A.full + B.full;
>> >>> > -
>> >>> > - return Sum;
>> >>> > -}
>> >>> > -
>> >>> > /* Conversion Functions */
>> >>> > static int GetReal (fInt A)
>> >>> > {
>> >>> > return (A.full >> SHIFT_AMOUNT);
>> >>> > }
>> >>> >
>> >>> > -/* Temporarily Disabled */
>> >>> > -static int GetRoundedValue(fInt A) /*For now, round the 3rd decimal
>> >>> > place */
>> >>> > -{
>> >>> > - /* ROUNDING TEMPORARLY DISABLED
>> >>> > - int temp = A.full;
>> >>> > - int decimal_cutoff, decimal_mask = 0x01FF;
>> >>> > - decimal_cutoff = temp & decimal_mask;
>> >>> > - if (decimal_cutoff > 0x147) {
>> >>> > - temp += 673;
>> >>> > - }*/
>> >>> > -
>> >>> > - return ConvertBackToInteger(A)/1; /*Temporary - in case
>> >>> > this was used somewhere else */
>> >>> > -}
>> >>> > -
>> >>> > -static fInt Multiply (int X, int Y)
>> >>> > -{
>> >>> > - fInt A, B, Product;
>> >>> > -
>> >>> > - A.full = X << SHIFT_AMOUNT;
>> >>> > - B.full = Y << SHIFT_AMOUNT;
>> >>> > -
>> >>> > - Product = fMultiply(A, B);
>> >>> > -
>> >>> > - return Product;
>> >>> > -}
>> >>> > -
>> >>> > static fInt Divide (int X, int Y)
>> >>> > {
>> >>> > fInt A, B, Quotient;
>> >>> > @@ -578,14 +529,6 @@ static int uPow(int base, int power)
>> >>> > return (base)*uPow(base, power - 1);
>> >>> > }
>> >>> >
>> >>> > -static fInt fAbs(fInt A)
>> >>> > -{
>> >>> > - if (A.partial.real < 0)
>> >>> > - return (fMultiply(A, ConvertToFraction(-1)));
>> >>> > - else
>> >>> > - return A;
>> >>> > -}
>> >>> > -
>> >>> > static int uAbs(int X)
>> >>> > {
>> >>> > if (X < 0)
>> >>> > --
>> >>> > 2.8.0.rc3
>> >>> >
>> >>> > ___
>> >>> > dri-devel mailing list
>> >>> > dri-devel at lists.freedesktop.org
>> >>> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>> >>
>> >>
>> >
>>
>
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160719/b9c36d45/attachment.html>
or the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160719/a5014bf8/attachment-0001.html>
Rename the CSI client device in the client_reg[] table to
"imx-ipuv3-csi".
Signed-off-by: Steve Longerbeam
---
drivers/gpu/ipu-v3/ipu-common.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/ipu-v3/ipu-common.c b/drivers/gpu/ipu-v3/ipu-common.c
index bb0377b..
The image converter kernel API supports conversion contexts and
job queues, so we should allow more than one handle to the IC, so
that multiple users can add jobs to the queue.
Note however that users that control the IC manually (that do not
use the image converter APIs but setup the IC task by h
This patch implements complete image conversion support to ipu-ic,
with tiling to support scaling to and from images up to 4096x4096.
Image rotation is also supported.
The internal API is subsystem agnostic (no V4L2 dependency except
for the use of V4L2 fourcc pixel formats).
Callers prepare for
There can be multiple IC tasks using the IRT, so the IRT needs
a separate use counter. Create a private ipu_irt_enable() to
enable the IRT module when any IC task requires rotation, and
ipu_irt_disable() when a task no longer needs the IRT.
Signed-off-by: Steve Longerbeam
---
v2:
- cleaned up i
The CSI data format was being programmed incorrectly for the
1x16 media bus formats. The CSI data format for 16-bit must
be bayer/generic (CSI_SENS_CONF_DATA_FMT_BAYER).
Suggested-by: Carsten Resch
Signed-off-by: Steve Longerbeam
---
drivers/gpu/ipu-v3/ipu-csi.c | 6 +-
1 file changed, 1 in
Set the sensor full frame based on whether the passed in mbus_fmt
is 720x480 (NTSC) or 720x576 (PAL).
Signed-off-by: Steve Longerbeam
---
drivers/gpu/ipu-v3/ipu-csi.c | 20 +---
1 file changed, 13 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/ipu-v3/ipu-csi.c b/drivers/
Adds the VDIC field input IDMAC channels. These channels
transfer fields F(n-1), F(n), and F(N+1) from memory to
the VDIC (channels 8, 9, 10 respectively).
Signed-off-by: Steve Longerbeam
---
v2:
- made the channel names more descriptive: "_PREV" instead of "_P", etc.
---
include/video/imx-ipu
Adds ipu_set_vdi_src_mux() that selects the VDIC input
(from CSI or memory).
Signed-off-by: Steve Longerbeam
---
v2:
- added macros for the VDI source select bits in register IPU_FS_PROC_FLOW1.
---
drivers/gpu/ipu-v3/ipu-common.c | 20
drivers/gpu/ipu-v3/ipu-prv.h| 6
Adds functions to link and unlink IDMAC source channels to sink
channels.
So far the following links are supported:
IPUV3_CHANNEL_IC_PRP_ENC_MEM -> IPUV3_CHANNEL_MEM_ROT_ENC
PUV3_CHANNEL_IC_PRP_VF_MEM -> IPUV3_CHANNEL_MEM_ROT_VF
IPUV3_CHANNEL_IC_PP_MEM -> IPUV3_CHANNEL_MEM_ROT_PP
More lin
Adds of-alias id to ipu_soc and retrieve with ipu_get_num().
Signed-off-by: Steve Longerbeam
---
drivers/gpu/ipu-v3/ipu-common.c | 8
drivers/gpu/ipu-v3/ipu-prv.h| 1 +
include/video/imx-ipu-v3.h | 1 +
3 files changed, 10 insertions(+)
diff --git a/drivers/gpu/ipu-v3/ipu-comm
Adds ipu_cpmem_get_burstsize().
Signed-off-by: Steve Longerbeam
---
drivers/gpu/ipu-v3/ipu-cpmem.c | 6 ++
include/video/imx-ipu-v3.h | 1 +
2 files changed, 7 insertions(+)
diff --git a/drivers/gpu/ipu-v3/ipu-cpmem.c b/drivers/gpu/ipu-v3/ipu-cpmem.c
index a36c35e..fcb7dc8 100644
--- a/
Adds ipu_cpmem_set_uv_offset(), to set planar U/V offsets.
Signed-off-by: Steve Longerbeam
---
drivers/gpu/ipu-v3/ipu-cpmem.c | 7 +++
include/video/imx-ipu-v3.h | 1 +
2 files changed, 8 insertions(+)
diff --git a/drivers/gpu/ipu-v3/ipu-cpmem.c b/drivers/gpu/ipu-v3/ipu-cpmem.c
index 64
Adds the Video Deinterlacer (VDIC) unit.
Signed-off-by: Steve Longerbeam
---
v2:
- removed include of module.h
- corrected V4L2 field type checks
- cleaned up use_count decrement in ipu_vdi_disable()
---
drivers/gpu/ipu-v3/Makefile | 2 +-
drivers/gpu/ipu-v3/ipu-common.c | 11 ++
driver
These updates to IPUv3 are needed for media staging drivers
for i.MX5/6 video capture and mem2mem.
Steve Longerbeam (13):
gpu: ipu-v3: Add Video Deinterlacer unit
gpu: ipu-cpmem: Add ipu_cpmem_set_uv_offset()
gpu: ipu-cpmem: Add ipu_cpmem_get_burstsize()
gpu: ipu-v3: Add ipu_get_num()
gp
Hi Markus,
On Fri, 2016-07-15 at 22:00 +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Fri, 15 Jul 2016 21:52:40 +0200
>
> The drm_fbdev_cma_hotplug_event() function tests whether its argument
> is NULL and then returns immediately.
> Thus the test around the call is not needed.
>
Hi Wei Yongjun,
On Tue, 2016-07-19 at 12:01 +, Wei Yongjun wrote:
> From: Wei Yongjun
>
> There is a error message within devm_ioremap_resource
> already, so remove the dev_err call to avoid redundant
> error message.
>
> Signed-off-by: Wei Yongjun
Acked-by: Alexey Brodkin
Hi Wei Yongjun,
On Tue, 2016-07-19 at 12:04 +, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Fixes the following sparse warnings:
>
> drivers/gpu/drm/arc/arcpgu_drv.c:52:5: warning:
> Â symbol 'arcpgu_gem_mmap' was not declared. Should it be static?
> drivers/gpu/drm/arc/arcpgu_drv.c:134:48:
(Adding Greg KH)
Hi Greg,
On 19 July 2016 at 17:45, Sumit Semwal wrote:
> Hi Greg,
>
>
> On 12 July 2016 at 23:38, Gustavo Padovan wrote:
>> From: Gustavo Padovan
>>
>> Create sync_file->fence to abstract the type of fence we are using for
>> each sync_file. If only one fence is present we use
Hi Greg,
On 12 July 2016 at 23:38, Gustavo Padovan wrote:
> From: Gustavo Padovan
>
> Create sync_file->fence to abstract the type of fence we are using for
> each sync_file. If only one fence is present we use a normal struct fence
> but if there is more fences to be added to the sync_file a f
On Tuesday, July 19, 2016 5:33:44 PM CEST Kalle Valo wrote:
> Arnd Bergmann writes:
>
> > On Monday, July 18, 2016 10:14:39 PM CEST Michal Marek wrote:
> >> On Wed, Jun 15, 2016 at 05:45:47PM +0200, Arnd Bergmann wrote:
> >> > When building with separate object directories and driver specific
> >
We had only DRM_INFO() and DRM_ERROR(), whereas the underlying printk()
provides several other useful intermediate levels such as NOTICE and
WARNING. So this patch fills out the set by providing both regular and
once-only macros for each of the levels INFO, NOTICE, and WARNING, using
a common under
Arnd Bergmann writes:
> On Monday, July 18, 2016 10:14:39 PM CEST Michal Marek wrote:
>> On Wed, Jun 15, 2016 at 05:45:47PM +0200, Arnd Bergmann wrote:
>> > When building with separate object directories and driver specific
>> > Makefiles that add additional header include paths, Kbuild adjusts
>
On Tue, Jul 19, 2016 at 5:25 PM, Daniel Vetter
wrote:
> On Tue, Jul 19, 2016 at 4:59 PM, Markus Heiser
> wrote:
>>
>> Am 19.07.2016 um 13:42 schrieb Daniel Vetter :
>>
>>> Unfortunately warnings generated after parsing in sphinx can end up
>>> with entirely bogus files and line numbers as source
On Tue, Jul 19, 2016 at 4:59 PM, Markus Heiser
wrote:
>
> Am 19.07.2016 um 13:42 schrieb Daniel Vetter :
>
>> Unfortunately warnings generated after parsing in sphinx can end up
>> with entirely bogus files and line numbers as sources. Strangely for
>> outright errors this is not a problem. Trying
On 07/19/2016 06:32 AM, Philipp Zabel wrote:
> Am Samstag, den 16.07.2016, 13:24 -0700 schrieb Steve Longerbeam:
> [...]
>>> Hmm, do you mean define something like a V4L2_MBUS_BT656_NEWAVMODE,
>>> and then add a new "newavmode" boolean DT binding parsed by
>>> v4l2_of_parse_endpoint()?
>>>
>>> I do
Am 19.07.2016 um 13:42 schrieb Daniel Vetter :
> Unfortunately warnings generated after parsing in sphinx can end up
> with entirely bogus files and line numbers as sources. Strangely for
> outright errors this is not a problem. Trying to convert warnings to
> errors also doesn't fix it.
>
> The
On Tue, Jul 19, 2016 at 7:32 AM, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Add the missing clk_disable_unprepare() before return in the
> error handling case.
>
> Signed-off-by: Wei Yongjun
Applied to drm-misc
> ---
> drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 ++
> 1 file change
On Tue, 19 Jul 2016 13:42:54 +0200
Daniel Vetter wrote:
> Unfortunately warnings generated after parsing in sphinx can end up
> with entirely bogus files and line numbers as sources. Strangely for
> outright errors this is not a problem. Trying to convert warnings to
> errors also doesn't fix it.
On Tue, Jul 19, 2016 at 8:45 AM, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Fixes the following sparse warning:
>
> drivers/gpu/drm/virtio/virtgpu_display.c:349:37: warning:
> symbol 'virtio_mode_config_helpers' was not declared. Should it be static?
>
> Signed-off-by: Wei Yongjun
Applied to
On Tue, Jul 19, 2016 at 2:02 PM, Alexey Brodkin
wrote:
> Hi Wei Yongjun,
>
> On Tue, 2016-07-19 at 12:01 +, Wei Yongjun wrote:
>> From: Wei Yongjun
>>
>> There is a error message within devm_ioremap_resource
>> already, so remove the dev_err call to avoid redundant
>> error message.
>>
>> Sig
On Tue, Jul 19, 2016 at 2:01 PM, Alexey Brodkin
wrote:
> Hi Wei Yongjun,
>
> On Tue, 2016-07-19 at 12:04 +, Wei Yongjun wrote:
>> From: Wei Yongjun
>>
>> Fixes the following sparse warnings:
>>
>> drivers/gpu/drm/arc/arcpgu_drv.c:52:5: warning:
>> symbol 'arcpgu_gem_mmap' was not declared.
Am Samstag, den 16.07.2016, 13:24 -0700 schrieb Steve Longerbeam:
[...]
> > Hmm, do you mean define something like a V4L2_MBUS_BT656_NEWAVMODE,
> > and then add a new "newavmode" boolean DT binding parsed by
> > v4l2_of_parse_endpoint()?
> >
> > I don't know if that would make sense given that this
Thanks to Ville for suggesting this as a potential solution to pipe
underruns on Skylake.
On Skylake all of the registers for configuring planes, including the
registers for configuring their watermarks, are double buffered. New
values written to them won't take effect until said registers are
"ar
just to be inline with the existing code (funcs:
mipi_dsi_dcs_get_power_mode, mipi_dsi_dcs_get_pixel_format) in
drivers/gpu/drm/drm_mipi_dsi.c followed the same for
mipi_dsi_dcs_get_display_brightness
We may need to change as you suggested for other two functions also.
please suggest.
On Tue,
On Tue, Jul 19, 2016 at 01:51:03PM +0100, Chris Wilson wrote:
> On Tue, Jul 19, 2016 at 12:44:22PM +, Wei Yongjun wrote:
> > From: Wei Yongjun
> >
> > Fixes the following sparse warning:
> >
> > drivers/gpu/drm/vgem/vgem_fence.c:75:24: warning:
> > symbol 'vgem_fence_ops' was not declared.
Update kernel interface with new I915_GETPARAM ioctl entries for
pooled EU and min no. of eus in a pool. Add a wrapping function
for each parameter. Userspace drivers need these values when decide
the thread count. This kernel enabled pooled eu by default for BXT
and for fused down 2x6 parts it is
connector_id in the uapi actually means drm_connector->base.id, which
is something entirely different. And ->index is also consistent with
plane/encoder/CRTCS and the various drm_*_index() functions.
While at it also improve/align the kerneldoc comment.
v2: Mention where those ids are from ...
v
On Tue, Jul 19, 2016 at 01:42:55PM +0200, Daniel Vetter wrote:
> These are the leftovers I could only track down using keep_warnings =
> True. For some of them we might want to update our style guide on how
> to reference structures and constants, not sure ...
>
> Cc: Markus Heiser
> Cc: Jonathan
Am 19.07.2016 um 14:15 schrieb Wei Yongjun:
> From: Wei Yongjun
>
> In case of error, the function debugfs_create_file() returns NULL
> pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test
> in the return value check should be replaced with NULL test.
> (defined(CONFIG_DEBUG_FS) make sur
Am 19.07.2016 um 14:15 schrieb Wei Yongjun:
> From: Wei Yongjun
>
> In case of error, the function debugfs_create_*() returns NULL
> pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test
> in the return value check should be replaced with NULL test.
> (defined(CONFIG_DEBUG_FS) make sure d
Am 19.07.2016 um 14:14 schrieb Wei Yongjun:
> From: Wei Yongjun
>
> In case of error, the function debugfs_create_*() returns NULL
> pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test
> in the return value check should be replaced with NULL test.
> (defined(CONFIG_DEBUG_FS) make sure d
On Tue, Jul 19, 2016 at 12:57:52PM +0200, Daniel Vetter wrote:
> connector_id in the uapi actually means drm_connector->base.id, which
> is something entirely different. And ->index is also consistent with
> plane/encoder/CRTCS and the various drm_*_index() functions.
>
> While at it also improve/
Am 19.07.2016 um 14:01 schrieb Wei Yongjun:
> From: Wei Yongjun
>
> Using list_move() instead of list_del() + list_add().
>
> Signed-off-by: Wei Yongjun
Reviewed-by: Christian König
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>
On Tue, Jul 19, 2016 at 12:44:22PM +, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Fixes the following sparse warning:
>
> drivers/gpu/drm/vgem/vgem_fence.c:75:24: warning:
> symbol 'vgem_fence_ops' was not declared. Should it be static?
>
> Signed-off-by: Wei Yongjun
Thanks,
Reviewed-by:
These are the leftovers I could only track down using keep_warnings =
True. For some of them we might want to update our style guide on how
to reference structures and constants, not sure ...
Cc: Markus Heiser
Cc: Jonathan Corbet
Cc: linux-doc at vger.kernel.org
Signed-off-by: Daniel Vetter
---
Unfortunately warnings generated after parsing in sphinx can end up
with entirely bogus files and line numbers as sources. Strangely for
outright errors this is not a problem. Trying to convert warnings to
errors also doesn't fix it.
The only way to get useful output out of sphinx to be able to ro
Hi, YT:
Some comments inline.
On Fri, 2016-07-15 at 18:07 +0800, YT Shen wrote:
> From: shaoming chen
>
> add dsi interrupt control
>
> Signed-off-by: shaoming chen
> ---
> drivers/gpu/drm/mediatek/mtk_dsi.c | 130
>
> 1 file changed, 130 insertions(+)
On Tue, Jul 19, 2016 at 1:19 PM, Matt Roper
wrote:
> On Tue, Jul 19, 2016 at 12:30:56PM -0400, Lyude wrote:
>> Now that we update the watermark values atomically, we still need to fix
>> the case of how we update watermarks when we haven't added or removed
>> pipes.
>>
>> When we haven't added or
On Tue, Jul 19, 2016 at 12:57:52PM +0200, Daniel Vetter wrote:
> connector_id in the uapi actually means drm_connector->base.id, which
> is something entirely different. And ->index is also consistent with
> plane/encoder/CRTCS and the various drm_*_index() functions.
>
> While at it also improve/
connector_id in the uapi actually means drm_connector->base.id, which
is something entirely different. And ->index is also consistent with
plane/encoder/CRTCS and the various drm_*_index() functions.
While at it also improve/align the kerneldoc comment.
v2: Mention where those ids are from ...
C
From: Wei Yongjun
Fixes the following sparse warning:
drivers/gpu/drm/virtio/virtgpu_display.c:349:37: warning:
symbol 'virtio_mode_config_helpers' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/virtio/virtgpu_display.c | 2 +-
1 file changed, 1 insert
From: Wei Yongjun
Fixes the following sparse warning:
drivers/gpu/drm/vgem/vgem_fence.c:75:24: warning:
symbol 'vgem_fence_ops' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/vgem/vgem_fence.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On Fri, Jul 15, 2016 at 04:18:13PM +0800, Chris Zhong wrote:
> Add support for cdn DP controller which is embedded in the rk3399
> SoCs. The DP is compliant with DisplayPort Specification,
> Version 1.3, This IP is compatible with the rockchip type-c PHY IP.
> There is a uCPU in DP controller, it n
Now that we update the watermark values atomically, we still need to fix
the case of how we update watermarks when we haven't added or removed
pipes.
When we haven't added or removed any pipes, we don't need to worry about
the ddb allocation changing. As such there's no order of flushing pipes
we
Thanks to Ville for suggesting this as a potential solution to pipe
underruns on Skylake.
On Skylake all of the registers for configuring planes, including the
registers for configuring their watermarks, are double buffered. New
values written to them won't take effect until said registers are
"ar
Unfortunately as a few of you are aware, Skylake is still very prone to pipe
underruns. Most of this comes from not doing things atomically enough (e.g.
needing to ensure that we update watermarks with other plane attributes, not
forcefully flushing pipes until we need to, etc.). Now that I've fina
From: Wei Yongjun
In case of error, the function debugfs_create_file() returns NULL
pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test
in the return value check should be replaced with NULL test.
(defined(CONFIG_DEBUG_FS) make sure debugfs is enabled)
Signed-off-by: Wei Yongjun
---
From: Wei Yongjun
In case of error, the function debugfs_create_*() returns NULL
pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test
in the return value check should be replaced with NULL test.
(defined(CONFIG_DEBUG_FS) make sure debugfs is enabled)
Signed-off-by: Wei Yongjun
---
dr
From: Wei Yongjun
In case of error, the function debugfs_create_*() returns NULL
pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test
in the return value check should be replaced with NULL test.
(defined(CONFIG_DEBUG_FS) make sure debugfs is enabled)
Signed-off-by: Wei Yongjun
---
dr
Hi Dave,
This is an update to the previous drm bridge pull request. The ADV7511
driver's conversion from slave encoder to bridge meant that its users
(the rcar-du kms driver) should use the bridge interface too. This pull
request now also contains a commit that updates the rcar-du's hdmi encoder
i
From: Wei Yongjun
Fixes the following sparse warnings:
drivers/gpu/drm/arc/arcpgu_drv.c:52:5: warning:
symbol 'arcpgu_gem_mmap' was not declared. Should it be static?
drivers/gpu/drm/arc/arcpgu_drv.c:134:48: warning:
Using plain integer as NULL pointer
drivers/gpu/drm/arc/arcpgu_drv.c:155:5:
From: Wei Yongjun
Using list_move() instead of list_del() + list_add().
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
i
From: Wei Yongjun
There is a error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/arc/arcpgu_drv.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/a
From: Wei Yongjun
Add the missing clk_disable_unprepare() before return in the
error handling case.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
b/drivers/gp
From: Wei Yongjun
Remove .owner field if calls are used which set it automatically.
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/dr
From: Wei Yongjun
Fix the reset_control_deassert() fail and clk_prepare_enable() fail
error handling of ade_power_up().
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/
On Fri, Jul 15, 2016 at 09:47:58PM +0200, Daniel Vetter wrote:
> Right now there's nothing, and kernel-doc produces a warning because
> of that. Remove it until we need it for a clean build.
>
> Cc: Laurent Pinchart
> Signed-off-by: Daniel Vetter
Pulled in the entire pile with Chris' irc-ack, a
Hi Dave
Here are some little fixes for rockchip drm, looks good for me, and
there is no doubt on them, So I'd like you can land them.
Thanks.
The following changes since commit 2d635fded22b71381b01e194a59544f2e8d71ef0:
Merge tag 'drm-vc4-next-2016-07-15' of
https://github.com/anholt/li
On Monday, July 18, 2016 10:14:39 PM CEST Michal Marek wrote:
> On Wed, Jun 15, 2016 at 05:45:47PM +0200, Arnd Bergmann wrote:
> > When building with separate object directories and driver specific
> > Makefiles that add additional header include paths, Kbuild adjusts
> > the gcc flags so that we i
On Tue, Jul 19, 2016 at 10:14:07AM +0200, Geert Uytterhoeven wrote:
> On Mon, Jul 18, 2016 at 4:45 PM, Daniel Vetter wrote:
> > On Mon, Jul 18, 2016 at 04:10:36PM +0200, SF Markus Elfring wrote:
> >> From: Markus Elfring
> >> Date: Mon, 18 Jul 2016 16:06:18 +0200
> >>
> >> The ttm_tt_destroy() fu
On Tue, Jul 19, 2016 at 12:30:56PM -0400, Lyude wrote:
> Now that we update the watermark values atomically, we still need to fix
> the case of how we update watermarks when we haven't added or removed
> pipes.
>
> When we haven't added or removed any pipes, we don't need to worry about
> the ddb
On Mon, Jul 18, 2016 at 4:45 PM, Daniel Vetter wrote:
> On Mon, Jul 18, 2016 at 04:10:36PM +0200, SF Markus Elfring wrote:
>> From: Markus Elfring
>> Date: Mon, 18 Jul 2016 16:06:18 +0200
>>
>> The ttm_tt_destroy() function tests whether its argument is NULL
>> and then returns immediately. Thus
On Tue, Jul 19, 2016 at 12:30:55PM -0400, Lyude wrote:
> Thanks to Ville for suggesting this as a potential solution to pipe
> underruns on Skylake.
>
> On Skylake all of the registers for configuring planes, including the
> registers for configuring their watermarks, are double buffered. New
> va
On Wed, Jul 13, 2016 at 11:23:29AM -0700, Sean Paul wrote:
> On Wed, Jul 13, 2016 at 9:39 AM, Chris Wilson
> wrote:
> > drm_connector_unregister_all() is automatically called by
> > drm_dev_unregister() and so the manual call can be dropped.
> >
> > Signed-off-by: Chris Wilson
> > Cc: Daniel Vet
On 2016å¹´07æ15æ¥ 21:35, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Fri, 15 Jul 2016 15:23:22 +0200
>
> The drm_gem_object_unreference_unlocked() function tests whether
> its argument is NULL and then returns immediately.
> Thus the test around the call is not needed.
>
> This issu
Thierry,
On 07/08/2016 05:57 AM, Thierry Reding wrote:
> On Thu, Jul 07, 2016 at 11:55:23PM +0200, Thierry Reding wrote:
>> On Sun, Jun 12, 2016 at 10:53:30AM +0800, Yakir Yang wrote:
>>> The LG LP097QX1-SPA1 is an 9.7", 2048x1536 (QXGA) TFT-LCD panel
>>> connected using eDP interfaces.
>>>
>>> Si
Hi Linus,
As mentioned Intel has gathered all the Kabylake fixes from -next,
which we've enabled in 4.7 for the first time, these are pretty
much limited in scope to only affects kabylake, which is hw that
isn't shipping yet. So I'm mostly okay with it going in now.
If we don't land this, it mig
Hi Linus,
Two i915 regression fixes.
Intel have submitted some Kabylake fixes I'll send separately, since
this is the first kernel with kabylake support and they don't go much
outside that area I think they should be fine.
Dave.
The following changes since commit 47ef4ad2684d380dd6d596140fb793
Dne 13.7.2016 v 12:48 Pavel Machek napsal(a)
Are there any updates on the status of this issue?
The current bugzilla report [1] marks this as a power management
issue. However, many reports indicate that it would only freeze
when running X, so it's not completely clear if
On Tue, Jul 19, 2016 at 5:42 AM, Vinay Simha wrote:
> just to be inline with the existing code (funcs:
> mipi_dsi_dcs_get_power_mode, mipi_dsi_dcs_get_pixel_format) in
> drivers/gpu/drm/drm_mipi_dsi.c followed the same for
> mipi_dsi_dcs_get_display_brightness
>
> We may need to change as you sugg
On Mon, Jul 18, 2016 at 4:28 AM, Vinay Simha BN wrote:
> Provide a small convenience wrapper that set/get the
> display brightness value
>
> Cc: John Stultz
> Cc: Sumit Semwal
> Cc: Archit Taneja
> Cc: Rob Clark
> Cc: Jani Nikula
> Cc: Thierry Reding
> Cc: Emil Velikov
> Signed-off-by: Vina
98 matches
Mail list logo
