On Thu, 28 Sep 2017, Jani Nikula wrote:
> This lets us configure a space separated list of URLs for each repo in
> drm_tip_repos, with all accepted protocols and sources, and the first
> one found gets picked. This way we don't have to have a complicated set
> of rules for converting between ssh,
Hi, Ulrich:
On Fri, 2017-09-29 at 15:09 +0200, Ulrich Hecht wrote:
> Forward-ported from chromeos-3.18 vendor kernel.
>
> Signed-off-by: Ulrich Hecht
I think this patch originate from [1]. And you should keep the
signed-off-by info so that we could know who contribute to this patch.
[1]
https:
Hi Meghana,
[auto build test WARNING on drm/drm-next]
[also build test WARNING on v4.14-rc3 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm-drm_of_fi
https://bugs.freedesktop.org/show_bug.cgi?id=93199
--- Comment #25 from os...@cetex.se ---
Forgot to mention this is ubuntu 17.04 and i intel-gpu-tools are version 1.17-1
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-
https://bugs.freedesktop.org/show_bug.cgi?id=93199
--- Comment #24 from os...@cetex.se ---
(In reply to Jari Tahvanainen from comment #23)
> Moving this from DRM/Intel to IGT, since it seems to be problem with that.
> Sean, Jonathan, Dimitry and Rami - please check if this problem is still
> valid
https://bugs.freedesktop.org/show_bug.cgi?id=103066
Bug ID: 103066
Summary: Empire boundaries do not show up in Stellaris.
Product: Mesa
Version: unspecified
Hardware: Other
OS: All
Status: NEW
Severity: n
From: Andrey Grodzovsky
Adding initial tests for locks detection when SW
scheduler FIFO is full.
The test works by submitting a batch of identical commands which make the CP
stall waiting for condition to become true. The condition is later satisfied
form a helper thread. Other events that happe
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc3 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm-drm_of_find_b
On Fri, Sep 29, 2017 at 08:22:57AM +, Chen-Yu Tsai wrote:
> The HDMI driver enables the bus and mod clocks in the bind function, but
> does not disable them if it then bails our due to any errors. Neither
> does it disable the clocks in the unbind function.
>
> Fix this by adding a proper erro
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc3 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm-drm_of_find_b
There are instances where the primary plane may have been disabled, look
through disabled planes as well to find primary plane to use for squash.
Signed-off-by: Adrian Salido
---
drmdisplaycompositor.cpp | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/drmdis
On 09/27/2017 06:20 AM, 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 for all heaps.
>
https://bugzilla.kernel.org/show_bug.cgi?id=196777
--- Comment #12 from Justin M. Forbes (jmfor...@linuxtx.org) ---
(In reply to Gerd Hoffmann from comment #11)
> (In reply to Justin M. Forbes from comment #10)
> > After going through these with a number of users:
> >
> > qxl: fix primary surface
https://bugs.freedesktop.org/show_bug.cgi?id=102955
--- Comment #3 from Marek Olšák ---
Workaround:
diff --git a/src/gallium/drivers/radeonsi/si_blit.c
b/src/gallium/drivers/radeonsi/si_blit.c
index 67972a2..dc71742 100644
--- a/src/gallium/drivers/radeonsi/si_blit.c
+++ b/src/gallium/drivers/ra
Fixes intermittent corruption of cmdstream dump.
Signed-off-by: Rob Clark
---
drivers/gpu/drm/msm/msm_rd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/msm/msm_rd.c b/drivers/gpu/drm/msm/msm_rd.c
index 5c087a4228ff..918c65e19964 100644
--- a/drivers/gpu
Hi Marco,
On Mon, 2017-10-02 at 10:45 -0300, Marco Franchi wrote:
> Improve the binding example by removing the '@di0' notation, which
> fixes the following build warning:
>
> Warning (unit_address_vs_reg): Node /display@di0 has a unit name, but
> no reg property
>
> Signed-off-by: Marco Franch
https://bugs.freedesktop.org/show_bug.cgi?id=102646
--- Comment #13 from Harry Wentland ---
I'm curious if you only see this with Stardew Valley? I've seen reports of
people having the same issue with Stardew Valley independent of their graphics
adapter. Seems to be a combination of game and wind
On Fri, Sep 29, 2017 at 6:19 PM, Maxime Ripard
wrote:
> On Fri, Sep 29, 2017 at 08:22:55AM +, Chen-Yu Tsai wrote:
>> static const struct sun4i_tcon_quirks sun5i_a13_quirks = {
>> - .has_unknown_mux = true,
>> - .has_channel_1 = true,
>> + .has_unknown_mux= true,
>> +
Hi Marco,
On Fri, 2017-09-22 at 15:05 -0300, Marco Franchi wrote:
> Improve the binding example by removing the '@di0' notation, which
> fixes the following build warning:
>
> Warning (unit_address_vs_reg): Node /display@di0 has a unit name, but
> no reg property
>
> Signed-off-by: Marco Franch
Hi, Ulrich:
This patch looks old-fashioned. Please follow up with [1].
[1]
http://lists.infradead.org/pipermail/linux-mediatek/2017-January/007654.html
Regards,
CK
On Fri, 2017-09-29 at 15:09 +0200, Ulrich Hecht wrote:
> Ported from chromeos-3.18 kernel.
>
> Signed-off-by: Ulrich Hecht
> ---
Hi Ulrich,
On 29/09/17 14:09, Ulrich Hecht wrote:
> This check breaks DRM initialization on Acer Chromebook R13, paddr is above
> 4GB. Haven't looked into this yet.
I think you mean "DRM initialisation was happening to still work with
silently truncated addresses..." ;)
Anyway, the Mediatek-spec
On Mon, 02 Oct 2017, Daniel Thompson wrote:
> On 02/10/17 09:49, Jani Nikula wrote:
>> On Mon, 02 Oct 2017, Daniel Thompson wrote:
>>> On 01/10/17 18:26, Meghana Madhyastha wrote:
-#ifdef CONFIG_OF
+#if defined CONFIG_OF && IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)
>>>
>>> The above com
On 02/10/17 09:49, Jani Nikula wrote:
On Mon, 02 Oct 2017, Daniel Thompson wrote:
On 01/10/17 18:26, Meghana Madhyastha wrote:
-#ifdef CONFIG_OF
+#if defined CONFIG_OF && IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)
The above comments are more important but why does this mix defined and
IS_ENAB
When using drm_of_panel_bridge_remove() we can simplify the
code and remove is_panel_bridge from dw_mipi_dsi structure.
Signed-off-by: Benjamin Gaignard
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
---
drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 5 +
1 file changed, 1 insertio
This function is the pendant of drm_of_find_panel_or_bridge()
to remove a previously allocated panel_bridge.
Given a specific port and endpoint it remove the panel bridge.
Since drm_panel_bridge_remove() will check that bridge parameter
is not NULL and is a real drm_panel_bridge and no a simple bri
With a call to drm_of_panel_bridge_remove() we could remove the bridge
without store it in ldtc internal driver structure.
Signed-off-by: Benjamin Gaignard
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
---
drivers/gpu/drm/stm/ltdc.c | 16 +---
drivers/gpu/drm/stm/ltdc.h |
With a call to drm_of_panel_bridge_remove() we could remove
the bridge without store it in vc4_dpi internal driver structure.
Signed-off-by: Benjamin Gaignard
---
drivers/gpu/drm/vc4/vc4_dpi.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/
The goal of this series is to simplify driver code when they need to clean up
a previously allocated panel bridge.
Few drivers have "is_panel_bridge" flag to be able to distinguish a
drm_panel_bridge from "simple" drm_bridge.
To remove this flag I propose to
- let drm_panel_bridge_remove() check if
Make sure that bridge parameter is not NULL and can be safely
cast into a panel_bridge structure.
Signed-off-by: Benjamin Gaignard
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
---
drivers/gpu/drm/bridge/panel.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --gi
On Mon, 02 Oct 2017, Daniel Thompson wrote:
> So I'm coming to this patchset cold but can you explain *why* something
> wants to call of_find_backlight_by_node() when there is no backlight
> support enabled. Why isn't the code that called is conditional on
> BACKLIGHT_CLASS_DEVICE?
>
> The unde
On Mon, 02 Oct 2017, Daniel Thompson wrote:
> On 01/10/17 18:26, Meghana Madhyastha wrote:
>> -#ifdef CONFIG_OF
>> +#if defined CONFIG_OF && IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)
>
> The above comments are more important but why does this mix defined and
> IS_ENABLED? Couldn't they both use d
Series pushed.
On Thu, 2017-09-28 at 18:40 +0200, Robert Foss wrote:
> This series removes the old thread-based synchronization utilities
> and
> replaces them with fences.
>
> It has been tested on various platforms, including
> etnaviv/freedreno/virgl.
>
> Robert Foss (5):
> drm_hwcomposer:
On Fri, Sep 08, 2017 at 11:39:44AM -0400, Sean Paul wrote:
> On Fri, Sep 8, 2017 at 11:35 AM, Daniel Vetter wrote:
> > fbdev is in maintenance only, except that it's still used by drm
> > through the drm fbdev emulation, to be able to use fbcon. And people
> > might want to sometimes extend fbcon
On Sun, Oct 01, 2017 at 03:30:44PM -0400, Jérémy Lefaure wrote:
> Using the ARRAY_SIZE macro improves the readability of the code. Also,
> it is not always useful to use a variable to store this constant
> calculated at compile time nor to re-invent the ARRAY_SIZE macro.
>
> Found with Coccinelle
On Sun, Oct 01, 2017 at 05:33:17PM +0200, Hans de Goede wrote:
> 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
On Mon, 02 Oct 2017, Daniel Vetter wrote:
> Also adding Jani, who looked at the backlight Kconfig mess in the past.
I've even sent patches to fix some of the dependency mess, but the
problem is social not technical. The problem is that people think
"select" is more convenient than "depends" becau
On Sun, Oct 01, 2017 at 05:33:16PM +0200, Hans de Goede wrote:
> 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_p
On Sun, Oct 01, 2017 at 05:33:14PM +0200, Hans de Goede wrote:
> Some x86 clamshell design devices use portrait tablet LCD panels and a
> display engine which cannot (transparently) rotate in hardware, so we
> need to rotate things in software / let user space deal with this.
>
> The fbcon code ha
On Sat, Sep 30, 2017 at 03:49:43PM +0200, Noralf Trønnes wrote:
>
> Den 30.09.2017 11.10, skrev Meghana Madhyastha:
> > Add select BACKLIGHT_LCD_SUPPORT, BACKLIGHT_CLASS_DEVICE to
> > the Kconfig of drm. This is required for the successful build of
> > drm_of_find_backlight helpers.
> >
> > Signe
On Sat, Sep 30, 2017 at 01:18:53PM +0800, kbuild test robot wrote:
> Hi Meghana,
>
> [auto build test WARNING on drm/drm-next]
> [also build test WARNING on v4.14-rc2 next-20170929]
> [if your patch is applied to the wrong git tree, please drop us a note to
> help improve the system]
>
> url:
On Fri, Sep 29, 2017 at 03:03:54PM +, Philippe CORNU wrote:
> Hi Benjamin,
> and many thanks for this cleanup patchset.
>
> Reviewed-by: Philippe Cornu
> Tested-by: Philippe Cornu
Can you pls either convert vc4 and dw-mipi too, or add an entry to our
todo list in Documentation/gpu/todo.rst?
On Fri, Sep 29, 2017 at 03:33:39PM +0530, Srishti Sharma wrote:
> Replace reference/unreference with get/put as it is consistent
> with the kernel coding style. Done using the following semantic
> patch by coccinelle.
>
> @r@
> expression e;
> @@
>
> -drm_gem_object_unreference_unlocked(e);
> +dr
On Fri, Sep 29, 2017 at 05:41:04PM +0200, Noralf Trønnes wrote:
>
> Den 29.09.2017 16.13, skrev Meghana Madhyastha:
> > On Fri, Sep 29, 2017 at 02:33:12PM +0200, Noralf Trønnes wrote:
> > > Den 29.09.2017 14.20, skrev Meghana Madhyastha:
> > > > On Fri, Sep 29, 2017 at 02:10:31PM +0200, Noralf Trø
On Sun, 01 Oct 2017, Jérémy Lefaure wrote:
> Using the ARRAY_SIZE macro improves the readability of the code. Also,
> it is not always useful to use a variable to store this constant
> calculated at compile time nor to re-invent the ARRAY_SIZE macro.
>
> Found with Coccinelle with the following se
On Sat, Sep 30, 2017 at 12:34:40PM -0600, Haneen Mohammed wrote:
> On Fri, Sep 29, 2017 at 02:29:31PM +0200, Noralf Trønnes wrote:
> >
> > Den 29.09.2017 09.25, skrev Daniel Vetter:
> > >On Wed, Sep 27, 2017 at 01:38:46AM -0600, Haneen Mohammed wrote:
> > >>Remove drmP.h as it is not needed anymor
https://bugzilla.kernel.org/show_bug.cgi?id=197103
Igor Gnatenko (i.gnatenko.br...@gmail.com) changed:
What|Removed |Added
See Also||https://bugs.
https://bugzilla.kernel.org/show_bug.cgi?id=197103
Bug ID: 197103
Summary: BUG: unable to handle kernel NULL pointer dereference
at 0008 / IP:
nouveau_fbcon_set_suspend_work+0x45/0xf0 [nouveau]
Product: Drivers
On Mon, 2 Oct 2017 09:01:31 +1100
"Tobin C. Harding" wrote:
> > In order to reduce the size of the To: and Cc: lines, each patch of the
> > series is sent only to the maintainers and lists concerned by the patch.
> > This cover letter is sent to every list concerned by this series.
>
> Why don
Hi everyone,
Using ARRAY_SIZE improves the code readability. I used coccinelle (I
made a change to the array_size.cocci file [1]) to find several places
where ARRAY_SIZE could be used instead of other macros or sizeof
division.
I tried to divide the changes into a patch per subsystem (excepted for
On Sun, Oct 01, 2017 at 03:30:38PM -0400, Jérémy Lefaure wrote:
> Hi everyone,
> Using ARRAY_SIZE improves the code readability. I used coccinelle (I
> made a change to the array_size.cocci file [1]) to find several places
> where ARRAY_SIZE could be used instead of other macros or sizeof
> divisio
Using the ARRAY_SIZE macro improves the readability of the code. Also,
it is not always useful to use a variable to store this constant
calculated at compile time nor to re-invent the ARRAY_SIZE macro.
Found with Coccinelle with the following semantic patch:
@r depends on (org || report)@
type T;
On Sat, Sep 30, 2017 at 2:26 PM, Julian Calaby wrote:
> Hi Chen-Yu,
>
> On Sat, Sep 30, 2017 at 3:58 PM, Chen-Yu Tsai wrote:
>> On Sat, Sep 30, 2017 at 1:35 PM, Julian Calaby
>> wrote:
>>> Hi Chen-Yu,
>>>
>>> On Fri, Sep 29, 2017 at 8:22 PM, Chen-Yu Tsai wrote:
On Fri, Sep 29, 2017 at 6:2
https://bugzilla.kernel.org/show_bug.cgi?id=196615
--- Comment #18 from klavkala...@gmail.com ---
Looks like it when browsing the source. It's in both 4.13 and 4.14-rc3. I hope
it can be removed again in time for the LTS release. For now I'm holding off
the upgrade to 4.13. I don't know if I'm get
53 matches
Mail list logo
