The PWM framework has clarified the concept of reference PWM config
(the platform dependent config retrieved from the DT or the PWM
lookup table) and real PWM state.
Use pwm_get_args() when the PWM user wants to retrieve this reference
config and not the current state.
This is part of the rework
The PWM framework has clarified the concept of reference PWM config
(the platform dependent config retrieved from the DT or the PWM
lookup table) and real PWM state.
Use pwm_get_args() when the PWM user wants to retrieve this reference
config and not the current state.
This is part of the rework
Add a ->get_state() function to the pwm_ops struct to let PWM drivers
initialize the PWM state attached to a PWM device.
Signed-off-by: Boris Brezillon
---
drivers/pwm/core.c | 3 +++
include/linux/pwm.h | 5 +
2 files changed, 8 insertions(+)
diff --git a/drivers/pwm/core.c b/drivers/pwm/
The current logic will disable the PWM clk even if the PWM was left
enabled by the bootloader (because it's controlling a critical device
like a regulator for example).
Keep the PWM clk enabled if the PWM is enabled to avoid any glitches.
Signed-off-by: Boris Brezillon
---
drivers/pwm/pwm-rockch
Implement ->get_state() instead of only initializing the polarity in
the probe function.
This implementation also takes care of keeping the PWM clk enabled if at
least one of the PWM exported by the PWM chip is already enabled, which
should prevent glitches.
Signed-off-by: Boris Brezillon
---
d
The PWM attached to a PWM regulator device might have been previously
configured by the bootloader.
Make sure the bootloader and linux config are in sync, and adjust the PWM
config if that's not the case.
Signed-off-by: Boris Brezillon
---
drivers/regulator/pwm-regulator.c | 50 +
pwm_config/enable/disable() have been deprecated in favor of
pwm_apply_state().
Replace all those calls with the equivalent pwm_get/apply_state().
Signed-off-by: Boris Brezillon
---
drivers/regulator/pwm-regulator.c | 54 ---
1 file changed, 34 insertions(+),
pwm_config/enable/disable() have been deprecated and should be replaced
by pwm_apply_state().
Signed-off-by: Boris Brezillon
---
drivers/input/misc/max77693-haptic.c | 23 +--
1 file changed, 17 insertions(+), 6 deletions(-)
diff --git a/drivers/input/misc/max77693-haptic.c
pwm_config/enable/disable() have been deprecated and should be replaced
by pwm_apply_state().
Signed-off-by: Boris Brezillon
---
drivers/input/misc/pwm-beeper.c | 46 +++--
1 file changed, 31 insertions(+), 15 deletions(-)
diff --git a/drivers/input/misc/pwm-
pwm_config/enable/disable() have been deprecated and should be replaced
by pwm_apply_state().
Signed-off-by: Boris Brezillon
---
drivers/video/backlight/lm3630a_bl.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/video/backlight/lm3630a_bl.c
b/drivers
Hi,
On Mon, Feb 15, 2016 at 07:09:36PM +0800, Yakir Yang wrote:
> Split the dp core driver from exynos directory to bridge directory,
> and rename the core driver to analogix_dp_*, rename the platform
> code to exynos_dp.
>
> Beside the new analogix_dp driver would export six hooks.
> "analogix_d
On 03/30, Boris Brezillon wrote:
> @@ -74,6 +74,23 @@ enum pwm_polarity {
> PWM_POLARITY_INVERSED,
> };
>
> +/**
> + * struct pwm_args - PWM arguments
> + * @period: reference period
> + * @polarity: reference polarity
> + *
> + * This structure describe board-dependent arguments attached
On Wed, Mar 30, 2016 at 10:03:31PM +0200, Boris Brezillon wrote:
> The PWM framework has clarified the concept of reference PWM config
> (the platform dependent config retrieved from the DT or the PWM
> lookup table) and real PWM state.
>
> Use pwm_get_args() when the PWM user wants to retrieve th
On Thu, Mar 03, 2016 at 04:40:42PM -0300, Gustavo Padovan wrote:
> From: Gustavo Padovan
Gustavo, can you resend both series of your android patches so I know I
have the latest ones to work with? Please also collect the acks that
people have provided so far.
thanks,
greg k-h
Hi Greg,
2016-03-30 Greg Kroah-Hartman :
> On Thu, Mar 03, 2016 at 04:40:42PM -0300, Gustavo Padovan wrote:
> > From: Gustavo Padovan
>
>
>
> Gustavo, can you resend both series of your android patches so I know I
> have the latest ones to work with? Please also collect the acks that
> peopl
On Wed, Mar 30, 2016 at 11:53:38PM -0300, Gustavo Padovan wrote:
> Hi Greg,
>
> 2016-03-30 Greg Kroah-Hartman :
>
> > On Thu, Mar 03, 2016 at 04:40:42PM -0300, Gustavo Padovan wrote:
> > > From: Gustavo Padovan
> >
> >
> >
> > Gustavo, can you resend both series of your android patches so I k
Hi all,
Today's linux-next merge of the drm-misc tree got a conflict in:
include/drm/drm_crtc.h
between commit:
5fff80bbdb6b ("drm/atomic: Allow for holes in connector state, v2.")
from Linus' tree and commit:
6c87e5c3ec6d ("drm: Rename drm_connector_unplug_all() to
drm_connector_unreg
vel/attachments/20160330/f2a67bfe/attachment.html>
eiving 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/20160330/bf22e421/attachment.html>
nts/20160330/20614adc/attachment.html>
On 29.03.2016 18:55, Julian Margetson wrote:
> On 3/28/2016 11:15 PM, Michel Dänzer wrote:
>> On 29.03.2016 08:47, Julian Margetson wrote:
>>> Seeing the following when booting kernel 4.6-rc1 on Acube Sam460ex
>>> Canyonlands board.
>>> This loops for a few times then the kernel boots.
>>> No prob
On Tue, 29 Mar 2016 19:23:12 +0200,
Russell King - ARM Linux wrote:
>
> On Tue, Mar 29, 2016 at 10:54:08AM +0200, Takashi Iwai wrote:
> > On Thu, 17 Mar 2016 13:22:29 +0100,
> > Jyri Sarha wrote:
> > >
> > > Add IEC958 channel status helper that gets the audio properties from
> > > snd_pcm_hw_par
On Tue, Mar 29, 2016 at 10:11:54AM -0400, Lyude Paul wrote:
> bump
>
> Could we get a reviewed-by for this patch? It's needed in addition to the
> patch
> series I sent for removing intel_dp_dpcd_read_wake() for the T560 to have it's
> monitors work properly on resume.
Applied, thanks.
-Daniel
On Tue, Mar 29, 2016 at 06:44:57PM +0200, Florian Zumbiehl wrote:
> Hi,
>
> > FBDEV_EMULATION vs. I915_FBDEV is probably a read herring, more likely is
> > that for some odd reason the very first modeset fails. Once X has
> > resurrect the screen, can you then switch to fbcon? Also please check in
On Tue, Mar 29, 2016 at 05:22:02PM +0300, Imre Deak wrote:
> On Tue, 2016-03-29 at 13:12 +0200, Daniel Vetter wrote:
> > This reverts commit e91abf80a0998f326107874c88d549f94839f13c.
> >
> > Since
> >
> > commit 24e4a8c3e8868874835b0f1ad6dd417341e99822
> > Author: John Stultz
> > Date:Â Â Â Wed
We have allowed migration for only LRU pages until now and it was
enough to make high-order pages. But recently, embedded system(e.g.,
webOS, android) uses lots of non-movable pages(e.g., zram, GPU memory)
so we have seen several reports about troubles of small high-order
allocation. For fixing the
On 03/30/16 09:21, Takashi Iwai wrote:
> On Tue, 29 Mar 2016 19:23:12 +0200,
> Russell King - ARM Linux wrote:
>>
>> On Tue, Mar 29, 2016 at 10:54:08AM +0200, Takashi Iwai wrote:
>>> On Thu, 17 Mar 2016 13:22:29 +0100,
>>> Jyri Sarha wrote:
Add IEC958 channel status helper that gets the a
On Tue, Mar 29, 2016 at 6:16 PM, Andy Lutomirski wrote:
> On Tue, Mar 29, 2016 at 12:49 AM, Andy Lutomirski
> wrote:
>> On Tue, Mar 29, 2016 at 12:43 AM, Daniel Vetter
>> wrote:
>>> On Tue, Mar 29, 2016 at 4:39 AM, Andy Lutomirski
>>> wrote:
AFAICT something got rather screwed up in i91
We don't need to keep empty callbacks for the (pre/post) enable/disable
drm_bridge ops anymore. Remove the nop callback used here for
pre_enable and post_disable ops.
Signed-off-by: Archit Taneja
---
drivers/gpu/drm/bridge/dw-hdmi.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drive
On 3/29/2016 12:04 PM, Daniel Vetter wrote:
> On Thu, Mar 24, 2016 at 12:19:32PM +0530, Archit Taneja wrote:
>>
>>
>> On 02/26/2016 03:21 PM, Laurent Pinchart wrote:
>>> Instead of forcing bridges to implement empty callbacks make them all
>>> optional.
>>
>> Acked-by: Archit Taneja
>
> Applied
This allows us to ditch a ton of ugly #ifdefs from a bunch of drm modeset
drivers.
v2: Make the dummy function actually return a sane value, spotted by
Ville.
v3: Because the patch is still in limbo there's no more drivers to
convert, noticed by Emil.
v4: Rebase once more, because hooray. I'll j
commit e2c8b8701e2d moved modeset locking inside resume/suspend
functions, but missed a code path only executed on lid close/open
on older hardware. The result was a deadlock when closing and
opening the lid without suspending on such hardware:
=
[ INF
It's less than last time around, but still some stragglers. And some fix bugs
reported in RH bugzilla even. Either way I'll throw them all into drm-misc if
there's no nacks within a week, assuming that this means an implicit ack.
Cheers, Daniel
Daniel Vetter (13):
drm/nouveau: Use unlocked gem
For drm_gem_object_unreference callers are required to hold
dev->struct_mutex, which these paths don't. Enforcing this requirement
has become a bit more strict with
commit ef4c6270bf2867e2f8032e9614d1a8cfc6c71663
Author: Daniel Vetter
Date: Thu Oct 15 09:36:25 2015 +0200
drm/gem: Check loc
For drm_gem_object_unreference callers are required to hold
dev->struct_mutex, which these paths don't. Enforcing this requirement
has become a bit more strict with
commit ef4c6270bf2867e2f8032e9614d1a8cfc6c71663
Author: Daniel Vetter
Date: Thu Oct 15 09:36:25 2015 +0200
drm/gem: Check loc
For drm_gem_object_unreference callers are required to hold
dev->struct_mutex, which these paths don't. Enforcing this requirement
has become a bit more strict with
commit ef4c6270bf2867e2f8032e9614d1a8cfc6c71663
Author: Daniel Vetter
Date: Thu Oct 15 09:36:25 2015 +0200
drm/gem: Check loc
For drm_gem_object_unreference callers are required to hold
dev->struct_mutex, which these paths don't. Enforcing this requirement
has become a bit more strict with
commit ef4c6270bf2867e2f8032e9614d1a8cfc6c71663
Author: Daniel Vetter
Date: Thu Oct 15 09:36:25 2015 +0200
drm/gem: Check loc
Doesn't protect anything at all.
With this patch nouveau is completely dev->struct_mutex free!
Cc: Ben Skeggs
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/nouveau/nouveau_fbcon.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_fbcon.c
b/drivers/gpu/
Simply forgotten about this when I was doing my general cleansing of
simple gem mmap offset functions. There's nothing but core functions
called here, and they all have their own protection already.
Aside: DRM_ERROR for userspace controlled input isn't great, but
that's for another patch.
v2: Use
The sg table isn't refcounted, there's no corresponding locking for
unmapping and drm_map_sg is ok with being called concurrently.
So drop the locking since it doesn't protect anything.
Cc: Inki Dae
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/exynos/exynos_drm_gem.c | 4
1 file chang
The only things this protects is reading ->flags and ->size, both of
which are invariant over the lifetime of an exynos gem bo. So no
locking needed at all (besides that, nothing protects the writers
anyway).
Aside: exynos_gem_obj->size is redundant with
exynos_gem_obj->base.size and probably shou
Doesn't protect anything at all, and probably just here because a long
time ago dev->struct_mutex was required to allocate gem objects.
With this patch exynos is completely struct_mutex free!
Cc: Inki Dae
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 22 -
With the previous two changes it doesn't protect anything any more.
v2: Use _unlocked unreference variant.
v3: Appease gcc noise.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/vgem/vgem_drv.c | 17 -
1 file changed, 4 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/
vgem doesn't have a shrinker or anything like that and drops backing
storage only at object_free time. There's no use in trying to be
clever and allocating backing storage delayed, it only causes trouble
by requiring locking.
Instead grab pages when we allocate the object right away.
v2: Fix comp
The offset manager already checks for existing offsets internally,
while holding suitable locks. We can drop this check.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/vgem/vgem_drv.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/
It's racy, creating mmap offsets is a slowpath, so better to remove it
to avoid drivers doing broken things.
The only user is i915, and it's ok there because everything (well
almost) is protected by dev->struct_mutex in i915-gem.
While at it add a note in the create_mmap_offset kerneldoc that
dri
Hi all,
Found this pile of cruft removal patches hiding. Bunch of them have been posted
already a few times, most of them are variations of the "hide dev->struct_mutex"
theme.
Feedback and review highly welcome, I'd like to get them all in.
Thanks, Daniel
Daniel Vetter (10):
drm/ttm: Remove T
Lots of arm drivers get this wrong and for most arm boards this is the
right thing actually. And anyway with most loaders you want to chase
sysfs links anyway to figure out which dri device you want.
This will fix dmesg noise for rockchip and sti.
Cc: Ilia Mirkin
Reported-by: Ilia Mirkin
Signed
For documentation and paranoia.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_sysfs.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_sysfs.c b/drivers/gpu/drm/drm_sysfs.c
index d503f8e8c2d1..43875cb35691 100644
--- a/drivers/gpu/drm/drm
This goes all the way back to the original KMS commit aeons ago
commit f453ba0460742ad027ae0c4c7d61e62817b3e7ef
Author: Dave Airlie
Date: Fri Nov 7 14:05:41 2008 -0800
DRM: add mode setting support
But it seems to be completely unused. Only i915 and nouveau even
register these properties,
It has a DRIVER_MODESET check to sure make it's not creating havoc
for drm drivers. Make that clear in the name too.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_agpsupport.c | 4 ++--
drivers/gpu/drm/drm_fops.c | 2 +-
drivers/gpu/drm/drm_pci.c| 2 +-
include/drm/drm_agps
It belongs right next to the addmap and rmmap functions really. And
for OCD consistency name it drm_legacy_getmap_ioctl.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_bufs.c | 52 ++
drivers/gpu/drm/drm_ioctl.c | 54 +-
Only two drivers implement this hook. vmwgfx (which doesn't need it
really) and legacy radeon (which since v1 has been nuked, yay).
v1: Rebase over radeon ums removal.
Cc: Thomas Hellstrom
Cc: Alex Deucher
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_drv.c | 2 +-
1 file changed, 1 in
Like in
commit 0e975980d435d58df2d430d688b8c18778b42218
Author: Peter Antoine
Date: Tue Jun 23 08:18:49 2015 +0100
drm: Turn off Legacy Context Functions
we need to again make an exception for nouveau, but everyone else
really doesn't need this.
Cc: Peter Antoine
Cc: Ben Skeggs
Signed-
And again make sure it's a no-op for modern drivers, again with the
exception of nouveau. Another case of dev->struct_mutex gone for
modern drivers!
v2: Also add a DRIVER_* check like for all other maps functions to
really short-circuit the code. And give drm_legacy_rmmap used by the
dev unregiste
On Sat, 26 Mar 2016, Paul Parsons wrote:
> The EDID 1.4 specification section 3.10.3.9 defines an Established Timings III
> descriptor (tag #F7h). The parsing of this descriptor by drm_est3_modes() is
> off by one byte: the offset of the first timing bitmap is 6, not 5.
>
> Signed-off-by: Paul Par
It tries to do fancy things with excluding agp support if ttm is
built-in, but agp isn't. Instead just express this depency like drm
does and use CONFIG_AGP everywhere.
Also use the neat Makefile magic to make the entire ttm_agp_backend
file optional.
v2: Use IS_ENABLED(CONFIG_AGP) as suggested b
Except for the ->lasclose driver callback evrything in drm_lastclose()
is all legacy cruft and can be hidden. Which means another
dev->struct_mutex site disappears entirely for modern drivers!
Also while at it change the return value of drm_lastclose to void
since it will always succeed. No one ch
Hi all,
Inspired by the color manager work, some prep work cleanup for the legacy gamma
code. Two more things I'd like to pull off:
- rework the fbdev emulation to use the main gamma interfaces, instead of
hand-rolling it's own.
- add some helpers to implement legacy gamma in terms of atomic and
Code stolen from gma500.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_crtc.c | 13 +
drivers/gpu/drm/gma500/psb_intel_display.c | 7 ---
2 files changed, 13 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.
DRM fbdev emulation only supports pallete_color with depth == 8, and
truecolor with depth > 8. Handling depth == 16 for palettes is hence
dead code, let's remove it.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_fb_helper.c | 33 ++---
1 file changed, 2 inserti
The fb helper private gamma_set/get functions are only required when
the driver supports paletted 8bit mode with fbdev. Armada uses 32bpp
unconditionally, so this is just dead code. It also doesn't do
anything really. Let's just remove it.
Cc: Russell King
Signed-off-by: Daniel Vetter
---
drive
Only really needed for fbdev emulation at 8bpp. And bochs doesn't do
that. And either way bochs only does 32bit rgb, so this is all pretty
much wasted dead code.
The only consideration is that we need to not set up any gamma size
either.
Cc: Gerd Hoffmann
Cc: Dave Airlie
Signed-off-by: Daniel V
The core does this for us already.
Cc: Dave Airlie
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/cirrus/cirrus_mode.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c
b/drivers/gpu/drm/cirrus/cirrus_mode.c
index d3d8d7bfcc57..0b1a411cb89e 100644
---
Again the fbdev emulation gamma_set/get functions are only needed for
drivers that try to also use 8bpp paletted mode. Which msm doesn't, so
this is dead code. Let's rip it out.
Cc: Rob Clark
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/msm/msm_fbdev.c | 14 --
1 file changed, 1
No need to confuse userspace like this.
Cc: Gerd Hoffmann
Cc: Dave Airlie
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/virtio/virtgpu_display.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c
b/drivers/gpu/drm/virtio/virtgpu_display.c
i
imx doesn't have any functions for setting the gamma table, so this is
completely defunct.
Not nice to lie to userspace, so let's stop!
Cc: Russell King
Cc: Philipp Zabel
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/imx/imx-drm-core.c | 4
1 file changed, 4 deletions(-)
diff --git a
qxl doesn't have any functions for setting the gamma table, so this is
completely defunct.
Not nice to lie to userspace, so let's stop!
Cc: Dave Airlie
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/qxl/qxl_display.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/qxl/qxl_
tegra doesn't have any functions to set gamma tables, so this is
completely defunct.
Not nice to lie to userspace, so let's stop!
Cc: Thierry Reding
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/tegra/dc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/tegra/dc.c b/drive
On Wed, Mar 30, 2016 at 11:08:33AM +0200, Bjørn Mork wrote:
> commit e2c8b8701e2d moved modeset locking inside resume/suspend
> functions, but missed a code path only executed on lid close/open
> on older hardware. The result was a deadlock when closing and
> opening the lid without suspending on
On Wed, Mar 30, 2016 at 02:53:24PM +0530, Archit Taneja wrote:
> We don't need to keep empty callbacks for the (pre/post) enable/disable
> drm_bridge ops anymore. Remove the nop callback used here for
> pre_enable and post_disable ops.
>
> Signed-off-by: Archit Taneja
Applied to drm-misc, thanks
Hi Jose,
On 28 March 2016 at 15:36, Jose Abreu wrote:
> This patch adds audio support for the ADV7511 HDMI transmitter
> using ALSA SoC.
>
> The code was ported from Analog Devices linux tree from
> commit 1770c4a1e32b ("Merge remote-tracking branch
> 'xilinx/master' into xcomm_zynq"), which is a
Hi Daniel,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.6-rc1 next-20160330]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Daniel-Vetter/Another-shot-at-cruft-removal
On Sun, 27 Mar 2016, Paul Parsons wrote:
> The EDID 1.4 specification section 3.10.3.9 defines an Established Timings III
> descriptor (tag #F7h). The parsing of this descriptor by drm_est3_modes() is
> off by one byte: the offset of the first timing bitmap is 6, not 5.
Dupe? Let's keep the discu
Op 30-03-16 om 11:08 schreef Bjørn Mork:
> commit e2c8b8701e2d moved modeset locking inside resume/suspend
> functions, but missed a code path only executed on lid close/open
> on older hardware. The result was a deadlock when closing and
> opening the lid without suspending on such hardware:
>
Ap
On 30 March 2016 at 10:26, Daniel Vetter wrote:
> This allows us to ditch a ton of ugly #ifdefs from a bunch of drm modeset
> drivers.
>
> v2: Make the dummy function actually return a sane value, spotted by
> Ville.
>
> v3: Because the patch is still in limbo there's no more drivers to
> convert,
On 30 March 2016 at 10:40, Daniel Vetter wrote:
> The sg table isn't refcounted, there's no corresponding locking for
> unmapping and drm_map_sg is ok with being called concurrently.
>
> So drop the locking since it doesn't protect anything.
>
> Cc: Inki Dae
> Signed-off-by: Daniel Vetter
> ---
On Wed, 30 Mar 2016 10:25:52 +0200,
Jyri Sarha wrote:
>
> On 03/30/16 09:21, Takashi Iwai wrote:
> > On Tue, 29 Mar 2016 19:23:12 +0200,
> > Russell King - ARM Linux wrote:
> >>
> >> On Tue, Mar 29, 2016 at 10:54:08AM +0200, Takashi Iwai wrote:
> >>> On Thu, 17 Mar 2016 13:22:29 +0100,
> >>> Jyri
On 30 March 2016 at 10:40, Daniel Vetter wrote:
> The offset manager already checks for existing offsets internally,
> while holding suitable locks. We can drop this check.
>
> Signed-off-by: Daniel Vetter
With s/dum_map/dump_map/ in the title this is
Reviewed-by: Emil Velikov
-Emil
On 30 March 2016 at 11:29, Emil Velikov wrote:
> On 30 March 2016 at 10:40, Daniel Vetter wrote:
>> The offset manager already checks for existing offsets internally,
>> while holding suitable locks. We can drop this check.
>>
>> Signed-off-by: Daniel Vetter
> With s/dum_map/dump_map/ in the tit
On 3/29/2016 11:49 PM, Michel Dänzer wrote:
> On 29.03.2016 18:55, Julian Margetson wrote:
>> On 3/28/2016 11:15 PM, Michel Dänzer wrote:
>>> On 29.03.2016 08:47, Julian Margetson wrote:
Seeing the following when booting kernel 4.6-rc1 on Acube Sam460ex
Canyonlands board.
This loop
on-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/20160330/0c67661a/attachment.sig>
On 30 March 2016 at 10:45, Daniel Vetter wrote:
> Like in
>
> commit 0e975980d435d58df2d430d688b8c18778b42218
> Author: Peter Antoine
> Date: Tue Jun 23 08:18:49 2015 +0100
>
> drm: Turn off Legacy Context Functions
>
> we need to again make an exception for nouveau, but everyone else
> rea
On 30 March 2016 at 10:45, Daniel Vetter wrote:
> Lots of arm drivers get this wrong and for most arm boards this is the
> right thing actually. And anyway with most loaders you want to chase
> sysfs links anyway to figure out which dri device you want.
>
> This will fix dmesg noise for rockchip a
On 30 March 2016 at 10:45, Daniel Vetter wrote:
> --- a/drivers/gpu/drm/Kconfig
> +++ b/drivers/gpu/drm/Kconfig
> @@ -83,6 +83,7 @@ config DRM_LOAD_EDID_FIRMWARE
> config DRM_TTM
> tristate
> depends on DRM
> + depends on (AGP || AGP=n)
Already part of config DRM, so we don
On 30.03.2016 11:40, Daniel Vetter wrote:
> For drm_gem_object_unreference callers are required to hold
> dev->struct_mutex, which these paths don't. Enforcing this requirement
> has become a bit more strict with
>
> commit ef4c6270bf2867e2f8032e9614d1a8cfc6c71663
> Author: Daniel Vetter
> Date:
Hi Daniel,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.6-rc1 next-20160330]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Daniel-Vetter/Another-shot-at-cruft-removal
It tries to do fancy things with excluding agp support if ttm is
built-in, but agp isn't. Instead just express this depency like drm
does and use CONFIG_AGP everywhere.
Also use the neat Makefile magic to make the entire ttm_agp_backend
file optional.
v2: Use IS_ENABLED(CONFIG_AGP) as suggested b
On Wed, Mar 30, 2016 at 11:51:18AM +0200, Daniel Vetter wrote:
> The fb helper private gamma_set/get functions are only required when
> the driver supports paletted 8bit mode with fbdev. Armada uses 32bpp
> unconditionally, so this is just dead code. It also doesn't do
> anything really. Let's just
On Wed, Mar 30, 2016 at 1:09 PM, Russell King - ARM Linux
wrote:
> On Wed, Mar 30, 2016 at 11:51:18AM +0200, Daniel Vetter wrote:
>> The fb helper private gamma_set/get functions are only required when
>> the driver supports paletted 8bit mode with fbdev. Armada uses 32bpp
>> unconditionally, so t
Hi Daniel,
[auto build test WARNING on drm/drm-next]
[also build test WARNING on v4.6-rc1 next-20160330]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Daniel-Vetter/drm-ttm-Remove
It tries to do fancy things with excluding agp support if ttm is
built-in, but agp isn't. Instead just express this depency like drm
does and use CONFIG_AGP everywhere.
Also use the neat Makefile magic to make the entire ttm_agp_backend
file optional.
v2: Use IS_ENABLED(CONFIG_AGP) as suggested b
The original patches have been done by Marek:
https://lists.freedesktop.org/archives/dri-devel/2016-January/099572.html
I have just adapt them to make zpos depend on plane and no more on drm core.
Since zpos range can be define per plane normalize function takes care of
the min and max before com
This patch adds support for generic plane's zpos property property with
well-defined semantics:
- added zpos properties to plane and plane state structures
- added helpers for normalizing zpos properties of given set of planes
- well defined semantics: planes are sorted by zpos values and then plan
remove private zpos property and use instead the generic new.
zpos range is now fixed per plane type and normalized before
being using in mixer.
Signed-off-by: Benjamin Gaignard
---
drivers/gpu/drm/sti/sti_mixer.c | 2 +-
drivers/gpu/drm/sti/sti_plane.c | 82 +++-
From: Thierry Reding
Commit 6c87e5c3ec6d ("drm: Rename drm_connector_unplug_all() to
drm_connector_unregister_all()") didn't update the Atmel HLCDC driver,
causing it to fail to build. While at it, remove unnecessary locking.
Cc: Boris Brezillon
Cc: Alexey Brodkin
Signed-off-by: Thierry Reding
for this already.
Thierry
-- 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/20160330/56446470/attachment.sig>
fs.
But given that I sent out almost the same patch earlier, and this does
fix a build failure:
Reviewed-by: Thierry Reding
-- 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/20160330/de8ab8ab/attachment.sig>
-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160330/d24f64f7/attachment.sig>
/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160330/011a3366/attachment.sig>
1 - 100 of 219 matches
Mail list logo
