linux icon indicating copy to clipboard operation
linux copied to clipboard

drm: Add DRM_MODE_TV_MODE_MONOCHROME

Open njhollinghurst opened this issue 1 year ago • 7 comments

Add this as a value for enum_drm_connector_tv_mode, represented by the string "MONOCHROME", to represent video with standard timings but no colour encoding or bursts. Define it to have no pedestal (since only NTSC-M calls for a pedestal).

Change default mode creation to acommodate the new tv_mode value which comprises both 525-line and 625-line formats.

njhollinghurst avatar Feb 15 '24 14:02 njhollinghurst

It's a draft for now as I'm not sure if it's something we want to commit ourselves to, or eventually upstream?

Hopefully I have mentioned the new tv_mode in all the places where it's needed?

Carefully worded to imply that it supports both 525-line and 625-line formats but is not necessarily limited to those two.

  • It might be vaguely useful in its own right;
  • It also solves my problem of how to hide/advertise "vintage" TV modes in RP1 VEC;
  • It does not solve the problem of more generic custom modes in the console FB.

njhollinghurst avatar Feb 15 '24 14:02 njhollinghurst

LGTM

6by9 avatar Feb 15 '24 17:02 6by9

I'm still a bit wary of merging it. Once the VEC driver (either of them) starts referring to this, they will be impossible to upstream without this change (or without reverting support for it).

njhollinghurst avatar Feb 15 '24 18:02 njhollinghurst

A late thought: Now I think "Mono" should be not all upper-case, since it's not an acronym. That would be more in keeping with the other property names in drm_connector.c. I hope the abbreviation from "Monochrome" will be acceptable.

njhollinghurst avatar Feb 16 '24 09:02 njhollinghurst

I'll create a patch to implement mono on vc4_vec and try upstreaming it. I'll query the ambiguity over having resolutions quoted in the descriptions but then omitting PAL60.

6by9 avatar Feb 16 '24 12:02 6by9

Sent upstream - https://patchwork.freedesktop.org/series/130023/

6by9 avatar Feb 16 '24 18:02 6by9

This branch and PR are somewhat orphaned now, but for completeness I've pushed an implementation for RP1 VEC.

njhollinghurst avatar Feb 19 '24 10:02 njhollinghurst

Superseded by #5962

njhollinghurst avatar Apr 25 '24 09:04 njhollinghurst