On 28 April 2016 at 07:45, Emil Velikov wrote:
> On 27 April 2016 at 10:47, Russell King - ARM Linux
> wrote:
>> On Thu, Apr 21, 2016 at 09:17:13PM +0100, Emil Velikov wrote:
>>> Dave Airlie pointed out that "polluting" the headers in a manner as seen
>>> with this series might not be too wise. D
Anyway, although i would have liked the stricter check and warning docs,
the v4 patch is ok with me:
Reviewed-by: Mario Kleiner
-mario
On 04/25/2016 08:32 AM, Maarten Lankhorst wrote:
> This function is useful for gen2 intel devices which have no frame
> counter, but need a way to determine th
Hello Andrew,
On Wed, Apr 27, 2016 at 01:20:35PM -0700, Andrew Morton wrote:
> On Wed, 27 Apr 2016 16:48:13 +0900 Minchan Kim wrote:
>
> > Recently, I got many reports about perfermance degradation in embedded
> > system(Android mobile phone, webOS TV and so on) and easy fork fail.
> >
> > The
On Thu, 2016-04-14 at 16:28 +0200, Thierry Reding wrote:
> On Sun, Apr 03, 2016 at 12:20:45PM +0800, Jitao Shi wrote:
> [...]
> > diff --git a/drivers/gpu/drm/bridge/parade-ps8640.c
> > b/drivers/gpu/drm/bridge/parade-ps8640.c
> > new file mode 100644
> > index 000..87f8bc7
> > --- /dev/null
>
Hi Dave,
After merging the drm tree, today's linux-next build (x86_64 allmodconfig)
failed like this:
drivers/gpu/drm/i915/intel_ddi.c: In function 'intel_prepare_ddi_buffer':
drivers/gpu/drm/i915/intel_ddi.c:447:15: error: 'struct drm_i915_private' has
no member named 'edp_low_vswing'
if (de
Hi Dave,
Have fixed the module compilation error and warnings in this commit:
3d76c7e5bbdd drm/hisilicon: Add designware dsi encoder driver
Please help to pull the new tag drm-hisilicon-next-2016-04-28 and let
me know if there is any problem.
Thanks,
-xinliang
The following changes since commi
and glxinfo.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/03eef73d/attachment.html>
Hi Dave,
V2 has fixed the module compilation error and warnings in this commit:
3d76c7e5bbdd drm/hisilicon: Add designware dsi encoder driver
Please help to try and let me know if there is any problem.
Thanks,
-xinliang
The following changes since commit 152ef5fa9e14e93e7efc43adad7dbcf35d7780f5
On Wed, 27 Apr 2016, Lyude wrote:
> For MST encoders, the encoder struct is stored in the intel_dp_mst
> struct, not a intel_digital_port struct.
>
> This fixes issues with hotplugging MST displays that support MST audio,
> where hotplugging had a surprisingly good chance of accidentally
> overwri
On Wed, Apr 27, 2016 at 09:24:52PM +0200, Noralf Trønnes wrote:
>
> Den 27.04.2016 21:20, skrev Daniel Vetter:
> >On Wed, Apr 27, 2016 at 08:16:32PM +0200, Noralf Trønnes wrote:
> >>This adds deferred io support to drm_fb_helper.
> >>The fbdev framebuffer changes are flushed using the callback
>
On Wed, Apr 27, 2016 at 09:49:49PM +0200, Noralf Trønnes wrote:
>
> Den 27.04.2016 21:24, skrev Daniel Vetter:
> >On Wed, Apr 27, 2016 at 08:16:29PM +0200, Noralf Trønnes wrote:
> >>This patchset adds fbdev deferred io support to drm_fb_helper and
> >>drm_fb_cma_helper.
> >>
> >>It channels fbde
Hello Thomas Hellstrom,
The patch d80efd5cb3de: "drm/vmwgfx: Initial DX support" from Aug 10,
2015, leads to the following static checker warning:
drivers/gpu/drm/vmwgfx/vmwgfx_so.c:335 vmw_view_add()
error: buffer overflow 'vmw_view_define_sizes' 3 <= 3
drivers/gpu/drm/vmwgfx/vm
ector_unregister_all() here.
Thanks, I'll fix that.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/0d08c8d6/attachment.sig>
On Wednesday 27 April 2016 08:05 PM, Alexey Brodkin wrote:
> Allocation of a frame buffer memory in a special memory region
> allows bypassing of so-called IO Coherency aperture
> which is typically set as a range 0x8z-0xAz.
>
> I.e. all data traffic to PGU bypasses IO Coherency block
> and saves
On Wed, 27 Apr 2016, Mihai DonÈu wrote:
> On Wed, 27 Apr 2016 20:36:13 +0300 Mihai DonÈu wrote:
>> On Wed, 27 Apr 2016 08:28:20 -0700 Rodrigo Vivi wrote:
>> > Hi Mihai,
>> >
>> > What platform do you have? HSW or BDW?
>>
>> I have an i7, Haswell CPU.
>>
>> > If you don't know please provide
On Thu, 28 Apr 2016, Stephen Rothwell wrote:
> Hi Dave,
>
> After merging the drm tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
>
> drivers/gpu/drm/i915/intel_ddi.c: In function 'intel_prepare_ddi_buffer':
> drivers/gpu/drm/i915/intel_ddi.c:447:15: error: 'struct drm_i91
--
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/69e53def/attachment.sig>
On Wed, 27 Apr 2016, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä
>
> Let's be nice and interrupt the dpcd aux-dev reads/writes when there's
> a signal pending. Much nicer if the user can hit ^C instead of having to
> sit around waiting for the read/write to finish.
>
> time dd
On Wed, Apr 27, 2016 at 11:07:02AM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Debug logging in this function does not provide any information
> apart that the userspace is calling an ioctl on the connector.
>
> There is not any info on the connector provided at all and
> since there
/free-electrons.com
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/2d553697/attachment.sig>
On Wed, Apr 27, 2016 at 12:11:47PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> The debug logging here can be very verbose in the kernel logs
> and provides no information which userspace doesn't have the
> access to already. Turn it off so kernel logs become more
> manageable.
>
> Si
With NV12, NV21, NV16 and NV61 formats, the size of one pixel from the
UV plane (plane #1) is one byte.
Indeed, these pixel formats use 4:2:x chroma subsampling: the chroma
pixels are sampled at half the luma: for 2 pixels, there are 1 Cb +
1 Cr = 2 bytes.
So for plane #1, the correct size is actu
The newly added failure path in amdgpu_mn_get() use the
wrong return type:
drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c: In function 'amdgpu_mn_get':
drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c:237:10: error: return makes pointer from
integer without a cast
This adds the necessary ERR_PTR() conversion.
Si
this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/06853cff/attachment.html>
On Thu, Apr 28, 2016 at 11:49:12AM +0300, Jani Nikula wrote:
> On Wed, 27 Apr 2016, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Let's be nice and interrupt the dpcd aux-dev reads/writes when there's
> > a signal pending. Much nicer if the user can hit ^C instead of ha
ves/dri-devel/attachments/20160428/f15e42ed/attachment.sig>
Kenneth Graunke wrote on 28.04.2016 03:05:
> On Wednesday, April 27, 2016 9:37:07 AM PDT Thorsten Leemhuis wrote:
>> Thorsten Leemhuis wrote on 26.04.2016 13:41:
>> > Lo! Below patch adds the PCI-ID for the Intel(R) Iris Graphics 550
> (Skylake
>>> GT3e mobile) to libdrm. It afaics is the last pie
On Thu 28-04-16 11:33:48, Arnd Bergmann wrote:
> The newly added failure path in amdgpu_mn_get() use the
> wrong return type:
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c: In function 'amdgpu_mn_get':
> drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c:237:10: error: return makes pointer
> from integer withou
On Thu, Apr 28, 2016 at 09:17:00AM +0300, Jani Nikula wrote:
> On Wed, 27 Apr 2016, Lyude wrote:
> > For MST encoders, the encoder struct is stored in the intel_dp_mst
> > struct, not a intel_digital_port struct.
> >
> > This fixes issues with hotplugging MST displays that support MST audio,
> > w
On Thu, Apr 28, 2016 at 10:40:39AM +0200, Fabien Dessenne wrote:
> With NV12, NV21, NV16 and NV61 formats, the size of one pixel from the
> UV plane (plane #1) is one byte.
>
> Indeed, these pixel formats use 4:2:x chroma subsampling: the chroma
> pixels are sampled at half the luma: for 2 pixels,
https://bugzilla.kernel.org/show_bug.cgi?id=117181
yves.dufournaud at gmail.com changed:
What|Removed |Added
Regression|No |Yes
--
You are receiving
Add linux-fbdev diffusion list in loop for patch-set review.
On 04/21/2016 05:29 PM, Arnaud POULIQUEN wrote:
> This patchset implements audio interface in HDMI drm driver. Implementation
> is based on
> ASoC generic hdmi codec driver( https://patchwork.kernel.org/patch/8713141/).
> It also prop
Add documentation for DT properties supported by
ps8640 DSI-eDP converter.
Signed-off-by: Jitao Shi
Acked-by: Rob Herring
Reviewed-by: Philipp Zabel
---
Changes since v14:
- change mode-sel-gpios as optional.
---
.../devicetree/bindings/display/bridge/ps8640.txt | 44
This patch adds drm_bridge driver for parade DSI to eDP bridge chip.
Signed-off-by: Jitao Shi
Reviewed-by: Daniel Kurtz
---
Changes since v14:
- update copyright info.
- change bridge_to_ps8640 and connector_to_ps8640 to inline function.
- fix some coding style.
- use sizeof as array counter
Add linux-fbdev diffusion list in loop for patch-set review.
On 04/21/2016 05:29 PM, Arnaud POULIQUEN wrote:
> Add the interface needed by audio hdmi-codec driver.
>
> Signed-off-by: Arnaud Pouliquen
> Acked-by: Benjamin Gaignard
> Acked-by: Vincent ABRIOU
> ---
> drivers/gpu/drm/sti/Kconfig
Add linux-fbdev diffusion list in loop for patch-set review.
On 04/21/2016 05:29 PM, Arnaud POULIQUEN wrote:
> Add helper functions to compute HDMI CTS and N parameters.
> Implementation is based on HDMI 1.4b specification.
>
> Signed-off-by: Arnaud Pouliquen
> Acked-by: Benjamin Gaignard
> Ack
On Thu, 28 Apr 2016, Ville Syrjälä wrote:
> On Thu, Apr 28, 2016 at 09:17:00AM +0300, Jani Nikula wrote:
>> On Wed, 27 Apr 2016, Lyude wrote:
>> > For MST encoders, the encoder struct is stored in the intel_dp_mst
>> > struct, not a intel_digital_port struct.
>> >
>> > This fixes issues with ho
On Wed, Apr 27, 2016 at 04:51:37PM +0530, Vaishali Thakkar wrote:
> Here, a location is reset to NULL before being passed to PTR_ERR.
> So, PTR_ERR should be called before its argument is reassigned
> to NULL. Further to simplify things use PTR_ERR_OR_ZERO instead
> of PTR_ERR and IS_ERR.
>
> Prob
On Wed, Apr 27, 2016 at 5:45 PM, Emil Velikov
wrote:
> On 27 April 2016 at 10:47, Russell King - ARM Linux
> wrote:
>> On Thu, Apr 21, 2016 at 09:17:13PM +0100, Emil Velikov wrote:
>>> Dave Airlie pointed out that "polluting" the headers in a manner as seen
>>> with this series might not be too
From: Thierry Reding
Some of the functions implemented are flagged as not having a prototype
defined when building with W=1. Include the header to avoid these build
warnings.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_atomic.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/dr
Here, a location is reset to NULL before being passed to PTR_ERR.
So, PTR_ERR should be called before its argument is reassigned
to NULL. Further to simplify things use PTR_ERR_OR_ZERO instead
of PTR_ERR and IS_ERR.
Problem found using Coccinelle.
Signed-off-by: Vaishali Thakkar
Reviewed-by: Eri
On Thursday 28 April 2016 06:23 PM, Eric Engestrom wrote:
> On Wed, Apr 27, 2016 at 04:51:37PM +0530, Vaishali Thakkar wrote:
>> Here, a location is reset to NULL before being passed to PTR_ERR.
>> So, PTR_ERR should be called before its argument is reassigned
>> to NULL. Further to simplify thin
On Tue, Apr 26, 2016 at 07:29:44PM +0200, Daniel Vetter wrote:
> No dev->struct_mutex anywhere to be seen.
>
> Cc: Russell King
Acked-by: Russell King
Thanks Daniel.
--
RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps do
On Tue, Apr 26, 2016 at 07:29:49PM +0200, Daniel Vetter wrote:
> No dev->struct_mutex anywhere to be seen.
>
> Cc: Christian Gmeiner
> Cc: Russell King
Acked-by: Russell King
Thanks Daniel.
--
RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
FTTC broadband for 0.8mile line
From: Gustavo Padovan
Hi,
This patchset sits on top of Sync ABI Rework v13:
https://www.spinics.net/lists/dri-devel/msg105667.html
The first eight clean up and prepare sync_file for de-staging. The last four
patches do the de-staging, moving files to drivers/dma-buf/ and include/linux/
plus ad
From: Gustavo Padovan
struct sync_merge_data already have documentation on top of the
struct definition. No need to duplicate it.
Signed-off-by: Gustavo Padovan
Reviewed-by: Maarten Lankhorst
Reviewed-by: Daniel Vetter
---
drivers/staging/android/uapi/sync.h | 10 +-
1 file changed,
From: Gustavo Padovan
These two functions are just wrappers for one line functions, they
call fd_install() and fput() respectively, so just get rid of them
and use fd_install() and fput() directly for more simplicity.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/stagi
From: Gustavo Padovan
To keep comments in line with drivers/dma-buf/ move all sync_file comments
to sync.c.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync.c | 26 +-
drivers/staging/android/sync.h | 31 ---
From: Gustavo Padovan
There is no plan in the near future to use this function outside of this
file so keep it as static.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync.c | 5 ++---
drivers/staging/android/sync.h | 2 --
2 files changed, 2 insertion
From: Gustavo Padovan
There is no plan in the near future to use this function outside of this
file so keep it as static.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync.c | 3 +--
drivers/staging/android/sync.h | 1 -
2 files changed, 1 insertion(+)
From: Gustavo Padovan
Simplifies the API to only receive the fence it needs to add to the
sync and create a name for the sync_file based on the fence context and
seqno.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync.c | 16 +---
dr
From: Gustavo Padovan
Move its functions and structs to their own file. Also moves function's
docs to the .c file.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/Makefile | 2 +-
drivers/staging/android/sync.c | 37
From: Gustavo Padovan
num_fences was missing a colon mark and sync_file_create() now have
better description.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync_file.c | 5 +++--
drivers/staging/android/sync_file.h | 2 +-
2 files changed, 4 insertions(
From: Gustavo Padovan
Fix checks reported by checkpatch.pl.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync_file.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/android/sync_file.c
b/drivers/staging/andr
From: Gustavo Padovan
Move sync_file headers file to include/ dir.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/staging/android/sync.h | 4 ++--
drivers/staging/android/sync_debug.c | 2 +-
drivers/staging/
From: Gustavo Padovan
sync_file is useful to connect one or more fences to the file. The file is
used by userspace to track fences between drivers that share DMA bufs.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
drivers/Kconfig | 2 ++
drive
From: Gustavo Padovan
Add entry in device-drivers.tmpl for sync_file documentation.
Signed-off-by: Gustavo Padovan
Reviewed-by: Daniel Vetter
---
Documentation/DocBook/device-drivers.tmpl | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/DocBook/device-drivers.tmpl
b/Docume
From: Gustavo Padovan
Add sync_file documentation on dma-buf-sync_file.txt
Reviewed-by: Daniel Vetter
---
Documentation/sync_file.txt | 69 +
1 file changed, 69 insertions(+)
create mode 100644 Documentation/sync_file.txt
diff --git a/Documentation/
On Wed, Apr 27, 2016 at 02:38:18PM +0200, Lucas Stach wrote:
> If the MMU map fails caused by an unaligned SG entry, the unmap path
> is called to undo all already setup SG mappings. When encountering the
> unaligned SG the unmap path hangs the kernel with a BUG(), while the
> error is recoverable
On Wed, Apr 27, 2016 at 02:39:20PM +0200, Lucas Stach wrote:
> This function will be changed to be called indirectly and this
> prototype change brings it in line with all the other indirect
> object calls.
Christian prefers passing around struct drm_gem_object rather than
the etnaviv_gem_object.
Am Donnerstag, den 28.04.2016, 14:55 +0100 schrieb Russell King - ARM
Linux:
> On Wed, Apr 27, 2016 at 02:38:18PM +0200, Lucas Stach wrote:
> > If the MMU map fails caused by an unaligned SG entry, the unmap path
> > is called to undo all already setup SG mappings. When encountering the
> > unalign
On Wed, Apr 27, 2016 at 10:45:54PM +0100, Emil Velikov wrote:
> On 27 April 2016 at 10:47, Russell King - ARM Linux
> wrote:
> > On Thu, Apr 21, 2016 at 09:17:13PM +0100, Emil Velikov wrote:
> >> Dave Airlie pointed out that "polluting" the headers in a manner as seen
> >> with this series might n
This reverts commit 1c278e5e3718d15475ec08ee2135f37a6b13361c.
If DRM_OMAP does not select OMAP2_DSS it is possible to build a kernel with
DRM_OMAP only and not selecting OMAP2_DSS. Since omapdrm depends on
OMAP2_DSS this will result on broken kernel build.
Signed-off-by: Peter Ujfalusi
---
driv
regulator_can_change_voltage() is deprecated and it's use is not necessary
as commit:
6a0028b3dd67b regulator: Deprecate regulator_can_change_voltage()
describers it clearly.
As there is no practical use of it it can be removed.
At this point the regulator_set_voltage() calls can not be removed as
Hi Vineet,
On Thu, 2016-04-28 at 09:56 +0530, Vineet Gupta wrote:
[snip]
> >
> > diff --git a/arch/arc/boot/dts/axc001.dtsi b/arch/arc/boot/dts/axc001.dtsi
> > index 420dcfd..ae6162d 100644
> > --- a/arch/arc/boot/dts/axc001.dtsi
> > +++ b/arch/arc/boot/dts/axc001.dtsi
> > @@ -95,6 +95,24 @@
>
On Thursday 28 April 2016 07:16 PM, Alexey Brodkin wrote:
>> > Note that the IOC start alignment needs to follow
>> > max(4k, size). What will be maximum size of frame buffer - 16M always !
> What do you mean by that?
For HS38, we intend to bypass the frame buffer traffic from IOC port. So the
fr
This mini-series allows to allocate frame buffer memory in desired
location. Allocation of a frame buffer memory in a special memory region
allows bypassing of so-called IO Coherency aperture which is typically set
as a range 0x8z-0xAz.
I.e. all data traffic to PGU bypasses IO Coherency block
and
Now when ARC supports reserved memory areas and
per-device coherent DMA allocations we may switch ARC PGU
to use of those dedicated areas.
One of the benefits we may move frame-buffer area out
from IO Coherency aperture and so significantly
reduce IOC utilization allowing less demanding
peripheral
Allocation of a frame buffer memory in a special memory region
allows bypassing of so-called IO Coherency aperture
which is typically set as a range 0x8z-0xAz.
I.e. all data traffic to PGU bypasses IO Coherency block
and saves its bandwidth for other peripherals.
Even though for AXS101 (which sor
On Thu, Apr 28, 2016 at 03:19:56PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Some of the functions implemented are flagged as not having a prototype
> defined when building with W=1. Include the header to avoid these build
> warnings.
>
> Signed-off-by: Thierry Reding
Applied to
2016-04-27 Daniel Stone :
> Hi,
>
> On 26 April 2016 at 21:48, Greg Hackmann wrote:
> > On 04/26/2016 01:05 PM, Daniel Vetter wrote:
> >> On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote:
> >>> What are they doing that can't stuff the fences into an array
> >>> instead of props?
>
On Thu, Apr 28, 2016 at 04:04:58PM +0200, Lucas Stach wrote:
> The observation was that the common code in iommu_map() rightfully
> rejected to map things, as mapping something unaligned to the page size
> is totally bogus.
Shouldn't iommu_map() detect this?
/*
* both the virtual
On Thu, Apr 28, 2016 at 11:36:44AM -0300, Gustavo Padovan wrote:
> 2016-04-27 Daniel Stone :
>
> > Hi,
> >
> > On 26 April 2016 at 21:48, Greg Hackmann wrote:
> > > On 04/26/2016 01:05 PM, Daniel Vetter wrote:
> > >> On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote:
> > >>> What a
2016-04-26 Chris Wilson :
> On Mon, Apr 25, 2016 at 07:33:21PM -0300, Gustavo Padovan wrote:
> > +static const char *fence_collection_get_timeline_name(struct fence *fence)
> > +{
> > + return "no context";
>
> "unbound" to distinguish from fence contexts within a timeline?
>
> > +static bool
Hi Vineet,
On Thu, 2016-04-28 at 19:26 +0530, Vineet Gupta wrote:
> On Thursday 28 April 2016 07:16 PM, Alexey Brodkin wrote:
> >
> > >
> > > >
> > > > Note that the IOC start alignment needs to follow
> > > > max(4k, size). What will be maximum size of frame buffer - 16M always !
> > What do y
Now that drm_fb_helper gets deferred io support, the
drm_fb_helper_sys_{fillrect,copyarea,imageblit} functions will schedule
a worker that will call the (struct drm_framebuffer *)->funcs->dirty()
function. This will break this driver so use the
sys_{fillrect,copyarea,imageblit} functions directly.
Now that drm_fb_helper gets deferred io support, the
drm_fb_helper_sys_{fillrect,copyarea,imageblit} functions will schedule
a worker that will call the (struct drm_framebuffer *)->funcs->dirty()
function. This will break this driver so use the
sys_{fillrect,copyarea,imageblit} functions directly.
This patchset adds fbdev deferred io support to drm_fb_helper and
drm_fb_cma_helper.
It channels fbdev mmap and fb_{write,fillrect,copyarea,imageblit} damage
through the (struct drm_framebuffer_funcs)->dirty callback on the
fb_helper framebuffer which will always run in process context.
I have al
This adds deferred io support to drm_fb_helper.
The fbdev framebuffer changes are flushed using the callback
(struct drm_framebuffer *)->funcs->dirty() by a dedicated worker
ensuring that it always runs in process context.
Signed-off-by: Noralf Trønnes
Reviewed-by: Daniel Vetter
---
Changes si
Export fb_deferred_io_mmap so drivers can change vma->vm_page_prot.
When the framebuffer memory is allocated using dma_alloc_writecombine()
instead of vmalloc(), I get cache syncing problems on ARM.
This solves it:
static int drm_fbdev_cma_deferred_io_mmap(struct fb_info *info,
This adds fbdev deferred io support if CONFIG_FB_DEFERRED_IO is enabled.
The driver has to provide a (struct drm_framebuffer_funcs *)->dirty()
callback to get notification of fbdev framebuffer changes.
If the dirty() hook is set, then fb_deferred_io is set up automatically
by the helper.
Two funct
Use the fbdev deferred io support in drm_fb_helper.
The (struct fb_ops *)->fb_{fillrect,copyarea,imageblit} functions will
now schedule a worker instead of being flushed directly like it was
previously (recorded when in atomic).
This patch has only been compile tested.
Signed-off-by: Noralf Trøn
Use the fbdev deferred io support in drm_fb_helper which mirrors the
one qxl has had.
This patch has only been compile tested.
Signed-off-by: Noralf Trønnes
Reviewed-by: Daniel Vetter
---
Changes since v2:
- The drm_clip_rect_{width/height} functions are dropped, so open code it
Changes since
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/d349b690/attachment.html>
2016-04-26 Daniel Vetter :
> On Mon, Apr 25, 2016 at 07:33:28PM -0300, Gustavo Padovan wrote:
> > From: Gustavo Padovan
> >
> > Support DRM out-fences creating a sync_file with a fence for each crtc
> > update with the DRM_MODE_ATOMIC_OUT_FENCE flag.
> >
> > We then send an struct drm_out_fence
On Thu, Apr 28, 2016 at 11:36:44AM -0300, Gustavo Padovan wrote:
> 2016-04-27 Daniel Stone :
>
> > Hi,
> >
> > On 26 April 2016 at 21:48, Greg Hackmann wrote:
> > > On 04/26/2016 01:05 PM, Daniel Vetter wrote:
> > >> On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote:
> > >>> What a
On Thu, Apr 28, 2016 at 6:56 PM, Ville Syrjälä
wrote:
>> - better for tracing, can identify the buffer/fence promptly
>
> Can fences be reused somehow while still attached to a plane, or ever?
> That might cause some oddness if you, say, leave a fence attached to one
> plane and then do a modes
On Thu, Apr 28, 2016 at 07:43:16PM +0200, Daniel Vetter wrote:
> On Thu, Apr 28, 2016 at 6:56 PM, Ville Syrjälä
> wrote:
> >> - better for tracing, can identify the buffer/fence promptly
> >
> > Can fences be reused somehow while still attached to a plane, or ever?
> > That might cause some odd
On Thu, Apr 28, 2016 at 7:55 PM, Gustavo Padovan
wrote:
> 2016-04-28 Ville Syrjälä :
>
>> On Thu, Apr 28, 2016 at 07:43:16PM +0200, Daniel Vetter wrote:
>> > On Thu, Apr 28, 2016 at 6:56 PM, Ville Syrjälä
>> > wrote:
>> > >> - better for tracing, can identify the buffer/fence promptly
>> > >
On Thu, Apr 28, 2016 at 07:56:19PM +0300, Ville Syrjälä wrote:
> On Thu, Apr 28, 2016 at 11:36:44AM -0300, Gustavo Padovan wrote:
> > 2016-04-27 Daniel Stone :
> >
> > > Hi,
> > >
> > > On 26 April 2016 at 21:48, Greg Hackmann wrote:
> > > > On 04/26/2016 01:05 PM, Daniel Vetter wrote:
> > > >
chment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/3c371083/attachment.html>
:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/79ed78ab/attachment.html>
On Wed, Apr 27, 2016 at 11:02:17PM +0300, Laurent Pinchart wrote:
> Hi Ville,
>
> On Wednesday 27 Apr 2016 20:29:24 Ville Syrjälä wrote:
> > On Wed, Apr 27, 2016 at 06:30:19PM +0300, Laurent Pinchart wrote:
> > > Hi Tomi,
> > >
> > > (CC'ing Daniel)
> > >
> > > Thank you for the patch.
> > >
HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/3f8c91e4/attachment.html>
il because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/41293bca/attachment.html>
Hi Dave,
The following changes since commit 928815245cbdaa611873424759d5e7a7293dd18b:
Merge tag 'drm-intel-fixes-2016-04-07' of
git://anongit.freedesktop.org/drm-intel into drm-fixes (2016-04-11 13:30:05
+1000)
are available in the git repository at:
git://people.freedesktop.org/~syeh/re
L:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/7f85a0e1/attachment.html>
On Thu, Apr 28, 2016 at 8:17 PM, Ville Syrjälä
wrote:
>> > - implicit fences also needs one fence per plane/fb, so it will be good to
>> >match with that.
>>
>> We would actually need a fence per object rather than per fb.
>
> I guess you could overcome this by automagically creating a merg
nee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160428/c690e77a/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=117151
--- Comment #1 from Parker Reed ---
Created attachment 214661
--> https://bugzilla.kernel.org/attachment.cgi?id=214661&action=edit
Full verbose lspci
--
You are receiving this mail because:
You are watching the assignee of the bug.
1 - 100 of 108 matches
Mail list logo