On some devices the LCD panel is mounted in the casing in such a way that
the up/top side of the panel does not match with the top side of the
device (e.g. it is mounted upside-down).
This commit adds the necessary infra for lcd-panel drm_connector-s to
have a "panel orientation" property to commu
Yes I agree it has to be started with the write transaction, according of PCIe
standard all write transaction are address routed, and I agree with Logan:
if in write transaction TLP the endpoint address written in header the TLP
should not touch CPU, the PCIe Switch has to route it to endpoint.
T
Hi,
On Fri, Oct 13, 2017 at 10:12:08AM -0700, Tony Lindgren wrote:
> * Tomi Valkeinen [171012 01:46]:
> > On 29/09/17 16:26, Sebastian Reichel wrote:
> > > Hi Tomi & Laurent,
> > >
> > > ping?
> >
> > I've been having quick glances at this every now and then, but I'm not
> > sure what to do wit
On some hardware the LCD panel is not mounted upright in the casing,
but rotated by 90 degrees. In this case we want the console to
automatically be rotated to compensate.
The drm subsys has a quirk table for this, use the
drm_get_panel_orientation_quirk function to get the panel orientation
and s
Apply the "panel orientation" drm connector prop to the primary plane so
that fbcon and fbdev using userspace programs display the right way up.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=94894
Signed-off-by: Hans de Goede
---
Changes in v2:
-New patch in v2 of this patch-set
Changes in
On Tue, Oct 17, 2017 at 11:06:22AM +0200, Maxime Ripard wrote:
> The TCON supports the LVDS interface to output to a panel or a bridge.
> Let's add support for it.
>
> Signed-off-by: Maxime Ripard
> ---
> drivers/gpu/drm/sun4i/Makefile | 1 +-
> drivers/gpu/drm/sun4i/sun4i_lvds.c | 183 +++
Hi All,
Here is v4 of my series to add a "panel orientation" property to
the drm-connector for the LCD panel to let userspace know about LCD
panels which are not mounted upright, as well as detecting upside-down
panels without needing quirks (like we do for 90 degree rotated screens).
New in v3:
Hi Maxime,
On 17 October 2017 at 20:06, Maxime Ripard
wrote:
> The various outputs the TCON can provide have different constraints on the
> dotclock divider. Let's make them configurable by the various mode_set
> functions.
>
> Signed-off-by: Maxime Ripard
> Reviewed-by: Chen-Yu Tsai
> ---
> d
Hi Hans,
On Mon, Oct 23, 2017 at 09:14:19AM +0200, Hans de Goede wrote:
> On some hardware the LCD panel is not mounted upright in the casing,
> but upside-down or rotated 90 degrees. In this case we want the console
> to automatically be rotated to compensate.
>
> The fbdev-driver may know about
Hello there,
Source code is
if ((temp & PIPEACONF_PIPE_STATE) == 1)
break;
but
$ fgrep PIPEACONF_PIPE_STATE `find drivers/gpu/drm/gma500 -name \*.h -print`
drivers/gpu/drm/gma500/psb_intel_reg.h:#define PIPEACONF_PIPE_STATE
(1 << 30)
$
Suggest new code
On 23/10/17 10:08 AM, David Laight wrote:
It is also worth checking that the hardware actually supports p2p transfers.
Writes are more likely to be supported then reads.
ISTR that some intel cpus support some p2p writes, but there could easily
be errata against them.
Ludwig mentioned a PCIe s
On Fri, Oct 20, 2017 at 01:02:03AM +0200, Noralf Trønnes wrote:
> This driver can use drm_fb_helper_lastclose() as its .lastclose callback.
> It can also use drm_fb_helper_output_poll_changed() as its
> .output_poll_changed callback.
>
> Cc: Russell King
Acked-by: Russell King
Thanks.
> Signe
From: Petrosyan Ludwig
> Sent: 22 October 2017 07:14
> Could be I have done is stupid...
> But at first sight it has to be simple:
> The PCIe Write transactions are address routed, so if in the packet header
> the other endpoint address
> is written the TLP has to be routed (by PCIe Switch to the
This is now all handled in the drivers and communicated through
fb_info.fbcon_rotate_hint.
Signed-off-by: Hans de Goede
---
drivers/video/fbdev/core/Makefile | 3 -
drivers/video/fbdev/core/fbcon.c| 4 +-
drivers/video/fbdev/core/fbcon.h| 6 --
drivers/vid
Ideally we could use the VBT for this, that would be simple, in
intel_dsi_init() check dev_priv->vbt.dsi.config->rotation, set
connector->display_info.panel_orientation accordingly and call
drm_connector_init_panel_orientation_property(), done.
Unfortunately vbt.dsi.config->rotation is always 0 ev
DRM core uses reference/unreference suffixes for refcounting
functions, but kernel uses get/put (e.g. kref_get/put()).
Replace reference/unreference with get/put for consistency
and also it's shorter.
The following cocci script was used to generate the patch:
@@
expression e;
@@
(
-drm_gem_objec
Hello Laurent,
thank you for getting back to me.
>
> Hi Fabrizio,
>
> Thank you for the patch.
>
> On Friday, 13 October 2017 18:22:19 EEST Fabrizio Castro wrote:
> > Add device tree bindings for r8a7743 and r8a7745 DUs.
> > r8a7743 DU is similar to the one from r8a7791, r8a7745 DU is similar
> >
Some x86 clamshell design devices use portrait tablet screens and a display
engine which cannot rotate in hardware, so the firmware just leaves things
as is and we cannot figure out that the display is oriented non upright
from the hardware.
So at least on x86, we need a quirk table for this. This
On some hardware the LCD panel is not mounted upright in the casing,
but upside-down or rotated 90 degrees. In this case we want the console
to automatically be rotated to compensate.
The fbdev-driver may know about the need to rotate. Add a new
fbcon_rotate_hint field to struct fb_info, which get
Hi Xinliang,
On 28 August 2017 at 10:27, Xinliang Liu wrote:
> Hi Daniel,
>
> On 28 August 2017 at 16:51, Daniel Vetter wrote:
> > On Mon, Aug 28, 2017 at 04:44:30PM +0800, Xinliang Liu wrote:
> >> Hi,
> >>
> >> On 15 August 2017 at 22:14, Peter Griffin
> wrote:
> >>
> >> > This patch fixes th
On 24/10/2017 00:46, Rob Herring wrote:
> On Tue, Oct 17, 2017 at 10:07:42AM +0200, Neil Armstrong wrote:
>> On reference boards and derivatives, the HDMI Logic is powered by an external
>> 5V regulator.
>> This regulator was set by the Vendor U-Boot, add optional support for it.
>>
>> Signed-off-b
Hi Linus,
Thank you for the patch.
On Friday, 20 October 2017 15:54:12 EEST Linus Walleij wrote:
> This extends the dumb VGA DAC bridge to handle the THS8134A
> and THS8134B VGA DACs in addition to those already handled.
>
> The THS8134A, THS8134B and as it turns out also THS8135 need to
> have
From: Markus Elfring
Date: Tue, 24 Oct 2017 10:34:56 +0200
Three update suggestions were taken into account
from static source code analysis.
Markus Elfring (3):
Use common error handling code in exynos_dsi_enable()
Use common error handling code in exynos_drm_gem_init()
Fix typos in five
From: Markus Elfring
Date: Tue, 24 Oct 2017 09:32:18 +0200
Add a jump target so that a bit of exception handling can be better reused
at the end of this function.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/exynos/exynos_drm_dsi.
From: Markus Elfring
Date: Tue, 24 Oct 2017 09:50:54 +0200
Adjust jump targets so that a bit of exception handling can be better
reused at the end of this function.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/exynos/exynos_drm_ge
On Tue, Oct 24, 2017 at 10:49 AM, SF Markus Elfring
wrote:
> From: Markus Elfring
> Date: Tue, 24 Oct 2017 09:32:18 +0200
>
> Add a jump target so that a bit of exception handling can be better reused
> at the end of this function.
>
> This issue was detected by using the Coccinelle software.
>
>
From: Markus Elfring
Date: Tue, 24 Oct 2017 10:20:20 +0200
The script "checkpatch.pl" pointed misspelled words out.
Thus fix the affected places.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +-
drivers/gpu/drm/exynos/exynos_drm_fimc.c | 2 +-
drivers/gpu/drm/
On Mon, 23 Oct 2017, David Binderman wrote:
> Hello there,
>
> Source code is
>
> if ((temp & PIPEACONF_PIPE_STATE) == 1)
> break;
>
> but
>
> $ fgrep PIPEACONF_PIPE_STATE `find drivers/gpu/drm/gma500 -name \*.h -print`
> drivers/gpu/drm/gma500/psb_intel_reg.h:#define PIPEACONF
2017-10-23 17:25 GMT+02:00 Ville Syrjala :
> From: Ville Syrjälä
>
> Since commit 632c6e4edef1 ("drm/vblank: Fix flip event vblank count")
> even drivers that don't implement accurate vblank timestamps will end
> up using drm_crtc_accurate_vblank_count(). That leads to a WARN every
> time drm_crtc
>> @@ -1401,11 +1400,14 @@ static void exynos_dsi_enable(struct drm_encoder
>> *encoder)
>> dsi->state &= ~DSIM_STATE_ENABLED;
>> exynos_dsi_set_display_enable(dsi, false);
>> drm_panel_unprepare(dsi->panel);
>> - pm_runtime_put_sync(ds
https://bugs.freedesktop.org/show_bug.cgi?id=103393
Samuel Pitoiset changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
We accidentally inverted an if statement and turned amd_powerplay_reset()
into a no-op.
Fixes: ae97988fc89e ("drm/amd/powerplay: tidy up ret checks in amd_powerplay.c
(v3)")
Signed-off-by: Dan Carpenter
---
IMHO, tidying unreadable double negatives is tricky and error prone but
still worth it in
On Tue, 24 Oct 2017, David Binderman wrote:
> Hello there,
>
>
> Code I looked at is in linux-4.14-rc6, released 20171023, so reasonably
>
> up to date.
>
>
> I did a further check on github.com/torvalds/linux and the code
>
> looks wrong there, too.
>
>
> So I don't see the fix you mentioned in e
On Tue, 24 Oct 2017, Dan Carpenter wrote:
> We accidentally inverted an if statement and turned amd_powerplay_reset()
> into a no-op.
>
> Fixes: ae97988fc89e ("drm/amd/powerplay: tidy up ret checks in
> amd_powerplay.c (v3)")
For some reason I looked up that commit, and it has *two* cases where
On Mon, Oct 23, 2017 at 12:47 PM, Aastha Gupta
wrote:
> For drivers which could have multiple instances, it is necessary to
> differentiate between which is which in the logs. Since
> DRM_INFO/WARN/ERROR don’t do this, drivers used dev_info/warn/err to
> make this differentiation. We now have DRM_
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #9 from Shih-Yuan Lee ---
I have tried acpi_osi="Windows 2009", "Windows 2012", "Windows 2013" and
"Windows 2015" on the latest mainline kernel 4.14rc6, and they all have the
same errors and halt the system.
The BIOS is also up to da
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #10 from Mike Lothian ---
Did this ever work for you?
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.freedesktop.org
h
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #11 from Shih-Yuan Lee ---
(In reply to Mike Lothian from comment #10)
> Did this ever work for you?
What do you mean by this?
--
You are receiving this mail because:
You are the assignee for the bug.__
On Tue, Oct 24, 2017 at 01:11:07PM +0300, Jani Nikula wrote:
> On Tue, 24 Oct 2017, Dan Carpenter wrote:
> > We accidentally inverted an if statement and turned amd_powerplay_reset()
> > into a no-op.
> >
> > Fixes: ae97988fc89e ("drm/amd/powerplay: tidy up ret checks in
> > amd_powerplay.c (v3)"
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #12 from Shih-Yuan Lee ---
BTW, this is a new Dell laptop in the development.
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #13 from Mike Lothian ---
I was meaning, is this a regression, as in it used to work with an older kernel
or mesa. If it's a new system perhaps not.
--
You are receiving this mail because:
You are the assignee for the bug._
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #14 from Shih-Yuan Lee ---
Yup, this is a new system. `DRI_PRIME=1 glxgears` never worked properly before.
--
You are receiving this mail because:
You are the assignee for the bug.___
dri
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #15 from Mike Lothian ---
Are there any changes when you boot the system with radeon.runpm=0, this will
mean the card never powers down
What distro are you running?
You mention trying older kernel version, did you try older mesa ve
https://bugs.freedesktop.org/show_bug.cgi?id=103370
--- Comment #16 from Mike Lothian ---
Do you also see the issue with amdgpu rather than using the radeon kernel
driver?
--
You are receiving this mail because:
You are the assignee for the bug.___
dr
On Tue, Oct 17, 2017 at 06:49:55AM -0400, Rob Clark wrote:
> We already have, as a result of upstreaming the gpu bindings,
> msm_clk_get() which will try to get the clock both without and with a
> "_clk" suffix. Use this in DSI code so we can drop the "_clk" suffix
> in bindings while maintaing ba
On Tue, Oct 17, 2017 at 06:49:56AM -0400, Rob Clark wrote:
> We already have, as a result of upstreaming the gpu bindings,
> msm_clk_get() which will try to get the clock both without and with a
> "_clk" suffix. Use this in eDP code so we can drop the "_clk" suffix
> in bindings while maintaing ba
On Tue, Oct 17, 2017 at 06:49:57AM -0400, Rob Clark wrote:
> We already have, as a result of upstreaming the gpu bindings,
> msm_clk_get() which will try to get the clock both without and with a
> "_clk" suffix. Use this in HDMI code so we can drop the "_clk" suffix
> in bindings while maintaing b
On Mon, Oct 23, 2017 at 11:35:42AM +0100, Daniel Thompson wrote:
> On 20/10/17 18:20, Sean Paul wrote:
> > On Wed, Oct 18, 2017 at 1:11 PM, Meghana Madhyastha
> > wrote:
> > > On Mon, Oct 16, 2017 at 02:26:00PM -0400, Sean Paul wrote:
> > > > On Fri, Oct 13, 2017 at 6:42 PM, Noralf Trønnes
> > >
From: Markus Elfring
Date: Tue, 24 Oct 2017 14:20:06 +0200
Add a jump target so that a call of the function "gvt_vgpu_err" is stored
only once at the end of this function implementation.
Replace two calls by goto statements.
This issue was detected by using the Coccinelle software.
Signed-off-b
On Tue, 24 Oct 2017, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 24 Oct 2017 14:20:06 +0200
>
> Add a jump target so that a call of the function "gvt_vgpu_err" is stored
> only once at the end of this function implementation.
> Replace two calls by goto statements.
>
> This issue
On 24/10/17 13:16, Sean Paul wrote:
On Mon, Oct 23, 2017 at 11:35:42AM +0100, Daniel Thompson wrote:
On 20/10/17 18:20, Sean Paul wrote:
On Wed, Oct 18, 2017 at 1:11 PM, Meghana Madhyastha
wrote:
On Mon, Oct 16, 2017 at 02:26:00PM -0400, Sean Paul wrote:
On Fri, Oct 13, 2017 at 6:42 PM, Nora
>> Add a jump target so that a call of the function "gvt_vgpu_err" is stored
>> only once at the end of this function implementation.
>> Replace two calls by goto statements.
>>
>> This issue was detected by using the Coccinelle software.
>
> I don't think this is an issue or an improvement.
Do y
Prep work for adding a debugfs file that dumps just submits which
trigger hangs/faults. In this case the bo may already be in the
MADV_DONTNEED state, but will be still on the active list (since
the submit hasn't completed yet). So the normal check that the
bo is in the WILLNEED state does not ap
Something that I've been using locally for a while, and found quite
helpful for debugging gallium driver issues:
1) Better messages about what process triggered a crash, which is
in particular particularly useful for piglit
2) Mechanism to dump just submits which triggered GPU hangs.
Eventu
Split into two instances, the existing $debugfs/rd which continues to
dump all submits, and $debugfs/hangrd which will be used to dump just
submits that cause gpu hangs (and eventually faults, but that will
require some iommu framework enhancements).
Signed-off-by: Rob Clark
---
drivers/gpu/drm/
Now that freedreno gallium driver defaults to using submit_queue task
(render reordering), just showing task->comm is not so useful (ie. it is
always "flush_queue:0"), so also dump the cmdline. This should also be
more useful for piglit/shader_runner.
Signed-off-by: Rob Clark
---
drivers/gpu/dr
For faults or hangs, it is nice to be able to include a bit more
information.
Signed-off-by: Rob Clark
---
drivers/gpu/drm/msm/msm_drv.h | 3 ++-
drivers/gpu/drm/msm/msm_gpu.c | 2 +-
drivers/gpu/drm/msm/msm_rd.c | 15 +--
3 files changed, 16 insertions(+), 4 deletions(-)
diff --
We need this if we want to dump the submit after cleanup (ie. from hang
or fault). But in the backoff/unpin case we want to clear them. So add
a flag so we can skip clearing the IOVAs in at cleanup.
Signed-off-by: Rob Clark
---
drivers/gpu/drm/msm/msm_gem_submit.c | 11 ++-
1 file chan
Note we need to move update_fences() to after msm_rd_dump_submit(),
otherwise the bo's referenced by the submit may no longer be valid.
Signed-off-by: Rob Clark
---
drivers/gpu/drm/msm/msm_gpu.c | 52 +--
1 file changed, 30 insertions(+), 22 deletions(-)
Markus,
I normally keep quiet on threads like this. I half agree with you. Yes, perhaps
a reportError function would be a good idea, but it seems that what you are
suggesting is trading an inline subroutine call for a spaghetti-code vondition.
The goto is used only if you do not have any code
From: Christian König
The amdgpu issue to also need signaled fences in the reservation objects should
be fixed by now.
Optimize the handling by replacing a signaled fence when adding a new
shared one.
Signed-off-by: Christian König
---
drivers/dma-buf/reservation.c | 18 +++---
1
From: Christian König
The amdgpu issue to also need signaled fences in the reservation objects
should be fixed by now.
Optimize the list by keeping only the not signaled yet fences around.
Signed-off-by: Christian König
---
drivers/dma-buf/reservation.c | 31 +--
1
From: Markus Elfring
Date: Tue, 24 Oct 2017 15:54:32 +0200
Two update suggestions were taken into account
from static source code analysis.
Markus Elfring (2):
Delete an unnecessary goto statement in intel_dp_sink_crc()
Use common error handling code in intel_dp_sink_crc_stop()
drivers/gpu
From: Markus Elfring
Date: Tue, 24 Oct 2017 15:15:20 +0200
A jump was specified for a location which was directly behind.
Thus remove such an unnecessary goto statement.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/i915/intel_dp.c
From: Markus Elfring
Date: Tue, 24 Oct 2017 15:40:47 +0200
Adjust jump targets so that a specific error code assignment
will be in the implementation only at the end of this function.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/i
The point of unwind code is to undo what was done earlier. If a
function allocates a list of things, using standard unwind style makes
it simpler, safer and more readable.
This isn't the case here. Instead of making the code more readable,
we're making it more convoluted. It's just that two out
https://bugs.freedesktop.org/show_bug.cgi?id=103384
--- Comment #4 from Max ---
Created attachment 135021
--> https://bugs.freedesktop.org/attachment.cgi?id=135021&action=edit
X.Org log file req
--
You are receiving this mail because:
You are the assignee for the bug._
> This isn't the case here.
I find your view interesting for further clarification somehow.
> Instead of making the code more readable, we're making it more convoluted.
Can the shown software refactoring usually help here?
> It's just that two out of three error messages happened to be the sa
On Tue, 2017-10-24 at 17:26 +0300, Dan Carpenter wrote:
> The point of unwind code is to undo what was done earlier. If a
> function allocates a list of things, using standard unwind style makes
> it simpler, safer and more readable.
>
> This isn't the case here. Instead of making the code more
>> … It's just that two out of three error
>> messages happened to be the same and Markus wants to save a bit of
>> memory by using the same string. The memory savings is not so big that
>> it's worth making the code less readable.
>
> I agree with Dan.
>
> It doesn't save any real memory eithe
On Tue, 2017-10-24 at 16:51 +0200, SF Markus Elfring wrote:
> Do you prefer to delegate the proposed software refactoring
> only to a corresponding optimiser?
yes.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/m
>> Do you prefer to delegate the proposed software refactoring
>> only to a corresponding optimiser?
>
> yes.
Will any applications around the semantic patch language
(Coccinelle software) fit also in the preferred tool category?
Regards,
Markus
___
dr
On Tue, Oct 24, 2017 at 01:51:45PM +0100, Daniel Thompson wrote:
> On 24/10/17 13:16, Sean Paul wrote:
> > On Mon, Oct 23, 2017 at 11:35:42AM +0100, Daniel Thompson wrote:
> > > On 20/10/17 18:20, Sean Paul wrote:
> > > > On Wed, Oct 18, 2017 at 1:11 PM, Meghana Madhyastha
> > > > wrote:
> > > > >
On Sat, Oct 21, 2017 at 05:26:33PM +0530, Meghana Madhyastha wrote:
> Move the helper functions enable_backlight and disable_backlight
> from tinydrm-helpers.c to backlight.h as static inline functions so
> that they can be used by other drivers.
>
> Signed-off-by: Meghana Madhyastha
Reviewed-by
On Sat, Oct 21, 2017 at 05:27:33PM +0530, Meghana Madhyastha wrote:
> Rename tinydrm_of_find_backlight to of_find_backlight and move
> it to linux/backlight.c so that it can be used by other drivers.
>
> Signed-off-by: Meghana Madhyastha
> ---
> Changes in v14:
> -s/backlight_get/of_find_backlig
On Sat, Oct 21, 2017 at 05:27:33PM +0530, Meghana Madhyastha wrote:
> Rename tinydrm_of_find_backlight to of_find_backlight and move
> it to linux/backlight.c so that it can be used by other drivers.
>
> Signed-off-by: Meghana Madhyastha
> ---
> Changes in v14:
> -s/backlight_get/of_find_backlig
[ Adding the dri-devel list ]
On 24/10/17 04:58 PM, Andrey Grodzovsky wrote:
> On 10/24/2017 10:36 AM, S, Shirish wrote:
>> On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote:
>>> On 10/24/2017 09:51 AM, S, Shirish wrote:
From: Shirish S
On some systems amdgpu_bo_gpu_offset seems to b
On Tue, Oct 24, 2017 at 10:09:15AM +0200, Neil Armstrong wrote:
> On 24/10/2017 00:46, Rob Herring wrote:
> > On Tue, Oct 17, 2017 at 10:07:42AM +0200, Neil Armstrong wrote:
> >> On reference boards and derivatives, the HDMI Logic is powered by an
> >> external
> >> 5V regulator.
> >> This regulat
On 24/10/17 16:38, Sean Paul wrote:
On Sat, Oct 21, 2017 at 05:27:33PM +0530, Meghana Madhyastha wrote:
Rename tinydrm_of_find_backlight to of_find_backlight and move
it to linux/backlight.c so that it can be used by other drivers.
Signed-off-by: Meghana Madhyastha
---
Changes in v14:
-s/bac
From: Markus Elfring
Date: Tue, 24 Oct 2017 17:55:43 +0200
Two update suggestions were taken into account
from static source code analysis.
Markus Elfring (2):
Use common error handling code in rcar_du_encoders_init()
Adjust 14 checks for null pointers
drivers/gpu/drm/rcar-du/rcar_du_crtc.
On Tue, Oct 17, 2017 at 08:17:59PM +0800, Chen-Yu Tsai wrote:
> From: Jonathan Liu
>
> The A10 has two TCONs that are similar to the ones found on other SoCs.
> Like the A31, TCON0 has a register used to mux the TCON outputs to the
> downstream encoders. The bit fields are slightly different.
>
From: Markus Elfring
Date: Tue, 24 Oct 2017 17:16:09 +0200
Add a jump target so that a bit of exception handling can be better reused
at the end of this function.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/rcar-du/rcar_du_kms.c
From: Markus Elfring
Date: Tue, 24 Oct 2017 17:47:37 +0200
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
The script “checkpatch.pl” pointed information out like the following.
Comparison to NULL could be written !…
Thus fix the affected source code pl
On 24/10/17 06:00 PM, Andrey Grodzovsky wrote:
> On 10/24/2017 11:43 AM, Michel Dänzer wrote:
>> On 24/10/17 04:58 PM, Andrey Grodzovsky wrote:
>>> On 10/24/2017 10:36 AM, S, Shirish wrote:
On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote:
> On 10/24/2017 09:51 AM, S, Shirish wrote:
>> F
On Tue, 24 Oct 2017, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 24 Oct 2017 15:40:47 +0200
>
> Adjust jump targets so that a specific error code assignment
> will be in the implementation only at the end of this function.
This is not an issue that needs to be "fixed".
>
> This
On Mon, Oct 23, 2017 at 05:55:37PM +0200, Benjamin Gaignard wrote:
> Instead a getting only one common device "/dev/ion" for
> all the heaps this patch allow to create one device
> entry ("/dev/ionX") per heap.
> Getting an entry per heap could allow to set security rules
> per heap and global ones
On 10/24/2017 11:43 AM, Michel Dänzer wrote:
[ Adding the dri-devel list ]
On 24/10/17 04:58 PM, Andrey Grodzovsky wrote:
On 10/24/2017 10:36 AM, S, Shirish wrote:
On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote:
On 10/24/2017 09:51 AM, S, Shirish wrote:
From: Shirish S
On some systems am
Quoting Kees Cook (2017-10-24 16:13:44)
> In preparation for unconditionally passing the struct timer_list pointer to
> all timer callbacks, switch to using the new timer_setup() and from_timer()
> to pass the timer pointer explicitly.
>
> Cc: Jani Nikula
> Cc: Joonas Lahtinen
> Cc: Rodrigo Vivi
On Tue, Oct 17, 2017 at 08:18:00PM +0800, Chen-Yu Tsai wrote:
> The HDMI controller in the A10 SoC is the same as the one currently
> supported in the A10s. It has slightly different setup parameters.
> Since these parameters are not thoroughly understood, we add support
> for this variant by copyi
On Tue, Oct 17, 2017 at 08:18:01PM +0800, Chen-Yu Tsai wrote:
> The A10 display pipeline has 2 frontends, 2 backends, and 2 TCONs.
> This patch adds support (or a compatible string in the frontend's
> case) for these components.
>
> The TCONs support directly outputting to CPU/RGB/LVDS LCD panels,
On Tue, Oct 17, 2017 at 08:18:02PM +0800, Chen-Yu Tsai wrote:
> From: Jonathan Liu
>
> The A20 display pipeline has 2 frontends, 2 backends, and 2 TCONs.
> This patch adds support (or a compatible string in the frontend's
> case) for these components.
>
> The TCONs support directly outputting to
On Tue, Oct 17, 2017 at 08:18:02PM +0800, Chen-Yu Tsai wrote:
> From: Jonathan Liu
>
> The A20 display pipeline has 2 frontends, 2 backends, and 2 TCONs.
> This patch adds support (or a compatible string in the frontend's
> case) for these components.
>
> The TCONs support directly outputting to
On Tue, Oct 17, 2017 at 08:18:03PM +0800, Chen-Yu Tsai wrote:
> From: Jonathan Liu
>
> The video PLLs are used directly by the HDMI controller. Export them so
> that we can use them in our DT node.
>
> Signed-off-by: Jonathan Liu
> Signed-off-by: Chen-Yu Tsai
> ---
> drivers/clk/sunxi-ng/ccu-
Den 23.10.2017 23.32, skrev Kristian Høgsberg:
On Mon, Oct 23, 2017 at 9:47 AM, Noralf Trønnes wrote:
Add debugfs file that dumps info about the framebuffers and its planes.
Also dump info about any connected gem object(s).
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_debugfs.c
Den 24.10.2017 17.42, skrev Sean Paul:
On Sat, Oct 21, 2017 at 05:27:33PM +0530, Meghana Madhyastha wrote:
Rename tinydrm_of_find_backlight to of_find_backlight and move
it to linux/backlight.c so that it can be used by other drivers.
Signed-off-by: Meghana Madhyastha
---
Changes in v14:
-s
Warn when refcount > 0 in drm_vblank_cleanup.
Signed-off-by: PrasannaKumar Muralidharan
---
No change in v2.
drivers/gpu/drm/drm_vblank.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c
index 70f2b95..3e61aeb 100644
--- a/drivers
In i915 driver unload drm_vblank_get is added to test whether
drm_vblank_cleanup refcount validation patch is working.
Signed-off-by: PrasannaKumar Muralidharan
---
Changes in v2:
Use drm_crtc_vblank_get instead of _put. In previous patch _put was wrongly
used.
drivers/gpu/drm/i915/i915_drv.c |
https://bugzilla.kernel.org/show_bug.cgi?id=193651
--- Comment #27 from Milo (milo...@gmail.com) ---
i have added it /etc/default/grub as follows:
GRUB_CMDLINE_LINUX_DEFAULT="nointremap quiet amdgpu.audio=0"
and then ran update-grub
however it still takes more than 5 minutes to boot
On Mon, Oct
From: Markus Elfring
Date: Tue, 24 Oct 2017 19:29:39 +0200
Two update suggestions were taken into account
from static source code analysis.
Markus Elfring (2):
Use common error handling code
Use an error code directly
drivers/gpu/drm/tegra/output.c | 13 +++--
1 file changed, 7 ins
From: Markus Elfring
Date: Tue, 24 Oct 2017 19:12:09 +0200
Add a jump target so that a bit of exception handling can be better reused
at the end of this function.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/tegra/output.c | 10 ++
1 - 100 of 126 matches
Mail list logo
