On 04/01/2016 09:35 AM, Tomi Valkeinen wrote:
>
> On 01/04/16 10:03, Tomi Valkeinen wrote:
>
>> So probably we could just fix hdmi_core_powerdown_disable(), so that it
>> sets PD to 1, which is what it was meant to do. This assumes that there
>> are no bad side effects having PD 1 even if the HDM
ves/dri-devel/attachments/20160410/1523f323/attachment.html>
This patch marks some compile-time constant tables 'const'.
The tables marked in this patch are the low hanging fruit
where little other changes were necesary to avoid casting
away constness etc. Also mark some tables that are private
to a file as static.
Signed-off-by: Nils Wallménius
---
driv
This marks the struct amdgpu_sched_ops const and
adjusts amd_sched_init to take a const pointer
for the ops param. The ops member of
struct amd_gpu_scheduler is also changed to const.
Signed-off-by: Nils Wallménius
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +-
drivers/gpu/drm/amd/am
All these are compile time constand and the
drm_debugfs_create/remove_files functions take a const
pointer argument.
Signed-off-by: Nils Wallménius
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h| 4 ++--
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++--
drivers/gpu/drm/amd/amdgpu/amdgpu_fen
As these arrays were of pointer to pointer type, they were
pointer to pointer to const. Make them pointer to const
pointer to const.
Signed-off-by: Nils Wallménius
---
.../drm/amd/powerplay/eventmgr/eventactionchains.c | 34 +++---
.../drm/amd/powerplay/eventmgr/eventmanagement.
These tables were initialized on stack on each call, avoid that
and save a little bit of text size.
Signed-off-by: Nils Wallménius
---
drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c | 2 +-
drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | 2 +-
drivers/gpu/drm/amd/powerplay/hwmgr/ppevvmath.h
Also adjust phm_construct_table to take a const pointer
Signed-off-by: Nils Wallménius
---
.../drm/amd/powerplay/hwmgr/cz_clockpowergating.c| 4 ++--
.../drm/amd/powerplay/hwmgr/cz_clockpowergating.h| 2 +-
drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c | 20 ++--
ks feeding the frontend and backend
> > > > +* ahb: the backend interface clock
> > > > +* mod: the backend module clock
> > > > +* ram: the backend DRAM clock
> > > > + - clock-names: the clock names mentioned above
> > > > + - resets: phandles to the reset controllers driving the backend
> > > > +
> > > > +Display Engine Pipeline
> > > > +---
> > > > +
> > > > +The display engine pipeline (and its entry point, since it can be
> > > > +either directly the backend or the frontend) is represented as an
> > > > +extra node.
> > > > +
> > > > +Required properties:
> > > > + - compatible: value must be one of:
> > > > +* allwinner,sun5i-a13-display-engine
> > > > + - allwinner,pipelines: list of phandle to the entry points of the
> > > > +pipelines (either to the frontend or backend)
> > >
> > > Seems like using FE or BE would be a function of your framebuffers'
> > > formats and shouldn't be defined in DT.
> >
> > Well, they are different IP blocks, so it should be defined in DT,
> > shouldn't it?
>
> They should, but allwinner,sun5i-a13-display-engine is not really an IP
> block. Again, what decides if you use the FE or not?
I'd say we would always want to use it, but we don't have support for
it right now. You can use the backend directly, and this is what we're
doing right now.
Thanks!
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: Digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160410/1c4b675d/attachment.sig>
lectrons
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: Digital signature
URL:
<https://lists.freedesktop.org/archives/d
https://bugzilla.kernel.org/show_bug.cgi?id=116101
Joe P. changed:
What|Removed |Added
Summary|"Userspace still has active |"RIP
|objects" and "trying to
.
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160410/a6d497a7/attachment.html>
12 matches
Mail list logo