DRM is pretty much the main user of this stuff (if we ignore the sysfs
interface used by X, i.e. by the same gang of people writing the drm
drivers). So shovel it into Dave's responsibility to avoid patches
getting lost on lkml.
With this get_maintainers will rout vgaarb and switcheroo patches
cor
in XXX sec" instead of "1 frame").
With older mesa egltri_screen still causes gpu resets and garbage.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20140526/5969df19/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=70411
Christoph Haag changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Mon, May 26, 2014 at 7:42 PM, Alexandre Courbot wrote:
> On Tue, May 27, 2014 at 10:07 AM, St?phane Marchesin
> wrote:
>> On Mon, May 26, 2014 at 5:02 PM, Alexandre Courbot
>> wrote:
>>> On Mon, May 26, 2014 at 6:21 PM, Lucas Stach
>>> wrote:
Am Montag, den 26.05.2014, 09:45 +0300 sch
https://bugzilla.kernel.org/show_bug.cgi?id=76861
--- Comment #8 from Sven Dziadek ---
So, I tried different values. First, I was not paying enough attention. Because
with value 112 and 113, there are very few lines appearing.
Value 111 is the one where I did not see any lines until now.
[drm:ra
On Mon, 26 May 2014, Ilia Mirkin wrote:
> On Mon, May 26, 2014 at 9:35 AM, Jani Nikula wrote:
>> Generated using semantic patch:
>>
>> @@
>> expression E;
>> @@
>>
>> - drm_get_connector_name(E)
>> + E->name
>>
>> Signed-off-by: Jani Nikula
>> ---
>> drivers/gpu/drm/nouveau/dispnv04/dac.c
On Mon, May 26, 2014 at 05:26:48PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Now that the vblank races are plugged, we can opt out of using
> the vblank disable timer and just let vblank interrupts get
> disabled immediately when the last reference is dropped.
>
>
On Mon, May 26, 2014 at 4:19 PM, Philipp Zabel
wrote:
[...]
> Cc: drm-devel at vger.kernel.org
Sorry about this unfortunate combination of me misremembering
dri-devel, noticing something is wrong, looking up and inserting the
correct address, and then, after a context switch, failing to remove
On Sat, May 24, 2014 at 02:30:23PM -0400, Rob Clark wrote:
> Signed-off-by: Rob Clark
> ---
> drivers/gpu/drm/msm/Makefile | 1 +
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | 57 ++--
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c | 6 ++
> drivers/gpu/drm/msm/mdp/mdp4/mdp
On Mon, May 26, 2014 at 6:12 PM, Rob Clark wrote:
> On Mon, May 26, 2014 at 11:24 AM, Daniel Vetter wrote:
>> On Mon, May 26, 2014 at 08:48:52AM -0400, Rob Clark wrote:
>>> On Mon, May 26, 2014 at 6:40 AM, Daniel Vetter wrote:
>>> > On Sat, May 24, 2014 at 02:30:09PM -0400, Rob Clark wrote:
>>>
ists.freedesktop.org/archives/dri-devel/attachments/20140526/12cb43d1/attachment.html>
From: Ville Syrj?l?
Make sure drm_vblank_get() never succeeds when called between
drm_vblank_off() and drm_vblank_on(). Borrow a trick from the
old drm_vblank_{pre,post}_modeset() functions and just bump
the refcount in drm_vblank_off() and drop it in drm_vblank_on().
When drm_vblank_get() encou
On Fri, May 23, 2014 at 6:24 PM, Lucas Stach wrote:
>> The best way to solve this issue would be to not use the BAR at all
>> since the memory behind these objects can be directly accessed by the
>> CPU. As such it would better be mapped using ttm_bo_kmap_ttm()
>> instead. But right now this is cl
On Mon, May 26, 2014 at 05:37:57PM +0200, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 06:23:05PM +0300, Ville Syrj?l? wrote:
> > On Mon, May 26, 2014 at 11:31:10AM +0200, Daniel Vetter wrote:
> > > On Sat, May 24, 2014 at 02:30:21PM -0400, Rob Clark wrote:
> > > > @@ -92,7 +100,18 @@ int drm_ato
On Mon, May 26, 2014 at 11:31:10AM +0200, Daniel Vetter wrote:
> On Sat, May 24, 2014 at 02:30:21PM -0400, Rob Clark wrote:
> > Break the mutable state of a crtc out into a separate structure
> > and use atomic properties mechanism to set crtc attributes. This
> > makes it easier to have some help
On Mon, May 26, 2014 at 06:46:30PM +0300, Ville Syrj?l? wrote:
> On Mon, May 26, 2014 at 05:37:57PM +0200, Daniel Vetter wrote:
> > On Mon, May 26, 2014 at 06:23:05PM +0300, Ville Syrj?l? wrote:
> > > On Mon, May 26, 2014 at 11:31:10AM +0200, Daniel Vetter wrote:
> > > > On Sat, May 24, 2014 at 02:
On Mon, May 26, 2014 at 11:49:59AM -0400, Rob Clark wrote:
> On Mon, May 26, 2014 at 11:35 AM, Daniel Vetter wrote:
> > On Mon, May 26, 2014 at 11:20:49AM -0400, Rob Clark wrote:
> >> On Mon, May 26, 2014 at 11:07 AM, Daniel Vetter wrote:
> >> > On Mon, May 26, 2014 at 5:04 PM, Rob Clark
> >> >
On Mon, May 26, 2014 at 5:02 PM, Alexandre Courbot wrote:
> On Mon, May 26, 2014 at 6:21 PM, Lucas Stach
> wrote:
>> Am Montag, den 26.05.2014, 09:45 +0300 schrieb Terje Bergstr?m:
>>> On 23.05.2014 17:40, Alex Courbot wrote:
>>> > On 05/23/2014 06:59 PM, Lucas Stach wrote:
>>> > So after checki
On Mon, May 26, 2014 at 02:46:32PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> The new watermaek update mechanism requires interrupts to work
> correctly. Because of this we need interrupts while disabling crtcs
> during suspend. So move the irq disable to happen a b
On Mon, May 26, 2014 at 06:23:05PM +0300, Ville Syrj?l? wrote:
> On Mon, May 26, 2014 at 11:31:10AM +0200, Daniel Vetter wrote:
> > On Sat, May 24, 2014 at 02:30:21PM -0400, Rob Clark wrote:
> > > @@ -92,7 +100,18 @@ int drm_atomic_set_event(struct drm_device *dev,
> > > struct drm_atomic
On Mon, May 26, 2014 at 11:20:49AM -0400, Rob Clark wrote:
> On Mon, May 26, 2014 at 11:07 AM, Daniel Vetter wrote:
> > On Mon, May 26, 2014 at 5:04 PM, Rob Clark wrote:
> >> hmm, ok, I had thought this case, thread B would get -EDEADLK because
> >> lock was held, and not his acquire ctx. If tha
ug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20140526/13503678/attachment.html>
On Mon, May 26, 2014 at 05:26:48PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Now that the vblank races are plugged, we can opt out of using
> the vblank disable timer and just let vblank interrupts get
> disabled immediately when the last reference is dropped.
>
>
From: Ville Syrj?l?
Now that the vblank races are plugged, we can opt out of using
the vblank disable timer and just let vblank interrupts get
disabled immediately when the last reference is dropped.
Gen2 is the exception since it has no hardware frame counter.
Signed-off-by: Ville Syrj?l?
---
From: Ville Syrj?l?
Add a flag to drm_device which will cause the vblank code to bypass the
disable timer and always disable the vblank interrupt immediately when
the last reference is dropped.
Signed-off-by: Ville Syrj?l?
---
drivers/gpu/drm/drm_irq.c | 6 +++---
include/drm/drmP.h|
On Mon, May 26, 2014 at 08:48:52AM -0400, Rob Clark wrote:
> On Mon, May 26, 2014 at 6:40 AM, Daniel Vetter wrote:
> > On Sat, May 24, 2014 at 02:30:09PM -0400, Rob Clark wrote:
> >> One more time, with feeling..
> >>
> >> Previous revision of series:
> >> http://lists.freedesktop.org/archives/dri
This issue was reported by coccicheck using the semantic patch
at scripts/coccinelle/api/memdup.cocci
Signed-off-by: Benoit Taine
---
Tested by compilation without errors.
drivers/gpu/drm/drm_edid.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_ed
These patches enhance kernel style usage, and allows smaller code while
preventing accidental code edits to produce overflows.
The semantic patch at scripts/coccinelle/api/memdup.cocci was used to
detect and edit this situation.
On Mon, May 26, 2014 at 5:04 PM, Rob Clark wrote:
> hmm, ok, I had thought this case, thread B would get -EDEADLK because
> lock was held, and not his acquire ctx. If that is not the case, then
> I would propose this:
>
> All places doing things the old way, must grab mode_config.mutex first
> cu
Hi
On Mon, May 26, 2014 at 3:35 PM, Jani Nikula wrote:
> As requested by David [1],[2].
>
> These are on top of drm-intel-nightly which carries the required core
> patches adding ->name field to drm_connector and drm_encoder. The i915
> patches could go in right away, the last patch need to wait
Hi YoungJun,
I am not famiilar with i80. Reading through this patch, it looks like
it works something like below for a page flip...
-> page_flip ioctl
exynos_drm_crtc_page_flip()
exynos_drm_crtc_mode_set_commit()
exynos_plane_mode_set()
exynos_drm_crtc_commit()
exynos_plane_commi
On Mon, May 26, 2014 at 07:35:45AM -0400, Rob Clark wrote:
> On Mon, May 26, 2014 at 5:31 AM, Daniel Vetter wrote:
> >> +struct drm_crtc_state *
> >> +drm_atomic_get_crtc_state(struct drm_crtc *crtc, struct drm_atomic_state
> >> *a)
> >> +{
> >> + struct drm_crtc_state *cstate;
> >> + int
OMAP DSS hardware supports changing the output port to which an overlay
manager's video stream goes. For example, DPI video stream can come from
any of the four overlay managers on OMAP5.
However, as it's difficult to manage the change in the driver, the
omapdss driver does not support that at the
On Mon, May 26, 2014 at 07:32:13AM -0400, Rob Clark wrote:
> On Mon, May 26, 2014 at 5:12 AM, Daniel Vetter wrote:
> > On Sat, May 24, 2014 at 02:30:20PM -0400, Rob Clark wrote:
> >> Break the mutable state of a plane out into a separate structure
> >> and use atomic properties mechanism to set pl
From: Ville Syrj?l?
v2: Drop the drm_vblank_off() (Daniel)
Use drm_crtc_vblank_{get,put}()
Signed-off-by: Ville Syrj?l?
---
drivers/gpu/drm/i915/intel_display.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/inte
On Mon, May 26, 2014 at 04:35:04PM +0200, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 07:56:50AM -0400, Rob Clark wrote:
> > On Mon, May 26, 2014 at 4:23 AM, Daniel Vetter wrote:
> > > On Sun, May 25, 2014 at 07:16:43PM -0400, Rob Clark wrote:
> > >> On Sun, May 25, 2014 at 6:10 PM, Daniel Vett
On Mon, May 26, 2014 at 03:22:45PM +0200, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 02:46:25PM +0300, ville.syrjala at linux.intel.com
> wrote:
> > From: Ville Syrj?l?
> >
> > Signed-off-by: Ville Syrj?l?
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 15 +++
> > 1 file ch
No longer used or needed as the structs have a name field.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_crtc.c | 20
include/drm/drm_crtc.h | 2 --
2 files changed, 22 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
index d986b3f
Generated using semantic patch:
@@
expression E;
@@
- drm_get_encoder_name(E)
+ E->name
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_crtc.c| 2 +-
drivers/gpu/drm/drm_crtc_helper.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc.c b
Generated using semantic patch:
@@
expression E;
@@
- drm_get_encoder_name(E)
+ E->name
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/i915_debugfs.c | 2 +-
drivers/gpu/drm/i915/intel_display.c | 14 +++---
drivers/gpu/drm/i915/intel_dp.c | 2 +-
3 files changed, 9 insert
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E->name
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/drm_crtc.c | 4 ++--
drivers/gpu/drm/drm_crtc_helper.c | 6 +++---
drivers/gpu/drm/drm_edid.c | 6 +++---
drivers/gpu/drm/drm_edid_load.c
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E->name
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/radeon/radeon_connectors.c | 19 ---
drivers/gpu/drm/radeon/radeon_display.c| 2 +-
2 files changed, 13 insertions(+), 8 deletions(-)
d
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E->name
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/nouveau/dispnv04/dac.c | 2 +-
drivers/gpu/drm/nouveau/dispnv04/dfp.c | 2 +-
drivers/gpu/drm/nouveau/dispnv04/disp.c | 2 +-
drivers/gpu/drm/n
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E->name
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/i915_debugfs.c | 4 ++--
drivers/gpu/drm/i915/i915_irq.c | 8
drivers/gpu/drm/i915/intel_crt.c | 2 +-
drivers/gpu/drm/i915/intel_
As requested by David [1],[2].
These are on top of drm-intel-nightly which carries the required core
patches adding ->name field to drm_connector and drm_encoder. The i915
patches could go in right away, the last patch need to wait for radeon
and nouveau patches to get merged, obviously.
BR,
Jani
On Mon, May 26, 2014 at 07:56:50AM -0400, Rob Clark wrote:
> On Mon, May 26, 2014 at 4:23 AM, Daniel Vetter wrote:
> > On Sun, May 25, 2014 at 07:16:43PM -0400, Rob Clark wrote:
> >> On Sun, May 25, 2014 at 6:10 PM, Daniel Vetter wrote:
> >> > On Sat, May 24, 2014 at 8:30 PM, Rob Clark
> >> > w
On Mon, May 26, 2014 at 03:21:41PM +0200, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 02:46:24PM +0300, ville.syrjala at linux.intel.com
> wrote:
> > From: Ville Syrj?l?
> >
> > Make sure drm_vblank_get() never succeeds when called between
> > drm_vblank_off() and drm_vblank_on(). Borrow a tr
This patch registers the two CSI platform devices per IPU.
Signed-off-by: Lucas Stach
Signed-off-by: Philipp Zabel
---
Changes since RFC:
- Rebased onto current staging-next
---
drivers/gpu/ipu-v3/ipu-common.c | 37 -
include/video/imx-ipu-v3.h | 1 +
IPU_CONF_..._EN bits are implementation details, not to be made public.
Add wrappers around ipu_module_enable/disable, so the CSI V4L2 driver
can enable/disable the CSI and SMFC modules.
Signed-off-by: Philipp Zabel
---
drivers/gpu/ipu-v3/ipu-common.c | 24
include/video
This function returns the currently active buffer (0 or 1)
of a double buffered IDMAC channel. It is to be used by the
CSI driver.
Signed-off-by: Philipp Zabel
---
drivers/gpu/ipu-v3/ipu-common.c | 9 +
include/video/imx-ipu-v3.h | 1 +
2 files changed, 10 insertions(+)
diff --git
The Sensor Multi Fifo Controller (SMFC) is used as a buffer between
the two CSIs (writing simultaneously) and up to four IDMAC channels.
Signed-off-by: Philipp Zabel
---
drivers/gpu/ipu-v3/Makefile | 2 +-
drivers/gpu/ipu-v3/ipu-common.c | 10 +
drivers/gpu/ipu-v3/ipu-prv.h| 6 +++
The i.MX Image Processing Unit (IPU) contains a number of image processing
blocks that sit right in the middle between DRM and V4L2. Some of the modules,
such as Display Controller, Processor, and Interface (DC, DP, DI) or CMOS
Sensor Interface (CSI) and their FIFOs could be assigned to either fram
Hi,
This is a rebased version of the earlier RFC series. It is
mostly about the first patch, which moves the IPUv3 core code
(drivers/staging/imx-drm/ipu-v3) to drivers/gpu.
host1x, which serves a similar purpose, already sits there.
I'd like to move the IPUv3 core code out of staging so that we
Am 26.05.2014 15:52, schrieb Daniel Vetter:
> On Mon, May 26, 2014 at 3:52 PM, Daniel Vetter wrote:
>> On Mon, May 26, 2014 at 04:35:39PM +0300, Jani Nikula wrote:
>>> As requested by David [1],[2].
>>>
>>> These are on top of drm-intel-nightly which carries the required core
>>> patches adding ->
On Mon, May 26, 2014 at 04:49:28PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> v2: Drop the drm_vblank_off() (Daniel)
> Use drm_crtc_vblank_{get,put}()
>
> Signed-off-by: Ville Syrj?l?
Reviewed-by: Daniel Vetter
> ---
> drivers/gpu/drm/i915/intel_display.c
On Mon, May 26, 2014 at 3:52 PM, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 04:35:39PM +0300, Jani Nikula wrote:
>> As requested by David [1],[2].
>>
>> These are on top of drm-intel-nightly which carries the required core
>> patches adding ->name field to drm_connector and drm_encoder. The i9
On Mon, May 26, 2014 at 04:35:39PM +0300, Jani Nikula wrote:
> As requested by David [1],[2].
>
> These are on top of drm-intel-nightly which carries the required core
> patches adding ->name field to drm_connector and drm_encoder. The i915
> patches could go in right away, the last patch need to w
On Mon, May 26, 2014 at 04:36:26PM +0300, Ville Syrj?l? wrote:
> On Mon, May 26, 2014 at 03:22:45PM +0200, Daniel Vetter wrote:
> > On Mon, May 26, 2014 at 02:46:25PM +0300, ville.syrjala at linux.intel.com
> > wrote:
> > > From: Ville Syrj?l?
> > >
> > > Signed-off-by: Ville Syrj?l?
> > > ---
On Mon, May 26, 2014 at 04:32:02PM +0300, Ville Syrj?l? wrote:
> On Mon, May 26, 2014 at 03:21:41PM +0200, Daniel Vetter wrote:
> > On Mon, May 26, 2014 at 02:46:24PM +0300, ville.syrjala at linux.intel.com
> > wrote:
> > > From: Ville Syrj?l?
> > >
> > > Make sure drm_vblank_get() never succeed
Updated drm documentation to include description of aspect ratio property.
HTML table structure created by Sagar Kamble.
Signed-off-by: Vandana Kannan
Cc: Sagar Kamble
Cc: Daniel Vetter
Cc: Thierry Reding
---
Documentation/DocBook/drm.tmpl | 31 +++
1 file changed,
Create and attach the drm property to set aspect ratio. If there is no user
specified value, then PAR_NONE/Automatic option is set by default. User can
select aspect ratio 4:3 or 16:9. The aspect ratio selected by user would
come into effect with a mode set.
v2: Modifications made based on changes
next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20140526/600e460a/attachment-0001.sig>
On Mon, May 26, 2014 at 2:59 PM, Rahul Sharma
wrote:
>
> Hi Inki,
>
> Please review this patch.
>
> Regards,
> Rahul Sharma
>
> On 23 May 2014 17:17, Rahul Sharma wrote:
> > From: Rahul Sharma
> >
> > Fimd probe is accessing fimd Registers without enabling the fimd
> > gate clocks. If FIMD cloc
In case user has specified an input for aspect ratio through the property,
then the user space value for PAR would take preference over the value from
CEA mode list.
v2: Thierry's review comments.
- Modified the comment "Populate..." as per review comments
Signed-off-by: Vandana Kannan
C
Added a property to enable user space to set aspect ratio.
This patch contains declaration of the property and code to create the
property.
v2: Thierry's review comments.
- Made aspect ratio enum generic instead of HDMI/CEA specfic
- Removed usage of temporary aspect_ratio variable
On Mon, May 26, 2014 at 02:46:31PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Declare a local struct drm_vblank_crtc * and use that
> instead of having to do dig it out via 'dev->vblank[crtc]'
> everywhere.
>
> Performed with the following coccinelle incantation,
>
Hi Jean,
Thank you for the patch.
On Monday 26 May 2014 14:52:13 Jean Delvare wrote:
> The shmobile DRM driver is only useful on SuperH and shmobile unless
> build testing. I am dropping the SuperH dependencies though because
> the driver doesn't even build there, so in practice it is an arm-only
On Mon, May 26, 2014 at 02:46:29PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> When drm_vblank_on() is called the hardware vblank counter may have
> been reset, so we can't trust that the old values sampled prior to
> drm_vblank_off() have anything to do with the new
On Mon, May 26, 2014 at 02:46:28PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> If the vblank irq has already been disabled (via the disable timer) when
> we call drm_vblank_off() sample the counter and timestamp one last time.
> This will make the sure that the user
On Mon, May 26, 2014 at 02:46:26PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Clearing the timestamps causes us to send zeroed timestamps to userspace
> if they get sent out in response to the drm_vblank_off(). It's better
> to send the very latest timestamp and cou
On Mon, May 26, 2014 at 02:46:25PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Signed-off-by: Ville Syrj?l?
> ---
> drivers/gpu/drm/i915/intel_display.c | 15 +++
> 1 file changed, 15 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c
On Mon, May 26, 2014 at 02:46:24PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Make sure drm_vblank_get() never succeeds when called between
> drm_vblank_off() and drm_vblank_on(). Borrow a trick from the
> old drm_vblank_{pre,post}_modeset() functions and just bump
On Mon, May 26, 2014 at 02:46:30PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrj?l?
>
> Make drm_vblank_put() disable the vblank interrupt immediately when the
> refcount drops to zero and drm_vblank_offdelay==0.
>
> Currently drm_vblank_put() would just leave vblank interru
The shmobile DRM driver is only useful on SuperH and shmobile unless
build testing. I am dropping the SuperH dependencies though because
the driver doesn't even build there, so in practice it is an arm-only
driver for now.
Signed-off-by: Jean Delvare
Cc: Laurent Pinchart
Cc: David Airlie
---
d
Hi Jean,
On Monday 26 May 2014 14:21:30 Jean Delvare wrote:
> On Mon, 26 May 2014 13:36:11 +0200, Laurent Pinchart wrote:
> > On Monday 26 May 2014 13:01:35 Jean Delvare wrote:
> > > The Renesas R-Car Display Unit driver is only useful on shmobile
> > > unless build testing. The LVDS output is use
From: Ville Syrj?l?
The new watermaek update mechanism requires interrupts to work
correctly. Because of this we need interrupts while disabling crtcs
during suspend. So move the irq disable to happen a bit later.
This also avoid clobbering the vblank.last count in case the
vblank interrupt was
From: Ville Syrj?l?
Declare a local struct drm_vblank_crtc * and use that
instead of having to do dig it out via 'dev->vblank[crtc]'
everywhere.
Performed with the following coccinelle incantation,
and a few manual whitespace cleanups:
@@
identifier func,member;
expression num_crtcs;
struct drm
From: Ville Syrj?l?
Make drm_vblank_put() disable the vblank interrupt immediately when the
refcount drops to zero and drm_vblank_offdelay==0.
Currently drm_vblank_put() would just leave vblank interrupts enabled all
the time if drm_vblank_offdelay==0. In case someone might still want that
behav
From: Ville Syrj?l?
When drm_vblank_on() is called the hardware vblank counter may have
been reset, so we can't trust that the old values sampled prior to
drm_vblank_off() have anything to do with the new values.
So update the .last count in drm_vblank_on() to make the first
drm_vblank_enable()
From: Ville Syrj?l?
If the vblank irq has already been disabled (via the disable timer) when
we call drm_vblank_off() sample the counter and timestamp one last time.
This will make the sure that the user space visible counter will account
for time between vblank irq disable and drm_vblank_off().
From: Ville Syrj?l?
Move drm_update_vblank_count() to avoid forward a declaration.
No functional change.
Signed-off-by: Ville Syrj?l?
---
drivers/gpu/drm/drm_irq.c | 128 +++---
1 file changed, 64 insertions(+), 64 deletions(-)
diff --git a/drivers/gpu/
From: Ville Syrj?l?
Clearing the timestamps causes us to send zeroed timestamps to userspace
if they get sent out in response to the drm_vblank_off(). It's better
to send the very latest timestamp and count instead.
Signed-off-by: Ville Syrj?l?
---
drivers/gpu/drm/drm_irq.c | 11 ---
1
From: Ville Syrj?l?
Signed-off-by: Ville Syrj?l?
---
drivers/gpu/drm/i915/intel_display.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 3da73ef..da318a7 100644
--- a/drivers/gpu/drm/i915/in
From: Ville Syrj?l?
Make sure drm_vblank_get() never succeeds when called between
drm_vblank_off() and drm_vblank_on(). Borrow a trick from the
old drm_vblank_{pre,post}_modeset() functions and just bump
the refcount in drm_vblank_off() and drop it in drm_vblank_on().
Hopefully the use of inmode
From: Ville Syrj?l?
Another vblank series with the following features:
- Plug a race between drm_vblank_off() and marking the crtc inactive
- Don't send zeroed vblank evens to userspace at drm_vblank_off()
- Have the user visible vblank counter account the entire time
when the crtc was active,
> Woody
>>
>>
>
> I pulled git://people.freedesktop.org/~deathsimple/linux drm-fixes-3.15
> from http://article.gmane.org/gmane.comp.video.dri.devel/106114 and my
> problem is gone ...
>
Also fixed in v3.15-rc7.
/Malte
-- next part --
A
Hi Laurent,
On Mon, 26 May 2014 13:36:11 +0200, Laurent Pinchart wrote:
> Thank you for the patch.
You're welcome. And thanks for your help :-)
> On Monday 26 May 2014 13:01:35 Jean Delvare wrote:
> > The Renesas R-Car Display Unit driver is only useful on shmobile
> > unless build testing. The
Hi Daniel,
On 26 May 2014 13:11, Daniel Kurtz wrote:
> On Mon, May 26, 2014 at 2:59 PM, Rahul Sharma
> wrote:
>>
>> Hi Inki,
>>
>> Please review this patch.
[snip]
>> > +
>> > + ret = clk_prepare_enable(ctx->lcd_clk);
>
> Hi Rahul,
>
> Can you explain why exactly we are "clearing windows"
https://bugzilla.kernel.org/show_bug.cgi?id=76861
--- Comment #7 from Christian K?nig ---
(In reply to Sven Dziadek from comment #5)
> Created attachment 137421 [details]
> patched with value 107
>
> Value 107 works for me. I am currently testing which is the highest value
> working for me. I th
https://bugzilla.kernel.org/show_bug.cgi?id=76861
--- Comment #6 from Sven Dziadek ---
Created attachment 137431
--> https://bugzilla.kernel.org/attachment.cgi?id=137431&action=edit
dmesg output for (still working) Kernel 3.15-rc1
--
You are receiving this mail because:
You are watching the a
https://bugzilla.kernel.org/show_bug.cgi?id=76861
--- Comment #5 from Sven Dziadek ---
Created attachment 137421
--> https://bugzilla.kernel.org/attachment.cgi?id=137421&action=edit
patched with value 107
Value 107 works for me. I am currently testing which is the highest value
working for me.
Hi Jean,
Thank you for the patch.
On Monday 26 May 2014 13:01:35 Jean Delvare wrote:
> The Renesas R-Car Display Unit driver is only useful on shmobile
> unless build testing. The LVDS output is useful on an even more
> reduced hardware set.
>
> Signed-off-by: Jean Delvare
> Cc: Laurent Pinchar
Hi,
On 05/26/2014 01:03 PM, Rafael J. Wysocki wrote:
> On Monday, May 26, 2014 12:03:43 PM Jingoo Han wrote:
>> On Thursday, May 22, 2014 6:02 PM, Lee Jones wrote:
>>> On Thursday, May 22, 2014 5:45 PM, Hans de Goede wrote:
On Thursday, May 22, 2014 8:31 AM, Rafael J. Wysocki wrote:
> On
de
> > +
> > +/*
> > + * Single "mode" entry. This describes one set of signal timings a
> > display can + * have in one setting. This struct can later be converted
> > to struct cmdmode + * (see include/video/cmdmode.h).
> > + */
> > +struct cmdmode_display_timing {
> > + u32 pixelclock;
> > +
> > + u32 hactive;
> > + u32 vactive;
> > +
> > + u32 cs_setup;
> > + u32 wr_setup;
> > + u32 wr_active;
> > + u32 wr_hold;
> > +};
> > +
> > +/*
> > + * This describes all timing settings a display provides.
> > + * The native_mode is the default setting for this display.
> > + * Drivers that can handle multiple cmdmodes should work with this struct
> > + * and convert each entry to the desired end result.
> > + */
> > +struct cmdmode_display_timings {
> > + unsigned int num_timings;
> > + unsigned int native_mode;
> > +
> > + struct cmdmode_display_timing **timings;
> > +};
> > +
> > +/* get one entry from struct cmdmode_display_timings */
> > +static inline struct cmdmode_display_timing*
> > +cmdmode_display_timings_get(const struct cmdmode_display_timings *cmdts,
> > + unsigned int index)
> > +{
> > + if (cmdts->num_timings > index)
> > + return cmdts->timings[index];
> > + else
> > + return NULL;
> > +}
> > +
> > +void cmdmode_display_timings_release(struct cmdmode_display_timings
> > *cmdts); +
> > +#endif /* __LINUX_CMDDMODE_DISPLAY_TIMING_H */
> > diff --git a/include/video/of_cmdmode.h b/include/video/of_cmdmode.h
> > new file mode 100644
> > index 000..fb7c6c7
> > --- /dev/null
> > +++ b/include/video/of_cmdmode.h
> > @@ -0,0 +1,19 @@
> > +/*
> > + * Copyright 2014 YoungJun Cho
> > + *
> > + * cmdmode of-helpers
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + */
> > +
> > +#ifndef __LINUX_OF_CMDMODE_H
> > +#define __LINUX_OF_CMDMODE_H
> > +
> > +struct device_node;
> > +struct cmdmode;
> > +
> > +int of_get_cmdmode(struct device_node *np, struct cmdmode *cm, int
> > index);
> > +
> > +#endif /* __LINUX_OF_CMDMODE_H */
> > diff --git a/include/video/of_cmdmode_display_timing.h
> > b/include/video/of_cmdmode_display_timing.h new file mode 100644
> > index 000..6be91ba
> > --- /dev/null
> > +++ b/include/video/of_cmdmode_display_timing.h
> > @@ -0,0 +1,26 @@
> > +/*
> > + * Copyright 2014 YoungJun Cho
> > + *
> > + * cmdmode display timings of helpers
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + */
> > +
> > +#ifndef __LINUX_OF_CMDMODE_DISPLAY_TIMING_H
> > +#define __LINUX_OF_CMDMODE_DISPLAY_TIMING_H
> > +
> > +struct device_node;
> > +struct cmdmode_display_timing;
> > +struct cmdmode_display_timings;
> > +
> > +#define OF_USE_CMDMODE_NATIVE_MODE -1
> > +
> > +int of_get_cmdmode_display_timing(struct device_node *np, const char
> > *name, +struct cmdmode_display_timing
> > *cmdt);
> > +struct cmdmode_display_timings*
> > + of_get_cmdmode_display_timings(struct device_node *np);
> > +int of_cmdmode_display_timings_exist(struct device_node *np);
> > +
> > +#endif /* __LINUX_OF_CMDMODE_DISPLAY_TIMING_H */
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20140526/e9af7502/attachment-0001.sig>
On Monday, May 26, 2014 12:03:43 PM Jingoo Han wrote:
> On Thursday, May 22, 2014 6:02 PM, Lee Jones wrote:
> > On Thursday, May 22, 2014 5:45 PM, Hans de Goede wrote:
> > > On Thursday, May 22, 2014 8:31 AM, Rafael J. Wysocki wrote:
> > > > On Wednesday, May 21, 2014 10:40 PM, Hans de Goede wrote:
The Renesas R-Car Display Unit driver is only useful on shmobile
unless build testing. The LVDS output is useful on an even more
reduced hardware set.
Signed-off-by: Jean Delvare
Cc: Laurent Pinchart
Cc: David Airlie
---
drivers/gpu/drm/rcar-du/Kconfig |2 ++
1 file changed, 2 insertions(+
On Sat, May 24, 2014 at 02:30:09PM -0400, Rob Clark wrote:
> One more time, with feeling..
>
> Previous revision of series:
> http://lists.freedesktop.org/archives/dri-devel/2014-March/055806.html
>
> And if you prefer, in git form:
> http://cgit.freedesktop.org/~robclark/linux/log/?h=cold-fusion
Hi Inki,
Please review this patch.
Regards,
Rahul Sharma
On 23 May 2014 17:17, Rahul Sharma wrote:
> From: Rahul Sharma
>
> Fimd probe is accessing fimd Registers without enabling the fimd
> gate clocks. If FIMD clocks are kept disabled in Uboot or disbaled
> during kernel boottime, the system
t; + return NULL;
> +}
> +
> +void cmdmode_display_timings_release(struct cmdmode_display_timings *cmdts);
> +
> +#endif /* __LINUX_CMDDMODE_DISPLAY_TIMING_H */
> diff --git a/include/video/of_cmdmode.h b/include/video/of_cmdmode.h
> new file mode 100644
> index 000..fb7c6c7
> --- /dev/null
> +++ b/include/video/of_cmdmode.h
> @@ -0,0 +1,19 @@
> +/*
> + * Copyright 2014 YoungJun Cho
> + *
> + * cmdmode of-helpers
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#ifndef __LINUX_OF_CMDMODE_H
> +#define __LINUX_OF_CMDMODE_H
> +
> +struct device_node;
> +struct cmdmode;
> +
> +int of_get_cmdmode(struct device_node *np, struct cmdmode *cm, int index);
> +
> +#endif /* __LINUX_OF_CMDMODE_H */
> diff --git a/include/video/of_cmdmode_display_timing.h
> b/include/video/of_cmdmode_display_timing.h
> new file mode 100644
> index 000..6be91ba
> --- /dev/null
> +++ b/include/video/of_cmdmode_display_timing.h
> @@ -0,0 +1,26 @@
> +/*
> + * Copyright 2014 YoungJun Cho
> + *
> + * cmdmode display timings of helpers
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#ifndef __LINUX_OF_CMDMODE_DISPLAY_TIMING_H
> +#define __LINUX_OF_CMDMODE_DISPLAY_TIMING_H
> +
> +struct device_node;
> +struct cmdmode_display_timing;
> +struct cmdmode_display_timings;
> +
> +#define OF_USE_CMDMODE_NATIVE_MODE -1
> +
> +int of_get_cmdmode_display_timing(struct device_node *np, const char *name,
> + struct cmdmode_display_timing *cmdt);
> +struct cmdmode_display_timings*
> + of_get_cmdmode_display_timings(struct device_node *np);
> +int of_cmdmode_display_timings_exist(struct device_node *np);
> +
> +#endif /* __LINUX_OF_CMDMODE_DISPLAY_TIMING_H */
> --
> 1.7.9.5
>
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20140526/479b6a2b/attachment-0001.sig>
On Mon, May 26, 2014 at 11:24 AM, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 08:48:52AM -0400, Rob Clark wrote:
>> On Mon, May 26, 2014 at 6:40 AM, Daniel Vetter wrote:
>> > On Sat, May 24, 2014 at 02:30:09PM -0400, Rob Clark wrote:
>> >> One more time, with feeling..
>> >>
>> >> Previous rev
1 - 100 of 163 matches
Mail list logo