eedesktop.org/archives/dri-devel/attachments/20160224/e0a8366b/attachment-0001.obj>
-8<
--
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/20160224/f3169240/attachment.html>
On Wed, Feb 24, 2016 at 01:06:39PM +0200, Tomi Valkeinen wrote:
> Hi,
>
> On 24/02/16 01:30, Stefan Agner wrote:
> > Any comments on this?
> >
> > Also added Manfred, Tomi and Boris to CC which previously attended in
> > similar discussions.
> >
> > Previous discussions:
> > http://thread.gmane.
On Tue, Feb 23, 2016 at 06:46:26PM +0200, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrjälä
>
> Try to detect the max TMDS clock limit for the DP++ adaptor (if any)
> and take it into account when checking the port clock.
>
> Note that as with the sink (HDMI vs. DVI) TMDS clock li
Use devm_kzalloc() and devm_kcalloc() for private data allocation at
driver load time.
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 4 +---
drivers/gpu/drm/tilcdc/tilcdc_drv.c| 19 +++
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 20 ++--
Initialize port device node pointer in the tilcdc crtc. Fixes "Falling
back to first CRTC" warning from tda998x driver.
The tda998x encoder driver calls drm_of_find_possible_crtcs() to
initialize possible_crtcs of struct drm_encoder. The crtc->port needs
to be initialized for drm_of_find_possible_
Disable the sync lost interrupt if it fires on every frame for 50
consecutive frames in a row. This is relatively sure sign of the sync
lost interrupt being stuck and firing on every frame even if the
display otherwise appears to work OK.
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilc
Add ratelimited prints on sync lost and FIFO underrun interrupts.
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 8
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 4 ++--
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_crtc.c
Removes the duplicate LCDC_INT_ENABLE_SET_REG-entry in registers array.
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c
b/drivers/gpu/drm/tilcdc/tilcdc_drv.c
index 964e192
Fix interrupt enable/disable code for version 2 tilcdc. In version 2
tilcdc there is a separate register for disabling interrupts. Writing
0 to enable registers bits does not have any effect. The interrupt
clear register works the same way, writing 1 to specific bit disables
the interrupt and writi
From: Tomi Valkeinen
Do not update the next frame buffer close to vertical blank. This is
to avoid situation when the frame changes between writing of
LCDC_DMA_FB_BASE_ADDR_0_REG and LCDC_DMA_FB_CEILING_ADDR_0_REG.
Signed-off-by: Tomi Valkeinen
[Added description to the patch]
Signed-off-by: Jy
From: Tomi Valkeinen
Get rid of complex ping-pong mechanism and replace it with simpler
single buffer flipping code.
The LCDC HW appears to be designed mainly static framebuffers in
mind. There are two modes of operation, either static single buffer,
or ping pong double buffering with two static
From: Tomi Valkeinen
Cleanup irq handling. Clear the irq status unconditionally and
restructure the status bit conditions.
Signed-off-by: Tomi Valkeinen
[Added description to the patch]
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 9 +
1 file changed, 5 inserti
From: Tomi Valkeinen
Remove broken error handling. The condition for handling the
LCDC_SYNC_LOST and LCDC_FIFO_UNDERFLOW could never be satisfied as the
LCDC_SYNC_LOST interrupt is not enabled. Also the requirement to have
both LCDC_SYNC_LOST and LCDC_FIFO_UNDERFLOW fired at once before
handling
From: Tomi Valkeinen
Split reset to a separate function and use usleep_range(250, 1000)
instead of msleep(1) to to keep the reset bit on long enough.
Signed-off-by: Tomi Valkeinen
[Added description to the patch, changed mdelay(500) to usleep_range(250, 1000)]
Signed-off-by: Jyri Sarha
---
dr
From: Tomi Valkeinen
Disable crtc on unload. Call tilcdc_crtc_dpms() with DRM_MODE_DPMS_OFF
in the beginning of unload function.
Signed-off-by: Tomi Valkeinen
[Added description to the patch]
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 ++
1 file changed, 2 insertion
From: Tomi Valkeinen
Cleanup runtime PM handling. Before the patch the usage of pm_runtime
calls was inconsistent and hard to follow. After the update the
pm_runtime calls are removed from set_scanout() and called around
major operations that access the HW. After the patch the DPMS code does
not
Allocate suspend/resume register storage based on the actual number
registers the driver is aware of. The static allocation for register
storage had fallen behind badly.
Reported-by: Michael Bode
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 21 -
drive
From: Grygorii Strashko
Fix build error when !CONFIG_CPU_FREQ
drivers/gpu/drm/tilcdc/tilcdc_drv.c: In function 'tilcdc_load':
drivers/gpu/drm/tilcdc/tilcdc_drv.c:327:1: error: label 'fail_put_clk' defined
but not used [-Werror=unused-label]
fail_put_clk:
^
Signed-off-by: Grygorii Strashko
Si
There is nothing special about tilcdc HW when the video memory is
concerned. Just using the standard drm helpers for implementation is
enough.
Signed-off-by: Jyri Sarha
---
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/drive
From: Darren Etheridge
The LCD controller must be deactivated and all DMA transactions stopped
when the suspend power state is entered otherwise the PRCM causes the L3
bus to get stuck in transition state.
This commit forces the lcdc to be shut down and waits for all pending DMA
transactions to
From: Darren Etheridge
The frame_done interrupt was only being enabled when the vsync
interrupts were being enabled by DRM. However the frame_done is
used to determine if the LCD controller has successfully completed
the raster_enable, raster_disable commands and the vsync interrupts
are not alw
From: Darren Etheridge
On BeagleBone Black if no HDMI monitor is connected and suspend
is requested a kernel panic will result:
root at am335x-evm:~# echo mem > /sys/power/state
[ 65.548710] PM: Syncing filesystems ... done.
[ 65.631311] Freezing user space processes ... (elapsed 0.006 seconds)
From: Dave Gerlach
Update tilcdc driver to set the state of the pins to:
- "default on resume
- "sleep" on suspend
By optionally putting the pins into sleep state in the suspend callback
we can accomplish two things.
- minimize current leakage from pins and thus save power,
- prevent the IP from
From: Tomi Valkeinen
LCDC hardware does not support fb pitch that is different (i.e. larger)
than the screen size. The driver currently does no checks for this, and
the results of too big pitch are are flickering and lower fps.
This issue easily happens when using libdrm's modetest tool with non
From: Darren Etheridge
Updating the tilcdc DRM driver code to calculate the LCD controller
pixel clock more accurately. Based on a suggested implementation by
Tomi Valkeinen.
The current code does not work correctly and produces wrong results
with many requested clock rates. It also oddly uses t
... and yet one more round.
Changes since v3:
- 21/22 "drm/tilcdc: Initialize crtc->port"
- Return an error and don't use scary WARN but simple dev_err() instead
- 22/22 "drm/tilcdc: Use devm_kzalloc() and devm_kcalloc() for private data"
- Tested module loading and unloading with all relevant
From: Thierry Reding
The error cleanup paths aren't quite correct and will crash upon
deferred probe.
Cc: stable at vger.kernel.org # v4.3+
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/nouveau/nouveau_platform.c | 2 +-
drivers/gpu/drm/nouveau/nvkm/engine/device/tegra.c | 40
Currently host1x-instanciated devices have their dma_ops left to NULL,
which makes any DMA operation (like buffer import) on ARM64 fallback
to the dummy_dma_ops and fail with an error.
This patch calls of_dma_configure() with the host1x node when creating
such a device, so the proper DMA operation
bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/f7761a52/attachment.html>
On 8 February 2016 at 18:12, Archit Taneja wrote:
> Hi,
>
> On 01/06/2016 09:33 AM, Chris Zhong wrote:
>>
>>
>> The rk3288 MIPI DSI is a Synopsys DesignWare MIPI DSI host controller
>> IP. This series adds support for a Synopsys DesignWare MIPI DSI host
>> controller DRM driver.
>>
>> The MIPI DSI
On 02/24/2016 01:04 AM, Thierry Reding wrote:
> * PGP Signed by an unknown key
>
> On Tue, Feb 23, 2016 at 03:25:54PM +0900, Alexandre Courbot wrote:
>> The default DMA mask covers a 32 bits address range, but tegradrm can
>> address more than that. Set the DMA mask to the actual addressable range
On 02/24/2016 12:28 AM, Thierry Reding wrote:
> * PGP Signed by an unknown key
>
> On Tue, Feb 23, 2016 at 03:25:53PM +0900, Alexandre Courbot wrote:
>> The current settings leaves the DRM device's dma_ops field NULL, which
>> makes it use the dummy DMA ops on arm64 and return an error whenever we
Hi Archit,
On Tuesday 23 February 2016 03:06 PM, Archit Taneja wrote:
>
>
> On 02/23/2016 12:57 AM, Rob Herring wrote:
>> On Mon, Feb 22, 2016 at 5:07 AM, Archit Taneja
>> wrote:
>>>
>>>
>>> On 02/22/2016 08:24 AM, Rob Herring wrote:
On Mon, Feb 15, 2016 at 12:23:26PM +0530, Archit T
e for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/cf42bf64/attachment.html>
Hi,
On 24 February 2016 at 16:01, Emil Velikov wrote:
> On 23 February 2016 at 23:56, Rob Clark wrote:
>> On Tue, Feb 23, 2016 at 6:29 PM, Emil Velikov
>> wrote:
>>> On 2 February 2016 at 23:37, Zach Reizner wrote:
The prime fd to handle ioctl was not used with rockchip before. Support
>
Hi Sonika,
A kernel bug report was opened against Ubuntu [0]. After a kernel
bisect, it was found that reverting the following commit resolved this bug:
commit 237ed86c693d8a8e4db476976aeb30df4deac74b
Author: Sonika Jindal
Date: Tue Sep 15 09:44:20 2015 +0530
drm/i915: Check live status
Hi Dave,
A few radeon and amdgpu fixes for 4.5. A few further fixes for the vblank
regressions in 4.4 and a couple of other minor fixes.
The following changes since commit 4fbbed46dca83bcba3a0170205110a06c116a8e6:
drm/nouveau: use post-decrement in error handling (2016-02-19 13:36:05 +1000)
24.02.2016 06:40, Jindal, Sonika напиÑав:
> Do you have logs for the failure with the single link hdmi cable and
> the register dump which you have given for the working case?
> If not, can you please capture the logs and register dump.
> Also which platform is this?
> If it is live status r
23.02.2016 17:08, Ville Syrjälä напиÑав:
> In the meantime I think as a workaround I think you could use
> something like video=HDMI-A-1:e on the kernel command line (not sure
> I got the connector name right for your system). I think that should
> result in the live status check to be ski
On 23 February 2016 at 23:56, Rob Clark wrote:
> On Tue, Feb 23, 2016 at 6:29 PM, Emil Velikov
> wrote:
>> Hi Zach,
>>
>> On 2 February 2016 at 23:37, Zach Reizner wrote:
>>> The prime fd to handle ioctl was not used with rockchip before. Support
>>> was added in order to support potential uses
Hi Alex,
On Wed, Feb 24, 2016 at 3:48 PM, Deucher, Alexander
wrote:
>> -Original Message-
>> From: Geert Uytterhoeven [mailto:geert+renesas at glider.be]
>> Sent: Wednesday, February 24, 2016 3:14 AM
>> To: David Airlie; Deucher, Alexander; Bayyavarapu, Maruthi
>> Cc: dri-devel at lists.f
--
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/5344aac2/attachment.html>
) but with AMD only configuration or
Intel IGP is the same.
--
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/20160224/cffe8
On Wed, Feb 24, 2016 at 11:48 AM, Jyri Sarha wrote:
> From: Tomi Valkeinen
>
> Get rid of complex ping-pong mechanism and replace it with simpler
> single buffer flipping code.
>
> The LCDC HW appears to be designed mainly static framebuffers in
> mind. There are two modes of operation, either st
lly when in combat) still persist for the first
couple of minutes after.
--
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/20
Include directories are provided by core already, adding them in driver
is redundand and causes warnings in case of out-of-tree build.
Signed-off-by: Andrzej Hajda
---
v2:
- fixed include in exynos_drm_iommu.c
- typo in commit message
drivers/gpu/drm/exynos/Makefile | 1 -
d
On 02/24/16 14:38, Tomi Valkeinen wrote:
>
> On 23/02/16 17:03, Jyri Sarha wrote:
>> Use devm_kzalloc() and devm_kcalloc() for private data allocation at
>> driver load time.
>>
>> Signed-off-by: Jyri Sarha
>> ---
>> drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 4 +---
>> drivers/gpu/drm/tilcdc/ti
> -Original Message-
> From: Geert Uytterhoeven [mailto:geert+renesas at glider.be]
> Sent: Wednesday, February 24, 2016 3:14 AM
> To: David Airlie; Deucher, Alexander; Bayyavarapu, Maruthi
> Cc: dri-devel at lists.freedesktop.org; linux-kernel at vger.kernel.org; Geert
> Uytterhoeven
> Sub
le, with all the kernel debug options
enabled, and a few load/unload module cycles?
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.freedes
Hi Vineet,
On Wed, 2016-02-24 at 04:36 +, Vineet Gupta wrote:
> On Tuesday 23 February 2016 07:22 PM, Jose Abreu wrote:
> > The defconfigs for the AXS boards were updated so that
> > ALSA SoC is enabled and also the audio for the ADV7511
> > HDMI transmitter.
> >
> > Signed-off-by: Jose Abreu
lication/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/89cfa974/attachment.sig>
https://bugzilla.kernel.org/show_bug.cgi?id=113051
Bug ID: 113051
Summary: Crash on resume
Product: Drivers
Version: 2.5
Kernel Version: 4.5rc5
Hardware: All
OS: Linux
Tree: Mainline
Status: NEW
. I'm not sure if that's only
for OMAP4, though.
I'll drop this patch too from the series, and spend a bit more time on
it. This is again something that's a bit tricky to reproduce and test.
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/20160224/4a1f509e/attachment.sig>
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/20160224/234d0685/attachment.sig>
mi
-- 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/20160224/6b4c18bd/attachment.sig>
Am Dienstag, den 23.02.2016, 15:30 -0800 schrieb Stefan Agner:
> Any comments on this?
None other that I'm all in favor. consider patch 2
Acked-by: Philipp Zabel
> Also added Manfred, Tomi and Boris to CC which previously attended in
> similar discussions.
>
> Previous discussions:
> http://thr
pt
doing a memcpy, ruining the performance.
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/20160224/5905fec9/attachment-0001.sig>
On Wed, Feb 24, 2016 at 9:58 AM, Geert Uytterhoeven
wrote:
> Hi Alex,
>
> On Wed, Feb 24, 2016 at 3:48 PM, Deucher, Alexander
> wrote:
>>> -Original Message-
>>> From: Geert Uytterhoeven [mailto:geert+renesas at glider.be]
>>> Sent: Wednesday, February 24, 2016 3:14 AM
>>> To: David Airli
On 2016-02-24 03:06, Tomi Valkeinen wrote:
> Hi,
>
> On 24/02/16 01:30, Stefan Agner wrote:
>> Any comments on this?
>>
>> Also added Manfred, Tomi and Boris to CC which previously attended in
>> similar discussions.
>>
>> Previous discussions:
>> http://thread.gmane.org/gmane.linux.kernel.api/128
On 02/24/2016 01:34 AM, Rob Herring wrote:
> On Tue, Feb 23, 2016 at 01:11:24PM +0200, Tomi Valkeinen wrote:
>>
>>
>> On 23/02/16 12:43, Archit Taneja wrote:
>>>
>>>
>>> On 02/23/2016 02:48 PM, Tomi Valkeinen wrote:
On 22/02/16 22:10, Rob Herring wrote:
>> If we want all DSI ho
On 2/23/2016 8:38 PM, Ville Syrjälä wrote:
> On Mon, Feb 22, 2016 at 02:32:32PM +0200, Oleksandr Natalenko wrote:
>> Ville, Daniel,
>>
>> any additional info I could provide? I have to return dual-link DVI
>> cable back, so let me know if I could reveal more details if necessary.
> Unfortunatel
On 02/12/2016 08:30 PM, Mario Kleiner wrote:
> Changes to drm_update_vblank_count() in Linux 4.4 broke the
> behaviour of the pre/post modeset functions as the new update
> code doesn't deal with hw vblank counter resets inbetween calls
> to drm_vblank_pre_modeset an drm_vblank_post_modeset, as it
Hey,
Op 18-02-16 om 13:59 schreef Ville Syrjälä:
> On Thu, Feb 18, 2016 at 01:43:11PM +0100, Daniel Vetter wrote:
>> On Thu, Feb 18, 2016 at 12:18:53PM +0100, Maarten Lankhorst wrote:
>>> Op 18-02-16 om 12:07 schreef Daniel Vetter:
On Thu, Feb 18, 2016 at 09:54:43AM +0100, Maarten Lankhorst
Now that only encoders can be stolen that are part of the state
steal_encoder no longer needs to inspect all connectors,
just those that are part of the atomic state.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic_helper.c | 11 ++-
1 file changed, 2 insertions(+), 9 del
The current check doesn't handle the case where we don't steal an
encoder, but keep it on the current connector. If we repurpose
disable_conflicting_encoders to do the checking, we just have
to reject the ones that conflict.
Signed-off-by: Maarten Lankhorst
Testcase: kms_setmode.invalid-clone-sin
Instead of failing with -EINVAL when conflicting encoders are found,
the legacy set_config will disable other connectors when encoders
conflict.
With the cleanup to update_output_state this is a lot easier to
implement. set_config only adds connectors to the state that are
modified, and because of
There's no need to have a separate function to get the crtc
which is stolen, this can already be found when actually
stealing the encoder.
drm_for_each_connector already checks for connection_mutex, so
use that macro now.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic_helper.c
Minor cleanup, connector and connector_state are always non-NULL here.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic_helper.c | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/drm_atomic_helper.c
b/drivers/gpu/drm/drm_atomic_h
With the addition of crtc_state->connector_mask other connectors from
different crtc's aren't needed any more to determine if a crtc has
connectors, so only call add_affected_connectors on the target crtc.
This allows a cleanup to first remove all current connectors, then
add all set->connectors to
After trying out various ways to handle encoder stealing better
I came up with a cleaner way.
The first patch cleans up update_output_state and only adds affected connectors.
This is required to determine which connectors are not part of the atomic state,
and allow disabling them if they have conf
By default, not only this driver is enabled on all platforms, but also
generic PM Domains and Multi-Function Devices.
Drop the "default y" to fix this.
Signed-off-by: Geert Uytterhoeven
---
drivers/gpu/drm/amd/acp/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/ac
now what your interest and strengths are. Thus listing
> those would be beneficial. This way people can come up with more
> 'personalised' ideas, should the ones on the wikis (short of links
> atm, sorry) do not suit you.
>
> Regards,
> Emil
>
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/7d24f035/attachment.html>
On 2/24/2016 3:41 AM, Lyude wrote:
> As it turns out, resuming DP MST is racey since we don't make sure MST
> is ready before we start modesetting, it just usually happens to be
> ready in time. This isn't the case on all systems, particularly a
> ThinkPad T560 with displays connected through the
Hi Ville,
We will look into this in sometime. Right now team is slightly loaded due to
project milestone.
Last time I looked into this, we dint have this HW to reproduce this issue.
Regards
Shashank
-Original Message-
From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
Sent: T
On Tuesday 23 February 2016 07:22 PM, Jose Abreu wrote:
> The defconfigs for the AXS boards were updated so that
> ALSA SoC is enabled and also the audio for the ADV7511
> HDMI transmitter.
>
> Signed-off-by: Jose Abreu
> ---
> arch/arc/configs/axs101_defconfig | 3 +++
> arch/arc/configs/axs
een marked as a duplicate of bug 79783 ***
--
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/20160224/ec65371f/attachment.html>
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/25bf24f8/attachment.html>
--
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/20160224/761b6b25/attachment.html>
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:48:08 Tomi Valkeinen wrote:
> omapdrm is missing a check on the validity of the rotation property.
> This leads to omapdrm possibly trying to use rotation on non-rotateable
> framebuffer, which causes the overlay setup to fail.
>
>
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:48:07 Tomi Valkeinen wrote:
> Before universal planes we had to have plane specific properties for the
> crtc too, as on the hardware level a crtc uses a plane. In other words,
> e.g. 'zorder' property was added to both planes and cr
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:48:05 Tomi Valkeinen wrote:
> The DSS hardware uses the same ROW_INC value for both Y and UV planes
> for NV12 format. This means that the pitches of the Y and UV planes have
> to match. omapdrm doesn't check this at the moment, and
ssignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160224/f6bc28ee/attachment-0001.html>
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:53 Tomi Valkeinen wrote:
> omap_crtc_wait_pending() waits until the config changes have been taken
> into use, usually at next vblank. The wait-timeout used is 50ms, which
> usually is enough, but in some rare cases not.
>
> As t
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:52 Tomi Valkeinen wrote:
> We no longer have the omapdrm plugin system for SGX, and we can thus
> remove the support for external memory and sync objects from omap_gem.c.
>
> Signed-off-by: Tomi Valkeinen
> ---
> drivers/gpu/dr
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:48 Tomi Valkeinen wrote:
> If the panel's enable fails, omap_encoder silently ignores the failure.
> omapdrm should really handle the failure, but unfortunately the whole
> encoder enable codepath is expected to always succeed.
>
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:47 Tomi Valkeinen wrote:
> omap_gem_dma_sync() calls dma_map_page() but does not check the possible
> error with dma_mapping_error(). If DMA-API debugging is enabled, the
> debug layer will give a warning if dma_mapping_error() ha
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:46 Tomi Valkeinen wrote:
> omap_gem_attach_pages() calls dma_map_page() but does not check the
> possible error with dma_mapping_error(). If DMA-API debugging is
> enabled, the debug layer will give a warning if dma_mapping_error(
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:45 Tomi Valkeinen wrote:
> OMAP4+ DSS has WBUNCOMPLETEERROR irq, which was not defined in the irq
> list. Add the define.
>
> Signed-off-by: Tomi Valkeinen
Acked-by: Laurent Pinchart
> ---
> include/video/omapdss.h | 1 +
>
Hi Tomi,
Thank you for the patch.
On Friday 19 February 2016 11:47:41 Tomi Valkeinen wrote:
> From: Jyri Sarha
>
> drm_atomic_get_plane_state() may return ERR_PTR. Handle
> drm_atomic_get_plane_state() return values right in
> omap_crtc_atomic_set_property().
>
> Signed-off-by: Jyri Sarha
> A
90 matches
Mail list logo
