diff options
author | Thierry Reding <treding@nvidia.com> | 2017-10-12 16:39:20 +0200 |
---|---|---|
committer | Thierry Reding <treding@nvidia.com> | 2017-12-13 13:42:02 +0100 |
commit | 268892cb63a822315921a8dab48ac3e4abf7dd03 (patch) | |
tree | 79771e9ecf1f579f2271bd4743ada16d1318877b /include/dt-bindings/clock/hi3516cv300-clock.h | |
parent | 5843f4e02fbe86a59981e35adc6cabebee46fdc0 (diff) | |
download | talos-obmc-linux-268892cb63a822315921a8dab48ac3e4abf7dd03.tar.gz talos-obmc-linux-268892cb63a822315921a8dab48ac3e4abf7dd03.zip |
drm/tegra: Sanitize format modifiers
The existing format modifier definitions were merged prematurely, and
recent work has unveiled that the definitions are suboptimal in several
ways:
- The format specifiers, except for one, are not Tegra specific, but
the names don't reflect that.
- The number space is split into two, reserving 32 bits for some
"parameter" which most of the modifiers are not going to have.
- Symbolic names for the modifiers are not using the standard
DRM_FORMAT_MOD_* prefix, which makes them awkward to use.
- The vendor prefix NV is somewhat ambiguous.
Fortunately, nobody's started using these modifiers, so we can still fix
the above issues. Do so by using the standard prefix. Also, remove TEGRA
from the name of those modifiers that exist on NVIDIA GPUs as well. In
case of the block linear modifiers, make the "parameter" smaller (4
bits, though only 6 values are valid) and don't let that leak into any
of the other modifiers.
Finally, also use the more canonical NVIDIA instead of the ambiguous NV
prefix.
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Diffstat (limited to 'include/dt-bindings/clock/hi3516cv300-clock.h')
0 files changed, 0 insertions, 0 deletions