drm/amdgpu/display: Multiple fixed-point implementations
Deucher, Alexander
Alexander.Deucher at amd.com
Sat Dec 10 22:25:22 UTC 2016
> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Hristo Venev
> Sent: Saturday, December 10, 2016 5:29 AM
> To: amd-gfx at lists.freedesktop.org
> Subject: drm/amdgpu/display: Multiple fixed-point implementations
>
> There are at least three implementations of fixed-point arithmetic in
> drm/amdgpu/display: fixed31_32 (signed 32-32), fixed32_32 (unsigned 32-
> 32) and bw_fixed (signed 40-24). I don't think fixed-point arithmetic
> implementations belong in drivers.
>
> There already are two implementations of fixed-point in drm: fixed20_12
> (unsigned 20-12) and drm_fixp (signed 32-32). The latter could be used
> instead of fixed31_32.
>
> Maybe we could use a standard fixed-point interface and share the
> source code between most (or all) of the implementations. This is one
> of the things that can be done before merging DC and is not too driver-
> specific. What do you guys think?
Sure. It would be nice to unify them especially if other drivers need them at some point.
Alex
More information about the amd-gfx
mailing list