On 16 December 2014 at 18:56, Daniel Vetter wrote:
> On Tue, Dec 16, 2014 at 04:34:39PM +1000, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> nouveau userspace back at 1.0.1 used to call the X server
>> DRIOpenDRMMaster interface even for DRI2 (doh!), this attempts
>> to map the sarea and fails if
Hello Thierry Reding,
This is a semi-automatic email about new static checker warnings.
The patch a52879e8d7cb: "drm/dsi: Add message to packet translator"
from Oct 16, 2014, leads to the following Smatch complaint:
drivers/gpu/drm/drm_mipi_dsi.c:328 mipi_dsi_create_packet()
warn: vari
Hello Dan,
Thanks for your review, I will fix it soon.
Do you use a script to find out these issues? I was interested in this
script.
Can you teach me how to find this script and use it?
regards,
Mark Yao
On 2014å¹´12æ16æ¥ 19:51, Dan Carpenter wrote:
> Hello Mark Yao,
>
> The patch 2048e3286
https://bugzilla.kernel.org/show_bug.cgi?id=85421
--- Comment #21 from Michel Dänzer ---
If there really is a significant difference in stability between Mesa 10.2.y
and 10.3.y, it would be interesting if you could isolate which change between
them made the difference.
However, from your descri
On 17.12.2014 08:05, Rob Clark wrote:
> The atomic modeset ioctl can be used to push any number of new values
> for object properties. The driver can then check the full device
> configuration as single unit, and try to apply the changes atomically.
>
> The ioctl simply takes a list of object IDs
On 12/15/2014 04:56 PM, Thierry Reding wrote:
> From: Thierry Reding
>
> The QXL driver duplicates part of the core's drm_vblank_count(), so it
> might as well use the core's variant for the extra goodies.
>
> Signed-off-by: Thierry Reding
> ---
> drivers/gpu/drm/qxl/qxl_drv.c | 7 +--
> 1
On 17.12.2014 01:53, Thierry Reding wrote:
> From: Thierry Reding
>
> When accessing the array of per-CRTC VBLANK structures we must always
> check that the index into the array is valid before dereferencing to
> avoid crashing.
>
> Signed-off-by: Thierry Reding
> ---
> drivers/gpu/drm/drm_irq
he assignee for the bug.
------ next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/0b612a6a/attachment.html>
On Wed, 17 Dec 2014 11:48:51 +0900
Michel Dänzer wrote:
> On 17.12.2014 08:05, Rob Clark wrote:
> > The atomic modeset ioctl can be used to push any number of new values
> > for object properties. The driver can then check the full device
> > configuration as single unit, and try to apply the ch
https://bugzilla.kernel.org/show_bug.cgi?id=89661
--- Comment #12 from Oded Gabbay ---
As I said, there is definitely a bug when compiling both radeon and amdkfd
inside the kernel.
I'm working on fixing it, but that could take a few days.
In the meantime, the only way to make it work without touc
ne process could go to sleep in pm_runtime_get_sync() or
clk_prepare_enable() and another process start running mixer_poweron()
concurrently, getting to the second mutex_lock() sooner than the first
process. So the lock being dropped between checking for ctx->powered and
setting it doesn't actually prevent a race.
Then again, nobody seems to have cared so far...
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/c3389dfd/attachment.sig>
m pretty sure I've
seen this work reliably with something like MFD where one module can
create a number of platform devices, and remove them again, just as
well.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/24ca9713/attachment-0001.sig>
I guess if you don't do anything else, then indeed the ipvr module will
stay around, but the above should work idempotently, that is you should
be able to repeat it an unlimited number of times and nothing should
break.
In fact you should be able to run the following in any permutation
without causing a crash:
# modprobe i915
# modprobe ipvr
# modprobe -r ipvr
# modprobe -r i915
If any permutation results in a crash you have a bug.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/beeacd74/attachment.sig>
ts.freedesktop.org/archives/dri-devel/attachments/20141217/e6f6e51a/attachment.html>
hment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/743c9bdb/attachment.html>
On 17.12.2014 16:20, Pekka Paalanen wrote:
> On Wed, 17 Dec 2014 11:48:51 +0900
> Michel Dänzer wrote:
>
>> On 17.12.2014 08:05, Rob Clark wrote:
>>> The atomic modeset ioctl can be used to push any number of new values
>>> for object properties. The driver can then check the full device
>>> con
On Wed, Dec 17, 2014 at 03:57:51AM +0100, Mario Kleiner wrote:
> On 12/15/2014 04:56 PM, Thierry Reding wrote:
> > From: Thierry Reding
> >
> > The QXL driver duplicates part of the core's drm_vblank_count(), so it
> > might as well use the core's variant for the extra goodies.
> >
> > Signed-off-
dri-devel/attachments/20141217/4f9a57ad/attachment-0001.html>
Hi Thierry,
Sorry for the late response.
I tried to address almost all your comments locally first.
More feedback below.
On 12/10/2014 09:16 PM, Thierry Reding wrote:
> On Wed, Dec 10, 2014 at 04:37:22PM +0800, Liu Ying wrote:
>> This patch adds i.MX MIPI DSI host controller driver support.
>> Cu
On 12/10/2014 10:03 PM, Thierry Reding wrote:
> On Wed, Dec 10, 2014 at 04:37:23PM +0800, Liu Ying wrote:
>> This patch adds support for Himax HX8369A MIPI DSI panel.
>>
>> Signed-off-by: Liu Ying
>> ---
>> .../devicetree/bindings/panel/himax,hx8369a.txt| 86 +++
>> drivers/gpu/drm/panel
--
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/cc9dea5a/attachment.html>
- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/f7825aa7/attachment.html>
ary, at least
> regarding the binding. If we ever get need to support multiple panels
> with the same driver IC the proper solution would be to abstract that
> code out into a helper library that is used from the various panel
> drivers.
>
> Thierry
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/74bc0699/attachment.html>
On Wed, Dec 17, 2014 at 05:44:33PM +0800, Liu Ying wrote:
> Hi Thierry,
>
> Sorry for the late response.
> I tried to address almost all your comments locally first.
> More feedback below.
>
> On 12/10/2014 09:16 PM, Thierry Reding wrote:
> >On Wed, Dec 10, 2014 at 04:37:22PM +0800, Liu Ying wrot
On Wed, Dec 17, 2014 at 06:31:13PM +0900, Michel Dänzer wrote:
> On 17.12.2014 16:20, Pekka Paalanen wrote:
> > On Wed, 17 Dec 2014 11:48:51 +0900
> > Michel Dänzer wrote:
> >
> >> On 17.12.2014 08:05, Rob Clark wrote:
> >>> The atomic modeset ioctl can be used to push any number of new values
https://bugzilla.kernel.org/show_bug.cgi?id=89661
Oded Gabbay changed:
What|Removed |Added
Attachment #160721|0 |1
is obsolete|
Hi Javier,
On Wednesday 17 December 2014 10:31:41 Javier Martinez Canillas wrote:
> On 12/16/2014 12:37 AM, Laurent Pinchart wrote:
> > Hi Javier,
> >
> >> Tomi and Laurent,
> >>
> >> You asked Ajay to change his series to use the video port and enpoints DT
> >> bindings instead of phandles, cou
https://bugzilla.kernel.org/show_bug.cgi?id=89661
--- Comment #14 from Oded Gabbay ---
I attached a new patch which should solve you the problem when compiling all
the drivers into the kernel image.
This is a hacky workaround, so this is not the final solution, but it will help
you continue with
https://bugzilla.kernel.org/show_bug.cgi?id=89661
Oded Gabbay changed:
What|Removed |Added
Attachment #160951|0 |1
is obsolete|
https://bugzilla.kernel.org/show_bug.cgi?id=86011
--- Comment #7 from JoaquÃn AramendÃa ---
Confirmed. It was also backported today to 3.17.y branch. So it's fixed for
good
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
From: Ville Syrjälä
We had a bug in i915 land recently where X passed in a zeroed mode with
mode_valid=1 to setcrtc. That didn't go down so well and caused a
div-by-zero in i915.
For a long time I've been thinking that we need some real checks to
validate the modes we feed into the hardware. I
From: Ville Syrjälä
Make drm_mode_validate_size() and drm_mode_validate_flag() deal with a
single mode instead of having each iterate through the mode list.
The hope is that in the future we might be able to share various mode
validation functions between modeset and get_modes.
Signed-off-by:
From: Ville Syrjälä
Make sure the timings of probed modes at least pass some very basic
sanity checks.
The checks include:
- clock,hdisplay,vdisplay are non zero
- sync pulse fits within the blanking period
- htotal,vtotal are big enough
I have not checked all the drivers to see if the mod
From: Ville Syrjälä
Currently userspace is allowed to pass in basiclly any kind of garbage
to setcrtc. Try to catch the cases where the timings make no sense
by passing the mode through drm_mode_validate_basic().
One concern here is that we now start to block some modes that have
worked in the
Hi Dave -
Final i915 fixes pull before rc1, majority of them cc: stable.
BR,
Jani.
The following changes since commit 9f49c37635d5c2a801f7670d5fbf0b25ec461f2c:
drm/i915: save/restore GMBUS freq across suspend/resume on gen4 (2014-12-11
15:31:59 +0200)
are available in the git repository at
Steve,
On Mon, Dec 15, 2014 at 10:29 PM, wrote:
> From: Jiada Wang
>
> On some monitors, high resolution modes are not working, exhibiting
> pixel column truncation problems (for example, 1280x1024 displays as
> 1280x1022).
>
> The function ipu_di_adjust_videomode() aims to fix these issues by
On Tue, Dec 16, 2014 at 06:05:29PM -0500, Rob Clark wrote:
> We can already have object properties, which technically can be fb's.
> Switch the property validation logic over to a get/put style interface
> so it can take a ref to refcnt'd objects, and then drop that ref after
> the driver has a cha
On Wed, Dec 17, 2014 at 08:56:25AM +0800, Mark yao wrote:
> Hello Dan,
>
> Thanks for your review, I will fix it soon.
> Do you use a script to find out these issues? I was interested in
> this script.
> Can you teach me how to find this script and use it?
The first one is a smatch warning. http
On Tue, Dec 16, 2014 at 06:05:32PM -0500, Rob Clark wrote:
> As we add properties for all the standard plane/crtc/connector
> attributes (in preperation for the atomic ioctl), we are going to want
> to handle core state in core (rather than per driver). Intercepting the
> core properties will be e
On Tue, Dec 16, 2014 at 06:05:33PM -0500, Rob Clark wrote:
> Since we won't be using the obj->properties->values[] array to shadow
> property values for atomic drivers, we are going to need a vfunc for
> getting prop values. Add that along w/ mandatory wrapper fxns.
>
> Signed-off-by: Rob Clark
On Tue, Dec 16, 2014 at 6:05 PM, Rob Clark wrote:
> Now that we have core atomic helpers, let's expose that to userspace!
> The first step here is to, for drivers supporting atomic, expose all
> the core plane/CRTC/connector state as properties. Once this is in
> place, all that remains is the at
On Tue, Dec 16, 2014 at 06:05:30PM -0500, Rob Clark wrote:
> Keep property pointer, instead of id, in per mode-object attachments.
> This will simplify things in later patches.
>
> Signed-off-by: Rob Clark
Reviewed-by: Daniel Vetter
> ---
> drivers/gpu/drm/drm_crtc.c | 17 -
>
On Tue, Dec 16, 2014 at 06:05:31PM -0500, Rob Clark wrote:
> For atomic drivers, we won't use the values array but instead shunt
> things off to obj->atomic_get_property(). So to simplify things make
> all read/write of properties values go through the accessors.
>
> Signed-off-by: Rob Clark
Re
On Tue, Dec 16, 2014 at 06:05:32PM -0500, Rob Clark wrote:
> As we add properties for all the standard plane/crtc/connector
> attributes (in preperation for the atomic ioctl), we are going to want
> to handle core state in core (rather than per driver). Intercepting the
> core properties will be e
On Tue, Dec 16, 2014 at 06:05:33PM -0500, Rob Clark wrote:
> Since we won't be using the obj->properties->values[] array to shadow
> property values for atomic drivers, we are going to need a vfunc for
> getting prop values. Add that along w/ mandatory wrapper fxns.
>
> Signed-off-by: Rob Clark
On Tue, Dec 16, 2014 at 06:05:34PM -0500, Rob Clark wrote:
> Once a driver is using atomic helpers for modeset, the next step is to
> switch over to atomic properties. To do this, plug in the helper
> function to your modeconfig funcs and be sure to implement the plane/
> crtc/connector atomic_{ge
Hi Steve,
I think this is a good idea. Two small comments below.
Am Montag, den 15.12.2014, 16:29 -0800 schrieb slongerbeam at gmail.com:
> From: Steve Longerbeam
>
> This patch changes struct ipu_di_signal_cfg to use struct videomode
> to define video timings and flags.
>
> Signed-off-by: Ste
Am Montag, den 15.12.2014, 16:29 -0800 schrieb slongerbeam at gmail.com:
> From: Steve Longerbeam
>
> The encoder ->mode_set() methods need to use the hw adjusted mode,
> not the original mode.
>
> Signed-off-by: Steve Longerbeam
> ---
> drivers/gpu/drm/imx/imx-hdmi.c |4 ++--
> dr
On Wed, Dec 17, 2014 at 4:31 AM, Michel Dänzer wrote:
> On 17.12.2014 16:20, Pekka Paalanen wrote:
>> On Wed, 17 Dec 2014 11:48:51 +0900
>> Michel Dänzer wrote:
>>
>>> On 17.12.2014 08:05, Rob Clark wrote:
The atomic modeset ioctl can be used to push any number of new values
for objec
On Wed, Dec 17, 2014 at 03:02:41PM +0100, Daniel Vetter wrote:
> On Tue, Dec 16, 2014 at 06:05:34PM -0500, Rob Clark wrote:
> > Once a driver is using atomic helpers for modeset, the next step is to
> > switch over to atomic properties. To do this, plug in the helper
> > function to your modeconfi
The following changes since commit 4e0cd68115620bc3236ff4e58e4c073948629b41:
drm: sti: fix module compilation issue (2014-12-15 17:07:57 +1000)
are available in the git repository at:
git://people.freedesktop.org/~tagr/linux tags/drm/tegra/for-3.19-rc1-fixes
for you to fetch changes up to 9
On Tue, Dec 16, 2014 at 06:05:35PM -0500, Rob Clark wrote:
> Getting ready to add a lot more standard properties for atomic.
>
> Signed-off-by: Rob Clark
Reviewed-by: Daniel Vetter
> ---
> drivers/gpu/drm/drm_crtc.c | 27 +--
> 1 file changed, 17 insertions(+), 10 dele
On Wed, Dec 17, 2014 at 9:02 AM, Daniel Vetter wrote:
> On Tue, Dec 16, 2014 at 06:05:34PM -0500, Rob Clark wrote:
>> Once a driver is using atomic helpers for modeset, the next step is to
>> switch over to atomic properties. To do this, plug in the helper
>> function to your modeconfig funcs and
On Tue, Dec 16, 2014 at 06:05:36PM -0500, Rob Clark wrote:
> We need to hold connection_mutex as we read the properties. Easiest
> thing is just widen the scope where connection_mutex is held.
>
> Signed-off-by: Rob Clark
I'm not yet sure on the story we want to have for getprop locking (and
wh
On Wed, Dec 17, 2014 at 9:06 AM, Daniel Vetter wrote:
> On Wed, Dec 17, 2014 at 03:02:41PM +0100, Daniel Vetter wrote:
>> On Tue, Dec 16, 2014 at 06:05:34PM -0500, Rob Clark wrote:
>> > Once a driver is using atomic helpers for modeset, the next step is to
>> > switch over to atomic properties. T
Hi Steve,
Please put me on Cc: for IPU and imx-drm related patches.
Am Montag, den 15.12.2014, 16:29 -0800 schrieb slongerbeam at gmail.com:
> From: Steve Longerbeam
>
> This patchset implements ->mode_fixup() in the imx ipuv3-crtc driver,
> using a new support function ipu_di_adjust_videomode(
On Tue, Dec 16, 2014 at 06:05:37PM -0500, Rob Clark wrote:
> Add functions to check core plane/crtc state.
>
> Signed-off-by: Rob Clark
> ---
> drivers/gpu/drm/drm_atomic.c| 87
> +
> drivers/gpu/drm/drm_atomic_helper.c | 47 ++--
> in
On Wed, Dec 17, 2014 at 09:15:12AM -0500, Rob Clark wrote:
> On Wed, Dec 17, 2014 at 9:06 AM, Daniel Vetter wrote:
> > On Wed, Dec 17, 2014 at 03:02:41PM +0100, Daniel Vetter wrote:
> >> On Tue, Dec 16, 2014 at 06:05:34PM -0500, Rob Clark wrote:
> >> > Once a driver is using atomic helpers for mod
On Wed, Dec 17, 2014 at 09:12:31AM -0500, Rob Clark wrote:
> But I am still not a huge fan of hiding props for atomic drivers via
> legacy getprop/setprop interfaces.. if nothing else it will prevent
> them from showing up in modetest. And it seems kind of inconsistent
> and unnecessary.
By defau
available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/c87d159f/attachment.sig>
On Tue, Dec 16, 2014 at 06:05:38PM -0500, Rob Clark wrote:
> Expose the core plane state as properties, so they can be updated via
> atomic ioctl.
>
> Signed-off-by: Rob Clark
Just comments about the lack of PROP_ATOMIC and one suggestion for a
comment. With that addressed this is Reviewed-by: D
On Tue, Dec 16, 2014 at 06:05:39PM -0500, Rob Clark wrote:
> Expose the core connector state as properties so it can be updated via
> atomic ioctl.
>
> Signed-off-by: Rob Clark
Oh, didn't realize that you're sharing config->prop_crtc_id with plane and
connector. Please add a comment about that t
On Tue, Dec 16, 2014 at 06:05:41PM -0500, Rob Clark wrote:
> The atomic modeset ioctl can be used to push any number of new values
> for object properties. The driver can then check the full device
> configuration as single unit, and try to apply the changes atomically.
>
> The ioctl simply takes
https://bugzilla.kernel.org/show_bug.cgi?id=89961
Bug ID: 89961
Summary: System Shutdown on High Load (Radeon)
Product: Drivers
Version: 2.5
Kernel Version: >3.13
Hardware: All
OS: Linux
Tree: Mainline
From: Thierry Reding
Signed-off-by: Thierry Reding
---
Documentation/DocBook/drm.tmpl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
index 8b6fe229dc9a..8a0b2a84f4bf 100644
--- a/Documentation/DocBook/drm.tmpl
From: Thierry Reding
Make drm_fb_helper_initial_config() return an int rather than a bool so
that the error can be properly propagated. While at it, update drivers
to propagate errors further rather than just ignore them.
Cc: David Airlie
Cc: Daniel Vetter
Cc: Patrik Jakobsson
Cc: Rob Clark
On Wed, Dec 17, 2014 at 04:13:17PM +0100, Thierry Reding wrote:
> From: Thierry Reding
>
> Signed-off-by: Thierry Reding
Applied to my drm misc tree.
-Daniel
> ---
> Documentation/DocBook/drm.tmpl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/DocBook/
From: Thierry Reding
Signed-off-by: Thierry Reding
---
Documentation/DocBook/drm.tmpl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
index 8b6fe229dc9a..8a0b2a84f4bf 100644
--- a/Documentation/DocBook/drm.tmpl
From: Thierry Reding
The struct drm_connector_funcs kerneldoc refers to a part of struct
drm_crtc_funcs that no longer exists.
Signed-off-by: Thierry Reding
---
include/drm/drm_crtc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_crtc.h b/include/drm/drm_c
From: Thierry Reding
The .irq and .irq_enabled fields are part of the VBLANK interrupt
handling infrastructure, so move them to the appropriate section within
the structure.
Signed-off-by: Thierry Reding
---
include/drm/drmP.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --g
From: Thierry Reding
The file refers to a bunch of structure declared in drm_crtc.h, so
include it to make sure the drm_crtc_helper.h header can be included
standalone.
Signed-off-by: Thierry Reding
---
include/drm/drm_crtc_helper.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include
From: Thierry Reding
There is already a section that describes the helpers implemented by
this module. Add the kerneldoc-generated structure descriptions to this
section.
While at it, add missing kerneldoc for the structures to avoid warnings
when generating the documentation.
Signed-off-by: Th
https://bugzilla.kernel.org/show_bug.cgi?id=89961
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment #1 fr
On Wed, Dec 17, 2014 at 6:56 AM, wrote:
> From: Ville Syrjälä
>
> We had a bug in i915 land recently where X passed in a zeroed mode with
> mode_valid=1 to setcrtc. That didn't go down so well and caused a
> div-by-zero in i915.
>
> For a long time I've been thinking that we need some real che
On Wed, Dec 17, 2014 at 12:09:47PM -0500, Alex Deucher wrote:
> On Wed, Dec 17, 2014 at 6:56 AM, wrote:
> > From: Ville Syrjälä
> >
> > We had a bug in i915 land recently where X passed in a zeroed mode with
> > mode_valid=1 to setcrtc. That didn't go down so well and caused a
> > div-by-zero
On Wed, Dec 17, 2014 at 04:41:40PM +0100, Thierry Reding wrote:
> From: Thierry Reding
>
> The struct drm_connector_funcs kerneldoc refers to a part of struct
> drm_crtc_funcs that no longer exists.
>
> Signed-off-by: Thierry Reding
All four pulled into drm misc branch, thanks.
-Daniel
> ---
Hi Dave,
Yeah a pull for one patch is a bit overkill but I started to assemble the
various patches for 3.20 in a branch for atomic props/ioctl and didn't
realize that this bugfix here at the beginnning of the branch should be in
3.19 (because msm is using the helpers arleady). So if you'd merge we
town-smoke-201412170928'
# extra tests on tree/branch robclark/atomic-properties
git bisect bad 820e68c3d3fb9c0ea036515c2e8535bc017c79fd # 15:20 0-
22 RFC: drm: Atomic modeset ioctl
# extra tests on tree/branch linus/master
git bisect good 603ba7e41bf5d405aba22294af5d075d8898176d # 15:29 60+
On 12/17/2014 10:37 AM, Ville Syrjälä wrote:
> On Wed, Dec 17, 2014 at 03:57:51AM +0100, Mario Kleiner wrote:
>> On 12/15/2014 04:56 PM, Thierry Reding wrote:
>>> From: Thierry Reding
>>>
>>> The QXL driver duplicates part of the core's drm_vblank_count(), so it
>>> might as well use the core's
https://bugzilla.kernel.org/show_bug.cgi?id=85421
--- Comment #22 from Hin-Tak Leung ---
Am just following the advice Alex gave in comment 2(try different versions of
mesa) and reporting on my experience.
It would appear that my problem is orthogonal to what the commit "radeonsi:
Disable asynchr
On Wed, Dec 17, 2014 at 10:39 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> Make drm_fb_helper_initial_config() return an int rather than a bool so
> that the error can be properly propagated. While at it, update drivers
> to propagate errors further rather than just ignore them.
>
> Cc: D
On Wed, Dec 17, 2014 at 10:41 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> There is already a section that describes the helpers implemented by
> this module. Add the kerneldoc-generated structure descriptions to this
> section.
>
> While at it, add missing kerneldoc for the structures to
On Wed, Dec 17, 2014 at 10:41 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> The .irq and .irq_enabled fields are part of the VBLANK interrupt
> handling infrastructure, so move them to the appropriate section within
> the structure.
>
> Signed-off-by: Thierry Reding
Reviewed-by: Alex Deu
On Wed, Dec 17, 2014 at 4:39 PM, Thierry Reding
wrote:
> From: Thierry Reding
>
> Make drm_fb_helper_initial_config() return an int rather than a bool so
> that the error can be properly propagated. While at it, update drivers
> to propagate errors further rather than just ignore them.
>
> Cc: Da
https://bugzilla.kernel.org/show_bug.cgi?id=89661
--- Comment #16 from Bernd Steinhauser ---
Thanks, I'll give it a try.
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=66761
KernelBug <3fdd1e5d at opayq.com> changed:
What|Removed |Added
CC||3fdd1e5d at opayq.com
https://bugzilla.kernel.org/show_bug.cgi?id=89961
--- Comment #2 from swoorupj at gmail.com ---
Created attachment 161091
--> https://bugzilla.kernel.org/attachment.cgi?id=161091&action=edit
dmesg output
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=89961
--- Comment #3 from swoorupj at gmail.com ---
Created attachment 161101
--> https://bugzilla.kernel.org/attachment.cgi?id=161101&action=edit
lspci -vnn output
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=89961
--- Comment #4 from swoorupj at gmail.com ---
Created attachment 16
--> https://bugzilla.kernel.org/attachment.cgi?id=16&action=edit
Xorg log file
--
You are receiving this mail because:
You are watching the assignee of the bug.
Hello Laurent and Tomi,
On 12/16/2014 12:37 AM, Laurent Pinchart wrote:
> Hi Javier,
>> Tomi and Laurent,
>>
>> You asked Ajay to change his series to use the video port and enpoints DT
>> bindings instead of phandles, could you please review his latest version?
>>
>> I guess is now too late for
dan carpenter
I've had a patch in my tree for almost two weeks now, I wonder why this
is still there...
Ah, forgot to push it of course. Done now.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size:
bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141217/882e2e69/attachment.sig>
92 matches
Mail list logo
