https://bugzilla.kernel.org/show_bug.cgi?id=196379
--- Comment #2 from Jani Nikula (jani.nik...@intel.com) ---
If you think this is a graphics bug, please file it at
https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
Otherwise please assign it to some other component than
https://bugs.freedesktop.org/show_bug.cgi?id=101837
--- Comment #4 from Lauri Kasanen ---
The DRM node must be opened (and kept open for the lifetime of the app) to make
the VRAM queries, or to make the register queries on kernels that prohibit
direct access (e.g. Ubuntu).
--
You are receiving
> -Original Message-
> From: Chris Wilson [mailto:ch...@chris-wilson.co.uk]
> Sent: Wednesday, July 19, 2017 7:20 PM
> To: Zhang, Tina ; intel-...@lists.freedesktop.org;
> intel-
> gvt-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org;
> ville.syrj...@linux.intel.com; zhen...@li
Hi Linus,
A bunch of fixes for rc2, two imx regressions, vc4 fix, dma-buf fix,
some displayport mst fixes, and an amdkfd fix.
Nothing too crazy, I assume we just haven't see much rc1 testing yet.
Dave.
The following changes since commit 5771a8c08880cdca3bfb4a3fc6d309d6bba20877:
Linux v4.13-r
On 17-07-14 22:10:15, Ville Syrjälä wrote:
On Fri, Jul 14, 2017 at 11:41:49AM -0700, Ben Widawsky wrote:
On 17-06-29 22:49:44, Ville Syrjälä wrote:
[snip]
>
>... but here it's ALIGN(formats_offset+formats_size). I think we should
>be aligning the same thing in both cases, or we add a BUILD_BUG
https://bugs.freedesktop.org/show_bug.cgi?id=101732
--- Comment #1 from Fede ---
More Details which might be relevant and might have missed the first time
around:
Other packages:
linux-firmware 20170422.ade8332-1
kernel 4.11.9-1
Xorg driver: xf86-video-ati 1:7.9.0-1
Have also seen the following
https://bugs.freedesktop.org/show_bug.cgi?id=101731
--- Comment #18 from Shmerl ---
Actually, I just experienced the freeze bug again. I guess it's somehow random,
and it's not truly gone :(
--
You are receiving this mail because:
You are the assignee for the bug.___
Hi all,
Today's linux-next merge of the drm tree got a conflict in:
drivers/gpu/drm/i915/intel_pm.c
between commit:
9cc5bb18bd0a ("drm/i915: Fix bad comparison in skl_compute_plane_wm")
from the drm-intel-fixes tree and commit:
eac2cb81fb87 ("drm/i915: cleanup fixed-point wrappers namin
Hi all,
The following conflict now exists between the drm and drm-intel trees.
On Thu, 20 Jul 2017 11:23:33 +1000 Stephen Rothwell
wrote:
>
> Today's linux-next merge of the drm-misc tree got a conflict in:
>
> drivers/gpu/drm/i915/i915_drv.c
>
> between commit:
>
> 99c539bef538 ("drm/i9
Hi Dave,
The following is now applicable to the drm and staging.current trees ...
On Wed, 19 Jul 2017 11:46:57 +1000 Stephen Rothwell
wrote:
>
> After merging the drm-misc tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/staging/vboxvideo/vbox_drv.c:235:2: e
Hi Dave,
The conflict below now exists between the drm-misc-fixes tree and the
drm tree.
On Tue, 18 Jul 2017 11:39:46 +1000 Stephen Rothwell
wrote:
>
> Today's linux-next merge of the drm-misc tree got a conflict in:
>
> drivers/gpu/drm/vc4/vc4_crtc.c
>
> between commit:
>
> 1ed134e6526b
On Thu, Jul 20, 2017 at 11:58 PM, Tobias Klausmann
wrote:
> Mh ok,
>
> paper over in nouveau_display_fini until Ben comes up with a better idea
> then?!
No paper needed, just don't call drm_vblank_off for the atomic case.
Not sure why that patch isn't landed yet, it should be simple.
-Daniel
>
>
Hi Dave,
Here's the first -misc-fixes pull for the current cycle. Note that I've based it
off Linus' tree, so you might want to ff drm-fixes before applying.
Pretty self-explanatory set of patches, I'm sure you have a good handle on the
mst group. The vc4 fix comes from drm-misc-next-fixes after t
Fixes:
../drivers/gpu/drm/drm_scdc_helper.c:203: ERROR: Unexpected indentation.
../drivers/gpu/drm/drm_scdc_helper.c:204: WARNING: Block quote ends without a
blank line; unexpected unindent.
Changes in v2:
- Property blockquote TMDS calculations so they look pretty (Daniel)
- Remove duplicate d
On Thu, Jul 20, 2017 at 03:01:16PM +0200, Maxime Ripard wrote:
> The current drm_atomic_helper_commit_tail helper works only if the CRTC is
> accessible, and documents an alternative implementation that is supposed to
> be used if that happens.
>
> That implementation is then duplicated by some dr
On Thu, Jul 20, 2017 at 01:47:43PM -0400, Sean Paul wrote:
> Fixes the following warnings when building docs:
> ../include/drm/drm_drv.h:553: warning: No description found for parameter
> 'debugfs_init'
> ../include/drm/drm_drv.h:553: warning: No description found for parameter
> 'gem_open_object
https://bugs.freedesktop.org/show_bug.cgi?id=97942
Ricardo Madrigal changed:
What|Removed |Added
Summary|[IGT] [BYT] [BXT] |[IGT] [BYT] [BXT] [BYT]
On Thu, Jul 20, 2017 at 01:47:41PM -0400, Sean Paul wrote:
> Fixes:
> ../drivers/gpu/drm/drm_scdc_helper.c:203: ERROR: Unexpected indentation.
> ../drivers/gpu/drm/drm_scdc_helper.c:204: WARNING: Block quote ends without a
> blank line; unexpected unindent.
>
> Signed-off-by: Sean Paul
> ---
>
https://bugs.freedesktop.org/show_bug.cgi?id=97942
--- Comment #7 from Ricardo Madrigal ---
The following tests fail on BYT with latest configuration
Test list
igt@gem_mmap_gtt@coherency
==
On Thu, Jul 20, 2017 at 01:47:40PM -0400, Sean Paul wrote:
> Fixes the following warnings when building docs:
> ../drivers/gpu/drm/drm_modes.c:1623: warning: No description found for
> parameter 'display'
> ../drivers/gpu/drm/drm_modes.c:1623: warning: Excess function parameter
> 'connector' desc
On Thu, Jul 20, 2017 at 05:07:48PM +0100, Liviu Dudau wrote:
> Explain better when the drm_modeset_acquire_ctx parameter can
> be skipped for drm_modeset_lock() call.
>
> Signed-off-by: Liviu Dudau
> Cc: Daniel Vetter
Applied, thanks.
-Daniel
> ---
> drivers/gpu/drm/drm_modeset_lock.c | 12 ++
On Fri, Jul 07, 2017 at 09:28:11AM -0400, Sean Paul wrote:
> On Thu, Jul 6, 2017 at 9:00 AM, Daniel Vetter wrote:
> > From: Thierry Reding
> >
> > The FB helper core now supports deferred setup, so the driver's custom
> > implementation can be removed.
> >
> > v2: Dont' resurrect drm_vblank_clean
Fixes the following warnings when building docs:
../include/drm/drm_drv.h:553: warning: No description found for parameter
'debugfs_init'
../include/drm/drm_drv.h:553: warning: No description found for parameter
'gem_open_object'
../include/drm/drm_drv.h:553: warning: No description found for par
Fixes the following warning when building docs:
../drivers/gpu/host1x/bus.c:50: warning: Excess function parameter 'driver'
description in 'host1x_subdev_add'
Signed-off-by: Sean Paul
---
drivers/gpu/host1x/bus.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/host1x/bus.c b/driv
Fixes:
../drivers/gpu/drm/drm_scdc_helper.c:203: ERROR: Unexpected indentation.
../drivers/gpu/drm/drm_scdc_helper.c:204: WARNING: Block quote ends without a
blank line; unexpected unindent.
Signed-off-by: Sean Paul
---
drivers/gpu/drm/drm_scdc_helper.c | 4 ++--
1 file changed, 2 insertions(+)
Fixes the following warnings when building docs:
../drivers/gpu/drm/drm_modes.c:1623: warning: No description found for
parameter 'display'
../drivers/gpu/drm/drm_modes.c:1623: warning: Excess function parameter
'connector' description in 'drm_mode_is_420_only'
Signed-off-by: Sean Paul
---
dri
I noticed these warnings as part of my compile testing and decided to fix them.
There are still a few warnings with the import/export prime helpers. They're
partially documented in drm_prime.c, so suggestions welcome on how to resolve
them without duplication (or perhaps duplication is best in thi
Pass in the array and not a pointer to the array to drm_dp_dpcd_read().
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/drm_dp_helper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c
index 213fb837e1c4..1d9e30
Before we interpret drm_dp_downstream_id() as a string, make sure it is
NULL terminated, even when drm_dp_downtsream_id() fails.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101660
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/drm_dp_helper.c | 3 ++-
1 file changed, 2 insertions(+),
On Thu, Jul 20, 2017 at 1:27 AM, Daniel Vetter wrote:
> On Wed, Jul 19, 2017 at 10:39:28AM -0700, Matthias Kaehlcke wrote:
>> Commit a21960339c8c ("drm/i915: Consistently use enum pipe for PCH
>> transcoders") misses some pieces, due to a problem with the patch
>> format, this patch adds the remai
Explain better when the drm_modeset_acquire_ctx parameter can
be skipped for drm_modeset_lock() call.
Signed-off-by: Liviu Dudau
Cc: Daniel Vetter
---
drivers/gpu/drm/drm_modeset_lock.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_modeset
When enabling lockdep debugging on Juno platform with HDLCD and TDA998x
I get the following warning from the system:
[ 25.990733] ==
[ 25.998637] WARNING: possible circular locking dependency detected
[ 26.006531] 4.13.0-rc1-00284-g28c0a682
On Thu, Jul 20, 2017 at 10:55 PM, Maxime Ripard
wrote:
> On Fri, Jul 14, 2017 at 12:14:37PM +0800, Chen-Yu Tsai wrote:
>> On Thu, Jul 13, 2017 at 10:13 PM, Maxime Ripard
>> wrote:
>> > It seems like the dotclock dividers are a bit less strict range, and can
>> > operate even with a smaller than 6
On Thu, Jul 20, 2017 at 04:57:12PM +0200, Daniel Vetter wrote:
> On Thu, Jul 20, 2017 at 4:40 PM, Liviu Dudau wrote:
> > On Thu, Jul 20, 2017 at 03:24:13PM +0100, Russell King - ARM Linux wrote:
> >> On Thu, Jul 20, 2017 at 03:19:10PM +0100, Liviu Dudau wrote:
> >> > On Thu, Jul 20, 2017 at 02:08:
Takashi Iwai writes:
> On Thu, 20 Jul 2017 06:17:58 +0200,
> Dave Airlie wrote:
> >
> > On 19 July 2017 at 00:43, Takashi Iwai wrote:
> > > From: Egbert Eich
> > >
> > > Add two more models G200_PCI and G200 for PCI device IDs 0x520 and
> > > 0x521, respectively. They need to retrieve t
On Thu, Jul 20, 2017 at 4:40 PM, Liviu Dudau wrote:
> On Thu, Jul 20, 2017 at 03:24:13PM +0100, Russell King - ARM Linux wrote:
>> On Thu, Jul 20, 2017 at 03:19:10PM +0100, Liviu Dudau wrote:
>> > On Thu, Jul 20, 2017 at 02:08:29PM +0100, Russell King - ARM Linux wrote:
>> > > On Thu, Jul 20, 2017
On Fri, Jul 14, 2017 at 12:14:37PM +0800, Chen-Yu Tsai wrote:
> On Thu, Jul 13, 2017 at 10:13 PM, Maxime Ripard
> wrote:
> > It seems like the dotclock dividers are a bit less strict range, and can
> > operate even with a smaller than 6 divider. Loose the boundaries a bit.
> >
> > Signed-off-by: M
On Tue, Jul 18, 2017 at 02:21:47PM +0100, Mark Brown wrote:
> On Mon, Jul 17, 2017 at 11:01:07AM +0200, Maxime Ripard wrote:
> > On Thu, Jul 13, 2017 at 05:01:42PM +0100, Mark Brown wrote:
>
> > > > This might be problematic if the clock to enable is stored in another
> > > > node.
> > > > Let's
On Wed, 2017-07-19 at 23:11 -0700, Manasi Navare wrote:
> On Tue, Jul 18, 2017 at 03:11:42PM +0300, Paul Kocialkowski wrote:
> > On Mon, 2017-06-26 at 15:32 +0300, Paul Kocialkowski wrote:
> > > After detecting an IRQ storm, hotplug detection will switch from
> > > irq-based detection to poll-based
On Thu, Jul 20, 2017 at 03:24:13PM +0100, Russell King - ARM Linux wrote:
> On Thu, Jul 20, 2017 at 03:19:10PM +0100, Liviu Dudau wrote:
> > On Thu, Jul 20, 2017 at 02:08:29PM +0100, Russell King - ARM Linux wrote:
> > > On Thu, Jul 20, 2017 at 01:54:04PM +0100, Liviu Dudau wrote:
> > > > On Thu, J
On Thu, 2017-07-20 at 01:04 +, Pandiyan, Dhinakaran wrote:
> On Mon, 2017-06-26 at 15:32 +0300, Paul Kocialkowski wrote:
> > After detecting an IRQ storm, hotplug detection will switch from
> > irq-based detection to poll-based detection. After a short delay or
> > when resetting storm detectio
Hi Laurent
On Tue, Jul 18, 2017 at 01:18:42PM +0300, Laurent Pinchart wrote:
> On Monday 17 Jul 2017 13:41:49 Rob Herring wrote:
> > On Thu, Jul 13, 2017 at 04:13:06PM +0200, Maxime Ripard wrote:
> > > The Allwinner SoCs usually come with a DSI encoder. Add a binding for it.
> > >
> > > Signed-of
When enabling lockdep debugging on Juno platform with HDLCD and TDA998x
I get the following warning from the system:
[ 25.990733] ==
[ 25.998637] WARNING: possible circular locking dependency detected
[ 26.006531] 4.13.0-rc1-00284-g28c0a682
Hi Rob,
On Mon, Jul 17, 2017 at 01:41:49PM -0500, Rob Herring wrote:
> On Thu, Jul 13, 2017 at 04:13:06PM +0200, Maxime Ripard wrote:
> > The Allwinner SoCs usually come with a DSI encoder. Add a binding for it.
> >
> > Signed-off-by: Maxime Ripard
> > ---
> > Documentation/devicetree/bindings/
On Thu, Jul 20, 2017 at 02:08:29PM +0100, Russell King - ARM Linux wrote:
> On Thu, Jul 20, 2017 at 01:54:04PM +0100, Liviu Dudau wrote:
> > On Thu, Jul 20, 2017 at 12:44:49PM +0100, Russell King - ARM Linux wrote:
> > > Actually, scrub that idea - drm_helper_probe_single_connector_modes()
> > > ca
This patch adds a flowchart to the drm-misc documentation to
help committers decide which branch is most appropriate for a
given patch.
Signed-off-by: Sean Paul
---
.gitignore | 1 +
Makefile | 2 +-
drm-misc-commit-flow.dot | 22 ++
drm-misc.r
Am Donnerstag, den 20.07.2017, 14:51 +0200 schrieb Daniel Vetter:
> It makes debugging a massive pain.
It is also considered very bad style to BUG the kernel on anything other
than filesystem eating catastrophic failures.
Reviewed-by: Lucas Stach
> Signed-off-by: Daniel Vetter
> Cc: Sumit Semw
On Tue, Jul 18, 2017 at 11:41:36AM +0800, Chen-Yu Tsai wrote:
> On Thu, Jul 13, 2017 at 10:13 PM, Maxime Ripard
> wrote:
> > Some channel0 setup has to be done, no matter what the output interface is
> > (RGB, CPU, LVDS). Move that code into a common function in order to avoid
> > duplication.
> >
On Tue, Jul 18, 2017 at 04:43:04PM -0500, Rob Herring wrote:
> Now that we have a custom printf format specifier, convert users of
> full_name to use %pOF instead. This is preparation to remove storing
> of the full path string for each node.
>
> Signed-off-by: Rob Herring
Hi Rob,
Thanks for sen
On Wed, Jul 19, 2017 at 08:37:06AM -0700, Gurchetan Singh wrote:
> On ChromeOS devices, readdir() processes the directory in
> the following order:
>
> -NAME- -TYPE-
> .n/a
> .. n/a
> vgem n/a
> card1 DRM_BUS_PLATFORM
> r
https://bugs.freedesktop.org/show_bug.cgi?id=101850
arkadiusz.hi...@intel.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee
Hi Chen-Yu,
On Fri, Jul 14, 2017 at 03:53:40PM +0800, Chen-Yu Tsai wrote:
> On Thu, Jul 13, 2017 at 10:13 PM, Maxime Ripard
> wrote:
> > Huarui Lighting makes display panel, add it to the list of panels.
>
> I could not find any information on "Huarui Lighting" within the
> context of LCD panels
https://bugs.freedesktop.org/show_bug.cgi?id=101850
arkadiusz.hi...@intel.com changed:
What|Removed |Added
Component|tests |IGT
QA Contact|pig.
On Fri, Jul 14, 2017 at 11:56:18AM +0800, Chen-Yu Tsai wrote:
> On Thu, Jul 13, 2017 at 10:13 PM, Maxime Ripard
> wrote:
> > Just like we did for the TCON enable and disable, for historical reasons we
> > used to rely on the encoders calling the TCON mode_set function, while the
> > CRTC has a cal
Hi Chen-Yu,
On Fri, Jul 14, 2017 at 11:40:07AM +0800, Chen-Yu Tsai wrote:
> > static void sun4i_rgb_encoder_mode_set(struct drm_encoder *encoder,
> > diff --git a/drivers/gpu/drm/sun4i/sun4i_tcon.c
> > b/drivers/gpu/drm/sun4i/sun4i_tcon.c
> > index d9791292553e..dc70bc2a42a5 100644
> > --- a/dri
On Fri, Jul 14, 2017 at 11:13:53AM +0800, Chen-Yu Tsai wrote:
> On Thu, Jul 13, 2017 at 10:12 PM, Maxime Ripard
> wrote:
> > Some options were not padded as they should, and the order in the Makefile
> > was chaotic. Fix that.
> >
> > Signed-off-by: Maxime Ripard
> > ---
> > drivers/gpu/drm/sun4
The current drm_atomic_helper_commit_tail helper works only if the CRTC is
accessible, and documents an alternative implementation that is supposed to
be used if that happens.
That implementation is then duplicated by some drivers. Instead of
documenting it, let's implement an helper that all the
The Allwinner backend has a commit bit in order to push the new
configuration to the actual hardware. We've always been using that bit.
However, we also should poll for that bit to clear, which we don't.
Accessing any register while a commit is pending is forbidden, and will for
example show a sym
The backend (planes) commit can only happen when the TCON (CRTC) is
enabled, which is not guaranteed with the default commit_tail helper.
Let's use the runtime_pm version that is designed specifically to deal with
that case.
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/sun4i/sun4i_framebuff
On the earlier Allwinner chips, with the first iteration of the display
engine, the backend commit bit needs to be polled before making any
register access to the backend.
Add an operation for that, that will be called in atomic_begin in order to
be sure to have that bit cleared before we do any m
In the earlier display engine designs, any register access while a commit
is pending is forbidden.
One of the symptoms is that reading a register will return another, random,
register value which can lead to register corruptions if we ever do a
read/modify/write cycle.
Signed-off-by: Maxime Ripar
On Thu, Jul 20, 2017 at 12:44:49PM +0100, Russell King - ARM Linux wrote:
> On Thu, Jul 20, 2017 at 12:38:53PM +0100, Russell King - ARM Linux wrote:
> > On Thu, Jul 20, 2017 at 12:04:50PM +0100, Liviu Dudau wrote:
> > > When enabling lockdep debugging on Juno platform with HDLCD and TDA998x
> > >
It makes debugging a massive pain.
Signed-off-by: Daniel Vetter
Cc: Sumit Semwal
Cc: Gustavo Padovan
Cc: linux-me...@vger.kernel.org
Cc: linaro-mm-...@lists.linaro.org
---
drivers/dma-buf/dma-fence.c | 4 ++--
include/linux/dma-fence.h | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(
Hi Philippe,
On Thu, 2017-07-20 at 14:19 +0200, Philippe CORNU wrote:
> Based on patch "Convert drivers to explicit reset API" from Philipp Zabel
>
> Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
> reset lines") started to transition the reset control request API calls
2017-07-20 14:05 GMT+02:00 Philippe CORNU :
> Version 2:
> - Add devm_reset_control_get_exclusive to follow explicit reset API
> - Add missing commit messages & reviewed-by.
>
> Version 1:
> - Initial commit
>
> The purpose of this set of patches is to clean up the drm stm driver.
>
> Philippe CORN
Hi Philippe,
On Thu, 2017-07-20 at 14:05 +0200, Philippe CORNU wrote:
> Use devm_reset_control_get_exclusive to avoid resource leakage (based
> on patch "Convert drivers to explicit reset API" from Philipp Zabel).
>
> Also use platform_get_resource, which is more usual and
> consistent with platf
This patch cleans up the Synopsys mipi dsi register list:
- rename registers according to the Synopsys documentation
(1.30 & 1.31)
- fix typos
- re-order registers for a better coherency
Signed-off-by: Philippe CORNU
Reviewed-by: Laurent Pinchart
---
drivers/gpu/drm/bridge/synopsys/dw-mipi-ds
Version 2:
- Put back Synopsys mipi dsi unused registers.
- Add devm_reset_control_get_exclusive to follow explicit reset API.
- Add a missing commit message & reviewed-by.
Version 1:
- Initial commit
The purpose of this set of patches is to clean up the mipi dsi dw
Synopsys drm bridge.
Philippe
Based on patch "Convert drivers to explicit reset API" from Philipp Zabel
Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
reset lines") started to transition the reset control request API calls
to explicitly state whether the driver needs exclusive or shared reset
control
Constify dw_mipi_dsi_bridge_funcs as these functions are not supposed
to change at runtime.
Signed-off-by: Philippe CORNU
Reviewed-by: Laurent Pinchart
---
drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/sy
On Thu, 20 Jul 2017 13:58:14 +0200,
Paul Menzel wrote:
>
> Dear Takashi,
>
>
> Thank you for posting these patches for review.
>
>
> Am Dienstag, den 18.07.2017, 16:43 +0200 schrieb Takashi Iwai:
> > From: Egbert Eich
> >
> > The depth/bpp handling for chips with limited memory in commit
> >
On Thu, Jul 20, 2017 at 1:43 PM, Michael Ellerman wrote:
> Matt Brown writes:
>> The cacheflush prototypes currently use start and stop values and each
>> call requires typecasting the address to an unsigned long.
>> This patch changes the cacheflush prototypes to follow the x86 style of
>> using
Constify dw_mipi_dsi_stm_phy_ops as these ops are not supposed
to change at runtime.
Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
---
drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
Constify drm funcs structures.
Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
---
drivers/gpu/drm/stm/ltdc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index 628825b..92e58ba 100644
--- a/driver
Lindent then checkpatch --strict cleanups
Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
---
drivers/gpu/drm/stm/ltdc.c | 172 ++---
1 file changed, 85 insertions(+), 87 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm
Rename the returned value from "res" to "ret" as it is more "readable".
Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
---
drivers/gpu/drm/stm/ltdc.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltd
Use devm_reset_control_get_exclusive to avoid resource leakage (based
on patch "Convert drivers to explicit reset API" from Philipp Zabel).
Also use platform_get_resource, which is more usual and
consistent with platform_get_irq called later.
Signed-off-by: Fabien Dessenne
Signed-off-by: Philipp
The GCR_PCPOL/DEPOL/VSPOL/HSPOL defines are sufficient to
describe the HS, VS, DE & PC signal polarities.
Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
---
drivers/gpu/drm/stm/ltdc.c | 28 ++--
1 file changed, 10 insertions(+), 18 deletions(-)
diff --git
Rename the platform driver name from "stm" to "stm32-display"
for a better readability in /sys/bus/platform/drivers entries.
Note: We keep "stm" as drm_driver.name because it is better
when using "modetest -M stm ..." (even if recent modetest patch
avoids using -M).
Signed-off-by: Philippe CORNU
Version 2:
- Add devm_reset_control_get_exclusive to follow explicit reset API
- Add missing commit messages & reviewed-by.
Version 1:
- Initial commit
The purpose of this set of patches is to clean up the drm stm driver.
Philippe CORNU (7):
drm/stm: drv: Rename platform driver name
drm/stm:
Dear Takashi,
Thank you for posting these patches for review.
Am Dienstag, den 18.07.2017, 16:43 +0200 schrieb Takashi Iwai:
> From: Egbert Eich
>
> The depth/bpp handling for chips with limited memory in commit
> 918be888d613 ("drm/mgag200: on cards with < 2MB VRAM default to
> 16-bit") was
On 19 July 2017 at 16:37, Gurchetan Singh wrote:
> On ChromeOS devices, readdir() processes the directory in
> the following order:
>
> -NAME- -TYPE-
> .n/a
> .. n/a
> vgem n/a
> card1 DRM_BUS_PLATFORM
> renderD129 D
https://bugs.freedesktop.org/show_bug.cgi?id=101837
--- Comment #3 from Emil Velikov ---
DanielP an orthogonal solution is to simply not use drmOpen. While it works,
sometimes, there's a lot of hidden gotchas.
Simply replace the pciaccess + drmOpen with drmDevice2 - see
libdrm/tests/drmdevice.c.
When enabling lockdep debugging on Juno platform with HDLCD and TDA998x
I get the following warning from the system:
[ 25.990733] ==
[ 25.998637] WARNING: possible circular locking dependency detected
[ 26.006531] 4.13.0-rc1-00284-g28c0a682
Quoting Tvrtko Ursulin (2017-07-20 11:09:53)
>
> On 19/07/2017 23:35, Christophe JAILLET wrote:
> > Goto the right label in case of error, otherwise there is a leak.
> > This has been introduced by c5cf9a9147ff. In this patch a goto has not been
> > updated.
> >
> > Fixes: c5cf9a9147ff ("drm/i915
On Thu, Jul 20, 2017 at 11:53:39AM +0200, Maxime Ripard wrote:
> Hi Daniel,
>
> On Tue, Jul 18, 2017 at 09:35:03AM +0200, Daniel Vetter wrote:
> > On Tue, Jul 18, 2017 at 9:07 AM, Maxime Ripard
> > wrote:
> > > On Mon, Jul 17, 2017 at 02:57:19PM +0800, Chen-Yu Tsai wrote:
> > >> On Mon, Jul 17, 2
On 19/07/2017 23:35, Christophe JAILLET wrote:
Goto the right label in case of error, otherwise there is a leak.
This has been introduced by c5cf9a9147ff. In this patch a goto has not been
updated.
Fixes: c5cf9a9147ff ("drm/i915: Create a kmem_cache to allocate struct i915_priolist
from")
Sign
2017-07-19 17:25 GMT+02:00 Philipp Zabel :
> Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
> reset lines") started to transition the reset control request API calls
> to explicitly state whether the driver needs exclusive or shared reset
> control behavior. Convert all dr
Hi Daniel,
On Tue, Jul 18, 2017 at 09:35:03AM +0200, Daniel Vetter wrote:
> On Tue, Jul 18, 2017 at 9:07 AM, Maxime Ripard
> wrote:
> > On Mon, Jul 17, 2017 at 02:57:19PM +0800, Chen-Yu Tsai wrote:
> >> On Mon, Jul 17, 2017 at 2:55 PM, Maxime Ripard
> >> wrote:
> >> > On Fri, Jul 14, 2017 at 04:
Hi Philippe,
On Wednesday 19 Jul 2017 09:11:44 Philippe CORNU wrote:
> On 07/18/2017 03:39 PM, Laurent Pinchart wrote:
> > On Tuesday 18 Jul 2017 13:43:52 Philippe CORNU wrote:
> >
> >> This patch cleans up the Synopsys mipi dsi register list:
> >> - remove unused registers
> >
> > Is the docume
On Thu, 20 Jul 2017 06:17:58 +0200,
Dave Airlie wrote:
>
> On 19 July 2017 at 00:43, Takashi Iwai wrote:
> > From: Egbert Eich
> >
> > Add two more models G200_PCI and G200 for PCI device IDs 0x520 and
> > 0x521, respectively. They need to retrieve the reference clock and
> > pclk min/max value
Hi Eric,
On 16/07/17 12:48, Hans Verkuil wrote:
> From: Hans Verkuil
>
> This patch adds support to VC4 for CEC.
>
> Thanks to Eric Anholt for providing me with the CEC register information.
>
> To prevent the firmware from eating the CEC interrupts you need to add this to
> your config.txt:
>
At destroying the connector, we need no longer to call
drm_connector_unregister() explicitly, as drm_dev_unregister()
unregisters all connectors too. Keeping it may cause a confusion by a
brainless copy-and-paste to other drivers, so let's rip it off now.
Suggested-by: Daniel Vetter
Signed-off-b
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
Please write a commit message
> Signed-off-by: Philippe CORNU
> ---
> drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
> b/drivers/gpu/drm/stm/dw_mipi
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
Commit message is missing here
> Signed-off-by: Philippe CORNU
> ---
> drivers/gpu/drm/stm/ltdc.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index f4ed
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
> Use devm_reset_control_get to avoid resource leakage.
> Also use platform_get_resource, which is more usual and
> consistent with platform_get_irq called later.
>
> Signed-off-by: Fabien Dessenne
> Signed-off-by: Philippe CORNU
Note that could be con
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
> The GCR_PCPOL/DEPOL/VSPOL/HSPOL defines are sufficient to
> describe the HS, VS, DE & PC signal polarities.
>
> Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
> ---
> drivers/gpu/drm/stm/ltdc.c | 28 ++--
> 1 f
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
> Constify drm funcs structures.
>
> Signed-off-by: Philippe CORNU
Reviewed-by: Benjamin Gaignard
> ---
> drivers/gpu/drm/stm/ltdc.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/g
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
> Lindent then checkpatch --strict cleanups
>
> Signed-off-by: Philippe CORNU
> ---
> drivers/gpu/drm/stm/ltdc.c | 172
> ++---
> 1 file changed, 85 insertions(+), 87 deletions(-)
>
> diff --git a/drivers/gpu/dr
2017-07-18 12:20 GMT+02:00 Philippe CORNU :
> Rename the platform driver name from "stm" to "stm32-display"
> for a better readability in /sys/bus/platform/drivers entries.
>
> Note: We keep "stm" as drm_driver.name because it is better
> when using "modetest -M stm ..." (even if recent modetest pa
1 - 100 of 114 matches
Mail list logo
