2015-08-27 17:31 GMT+09:00 Joonyoung Shim :
> If exynos_drm_gem_init() is failed, the result is ERR_PTR, so we should
> just return the result. If not, wrong porinter will be referenced from
> err label.
>
> Reported-by: Dan Carpenter
> Signed-off-by: Joonyoung Shim
> ---
> drivers/gpu/drm/exyno
Hi Dave,
This is a second pull-request which adds last part of
atomic modeset/pageflip support, render node support,
clean-up, and fix-up.
Please, kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit 92cffd56b21c825579f3b37bc7803e4c37073076
The change adds support of internal HDMI I2C master controller, this
subdevice is used by default, if "ddc-i2c-bus" DT property is omitted.
The main purpose of this functionality is to support reading EDID from
an HDMI monitor on boards, which don't have an I2C bus connected to
DDC pins.
The curr
||
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150831/b6527edb/attachment-0001.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150831/e3bab210/attachment.html>
For state->fb may be NULL in fsl_dcu_drm_plane_atomic_check function,
if so, return -EINVAL. No need check in fsl_dcu_drm_plane_atomic_update
anymore.
Signed-off-by: Jianwei Wang
---
drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --g
On 08/28/2015 11:48 PM, Rob Clark wrote:
> On Fri, Aug 28, 2015 at 3:56 AM, Archit Taneja
> wrote:
>>
>>
>> On 08/27/2015 10:36 AM, Archit Taneja wrote:
>>>
>>>
>>>
>>> On 08/26/2015 08:42 PM, hali at codeaurora.org wrote:
>
> 2015-08-26 9:55 GMT-04:00 :
>>
>> Hi Archit,
>>
Switch update interrupt mask bit with regmap_update_bits, and clear
interrupt status by writing 1 to relevant bit before setting mask in
fsl_dcu_drm_irq_init function.
Signed-off-by: Jianwei Wang
---
drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 26 +++---
1 file changed, 7 ins
Op 27-08-15 om 17:22 schreef Daniel Vetter:
> On Thu, Aug 27, 2015 at 03:36:09PM +0100, Daniel Stone wrote:
>> Hi,
>>
>> On 6 August 2015 at 13:49, Daniel Vetter wrote:
>>> On Thu, Aug 06, 2015 at 01:19:35PM +0200, Maarten Lankhorst wrote:
Op 06-08-15 om 11:47 schreef Daniel Stone:
> On 3
On Thu, Aug 27, 2015 at 06:25:29PM +0200, Oleg Nesterov wrote:
> It is hardly possible to enumerate all problems with block_all_signals()
> and unblock_all_signals(). Just for example,
>
> 1. block_all_signals(SIGSTOP/etc) simply can't help if the caller is
>multithreaded. Another thread can d
crubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150831/844c9782/attachment.html>
For DCU support atmost 16 layers(on ls1021a) or 64 layers(on vf610),
add (total_layer - 1) overlay planes.
Signed-off-by: Jianwei Wang
---
drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 27 ++-
1 file changed, 26 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/f
Hi Vladimir,
Am Montag, den 31.08.2015, 00:34 +0300 schrieb Vladimir Zapolskiy:
> The change adds support of internal HDMI I2C master controller, this
> subdevice is used by default, if "ddc-i2c-bus" DT property is omitted.
I think this should be mentioned in
Documentation/devicetree/bindings/drm
Commit ec9f932ed41622d120de52a5b525e4d77b9ef17e
"drm/atomic: Cleanup on error properly in the atomic ioctl."
cleaned up some error paths, but didn't fix the TEST_ONLY path.
In the check only case plane->fb shouldn't be updated, and
the vblank events should be cleared as on failure.
Changes since v
On Mon, Aug 31, 2015 at 1:15 AM, Archit Taneja
wrote:
>
>
> On 08/28/2015 11:48 PM, Rob Clark wrote:
>>
>> On Fri, Aug 28, 2015 at 3:56 AM, Archit Taneja
>> wrote:
>>>
>>>
>>>
>>> On 08/27/2015 10:36 AM, Archit Taneja wrote:
On 08/26/2015 08:42 PM, hali at codeaurora.org
Hi all,
On 18 August 2015 at 08:01, Inki Dae wrote:
> From: Joonyoung Shim
>
> This patch allows clients who want to use render node to access
> rendering relevant ioctls - g2d, post processor and gem allocation.
>
I seem to recall Dave and others saying that we must have users of
these interfac
From: Ville Syrjälä
Make generic_edid_names[] const since it's supposed to be immutable.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/drm_edid_load.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
in
From: Ville Syrjälä
drivers/gpu/drm/i2c/ch7006.ko:
-.text5913
+.text5897
-.rodata 664
+.rodata 7256
-.data6992
+.data 416
Signed-off-by: Ville Syrjälä
From: Ville Syrjälä
Moves a bunch of junk to .rodata from .data.
drivers/gpu/drm/armada/armada.ko:
-.rodata 1040
+.rodata 1100
-.data1156
+.data1096
drivers/gpu/drm/atmel-hlcdc/atmel-hlcdc-dc.ko:
-.tex
From: Ville Syrjälä
Make the mode names passed to drm_mode_create_tv_properties() const.
drivers/gpu/drm/i2c/ch7006.ko:
-.rodata 596
+.rodata 664
-.data7064
+.data6992
drivers/gpu/drm/nouveau/nouveau
Hi Emil,
On 2015ë
08ì 31ì¼ 20:58, Emil Velikov wrote:
> Hi all,
>
> On 18 August 2015 at 08:01, Inki Dae wrote:
>> From: Joonyoung Shim
>>
>> This patch allows clients who want to use render node to access
>> rendering relevant ioctls - g2d, post processor and gem allocation.
>>
> I seem t
On 2015ë
08ì 24ì¼ 23:13, Tobias Jakobi wrote:
> The amount of commands (regular and GEM) doesn't depend
> on the input here.
>
> Signed-off-by: Tobias Jakobi
> ---
> exynos/exynos_fimg2d.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fi
On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
> Move the parameter validation before buffer space checking
> so that we can exit early if it fails.
> Also don't reset the G2D context anymore in this situation
> (since the buffers are not partially submitted).
>
> Signed-off-by: Tobias Jakobi
endering should look like?
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150831/a84eff7c/attachment-0001.html>
On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
> The G2D headers define a number of modes through enums
> (like e.g. color, select, repeat, etc.).
>
> This introduces g2d_validate_select_mode() and
> g2d_validate_blending_op() which validate a
> select mode or blending operation respectively.
On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
> Move parameter validation to the top and also validate
> the select mode of the source image and the requested
> blending operation before starting command submission.
>
> Signed-off-by: Tobias Jakobi
> ---
> exynos/exynos_fimg2d.c | 66
> +++
On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
> Apply the same transformation as in g2d_blend().
>
> Signed-off-by: Tobias Jakobi
> ---
> exynos/exynos_fimg2d.c | 67
> +-
> 1 file changed, 39 insertions(+), 28 deletions(-)
>
> diff --git a/
On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
> We now validate the blending mode via g2d_validate_mode()
> prior to feeding it to g2d_get_blend_op().
>
> Signed-off-by: Tobias Jakobi
> ---
> exynos/exynos_fimg2d.c | 5 -
> 1 file changed, 5 deletions(-)
>
> diff --git a/exynos/exynos_
From: "monk.liu"
Signed-off-by: monk.liu
Signed-off-by: Alex Deucher
---
amdgpu/amdgpu_device.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/amdgpu/amdgpu_device.c b/amdgpu/amdgpu_device.c
index 75b12e2..e5a923e 100644
--- a/amdgpu/amdgpu_device.c
+++ b/amdgpu/amdgpu_device.c
@@ -132,
On Mon, Aug 31, 2015 at 8:09 AM, wrote:
> From: Ville Syrjälä
>
> Make generic_edid_names[] const since it's supposed to be immutable.
>
> Signed-off-by: Ville Syrjälä
For the series:
Reviewed-by: Alex Deucher
> ---
> drivers/gpu/drm/drm_edid_load.c | 2 +-
> 1 file changed, 1 insertion
On Thu, Aug 27, 2015 at 10:23 AM, wrote:
> From: Ville Syrjälä
>
> AUX addresses are 20 bits long. Send out the entire address instead of
> just the low 16 bits.
>
> Cc: Alex Deucher
> Cc: "Christian König"
> Signed-off-by: Ville Syrjälä
Applied. thanks!
Alex
> ---
> drivers/gpu/drm
AUX addresses are 20 bits long. Send out the entire address instead of
just the low 16 bits.
Port of:
drm/radeon/atom: Send out the full AUX address
to radeon non-atom aux path
Signed-off-by: Alex Deucher
Cc: stable at vger.kernel.org
---
drivers/gpu/drm/radeon/radeon_dp_auxch.c | 4 ++--
1 fil
AUX addresses are 20 bits long. Send out the entire address instead of
just the low 16 bits.
Port of:
drm/radeon/atom: Send out the full AUX address
to amdgpu
Signed-off-by: Alex Deucher
Cc: stable at vger.kernel.org
---
drivers/gpu/drm/amd/amdgpu/atombios_dp.c | 3 ++-
1 file changed, 2 insert
On 31.08.2015 17:19, Alex Deucher wrote:
> AUX addresses are 20 bits long. Send out the entire address instead of
> just the low 16 bits.
>
> Port of:
> drm/radeon/atom: Send out the full AUX address
> to radeon non-atom aux path
>
> Signed-off-by: Alex Deucher
> Cc: stable at vger.kernel.org
Rev
t was scrubbed...
Name: 0001-drm-amdgpu-use-ttm_pool-instead-of-ttm_dma.patch
Type: text/x-patch
Size: 1320 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150831/e403de42/attachment.bin>
v2: keep the bo_gem declaration in exec2() within the loop (Chris)
Cc: Chris Wilson
Cc: intel-gfx at lists.freedesktop.org
Signed-off-by: Emil Velikov
---
intel/intel_bufmgr_fake.c | 2 +-
intel/intel_bufmgr_gem.c | 7 +++
intel/intel_decode.c | 7 ++-
3 files changed, 6 insertion
Just like we do for the original exec()
v2: move bo_gem declaration to the top of the function.
Cc: Chris Wilson
Cc: intel-gfx at lists.freedesktop.org
Signed-off-by: Emil Velikov
---
intel/intel_bufmgr_gem.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/intel/inte
On Mon, Aug 31, 2015 at 07:13:12PM +0100, Emil Velikov wrote:
> v2: keep the bo_gem declaration in exec2() within the loop (Chris)
>
> Cc: Chris Wilson
> Cc: intel-gfx at lists.freedesktop.org
> Signed-off-by: Emil Velikov
Reviewed-by: Chris Wilson
-Chris
--
Chris Wilson, Intel Open Source Te
On Mon, Aug 31, 2015 at 07:14:12PM +0100, Emil Velikov wrote:
> Just like we do for the original exec()
>
> v2: move bo_gem declaration to the top of the function.
>
> Cc: Chris Wilson
> Cc: intel-gfx at lists.freedesktop.org
> Signed-off-by: Emil Velikov
> ---
> intel/intel_bufmgr_gem.c | 7 +
On 31 August 2015 at 13:36, Inki Dae wrote:
> Hi Emil,
>
> On 2015ë
08ì 31ì¼ 20:58, Emil Velikov wrote:
>> Hi all,
>>
>> On 18 August 2015 at 08:01, Inki Dae wrote:
>>> From: Joonyoung Shim
>>>
>>> This patch allows clients who want to use render node to access
>>> rendering relevant ioctls
On 31 August 2015 at 14:18, Inki Dae wrote:
> On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
>> The G2D headers define a number of modes through enums
>> (like e.g. color, select, repeat, etc.).
>>
>> This introduces g2d_validate_select_mode() and
>> g2d_validate_blending_op() which validate a
On 31 August 2015 at 14:25, Inki Dae wrote:
> On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
>> We now validate the blending mode via g2d_validate_mode()
>> prior to feeding it to g2d_get_blend_op().
>>
>> Signed-off-by: Tobias Jakobi
>> ---
>> exynos/exynos_fimg2d.c | 5 -
>> 1 file cha
On 31 August 2015 at 19:26, Chris Wilson wrote:
> On Mon, Aug 31, 2015 at 07:14:12PM +0100, Emil Velikov wrote:
>> Just like we do for the original exec()
>>
>> v2: move bo_gem declaration to the top of the function.
>>
>> Cc: Chris Wilson
>> Cc: intel-gfx at lists.freedesktop.org
>> Signed-off-b
On Mon, Aug 31, 2015 at 08:01:21PM +0100, Emil Velikov wrote:
> > Reusing bo_gem here is a little worrying as it would be very easy for
> > someone to add code to the end of the function thinking that bo_gem
> > still was the batch.
> >
> Doesn't this concert apply to drm_intel_gem_bo_exec() as wel
Hello!
Inki Dae wrote:
> On 2015ë
08ì 24ì¼ 23:13, Tobias Jakobi wrote:
>> The amount of commands (regular and GEM) doesn't depend
>> on the input here.
>>
>> Signed-off-by: Tobias Jakobi
>> ---
>> exynos/exynos_fimg2d.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/exynos/
Hello,
Emil Velikov wrote:
> On 31 August 2015 at 14:18, Inki Dae wrote:
>> On 2015ë
08ì 24ì¼ 23:14, Tobias Jakobi wrote:
>>> The G2D headers define a number of modes through enums
>>> (like e.g. color, select, repeat, etc.).
>>>
>>> This introduces g2d_validate_select_mode() and
>>> g2d_va
Emil Velikov wrote:
> On 31 August 2015 at 13:36, Inki Dae wrote:
>> Hi Emil,
>>
>> On 2015ë
08ì 31ì¼ 20:58, Emil Velikov wrote:
>>> Hi all,
>>>
>>> On 18 August 2015 at 08:01, Inki Dae wrote:
From: Joonyoung Shim
This patch allows clients who want to use render node to acces
...to minimise misuse of bo_gem.
If the variable is declared at the top of the function and then used
for two (or more) different contexts this can cause confusion and errors.
Just introduce a wrapper, which can be used in a once off situations.
Suggested-by: Chris Wilson
Cc: Chris Wilson
Signe
On Mon, Aug 31, 2015 at 08:47:17PM +0100, Emil Velikov wrote:
> ...to minimise misuse of bo_gem.
> If the variable is declared at the top of the function and then used
> for two (or more) different contexts this can cause confusion and errors.
>
> Just introduce a wrapper, which can be used in a o
On 31 August 2015 at 20:27, Tobias Jakobi
wrote:
> Hello!
>
> Inki Dae wrote:
>> On 2015ë
08ì 24ì¼ 23:13, Tobias Jakobi wrote:
>>> +if (g2d_check_space(ctx, 7, 1))
>>> +return -ENOSPC;
>>
>> You can make 3 and 4 patches to one. These should be same patch.
> Hmm, so which 3 (
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150831/ed03a6e8/attachment.html>
On 07/15/2015 04:57 AM, Lukas Wunner wrote:
> Commit 92122789b2d6 ("drm/i915: preserve SSC if previously set v3")
> added code to intel_modeset_gem_init to override the SSC status read
> from VBT with the SSC status set by BIOS.
>
> However, intel_modeset_gem_init is invoked *after* intel_modeset_
Hi Romain,
å¨ 08/30/2015 08:16 PM, Romain Perier åé:
> Hi,
>
> Could you rebase your serie onto linux-next or 4.2-rc8 ? it does not
> apply here...
Thanks for try to applied, and feel sorry for that failed.
This v3 series was rebased on github.com/torvalds/linux.git, so I should
rebase on k
On 08/25/2015 07:09 AM, Ben Skeggs wrote:
> On 18 August 2015 at 04:04, Hauke Mehrtens wrote:
>> On 08/08/2015 07:01 PM, Hauke Mehrtens wrote:
>>> These patches are adding support for outputting 2560x1440 at 56 over HDMI.
>>> This needs a pixel clock of 225 MHz which was not supported before.
>>>
On Sat, 29 Aug 2015, Rob Herring wrote:
> Thomas,
>
> As requested, here are the remaining patches for killing off
> set_irq_flags which have not been picked up. The rest of the
> series has been picked up and are in -next.
I pick it up, stick it into tip irq/urgent let it brew for a couple of
55 matches
Mail list logo
