Hi Dave,
This 2nd pull request includes one patch I missed, some cleanups
and fixups.
Summary:
- expose HDMI-PHY clock to other drivers.
. this patch was included in below patch series but I missed.
http://www.spinics.net/lists/dri-devel/msg103097.html
- some fixu
bbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/8cb9c99a/attachment-0001.html>
On Mon, 2016-05-09 at 15:39 -0700, Stephen Boyd wrote:
> On 05/09, Stephen Boyd wrote:
> >
> >
> > Ok I applied this one to clk-next.
> >
> And I squashed this in to silence the following checker warning.
>
> drivers/clk/sunxi/clk-sun4i-display.c:110:33: warning: Variable
> length array is used
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/20160510/65d6f67f/attachment.html>
the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/a5ee2fbc/attachment-0001.html>
Den 10.05.2016 16:18, skrev Daniel Vetter:
> On Tue, May 10, 2016 at 4:04 PM, Noralf Trønnes
> wrote:
>> Den 10.05.2016 11:14, skrev Jyri Sarha:
>>> On 05/10/16 09:34, Daniel Vetter wrote:
>> There's a drm_simple_display_pipe floating around which seems perfectly
suited to tilcdc.
From: Huang Rui
Actually, pp_sw_init executes pptable_init and backend_init orderly if
they are initialized successfully. So rewrite it to make code more
readable.
Reviewed-by: Alex Deucher
Signed-off-by: Huang Rui
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/amd_powerplay.c
On Tuesday 10 May 2016 16:18:54 Daniel Vetter wrote:
> On Tue, May 10, 2016 at 4:04 PM, Noralf Trønnes
> wrote:
> > Den 10.05.2016 11:14, skrev Jyri Sarha:
> >> On 05/10/16 09:34, Daniel Vetter wrote:
> >>> There's a drm_simple_display_pipe floating around which seems
> >>> perfectly sui
On Tue, May 10, 2016 at 5:11 PM, Laurent Pinchart
wrote:
>> >> Hmmm, too bad it wants to own its encoder. LCDC on Beaglebone-Black
>> >> needs the componentized external tda998x driver. So at least in its
>> >> current form the drm_simple_display_pipe wont work for tilcdc.
>> >>
>> >> It may not b
https://bugzilla.kernel.org/show_bug.cgi?id=117861
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment #1 f
This patch allows panels to set pixel clock and data enable pin polarity
other than the default of driving data at the rising pixel clock edge and
active high display enable.
Signed-off-by: Philipp Zabel
---
drivers/gpu/drm/imx/imx-drm-core.c | 9 ++---
drivers/gpu/drm/imx/imx-drm.h
Combined with the previous commit, this fixes all of the sparse warnings
in drm/rockchip.
Signed-off-by: John Keeping
---
drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 4 ++--
drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 6 +++---
drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 4 ++--
drivers/gp
This fixes the following sparse warnings:
drivers/gpu/drm/rockchip/rockchip_drm_fb.c:32:23: warning: symbol
'rockchip_fb_get_gem_obj' was not declared. Should it be static?
drivers/gpu/drm/rockchip/rockchip_drm_fb.c:315:24: warning: symbol
'rockchip_drm_framebuffer_init' was not declared. Should
drm_encoder_slave is not used in this file.
Signed-off-by: John Keeping
---
drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
index 801110f65a63..0665fb915579 1
The first patch wasn't found by sparse, but is something I noticed while
in the area. The following two fix all of the sparse warnings inside
drm/rockchip.
John Keeping (3):
drm/rockchip: dw_hdmi: remove unused #include
drm/rockchip: fb: add missing header
drm/rockchip: fix "should it be st
Hi Dave,
The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
are available in the git repository at:
git://anongit.freedesktop.org/tegra/linux tags/drm/panel/for-4.7-rc1
for you to fetch changes up to 69b3478efeed79059d
Since commit 9f0ba539d13a ("drm/gem: support BO freeing without
dev->struct_mutex"), drivers that are struct_mutex free are supposed
to use the gem_free_object_unlocked callback.
Fixes: 119f5173628a ("drm/mediatek: Add DRM Driver for Mediatek SoC MT8173.")
Reported-by: Daniel Vetter
Signed-off-by
Hi Dave,
The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
are available in the git repository at:
git://anongit.freedesktop.org/tegra/linux tags/drm/tegra/for-4.7-rc1
for you to fetch changes up to 057eab2013ecbba2e77229
2016ë
05ì 10ì¼ 16:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> On 05/10/2016 08:24 AM, Inki Dae wrote:
>> Hi Andrzej,
>>
>> 2016ë
05ì 10ì¼ 15:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>>> Hi Inki,
>>>
>>>
>>> On 05/10/2016 07:31 AM, Inki Dae wrote:
Hi Andrzej,
2016ë
04ì 29ì¼ 2
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/f7ce23a5/attachment.sig>
behind
it could possibly change later...
A "if (crtc->is_hw_enabled)" would be much more readable.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/5712104b/attachment.sig>
On Tue, May 10, 2016 at 4:04 PM, Noralf Trønnes wrote:
> Den 10.05.2016 11:14, skrev Jyri Sarha:
>>
>> On 05/10/16 09:34, Daniel Vetter wrote:
>
> There's a drm_simple_display_pipe floating around which seems perfectly
>>>
>>> suited to tilcdc. It's meant for the case where you ha
This patch changes GPL license to X11/MIT.
Signed-off-by: Inki Dae
---
exynos/exynos_fimg2d.c | 21 +
exynos/exynos_fimg2d.h | 21 +
exynos/fimg2d_reg.h| 21 +
tests/exynos/exynos_fimg2d_event.c |
k &= ~DISPC_IRQ_SYNC_LOST_DIGIT;
> -
> - omap_irq_register(dev, error_handler);
This makes me wonder is the previous patch correct... It doesn't ignore
the SYNC_LOST_DIGIT. Oh, but is this ignore only for the error handler
that only prints. Ah, confusing =).
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/61e34ec0/attachment.sig>
Den 10.05.2016 11:14, skrev Jyri Sarha:
> On 05/10/16 09:34, Daniel Vetter wrote:
There's a drm_simple_display_pipe floating around which seems perfectly
>> suited to tilcdc. It's meant for the case where you have 1 plane, 1 crtc
>> and 1 encoder maybe linking to different connectors.
ed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/7a3b5629/attachment.sig>
2016ë
04ì 27ì¼ 02:29ì Daniel Vetter ì´(ê°) ì´ ê¸:
> No dev->struct_mutex anywhere to be seen.
Acked-by: Inki Dae
Thanks,
Inki Dae
>
> Cc: Inki Dae
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion
Hi Daniel,
2016ë
04ì 27ì¼ 20:38ì Daniel Vetter ì´(ê°) ì´ ê¸:
> It's an optional hook. Might be needed for frontbuffer rendering on
> manual upload displays, but a simple TODO doesn't explain at all what
> needs to be done or why.
We have a plan for partial update support but not now ye
t;>> omapdss-$(CONFIG_OMAP2_DSS_HDMI_COMMON) += hdmi_common.o hdmi_wp.o
>>> hdmi_pll.o \
>>> hdmi_phy.o
>>> +ifeq ($(CONFIG_OMAP2_DSS_HDMI_CEC),y)
>>> + omapdss-$(CONFIG_OMAP2_DSS_HDMI_COMMON) += hdmi_cec.o
>>> +endif
>>
>> The file should be hdmi4_cec.o, as it's for omap4. And why the ifeq?
>> Isn't just
>>
>> omapdss-$(OMAP4_DSS_HDMI_CEC) += hdmi4_cec.o
>
> OMAP4_DSS_HDMI_CEC is a bool, not a tristate.
Yes, and that's fine. You're not compiling hdmi4_cec.o as a module,
you're compiling it into omapdss.o. Objects are added with "omapdss-y +=
xyz" to omapdss.o.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/2e2d37a4/attachment.sig>
| 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Thierry Reding
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-de
Hi Andrzej,
2016ë
05ì 10ì¼ 15:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> Hi Inki,
>
>
> On 05/10/2016 07:31 AM, Inki Dae wrote:
>> Hi Andrzej,
>>
>> 2016ë
04ì 29ì¼ 22:42ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>>> It seems trigger cannot be configured too early, otherwise it does not work
>>>
This patch aims to replace the roll-your-own seqlock implementation with
full-blown seqlock'. We also remove the timestamp ring-buffer in favour
of single timestamp/count pair protected by a seqlock. In turn this
means we can now increment the vblank freely without the need for
clamping.
v2:
- r
writing the configs to the registers and starting the
transfer.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/782aba8d/attachment.sig>
Hello Flora Cui,
The patch 2cc0c0b5cd4d: "drm/amdgpu: change ELM/BAF to
Polaris10/Polaris11" from Mar 14, 2016, leads to the following static
checker warning:
drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/polaris10_hwmgr.c:3620
polaris10_get_pp_table_entry_callback_func()
warn: b
On 6 May 2016 at 19:38, Lucas Stach wrote:
> Hi Dave,
>
> not much new stuff this time. A (micro-)optimization to allow the
> hangcheck timer to be coalesced with other wakeups in the system and a
> fix to handle mmaping of prime imported and userptr buffers correctly. I
> don't think we have seen
nown;
> }
>
Thanks, I've picked this up for 4.8.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/6476afc4/attachment.sig>
omi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/4e7c1aa1/attachment-0001.sig>
On 9 May 2016 at 18:17, Daniel Vetter wrote:
> On Wed, May 04, 2016 at 02:26:42PM -0400, Alex Deucher wrote:
>> From: Chunming Zhou
>>
>> The release of the vmid owner was not handled
>> correctly. We need to take the lock and walk
>> the lru list.
>>
>> Signed-off-by: Chunming Zhou
>> Reviewed
Hello Alex Deucher,
The patch d525eb8d2e67: "drm/amdgpu: add ELM/BAF DCE11 configs (v2)"
from Oct 14, 2015, leads to the following static checker warning:
drivers/gpu/drm/amd/amdgpu/dce_v11_0.c:1631 dce_v11_0_audio_init()
error: buffer overflow 'pin_offsets' 7 <= 7
drivers/gpu/dr
> +
> hdmi_pll_uninit(&hdmi.pll);
>
> pm_runtime_disable(&pdev->dev);
> diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi_cec.c
> b/drivers/gpu/drm/omapdrm/dss/hdmi_cec.c
> new file mode 100644
> index 000..d4309df
> --- /dev/null
> +++ b/drivers/gpu/drm/omapdrm/dss/hdmi_cec.c
> @@ -0,0 +1,329 @@
> +/*
> + * HDMI CEC
> + *
> + * Based on the CEC code from hdmi_ti_4xxx_ip.c from Android.
> + *
> + * Copyright (C) 2010-2011 Texas Instruments Incorporated -
> http://www.ti.com/
> + * Authors: Yong Zhi
> + * Mythri pk
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License version 2 as published
> by
> + * the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful, but
> WITHOUT
> + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
> + * more details.
> + */
> +
> +#include
> +#include
> +#include
> +#include
> +#include
> +#include
> +
> +#include "dss.h"
> +#include "hdmi.h"
> +
> +#define HDMI_CORE_CEC_RETRY200
> +
> +void hdmi_cec_transmit_fifo_empty(struct hdmi_cec_data *cec, u32 stat1)
> +{
> + if (stat1 & 2) {
> + u32 dbg3 = hdmi_read_reg(cec->base, HDMI_CEC_DBG_3);
This is a debug register. I haven't looked at how CEC is to be used, but
using a debug register looks a bit suspicious =).
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/404975df/attachment.sig>
e.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/6f1653e2/attachment-0001.sig>
On Tue, May 10, 2016 at 2:35 PM, Tobias Jakobi
wrote:
>
> Hi Inki,
>
> I guess you need some kind of tag from me here? I'm not sure if it
> should be Acked-by or Reviewed-by.
Also other people contributed there in the past (in the previous
license) so I guess consent from all of them is required.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/c0d26d45/attachment.sig>
Hi Inki,
I guess you need some kind of tag from me here? I'm not sure if it
should be Acked-by or Reviewed-by.
@Emil: You're the expert, what should it be?
I also would suggest to change the commit description to _why_ this
change is needed (and not just copy the commit subject).
With best wis
Hi Andrzej,
2016ë
04ì 29ì¼ 22:42ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> It seems trigger cannot be configured too early, otherwise it does not work in
> case of panel. The patch fixes also trigger flag logic, previously HW-TRIGGER
> flag was cleared in case of panel - as a result panel used alw
Hi Tomi,
On 05/10/16 14:01, Tomi Valkeinen wrote:
> Hi Hans,
>
> On 29/04/16 12:39, Hans Verkuil wrote:
>> From: Hans Verkuil
>>
>> Signed-off-by: Hans Verkuil
>> ---
>> arch/arm/boot/dts/omap4-panda-a4.dts | 2 +-
>> arch/arm/boot/dts/omap4-panda-es.dts | 2 +-
>> arch/arm/boot/dts/om
ct is trivial, though. Do you want me
to resolve it and apply, or do you want to keep this in your series?
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/808a7703/attachment.sig>
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/58fc38e7/attachment.sig>
c: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/395b01d4/attachment.sig>
Hello rezhu,
The patch c81726252ec9: "drm/amd/powerplay: add smu support for
ellesmere/baffin" from Nov 10, 2015, leads to the following static
checker warning:
drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/polaris10_smumgr.c:978
polaris10_smum_init()
warn: returning -1 instead
> #include
> #include
> +#include
> #include
> #include
> #include
Thanks, queued for omapdrm fixes.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP
Hello Eric Huang,
The patch 5632708f4452: "drm/amd/powerplay: add dpm force multiple
levels on cz/tonga/fiji/polaris (v2)" from Apr 12, 2016, leads to the
following static checker warning:
drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c:368 amdgpu_set_pp_dpm_sclk()
warn: why is the last el
Hi Dan,
It is for avoiding the last char of newline. I will change it.
Thanks,
Eric
On 16-05-10 06:51 AM, Dan Carpenter wrote:
> Hello Eric Huang,
>
> The patch 5632708f4452: "drm/amd/powerplay: add dpm force multiple
> levels on cz/tonga/fiji/polaris (v2)" from Apr 12, 2016, leads to the
> foll
Thanks Eric!
On 2016-05-10 01:06 PM, Eric Anholt wrote:
> Robert Foss writes:
>
>> On 2016-05-03 03:22 PM, Eric Anholt wrote:
>>> robert.foss at collabora.com writes:
>>>
From: Robert Foss
As per the documentation in drm_crtc.h, atomic_commit should return
-EBUSY if an asycnh
On Tue, May 10, 2016 at 12:44:22PM +0300, Mika Kahola wrote:
> On Tue, 2016-05-03 at 16:28 +0200, Daniel Vetter wrote:
> > On Tue, May 03, 2016 at 04:23:34PM +0300, Ville Syrjälä wrote:
> > > On Tue, May 03, 2016 at 02:46:36PM +0300, Mika Kahola wrote:
> > > > Prep work to improve DP branch devic
On Tue, 2016-05-03 at 16:28 +0200, Daniel Vetter wrote:
> On Tue, May 03, 2016 at 04:23:34PM +0300, Ville Syrjälä wrote:
> > On Tue, May 03, 2016 at 02:46:36PM +0300, Mika Kahola wrote:
> > > Prep work to improve DP branch device handling.
> > >
> > > Filter out a mode that exceeds the max pixel
On 2016-05-10 08:07, Philipp Zabel wrote:
> This patch allows panels to set pixel clock and data enable pin polarity
> other than the default of driving data at the rising pixel clock edge and
> active high display enable.
>
> Signed-off-by: Philipp Zabel
> ---
> drivers/gpu/drm/imx/imx-drm-
On Tue, 10 May 2016, Dave Airlie wrote:
> From: Dave Airlie
>
> We need to use this for validating modeline additions.
>
> Signed-off-by: Dave Airlie
> ---
> drivers/gpu/drm/drm_edid.c | 44 ++--
> 1 file changed, 26 insertions(+), 18 deletions(-)
>
> dif
On 05/10/16 09:34, Daniel Vetter wrote:
>>> There's a drm_simple_display_pipe floating around which seems perfectly
>>> > > suited to tilcdc. It's meant for the case where you have 1 plane, 1 crtc
>>> > > and 1 encoder maybe linking to different connectors. And it takes care
>>> > > of
>>> > > all
On Tue, 10 May 2016, Dave Airlie wrote:
> From: Tomas Bzatek
>
> This will iterate over all DisplayID blocks found in the buffer.
> Previously only the first block was parsed.
>
> https://bugs.freedesktop.org/show_bug.cgi?id=95207
>
> Signed-off-by: Tomas Bzatek
> Signed-off-by: Dave Airlie
> -
On Tue, 10 May 2016, Jani Nikula wrote:
> On Tue, 10 May 2016, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> This just makes the code easier to follow.
>>
>> Signed-off-by: Dave Airlie
>
> Reviewed-by: Jani Nikula
Ooops, one thing needs fixing below.
>
>> ---
>> drivers/gpu/drm/drm_edid.c |
On 2016-05-03 03:22 PM, Eric Anholt wrote:
> robert.foss at collabora.com writes:
>
>> From: Robert Foss
>>
>> As per the documentation in drm_crtc.h, atomic_commit should return
>> -EBUSY if an asycnhronous update is requested and there is an earlier
>> update pending.
>>
>> Note: docs cited he
From: Dave Airlie
The tiled 5K Dell monitor appears to be hiding it's tiled mode
inside the displayid timings block, this patch parses this
blocks and adds the modes to the modelist.
v1.1: add missing __packed.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95207
Signed-off-by: Dave Air
From: Dave Airlie
We need to use this for validating modeline additions.
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/drm_edid.c | 44 ++--
1 file changed, 26 insertions(+), 18 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm
From: Tomas Bzatek
This will iterate over all DisplayID blocks found in the buffer.
Previously only the first block was parsed.
https://bugs.freedesktop.org/show_bug.cgi?id=95207
Signed-off-by: Tomas Bzatek
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/drm_edid.c | 30 ++
From: Dave Airlie
This just makes the code easier to follow.
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/drm_edid.c | 111 -
1 file changed, 59 insertions(+), 52 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
in
>>
>> +struct displayid_detailed_timings_1 {
>> + u8 pixel_clock[3];
>> + u8 flags;
>> + u8 hactive[2];
>> + u8 hblank[2];
>> + u8 hsync[2];
>> + u8 hsw[2];
>> + u8 vactive[2];
>> + u8 vblank[2];
>> + u8 vsync[2];
>> + u8 vsw[2];
>
> An alternative would be t
>>> index 623b4e9..042f9fc 100644
>>> --- a/include/drm/drm_displayid.h
>>> +++ b/include/drm/drm_displayid.h
>>> @@ -52,7 +52,8 @@
>>> #define PRODUCT_TYPE_DIRECT_DRIVE 6
>>>
>>> struct displayid_hdr {
>>> -u8 rev;
>>> +u8 rev:4;
>>> +u8 ver:4;
>>> u8 bytes;
>>> u8 prod_id;
On Tue, 10 May 2016, Dave Airlie wrote:
> From: Dave Airlie
>
> This just makes the code easier to follow.
>
> Signed-off-by: Dave Airlie
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/drm_edid.c | 111
> -
> 1 file changed, 59 insertions(+), 52
at.
So, in the end, all I know is that describing pixel formats well is not
easy =).
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/e800ce7a/attachment-0001.sig>
Hi Daniel, Thank you for your patch.
On 6 May 2016 at 16:28, Daniel Vetter wrote:
> Also, the unbind function is totally not sufficient, and it's calling
> the deprecated drm_put_dev. Please look at what other drivers are
> doing and fix things up.
>
> Cc: Xinliang Liu
> Cc: Xinwei Kong
> Sign
On 9 May 2016 at 14:44, Daniel Vetter wrote:
> On Mon, May 09, 2016 at 10:32:07AM +0800, Xinliang Liu wrote:
>> Remove deprecated drm_put_dev.
>> Clean up everything needed in unbind.
>> Thanks to Daniel Vetter, this issue is reported by him.
>>
>> Signed-off-by: Xinliang Liu
>> Reported-by: Dani
Am 10.05.2016 um 07:05 schrieb Dave Airlie:
> On 9 May 2016 at 18:17, Daniel Vetter wrote:
>> On Wed, May 04, 2016 at 02:26:42PM -0400, Alex Deucher wrote:
>>> From: Chunming Zhou
>>>
>>> The release of the vmid owner was not handled
>>> correctly. We need to take the lock and walk
>>> the lru l
.
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/cfffbd03/attachment.sig>
regarding this patch.
> Does anyone have objections to it being merged?
I merged it to drm-vc4-next last night, actually :)
-- next part ------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/20e1c636/attachment.sig>
Hi Dave,
On Fri, 2016-04-29 at 11:36 +, Alexey Brodkin wrote:
> Hi Dave,
>
> Please pull this mini-series that allows ARC PGU to use
> dedicated memory location as framebuffer backing storage.
>
> v2 of that series was reviewed here
> https://lists.freedesktop.org/archives/dri-devel/2016-Apr
mode_info.audio.pin[i], false);
> 1636 }
> 1637
> 1638 return 0;
> 1639 }
>
> regards,
> dan carpenter
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesk
or the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/9f2b96bb/attachment.html>
It is a bug. I will fix it.
Best Regars
Rex
-Original Message-
From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
Sent: Tuesday, May 10, 2016 3:42 AM
To: Zhu, Rex
Cc: dri-devel at lists.freedesktop.org
Subject: re: drm/amd/powerplay: enable dpm for baffin.
Hello Rex Zhu,
This is a s
is a huge change, and probably
requires rewriting good part of omapdrm and omapdss drivers. And then
porting all the encoder and panel drivers to DRM.
So I'm not sure when I can start on that work, but even when I do, I
expect it to take a rather long time. But I know my life would be easier
after that change, so I very much like to make it happen.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/dcbbc0a3/attachment.sig>
On 05/10/2016 08:24 AM, Inki Dae wrote:
> Hi Andrzej,
>
> 2016ë
05ì 10ì¼ 15:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>> Hi Inki,
>>
>>
>> On 05/10/2016 07:31 AM, Inki Dae wrote:
>>> Hi Andrzej,
>>>
>>> 2016ë
04ì 29ì¼ 22:42ì Andrzej Hajda ì´(ê°) ì´ ê¸:
It seems trigger cannot be co
https://bugzilla.kernel.org/show_bug.cgi?id=117931
--- Comment #2 from Eugene A. Shatokhin ---
Created attachment 215721
--> https://bugzilla.kernel.org/attachment.cgi?id=215721&action=edit
Xorg.0.log after resume
--
You are receiving this mail because:
You are watching the assignee of the bu
https://bugzilla.kernel.org/show_bug.cgi?id=117931
--- Comment #1 from Eugene A. Shatokhin ---
Created attachment 215711
--> https://bugzilla.kernel.org/attachment.cgi?id=215711&action=edit
lspci -vnn
--
You are receiving this mail because:
You are watching the assignee of the bug.
some don't
--
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/20160510/8c7e9b67/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=117931
Bug ID: 117931
Summary: Black screen after resume from hibernate on HP
Pavilion dv7 with 2 AMD GPUs
Product: Drivers
Version: 2.5
Kernel Version: 4.6-rc5 (mainline), 3.14.53, 4.4
On Mon, May 09, 2016 at 11:07:53PM +0300, Laurent Pinchart wrote:
> Hi Ville,
>
> On Monday 09 May 2016 15:07:21 Ville Syrjälä wrote:
> > On Mon, May 09, 2016 at 02:43:02PM +0300, Laurent Pinchart wrote:
> > > On Saturday 07 May 2016 21:11:23 Ville Syrjälä wrote:
> > >> On Fri, May 06, 2016 at
On Mon, May 09, 2016 at 08:37:39PM +0200, Noralf Trønnes wrote:
>
> Den 09.05.2016 16:46, skrev Daniel Vetter:
> >On Thu, May 05, 2016 at 03:24:33PM +0200, Noralf Trønnes wrote:
> >>Provides helper functions for drivers that have a simple display
> >>pipeline. Plane, crtc and encoder are collaps
On Mon, May 09, 2016 at 09:19:22PM +0200, Noralf Trønnes wrote:
>
> Den 05.05.2016 18:23, skrev Daniel Vetter:
> >On Thu, May 05, 2016 at 03:24:32PM +0200, Noralf Trønnes wrote:
> >>Make drm_encoder_helper_funcs and it's functions optional to avoid
> >>having dummy functions.
> >>
> >>Signed-off
On Tue, May 10, 2016 at 12:29:23AM +0300, Jyri Sarha wrote:
> On 05/09/16 17:42, Daniel Vetter wrote:
> >> > It's not clear to me if a (primary) plane is required with atomic
> >> > universal planes and modesetting or not... I guess it is, when thinking
> >> > of how e.g. a framebuffer is configure
text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/88855f6a/attachment.sig>
On Tue, May 10, 2016 at 12:29:23AM +0300, Jyri Sarha wrote:
> On 05/09/16 17:42, Daniel Vetter wrote:
> >> > It's not clear to me if a (primary) plane is required with atomic
> >> > universal planes and modesetting or not... I guess it is, when thinking
> >> > of how e.g. a framebuffer is configure
Hi Inki,
On 05/10/2016 07:31 AM, Inki Dae wrote:
> Hi Andrzej,
>
> 2016ë
04ì 29ì¼ 22:42ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>> It seems trigger cannot be configured too early, otherwise it does not work
>> in
>> case of panel. The patch fixes also trigger flag logic, previously HW-TRIGGER
>>
.
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/27fac386/attachment.html>
On 05/09/16 17:42, Daniel Vetter wrote:
>> > It's not clear to me if a (primary) plane is required with atomic
>> > universal planes and modesetting or not... I guess it is, when thinking
>> > of how e.g. a framebuffer is configured to be shown on a screen when
>> > using the atomic modesetting uap
plane implementation you added is not
> complex, but is it something that should be implemented with DRM helper
> funcs?
>
> Tomi
>
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160510/0f1f2400/attachment.sig>
95 matches
Mail list logo
