Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On 13/10/17 18:09, Tony Lindgren wrote:
>> Looks like today's next build fails if omapdrm is enabled as modules
>> with:
>>
>> depmod: ERROR: Cycle detected: drm_kms_h
On 16 October 2017 at 16:41, Thierry Reding wrote:
> On Mon, Oct 16, 2017 at 02:29:07PM +1000, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> This adds the infrastructure needed to quirk displays
>> using edid and to mark them a non-standard.
>>
>> A non-standard display is one which doesn't work
On Mon, Oct 16, 2017 at 02:29:09PM +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> This uses the EDID info from my HTC Vive to mark it as
> non-standard.
>
> Signed-off-by: Dave Airlie
> ---
> drivers/gpu/drm/drm_edid.c | 3 +++
> 1 file changed, 3 insertions(+)
I guess I was being naive fo
On Mon, Oct 16, 2017 at 02:29:07PM +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> This adds the infrastructure needed to quirk displays
> using edid and to mark them a non-standard.
>
> A non-standard display is one which doesn't work like
> a normal rectangular monitor or requires some trans
Hi：
Thanks for the patch. This problem has been fixed on staging branch with:
a8e18bfc854141621a29600a6c13e5f40c6e9f19
drm/i915/gvt: Refine shadow batch buffer
Thanks,
Zhi.
-Original Message-
From: Christos Gkekas [mailto:chris.ge...@gmail.com]
Sent: Monday, October 16, 2017 1:18 A
From: Dave Airlie
We don't want fbcon to get used on non-standard dislays,
don't pass them as enabled connectors to the fb helper setup.
This prevents my HMD from getting disorted fbcon, and from
affecting other displays console.
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/drm_fb_helper.c
From: Dave Airlie
This adds the infrastructure needed to quirk displays
using edid and to mark them a non-standard.
A non-standard display is one which doesn't work like
a normal rectangular monitor or requires some transformation
of the output by the rendering process to make sense.
This is me
From: Dave Airlie
This uses the EDID info from my HTC Vive to mark it as
non-standard.
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/drm_edid.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 4225052def37..b491b3c3cc1f 10064
Sean Paul writes:
> Sphinx won't pick these up, and will issue warnings. Please update to @
> instead of \param
>
> If you want to try it out:
> make htmldocs
Yeah, I was attempting to emulate the existing style. I suggest that a
general cleanup to fix the docstrings should be in a separate patc
https://bugs.freedesktop.org/show_bug.cgi?id=99801
--- Comment #15 from Matthew Treinish ---
I tested the most recent kernel from the drm-next-4.15-dc branch found at:
https://cgit.freedesktop.org/~agd5f/linux/log/?h=drm-next-4.15-dc (commit
b175d392cfb28a9d260904bbb330917efe039331 ) The probl
Hi Dave,
Just two regression fixups to potential use-after-free and
NULL pointer dereference issues in suspend/resume.
Please kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit a480f30846d19b50106b3243d9d48683d2966249:
Merge tag 'drm-in
2017년 10월 12일 18:51에 Sylwester Nawrocki 이(가) 쓴 글:
> On 09/26/2017 04:17 PM, Sylwester Nawrocki wrote:
>> The hdmi-codec interface added in this patch is required to properly
>> support HDMI audio. Currently the audio part of the SoC internal
>> HDMI transmitter is configured with fixed values, wh
Hi Laurent,
On Fri, Oct 13, 2017 at 05:59:17PM +0300, Laurent Pinchart wrote:
> To simplify implementation of debugfs seq_file show handlers, the driver
> passes the pointer to the show function through the debugfs_create_file
> data pointer. This prevents using the pointer to pass driver private
https://bugs.freedesktop.org/show_bug.cgi?id=103100
--- Comment #2 from Gregor Münch ---
I tested again with yesterdays git and newer Kernel:
OpenGL renderer string: AMD Radeon HD 7900 Series (TAHITI / DRM 3.21.0 /
4.14.0-2-drm-next-dc-git, LLVM 6.0.0)
OpenGL core profile version string: 4.5 (Cor
https://bugs.freedesktop.org/show_bug.cgi?id=92827
--- Comment #4 from dwagner ---
JFYI: HDMI audio output works fine with amdgpu as present in current
https://cgit.freedesktop.org/~agd5f/linux/log/?h=amd-staging-drm-next
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=103277
--- Comment #2 from Jordan L ---
(In reply to dwagner from comment #0)
> After I updated my kernel to
> https://cgit.freedesktop.org/~agd5f/linux/log/?h=amd-staging-drm-next as of
> today (latest commit at this time:
> 1c630e83443a0f271c192ecfa
Add vmalloc buffer object helper that can be useful for modesetting
drivers, particularly the framebuffer flushing kind.
Signed-off-by: Noralf Trønnes
---
Documentation/gpu/drm-kms-helpers.rst | 12 ++
drivers/gpu/drm/Kconfig | 7 +
drivers/gpu/drm/Makefile|
Add drm_gem_fb_debugfs_show() function to provide a debugfs
representation of the framebuffer and GEM object(s).
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_gem_framebuffer_helper.c | 45
include/drm/drm_gem_framebuffer_helper.h | 6
2 files chang
Make functions tolerate that the drm_fb_helper argument is NULL.
This is useful for drivers that continue probing when fbdev emulation
fails and not having to do this check themselves.
Update docs for functions that already handles this.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_h
vmalloc BO's gives us cached reads, so no need to prefetch in that case.
Prefetching gives a ~20% speedup on a cma buffer using the mi0283qt
driver on a Raspberry Pi 1.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c | 54 ++
1 file change
This adds some simple init/fini helpers for drivers that don't need
anything special in their fbdev emulation setup.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_helper.c | 163
include/drm/drm_fb_helper.h | 22 ++
2 files changed, 18
This adds helpers for the drm_driver->last_close and the
drm_mode_config_funcs->output_poll_changed callbacks.
Signed-off-by: Noralf Trønnes
---
drivers/gpu/drm/drm_fb_helper.c | 32
include/drm/drm_fb_helper.h | 10 ++
2 files changed, 42 insertions(
This patchset adds a library for vmalloc buffer objects and makes use of
it in tinydrm.
The reason I want to move away from the cma helper, is that it restricts
which drivers to PRIME import from, since cma requires the buffer to be
physically continuous. Initially I looked at udl and decided to u
drm_fb_helper is *the* way of doing fbdev emulation so add a pointer to
struct drm_device. This makes it possible to add callback helpers for
.last_close and .output_poll_changed further reducing fbdev emulation
footprint in drivers.
Signed-off-by: Noralf Trønnes
---
include/drm/drm_device.h | 8
Use the vmalloc BO helper instead of the cma helper to be able to PRIME
import from more drivers. The cma helper can only import physically
continuous buffers, but tinydrm only requires the buffer to be virtually
continuous.
This switch also makes it possible to use the drm_fb_helper_lastclose()
h
* Tony Lindgren [171013 07:59]:
> Hi Tomi,
>
> Looks like today's next build fails if omapdrm is enabled as modules
> with:
>
> depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
> depmod: ERROR: Found 2 modules in dependency cycles!
> make: *** [Makefile:1261: _modinst_post]
Le 12/10/2017 à 18:25, Bartlomiej Zolnierkiewicz a écrit :
[ added dri-devel ML to cc: ]
On Tuesday, September 12, 2017 07:39:30 AM Christophe JAILLET wrote:
If 'dmam_alloc_attrs()' fails, we must go through the error handling code,
as done elsewhere in this function. Otherwise, there is a reso
Add device tree bindings for r8a7743 and r8a7745 DUs.
r8a7743 DU is similar to the one from r8a7791, r8a7745 DU is similar
to the one from r8a7794.
Signed-off-by: Fabrizio Castro
Reviewed-by: Biju Das
---
.../devicetree/bindings/display/renesas,du.txt | 30 --
1 file cha
Done with following coccinelle patch
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T*)x)->f
|
- (T*)
e
)
Signed-off-by: Harsha Sharma
---
Changes in v4:
-Removed git diff warning "No newline at end of file"
Changes in v3:
-Removed unnecessary line
Hi,
On Fri, Oct 13, 2017 at 05:59:14PM +0300, Laurent Pinchart wrote:
> The connector drivers need a handle to the source they are connected to
> in order to control the source.
>
> All drivers get that handle at probe time, resulting in probe deferral
> when the source hasn't been probed yet. Ho
Hi,
On Fri, Oct 13, 2017 at 05:59:13PM +0300, Laurent Pinchart wrote:
> The connector drivers need a handle to the source they are connected to
> in order to control the source.
>
> All drivers get that handle at probe time, resulting in probe deferral
> when the source hasn't been probed yet. Ho
Hi,
On Fri, Oct 13, 2017 at 05:58:58PM +0300, Laurent Pinchart wrote:
> The omap_gem_resume() function is internal to the driver. Pass it a
> drm_device pointer that the caller already has instead of looking it up
> from device data.
>
> Signed-off-by: Laurent Pinchart
Reviewed-by: Sebastian Re
Add support for the R8A7743 DU (which is very similar to the R8A7791 DU);
it has 1 DPAD (RGB) output and 1 LVDS output.
Signed-off-by: Fabrizio Castro
Reviewed-by: Biju Das
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/driv
Hi,
On Fri, Oct 13, 2017 at 05:59:09PM +0300, Laurent Pinchart wrote:
> The tvc_of_match variable is never referenced before its definition.
> Remove the forward declaration.
>
> Signed-off-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/gpu/drm/omapdrm/disp
Hi,
On Fri, Oct 13, 2017 at 05:59:00PM +0300, Laurent Pinchart wrote:
> There's no need for the omapdss-base code to be part of a separate
> module. Merge it with the omapdss module. This allows removing the
> exports for internal symbols.
>
> Signed-off-by: Laurent Pinchart
> ---
> drivers/gpu
Hi,
On Fri, Oct 13, 2017 at 05:59:12PM +0300, Laurent Pinchart wrote:
> No panel is instantiated through platform data anymore, there is no
> need to check for OF node presence.
>
> Signed-off-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/gpu/drm/omapdrm/d
Hi,
On Fri, Oct 13, 2017 at 05:59:02PM +0300, Laurent Pinchart wrote:
> The function isn't used outside of its compilation unit, make it static.
>
> Signed-off-by: Laurent Pinchart
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/gpu/drm/omapdrm/dss/dss.c | 4 +++-
> drivers/gpu/
of_fdt_unflatten_tree() already sets the flag on this node to
OF_DETACHED, because of_fdt_unflatten_tree() calls
__unflatten_device_tree() with the detached bool set to true.
Cc: Rob Herring
Cc: Frank Rowand
Signed-off-by: Stephen Boyd
---
drivers/gpu/drm/tilcdc/tilcdc_slave_compat.c | 1 -
1
Replace use of list_for_each with list_for_each_entry to simplify the
code and remove variables that are used only in list_for_each.
Done with following coccinelle patch:
@r@
identifier fn,i,f,p;
expression e;
iterator name list_for_each, list_for_each_entry;
type T;
@@
fn(...) {
++ T *i;
<+...
On Thu, Oct 12, 2017 at 11:35:12AM +0100, Chris Wilson wrote:
> Quoting Haneen Mohammed (2017-10-12 03:32:53)
> > diff --git a/drivers/gpu/drm/drm_debug.c b/drivers/gpu/drm/drm_debug.c
> > new file mode 100644
> > index 000..a79593f
> > --- /dev/null
> > +++ b/drivers/gpu/drm/drm_debug.c
> > @@
Replace use of list_for_each with list_for_each_entry to simplify the
code and remove variables that are used only in list_for_each.
Done with following coccinelle patch:
@r@
identifier fn,i,f,p;
expression e;
iterator name list_for_each, list_for_each_entry;
type T;
@@
fn(...) {
++ T *i;
<+...
On Tue, 2017-09-12 at 09:56 -0700, Greg Kroah-Hartman wrote:
> 4.4-stable review patch. If anyone has any objections, please let me
> know.
>
> --
>
> From: John Stultz
>
> commit 518cb7057a59b9441336d2e88a396d52b6ab0cce upstream.
>
> I was recently seeing issues with EDID pro
Replace instances of drm_framebuffer_reference/unreference() with
*_get/put() suffixes and drm_dev_unref with *_put() suffix
because get/put is shorter and consistent with the
kernel use of *_get/put suffixes.
Done with following coccinelle semantic patch
@@
expression ex;
@@
(
-drm_framebuffer_u
Hi,
On Fri, Oct 13, 2017 at 05:59:10PM +0300, Laurent Pinchart wrote:
> No connector is instantiated through platform data anymore, there is no
> need to check for OF node presence.
>
> Signed-off-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/gpu/drm/omapd
Signed-off-by: Fabrizio Castro
Reviewed-by: Biju Das
---
drivers/clk/renesas/r8a7745-cpg-mssr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/renesas/r8a7745-cpg-mssr.c
b/drivers/clk/renesas/r8a7745-cpg-mssr.c
index 9e2360a..2859504 100644
--- a/drivers/clk/renesas/r8a7745-cpg
Add support for the R8A7745 DU (which is very similar to the R8A7794 DU);
it has 2 RGB outputs.
Signed-off-by: Fabrizio Castro
Reviewed-by: Biju Das
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/drivers/gpu/drm/rcar-du/rcar
Done with following coccinelle patch
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T*)x)->f
|
- (T*)
e
)
Signed-off-by: Harsha Sharma
---
Changes in v2:
-Remove unnecessary parentheses
-Remove one more useless cast
drivers/gpu/drm/amd/powerplay
Hi,
On Fri, Oct 13, 2017 at 05:59:01PM +0300, Laurent Pinchart wrote:
> When merging the omapdrm and omapdss drivers the omapdrm virtual
> platform device will disappear, and the omapdss platform device will be
> used for DMA memory allocation. To prepare for that, set the DMA
> coherent mask for
Prefer kcalloc over kzalloc to allocate an array.
This patch fixes checkcpatch issue.
Signed-off-by: Harsha Sharma
---
Changes in v2:
-kcalloc will take 3 arguments
drivers/gpu/drm/drm_crtc_helper.c | 4 ++--
drivers/gpu/drm/drm_fb_helper.c| 2 +-
drivers/gpu/drm/drm_plane_helper.c | 2 +-
Hi,
On Fri, Oct 13, 2017 at 05:59:08PM +0300, Laurent Pinchart wrote:
> When merging the omapdrm and omapdss drivers there will be not omapdrm
> platform device anymore, and thus no associated probe and remove
> functions. To prepare for that, split all the initialization code from
> the probe fun
Hi,
On Fri, Oct 13, 2017 at 05:59:15PM +0300, Laurent Pinchart wrote:
> The encoder drivers need a handle to the source they are connected to in
> order to control the source.
>
> All drivers get that handle at probe time, resulting in probe deferral
> when the source hasn't been probed yet. Howe
* 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 with the series.
>
> We have one work item that more or less overrides everything but
In order to be able to define du nodes in r8a7743 and r8a7745 device
trees, we need to define data structures, compatible strings, missing
clock, and update the dt-bindings.
I'll send out the patches to add the du nodes in r8a774[35] device trees
once the patches in this series get accepted.
Best
Hi Rob,
On 10/02/17 20:53, frowand.l...@gmail.com wrote:
> From: Frank Rowand
>
> I have found the device tree overlay code to be difficult to read and
> maintain. This patch series attempts to improve that situation.
>
> The cleanup includes some changes visible to users of overlays. The
> o
Done with following coccinelle patch
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T*)x)->f
|
- (T*)
e
)
Signed-off-by: Harsha Sharma
---
Changes in v3:
-Removed unnecessary lines
-Remove more useless casts
Changes in v2:
-Remove unnecessary pare
Hi,
On Fri, Oct 13, 2017 at 05:59:07PM +0300, Laurent Pinchart wrote:
> The kernel favours 'unsigned int' over plain 'unsigned'. Replace all
> occurences of the latter by the former. This avoid lots of checkpatch
> complaints in patches that touch lines where a plain 'unsigned' is used.
>
> Signe
Hi,
On Fri, Oct 13, 2017 at 05:59:04PM +0300, Laurent Pinchart wrote:
> The venc_probe_of() function has an error cleanup path that returns
> success instead of an error code. Fix it.
>
> Signed-off-by: Laurent Pinchart
> ---
Maybe this should be tagged for stable backporting?
Reviewed-by: Seb
Hi,
On Fri, Oct 13, 2017 at 05:59:03PM +0300, Laurent Pinchart wrote:
> The dpi_data structure port_initialized field is used to check in the
> cleanup path whether the DPI has been initialized. This can be performed
> through the associated device_node data field instead. Remove the
> port_initia
Hi,
On Fri, Oct 13, 2017 at 05:59:11PM +0300, Laurent Pinchart wrote:
> No encoder is instantiated through platform data anymore, there is no
> need to check for OF node presence.
>
> Signed-off-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/gpu/drm/omapdrm
On Mon, Oct 9, 2017 at 5:36 PM, Harsha Sharma
wrote:
> Replace instances of drm_framebuffer_reference/unreference() with
> *_get/put() suffixes and drm_dev_unref with *_put() suffix
> because get/put is shorter and consistent with the
> kernel use of *_get/put suffixes.
> Done with following cocci
Hi,
On Fri, Oct 13, 2017 at 05:58:59PM +0300, Laurent Pinchart wrote:
> The omap_dss_find_device() function is unused. Remove it.
>
> Signed-off-by: Laurent Pinchart
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/gpu/drm/omapdrm/dss/display.c | 14 --
> drivers/gpu/
Hi,
On Fri, Oct 13, 2017 at 05:58:57PM +0300, Laurent Pinchart wrote:
> There's no need to print an error message on probe deferral, that's a
> normal situation. Probe deferral debugging can be performed by enabling
> the related debug messages in the drivers core.
>
> Signed-off-by: Laurent Pinc
Done with following coccinelle patch
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T*)x)->f
|
- (T*)
e
)
Signed-off-by: Harsha Sharma
---
drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c| 6 +++---
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c
Hi,
On Fri, Oct 13, 2017 at 05:59:06PM +0300, Laurent Pinchart wrote:
> The standard kernel integer types are [us]{8,16,32}. Use them instead of
> the u?int{8,16,32}_t types.
>
> Signed-off-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/gpu/drm/omapdrm/omap
64 matches
Mail list logo
