es
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160919/a15eff4b/attachment-0001.gz>
Hi, YT:
On Mon, 2016-09-12 at 20:01 +0800, YT Shen wrote:
> This patch add support for the Mediatek MT2701 DISP subsystem.
> There is only one OVL engine in MT2701.
>
> Signed-off-by: YT Shen
> ---
> drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 6 ++
> drivers/gpu/drm/mediatek/mtk_disp_rd
Hi Dave,
There was my mistake reported below when I merged one fixup manually.
http://www.spinics.net/lists/dri-devel/msg118283.html
Fixed.
Thanks,
Inki Dae
2016ë
09ì 18ì¼ 23:12ì Inki Dae ì´(ê°) ì´ ê¸:
> Hi Dave,
>
>Just fixup to runtime pm usage and some cleanups.
>
>Plea
L:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160919/30c21a9b/attachment.html>
Hi Sean,
On Sat, Sep 17, 2016 at 2:22 AM, Sean Paul wrote:
>
> Instead of assigning device managed resources to local variables,
> keep track of them in the vop struct.
Why this patch?
Is it fixing an issue?
Or, is it preparing for some future use of ahb_rst outside of vop_initial?
I thought th
On Sat, Sep 17, 2016 at 2:22 AM, Sean Paul wrote:
> If create_crtc fails in vop bind, ensure the irq refcount is zeroed
> back out before exiting.
>
> Signed-off-by: Sean Paul
Reviewed-by: Daniel Kurtz
> ---
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 6 +-
> 1 file changed, 5 inserti
On Sat, 17 Sep 2016, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Fixes the following sparse warning:
>
> drivers/gpu/drm/i915/intel_dp.c:1527:5: warning:
> symbol 'intel_dp_compute_bpp' was not declared. Should it be static?
>
> Signed-off-by: Wei Yongjun
> ---
> drivers/gpu/drm/i915/intel_dp.c
On 18.09.2016 19:42, Noralf Trønnes wrote:
>
> Den 18.09.2016 15:48, skrev poma:
>> Fix for DisplayLink GPU USB2.0 device X server screen update
>>
>> Within X server on top of DisplayLink GPU USB2.0 device,
>> screen content is not refreshed i.e. updated,
>> which is the most basic functionality
Am Freitag, den 16.09.2016, 20:38 +0300 schrieb Vladimir Zapolskiy:
> Hi Philipp,
[...]
> >> as far as I know David accepts pull requests from you, can you please
> >> create and send a pull request for v4.9 containing these changes?
> >>
> >> https://patchwork.kernel.org/patch/9284717/ -- with
Hi Dave,
Just random misc stuff that Sean/Sumit&Archit picked up while I relaxed.
Well except for one commit:
commit a988588b1806b40ae115fe1c9ab38706fd1a7c2b
Author: Kristian H. Kristensen
Date: Tue Sep 13 14:20:45 2016 -0700
drm: Only use compat ioctl for addfb2 on X86/IA64
Pls also che
Hi Dave,
this tag contains support for the I2C master controller contained in the
HDMI TX IP core, for those boards that don't allow to mux their DDC pins
to SoC I2C controllers. This will make the dw-hdmi driver register its
internal I2C master if the ddc-i2c-bus property is not set on the device
On Wed, Sep 14, 2016 at 1:05 PM, Philipp Zabel
wrote:
> Hi Dave,
>
> Am Montag, den 29.08.2016, 17:44 +0200 schrieb Daniel Vetter:
>> On Mon, Aug 29, 2016 at 4:59 PM, Philipp Zabel
>> wrote:
>> > Am Montag, den 29.08.2016, 12:53 +0200 schrieb Philipp Zabel:
>> >> Am Freitag, den 26.08.2016, 15:
On 09/16/16 18:02, Tony Lindgren wrote:
> * Jyri Sarha [160916 04:50]:
>> > These patches complete the am335x LCDC color errata fix[1]. The
>> > functional patches are now queued for v4.9.
>> >
>> > The patch for am335x-boneblack.dts is delayed until v4.10 or v4.9-rc
>> > phase to avoid conflickt
Hi Dave,
please consider merging this tag containing the latest active plane
reconfiguration patches, several fixes and cleanups, and image
conversion support for the V4L2 drivers to build upon.
The active plane reconfiguration patch (v4) will conflict with the v2
patch that already went into v4.
On Monday, September 19, 2016 3:28:39 PM CEST Baoyou Xie wrote:
> We get a few warnings when building kernel with W=1:
> drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/tonga_smumgr.c:146:5: warning:
> no previous prototype for 'tonga_program_jump_on_start' [-Wmissing-prototypes]
> drivers/gpu/drm/
On 19 September 2016 at 17:31, Philipp Zabel wrote:
> Hi Dave,
>
> please consider merging this tag containing the latest active plane
> reconfiguration patches, several fixes and cleanups, and image
> conversion support for the V4L2 drivers to build upon.
>
> The active plane reconfiguration patc
Hi Linus,
One important drm 32/64 ABI fix came in so I'll dequeue what I have, the rest is
just exynos runtime pm fixes, but the net removal of code seems like a
win to me.
I'm going to be sporadic this week due to school holidays, so if anything urgent
turns up, Daniel will take care of it.
Dav
On 09/17/16 01:17, Nishanth Menon wrote:
> On 09/16/2016 06:50 AM, Jyri Sarha wrote:
> [..]
>> +blue-and-red-wiring = "crossed";
>
> Dumb question: Why is this a Linux generic property - does it make sense
> to have "blue-and-red-wiring" as a Linux generic property? I mean, it
> reminds me lit
Am 19.09.2016 um 09:34 schrieb Arnd Bergmann:
> On Monday, September 19, 2016 3:28:39 PM CEST Baoyou Xie wrote:
>> We get a few warnings when building kernel with W=1:
>> drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/tonga_smumgr.c:146:5:
>> warning: no previous prototype for 'tonga_program_jump_
On Monday, September 19, 2016 9:59:56 AM CEST Christian König wrote:
> Am 19.09.2016 um 09:34 schrieb Arnd Bergmann:
> > On Monday, September 19, 2016 3:28:39 PM CEST Baoyou Xie wrote:
> >> We get a few warnings when building kernel with W=1:
> >> drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/ton
Hi Baoyou,
On Mon, Sep 19, 2016 at 03:28:39PM +0800, Baoyou Xie wrote:
> We get a few warnings when building kernel with W=1:
> drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/tonga_smumgr.c:146:5: warning:
> no previous prototype for 'tonga_program_jump_on_start' [-Wmissing-prototypes]
> drivers/
Am Montag, den 19.09.2016, 17:37 +1000 schrieb Dave Airlie:
> On 19 September 2016 at 17:31, Philipp Zabel
> wrote:
> > Hi Dave,
> >
> > please consider merging this tag containing the latest active plane
> > reconfiguration patches, several fixes and cleanups, and image
> > conversion support fo
On Mon, 19 Sep 2016, Dave Airlie wrote:
> Hi Linus,
>
> One important drm 32/64 ABI fix came in so I'll dequeue what I have, the rest
> is
> just exynos runtime pm fixes, but the net removal of code seems like a
> win to me.
>
> I'm going to be sporadic this week due to school holidays, so if any
he git repository at:
git://anongit.freedesktop.org/drm-intel tags/drm-intel-next-2016-09-19
for you to fetch changes up to 6e05f3d3b9298a56d6f1acb474a75cf14a17c31e:
drm/i915: Update DRIVER_DATE to 20160919 (2016-09-19 09:26:08 +0200)
---
On Mon, 19 Sep 2016, Jani Nikula wrote:
> On Mon, 19 Sep 2016, Dave Airlie wrote:
>> Hi Linus,
>>
>> One important drm 32/64 ABI fix came in so I'll dequeue what I have, the
>> rest is
>> just exynos runtime pm fixes, but the net removal of code seems like a
>> win to me.
>>
>> I'm going to be s
gory.
> Would it be feasible to combine the two, and support both devices
> with a single driver ?
I don't see why not. But converting the renesas DRM driver seems a bit
out of scope, and the ADV compatible can definitely be added later.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel 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/20160919/22fa2cec/attachment-0001.sig>
Am 18.09.2016 um 15:12 schrieb Baoyou Xie:
> We get 6 warnings when building kernel with W=1:
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c:629:6: warning: no previous prototype
> for 'dce_v8_0_disable_dce' [-Wmissing-prototypes]
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c:730:6: warning: no previous proto
Am 18.09.2016 um 15:37 schrieb Baoyou Xie:
> In amdgpu_gfx.h, the declaration of amdgpu_gfx_parse_disable_cu()
> is incorrect.
>
> Signed-off-by: Baoyou Xie
Ups, nice catch.
Patch is Reviewed-by: Christian König .
Regards,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 4 +++-
Am 18.09.2016 um 16:09 schrieb Baoyou Xie:
> We get 7 warnings when building kernel with W=1:
> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous
> prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes]
> drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c:1548:5: warning: n
3c494a57e2ee9] [AMDGPU]
Refactor MUBUF/MTBUF instructions
--
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/20160919/e58dbf67/attachment.html>
Am 18.09.2016 um 16:13 schrieb Baoyou Xie:
> We get 2 warnings when building kernel with W=1:
> drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:146:5: warning: no previous
> prototype for 'pool_to_domain' [-Wmissing-prototypes]
> drivers/gpu/drm/amd/amdgpu/cz_smc.c:104:5: warning: no previous prototype
Am 18.09.2016 um 16:29 schrieb Baoyou Xie:
> We get 2 warnings when building kernel with W=1:
> drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c:502:10: warning: no previous prototype
> for 'init_cond_exec' [-Wmissing-prototypes]
> drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c:514:6: warning: no previous prototype
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160919/d25f8ab5/attachment.html>
On Thu, Sep 01, 2016 at 02:48:35PM +0200, David Herrmann wrote:
> Rather than doing drm_file allocation/destruction right in the fops, lets
> provide separate helpers. This decouples drm_file management from the
> still-mandatory drm-fops. It prepares for use of drm_file without the
> fops, both by
On Thu, Sep 01, 2016 at 02:48:31PM +0200, David Herrmann wrote:
> Hey
>
> The remaining cleanup patches pending on dri-devel in one batch. Random
> cleanups
> all over the place. Should all be straightforward.
Except for patch 4 all applied to drm-misc, thanks.
-Daniel
>
> Thanks
> David
>
>
On Mon, Sep 19, 2016 at 01:56:56PM +0200, Daniel Vetter wrote:
> On Thu, Sep 01, 2016 at 02:48:31PM +0200, David Herrmann wrote:
> > Hey
> >
> > The remaining cleanup patches pending on dri-devel in one batch. Random
> > cleanups
> > all over the place. Should all be straightforward.
>
> Except
The only multi-planar format supported by the driver is NV12, there will
thus never be more than two planes per framebuffer.
Signed-off-by: Laurent Pinchart
Reviewed-by: Tomi Valkeinen
---
drivers/gpu/drm/omapdrm/omap_fb.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/
Hello,
Here's the third version of my current stack of pending patches for the
omapdrm driver.
All comments received for v2 have been considered and patches updated where
applicable. Changes since v2 include
- Rebased on top of Dave's latest drm-next branch
- Replaced "drm: omapdrm: Use atomic s
The driver stores in a custom structure named format several pieces of
information about the format that are available in the DRM core. Remove
them and get the information from the DRM core instead.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_fb.c | 91 --
Don't print userspace parameters validation failures as error messages
to avoid giving userspace the ability to flood the kernel log.
Signed-off-by: Laurent Pinchart
Reviewed-by: Tomi Valkeinen
---
drivers/gpu/drm/omapdrm/omap_fb.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions
As the FIFO underflow IRQ handler just prints an error message to the
kernel log, simplify the code by not registering one IRQ handler per
plane but print the messages directly from the main IRQ handler.
Signed-off-by: Laurent Pinchart
---
Changes since v1:
- Only register error IRQs that exist
The omapdrm DSS manager enable/disable operations check the DSS manager
state to avoid double enabling/disabling. Move that code to the DSS
manager to decrease the dependency of the DRM layer to the DSS layer.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/dss/dispc.c | 1 -
driver
The DRM core supports skipping plane update for inactive CRTCs for
hardware that don't need it or can't cope with it. That's our case, so
use the DRM core infrastructure instead of reinventing it.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_crtc.c | 8 +++-
drivers/gpu/d
The flag will need to be accessed atomically in the vblank interrupt
handler, memory barriers won't be enough.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_crtc.c | 28 +---
1 file changed, 21 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm
The function is only used in omap_irq.c, move it there and make it
static.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_crtc.c | 7 ---
drivers/gpu/drm/omapdrm/omap_drv.h | 1 -
drivers/gpu/drm/omapdrm/omap_irq.c | 7 ++-
3 files changed, 6 insertions(+), 9 deletion
Instead of conditioning planes update based on the DD manager state, use
the enabled field newly added to the omap_crtc structure. This reduces
the dependency from the DRM layer to the DSS layer.
The enabled field is a transitory measure, the implementation should use
the CRTC atomic state instead
Instead of going through a complicated registration mechanism, just
call the OCP error IRQ handler directly from the main IRQ handler.
Signed-off-by: Laurent Pinchart
---
Changes since v1:
- Rename IRQ handler to omap_irq_ocp_error_handler()
- Replace hex error value with "OCP error" message
---
Instead of going through a complicated private IRQ registration
mechanism, handle the vblank interrupt activation with the standard
drm_crtc_vblank_get() and drm_crtc_vblank_put() mechanism. This will let
the DRM core keep the vblank interrupt enabled as long as needed to
update the frame counter.
The IRQ registration functions are not used outside of their compilation
unit, make them static. As the __omap_irq_(un)register() functions are
only called by their omap_irq_(un)register() counterparts, merge them
together.
Signed-off-by: Laurent Pinchart
---
Changes since v1:
- Split the omap_d
Now that the IRQ list is used for IRQ wait only we can merge
omap_drm_irq and omap_irq_wait and simplify the implementation.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_drv.h | 17 +--
drivers/gpu/drm/omapdrm/omap_irq.c | 94 ++
2 file
Move the list of pending IRQ wait instances to the omap_drm_private
structure and the wait queue head to the IRQ wait structure.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_drv.h | 3 ++-
drivers/gpu/drm/omapdrm/omap_irq.c | 42 --
2 file
Merge the single-user objects_lookup inline function into its caller,
allowing reuse of the error code path.
Signed-off-by: Laurent Pinchart
Reviewed-by: Tomi Valkeinen
---
drivers/gpu/drm/omapdrm/omap_drv.h | 25 -
drivers/gpu/drm/omapdrm/omap_fb.c | 29 +++
The hardware requires all planes to have an identical pitch in number of
pixels. Given that all supported formats use the same number of bytes
per pixel in all planes, framebuffer creation checks can be simplified.
The implementations assumes that no format use more than two planes
which is true wi
Instead of going through a complicated registration mechanism, just
expose the CRTC error IRQ function and call it directly from the main
IRQ handler.
Signed-off-by: Laurent Pinchart
Reviewed-by: Tomi Valkeinen
---
drivers/gpu/drm/omapdrm/omap_crtc.c | 12 ++--
drivers/gpu/drm/omapdrm/o
The vblank interrupt is disabled after one occurrence, preventing the
atomic update event from being processed twice. However, this also
prevents the software frame counter from being updated correctly that
would require vblank interrupts to be kept enabled while the CRTC is
active.
In preparation
The IRQ wait functions are called from the DSS enable and disable
operations only, where the DISPC is guaranteed to be enabled. There's no
need for manual DISPC power management there.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/omap_irq.c | 4
1 file changed, 4 deletions(-)
The only omap_drm_irq handler doesn't use the irqstatus parameter passed
to the function. Remove it.
Signed-off-by: Laurent Pinchart
---
Changes since v1:
- New patch
---
drivers/gpu/drm/omapdrm/omap_drv.h | 2 +-
drivers/gpu/drm/omapdrm/omap_irq.c | 4 ++--
2 files changed, 3 insertions(+), 3
The addition of the debugfs info created references to seq_puts() and
seq_printf(), but relied on the debugfs header to be included implicitly,
which apparently doesn't happen all the time, as seen from this randconfig
build output:
drivers/gpu/drm/drm_dp_helper.c: In function 'drm_dp_downstream_d
On Tue, Sep 06, 2016 at 12:59:31PM -0400, Sean Paul wrote:
> On Wed, Aug 31, 2016 at 12:09 PM, Daniel Vetter
> wrote:
> > Just pure code movement, cleanup and polish will happen in later
> > patches.
> >
> > v2: Don't forget all the ioctl! To extract those cleanly I decided to
> > put check_src_c
On Fri, Sep 02, 2016 at 03:00:38PM +0530, Archit Taneja wrote:
>
>
> On 8/31/2016 9:39 PM, Daniel Vetter wrote:
> > Big thing is untangling and carefully documenting the different uapi
> > types of planes. I also sprinkled a few more cross references around
> > to make this easier to discover.
>
On Mon, 19 Sep 2016, Arnd Bergmann wrote:
> The addition of the debugfs info created references to seq_puts() and
> seq_printf(), but relied on the debugfs header to be included implicitly,
> which apparently doesn't happen all the time, as seen from this randconfig
> build output:
>
> drivers/gpu
From: Ville Syrjälä
I got a bit fed up from the sparse noise during a full build,
so I tried to shut some of it up. Mostly missing statics or function
declarations.
I didn't touch the main offender (radeon/amdgpu) since there was enough
of this stuff in the other drivers as well. Nouveau was p
From: Ville Syrjälä
drm/drm_atomic_helper.c:1696:6: warning: symbol 'plane_crtc_active' was not
declared. Should it be static?
Cc: Daniel Vetter
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/drm_atomic_helper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/driver
From: Ville Syrjälä
drm/drm_blend.c:207:5: warning: symbol 'drm_atomic_normalize_zpos' was not
declared. Should it be static?
Cc: Marek Szyprowski
Cc: Benjamin Gaignard
Cc: Laurent Pinchart
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/drm_blend.c | 2 +-
1 file changed, 1 insertion
From: Ville Syrjälä
drm/ast/ast_drv.c:36:5: warning: symbol 'ast_modeset' was not declared. Should
it be static?
drm/ast/ast_ttm.c:227:22: warning: symbol 'ast_bo_driver' was not declared.
Should it be static?
Cc: Dave Airlie
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/ast/ast_drv.
From: Ville Syrjälä
drm/nouveau/dispnv04/overlay.c:496:1: warning: symbol 'nouveau_overlay_init'
was not declared. Should it be static?
drm/nouveau/nouveau_connector.c:63:5: warning: symbol 'nouveau_hdmimhz' was not
declared. Should it be static?
drm/nouveau/nouveau_display.c:96:1: warning: s
From: Ville Syrjälä
drm/drm_fb_helper.c:2306:12: warning: symbol 'drm_fb_helper_modinit' was not
declared. Should it be static?
While at it, move the lefover static inline to the right place.
Cc: Daniel Vetter
Cc: Sean Paul
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/drm_crtc_help
From: Ville Syrjälä
drm/bochs/bochs_mm.c:196:22: warning: symbol 'bochs_bo_driver' was not
declared. Should it be static?
drm/bochs/bochs_kms.c:181:5: warning: symbol 'bochs_connector_get_modes' was
not declared. Should it be static?
Cc: Gerd Hoffmann
Signed-off-by: Ville Syrjälä
---
dr
From: Ville Syrjälä
drm/cirrus/cirrus_drv.c:18:5: warning: symbol 'cirrus_modeset' was not
declared. Should it be static?
drm/cirrus/cirrus_ttm.c:227:22: warning: symbol 'cirrus_bo_driver' was not
declared. Should it be static?
Cc: Dave Airlie
Signed-off-by: Ville Syrjälä
---
drivers/gp
From: Ville Syrjälä
drm/mgag200/mgag200_drv.c:24:5: warning: symbol 'mgag200_modeset' was not
declared. Should it be static?
drm/mgag200/mgag200_ttm.c:227:22: warning: symbol 'mgag200_bo_driver' was not
declared. Should it be static?
Cc: Dave Airlie
Signed-off-by: Ville Syrjälä
---
driv
From: Ville Syrjälä
drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c:96:22: warning: symbol 'get_connector' was
not declared. Should it be static?
drm/msm/mdp/mdp4/mdp4_plane.c:84:5: warning: symbol 'mdp4_plane_set_property'
was not declared. Should it be static?
drm/msm/mdp/mdp5/mdp5_cfg.c:25:26: warnin
From: Ville Syrjälä
drm/rockchip/rockchip_drm_drv.c:312:6: warning: symbol
'rockchip_drm_fb_suspend' was not declared. Should it be static?
drm/rockchip/rockchip_drm_drv.c:321:6: warning: symbol 'rockchip_drm_fb_resume'
was not declared. Should it be static?
drm/rockchip/rockchip_drm_fbdev.c:
From: Ville Syrjälä
drm/sti/sti_mixer.c:361:6: warning: symbol 'sti_mixer_set_matrix' was not
declared. Should it be static?
drm/sti/sti_gdp.c:476:5: warning: symbol 'sti_gdp_field_cb' was not declared.
Should it be static?
drm/sti/sti_gdp.c:885:24: warning: symbol 'sti_gdp_plane_helpers_func
From: Ville Syrjälä
drm/sun4i/sun4i_tv.c:181:21: warning: symbol 'ntsc_video_levels' was not
declared. Should it be static?
drm/sun4i/sun4i_tv.c:185:21: warning: symbol 'pal_video_levels' was not
declared. Should it be static?
drm/sun4i/sun4i_tv.c:189:21: warning: symbol 'ntsc_burst_levels' w
From: Ville Syrjälä
drm/tilcdc/tilcdc_tfp410.c:385:24: warning: symbol 'tfp410_driver' was not
declared. Should it be static?
drm/tilcdc/tilcdc_tfp410.c:395:12: warning: symbol 'tilcdc_tfp410_init' was not
declared. Should it be static?
drm/tilcdc/tilcdc_tfp410.c:400:13: warning: symbol 'tilc
From: Ville Syrjälä
drm/arm/malidp_planes.c:49:24: warning: symbol 'malidp_duplicate_plane_state'
was not declared. Should it be static?
drm/arm/malidp_planes.c:66:6: warning: symbol 'malidp_destroy_plane_state' was
not declared. Should it be static?
Cc: Liviu Dudau
Cc: Brian Starkey
Cc: M
On 19 September 2016 at 14:33, wrote:
> --- a/drivers/gpu/drm/msm/msm_fb.c
> +++ b/drivers/gpu/drm/msm/msm_fb.c
> @@ -132,7 +132,7 @@ const struct msm_format *msm_framebuffer_format(struct
> drm_framebuffer *fb)
> struct drm_framebuffer *msm_framebuffer_create(struct drm_device *dev,
>
Hello,
I had these two patches in my private tree for some time. Might as well ask if
there is any interested in merging these.
The first is just some unification on how booleans are used in the mixer code.
The second one reduces register manipulation by moving stuff to the atomic
flush call.
The mixer context struct already has a 'flags' field, so
we can use it to store the 'interlace', 'vp_enabled' and
'has_sclk' booleans.
We use the non-atomic helper functions to access these bits.
Signed-off-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_mixer.c | 54 +++-
> -Original Message-
> From: Baoyou Xie [mailto:baoyou.xie at linaro.org]
> Sent: Sunday, September 18, 2016 10:29 AM
> To: Deucher, Alexander; Koenig, Christian; airlied at linux.ie; Zhou,
> David(ChunMing); Liu, Monk; Zhu, Rex; Huang, JinHuiEric; Cui, Flora
> Cc: dri-devel at lists.freede
> -Original Message-
> From: SF Markus Elfring [mailto:elfring at users.sourceforge.net]
> Sent: Sunday, September 18, 2016 12:53 PM
> To: dri-devel at lists.freedesktop.org; Deucher, Alexander; Koenig, Christian;
> Zhou, David(ChunMing); David Airlie; Liu, Monk; StDenis, Tom
> Cc: LKML; ke
Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once
in mixer_cfg_layer().
Trigger this via atomic flush.
Signed-off-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_mixer.c | 104 ++
drivers/gpu/drm/exynos/regs-mixer.h | 2 +
2 files changed, 69 in
On Tue, Sep 06, 2016 at 12:59:39PM -0400, Sean Paul wrote:
> On Wed, Aug 31, 2016 at 12:09 PM, Daniel Vetter
> wrote:
> > Some were still left in drm_crtc.h. Also include drm_edid.h in the
> > rst files.
> >
> > Signed-off-by: Daniel Vetter
>
> Reviewed-by: Sean Paul
Merged up to this patch,
On Fri, Sep 16, 2016 at 01:06:36PM +0300, Jani Nikula wrote:
>drivers/gpu/drm/drm_dp_helper.c: In function 'drm_dp_downstream_debug':
> >> drivers/gpu/drm/drm_dp_helper.c:551:2: error: implicit declaration of
> >> function 'seq_printf' [-Werror=implicit-function-declaration]
> seq_printf(m
On Mon, Sep 19, 2016 at 04:33:44PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrjälä
>
> drm/drm_fb_helper.c:2306:12: warning: symbol 'drm_fb_helper_modinit' was not
> declared. Should it be static?
>
> While at it, move the lefover static inline to the right place.
>
> C
Acked-by: Vincent Abriou
On 09/19/2016 03:33 PM, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä
>
> drm/sti/sti_mixer.c:361:6: warning: symbol 'sti_mixer_set_matrix' was not
> declared. Should it be static?
> drm/sti/sti_gdp.c:476:5: warning: symbol 'sti_gdp_field_cb' was not d
org/archives/dri-devel/attachments/20160919/e6d00c1c/attachment-0001.html>
---
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160919/61aca785/attachment.html>
se:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160919/12916da5/attachment.html>
s://lists.freedesktop.org/archives/dri-devel/attachments/20160919/3e20a7e9/attachment.html>
On Sun, 2016-09-18 at 13:35 +0200, Thorsten Leemhuis wrote:
> Hi! James & Paulo: What's the current status of this?
No, the only interaction has been the suggestion below for a revert,
which didn't fix the problem.
> Was this issue discussed elsewhere or even fixed in between? Just
> asking, be
.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160919/73bd5f34/attachment.html>
On Mon, 19 Sep 2016, Emil Velikov wrote:
> On 19 September 2016 at 14:33, wrote:
>
>> --- a/drivers/gpu/drm/msm/msm_fb.c
>> +++ b/drivers/gpu/drm/msm/msm_fb.c
>> @@ -132,7 +132,7 @@ const struct msm_format *msm_framebuffer_format(struct
>> drm_framebuffer *fb)
>> struct drm_framebuffer *msm_fr
From: Markus Elfring
Date: Mon, 19 Sep 2016 17:47:37 +0200
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (5):
Use kmalloc_array() in drm_legacy_addbufs_pci()
Replace two kzalloc() calls by kcalloc() in drm_legacy_addbufs_pci()
Replace a k
From: Markus Elfring
Date: Mon, 19 Sep 2016 17:07:06 +0200
A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle software.
From: Markus Elfring
Date: Mon, 19 Sep 2016 17:17:34 +0200
The script "checkpatch.pl" can point information out like the following.
WARNING: Prefer kcalloc over kzalloc with multiply
Thus fix the affected source code places.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/drm_bufs.c | 4 ++
From: Markus Elfring
Date: Mon, 19 Sep 2016 17:24:20 +0200
The script "checkpatch.pl" can point information out like the following.
WARNING: Prefer kcalloc over kzalloc with multiply
Thus fix the affected source code place.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/drm_bufs.c | 2 +-
From: Markus Elfring
Date: Mon, 19 Sep 2016 17:30:31 +0200
The script "checkpatch.pl" can point information out like the following.
WARNING: Prefer kcalloc over kzalloc with multiply
Thus fix the affected source code place.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/drm_bufs.c | 3 +--
From: Markus Elfring
Date: Mon, 19 Sep 2016 17:37:27 +0200
Adjust jump labels according to the current Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/drm_bufs.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/
On 19 September 2016 at 16:33, Jani Nikula
wrote:
> On Mon, 19 Sep 2016, Emil Velikov wrote:
>> On 19 September 2016 at 14:33, wrote:
>>
>>> --- a/drivers/gpu/drm/msm/msm_fb.c
>>> +++ b/drivers/gpu/drm/msm/msm_fb.c
>>> @@ -132,7 +132,7 @@ const struct msm_format *msm_framebuffer_format(struct
1 - 100 of 177 matches
Mail list logo
